Re: [dpdk-dev] [PATCH 1/2] net/virtio: do not re-enter clean up routines

2018-11-02 Thread Ferruh Yigit
On 11/2/2018 4:48 PM, Maxime Coquelin wrote: > > > On 11/2/18 4:19 PM, Chas Williams wrote: >> >> >> On 11/02/2018 10:33 AM, Ferruh Yigit wrote: >>> On 11/1/2018 2:45 PM, Luca Boccassi wrote: On Mon, 2017-07-17 at 19:05 -0400, Charles (Chas) Williams wrote: > .dev_uninit calls .dev_stop

Re: [dpdk-dev] [PATCH 1/2] net/virtio: do not re-enter clean up routines

2018-11-02 Thread Maxime Coquelin
On 11/2/18 4:19 PM, Chas Williams wrote: On 11/02/2018 10:33 AM, Ferruh Yigit wrote: On 11/1/2018 2:45 PM, Luca Boccassi wrote: On Mon, 2017-07-17 at 19:05 -0400, Charles (Chas) Williams wrote: .dev_uninit calls .dev_stop and .dev_close.  The work that is done in those routines doesn't ne

Re: [dpdk-dev] [PATCH 1/2] net/virtio: do not re-enter clean up routines

2018-11-02 Thread Chas Williams
On 11/02/2018 10:33 AM, Ferruh Yigit wrote: On 11/1/2018 2:45 PM, Luca Boccassi wrote: On Mon, 2017-07-17 at 19:05 -0400, Charles (Chas) Williams wrote: .dev_uninit calls .dev_stop and .dev_close. The work that is done in those routines doesn't need repeated. Use started and opened to trac

Re: [dpdk-dev] [PATCH 1/2] net/virtio: do not re-enter clean up routines

2018-11-02 Thread Ferruh Yigit
On 11/1/2018 2:45 PM, Luca Boccassi wrote: > On Mon, 2017-07-17 at 19:05 -0400, Charles (Chas) Williams wrote: >> .dev_uninit calls .dev_stop and .dev_close.  The work that is done in >> those routines doesn't need repeated.  Use started and opened to >> track >> the adapter's status. >> >> Signed-

Re: [dpdk-dev] [PATCH 1/2] net/virtio: do not re-enter clean up routines

2018-11-02 Thread Luca Boccassi
On Fri, 2018-11-02 at 12:03 +0100, Maxime Coquelin wrote: > > On 11/2/18 10:57 AM, Maxime Coquelin wrote: > > > > > > On 11/1/18 3:45 PM, Luca Boccassi wrote: > > > On Mon, 2017-07-17 at 19:05 -0400, Charles (Chas) Williams wrote: > > > > .dev_uninit calls .dev_stop and .dev_close.  The work tha

Re: [dpdk-dev] [PATCH 1/2] net/virtio: do not re-enter clean up routines

2018-11-02 Thread Maxime Coquelin
On 11/2/18 10:57 AM, Maxime Coquelin wrote: On 11/1/18 3:45 PM, Luca Boccassi wrote: On Mon, 2017-07-17 at 19:05 -0400, Charles (Chas) Williams wrote: .dev_uninit calls .dev_stop and .dev_close.  The work that is done in those routines doesn't need repeated.  Use started and opened to trac

Re: [dpdk-dev] [PATCH 1/2] net/virtio: do not re-enter clean up routines

2018-11-02 Thread Maxime Coquelin
On 11/1/18 3:45 PM, Luca Boccassi wrote: On Mon, 2017-07-17 at 19:05 -0400, Charles (Chas) Williams wrote: .dev_uninit calls .dev_stop and .dev_close.  The work that is done in those routines doesn't need repeated.  Use started and opened to track the adapter's status. Signed-off-by: Chas Wi

Re: [dpdk-dev] [PATCH 1/2] net/virtio: do not re-enter clean up routines

2018-11-01 Thread Luca Boccassi
On Mon, 2017-07-17 at 19:05 -0400, Charles (Chas) Williams wrote: > .dev_uninit calls .dev_stop and .dev_close.  The work that is done in > those routines doesn't need repeated.  Use started and opened to > track > the adapter's status. > > Signed-off-by: Chas Williams > --- >  drivers/net/virtio

[dpdk-dev] [PATCH 1/2] net/virtio: do not re-enter clean up routines

2017-07-17 Thread Charles (Chas) Williams
.dev_uninit calls .dev_stop and .dev_close. The work that is done in those routines doesn't need repeated. Use started and opened to track the adapter's status. Signed-off-by: Chas Williams --- drivers/net/virtio/virtio_ethdev.c | 15 --- drivers/net/virtio/virtio_pci.h| 4 +++