Re: [dpdk-dev] [PATCH] net/failsafe: do not probe device if plugged out

2017-07-18 Thread Thomas Monjalon
18/07/2017 11:39, Ferruh Yigit: > On 7/13/2017 10:17 AM, Thomas Monjalon wrote: > > 13/07/2017 10:14, Gaëtan Rivet: > >> On Thu, Jul 13, 2017 at 08:52:33AM +0200, Thomas Monjalon wrote: > >>> 12/07/2017 22:39, Gaëtan Rivet: > Hi Thomas, > > Nice idea. A few remarks below: > > >>

Re: [dpdk-dev] [PATCH] net/failsafe: do not probe device if plugged out

2017-07-18 Thread Ferruh Yigit
On 7/13/2017 10:17 AM, Thomas Monjalon wrote: > 13/07/2017 10:14, Gaëtan Rivet: >> On Thu, Jul 13, 2017 at 08:52:33AM +0200, Thomas Monjalon wrote: >>> 12/07/2017 22:39, Gaëtan Rivet: Hi Thomas, Nice idea. A few remarks below: On Wed, Jul 12, 2017 at 08:28:12PM +0200, Thoma

Re: [dpdk-dev] [PATCH] net/failsafe: do not probe device if plugged out

2017-07-13 Thread Thomas Monjalon
13/07/2017 10:14, Gaëtan Rivet: > On Thu, Jul 13, 2017 at 08:52:33AM +0200, Thomas Monjalon wrote: > > 12/07/2017 22:39, Gaëtan Rivet: > > > Hi Thomas, > > > > > > Nice idea. A few remarks below: > > > > > > On Wed, Jul 12, 2017 at 08:28:12PM +0200, Thomas Monjalon wrote: > > > > FOREACH_

Re: [dpdk-dev] [PATCH] net/failsafe: do not probe device if plugged out

2017-07-13 Thread Gaëtan Rivet
On Thu, Jul 13, 2017 at 08:52:33AM +0200, Thomas Monjalon wrote: > 12/07/2017 22:39, Gaëtan Rivet: > > Hi Thomas, > > > > Nice idea. A few remarks below: > > > > On Wed, Jul 12, 2017 at 08:28:12PM +0200, Thomas Monjalon wrote: > > > FOREACH_SUBDEV(sdev, i, dev) { > > > if (sdev->state

Re: [dpdk-dev] [PATCH] net/failsafe: do not probe device if plugged out

2017-07-12 Thread Thomas Monjalon
12/07/2017 22:39, Gaëtan Rivet: > Hi Thomas, > > Nice idea. A few remarks below: > > On Wed, Jul 12, 2017 at 08:28:12PM +0200, Thomas Monjalon wrote: > > FOREACH_SUBDEV(sdev, i, dev) { > > if (sdev->state != DEV_PARSED) > > continue; > > da = &sdev-

Re: [dpdk-dev] [PATCH] net/failsafe: do not probe device if plugged out

2017-07-12 Thread Gaëtan Rivet
Hi Thomas, Nice idea. A few remarks below: On Wed, Jul 12, 2017 at 08:28:12PM +0200, Thomas Monjalon wrote: > When probing a sub-devices which does not exist in the system, > some errors are logged: > EAL: Cannot find unplugged device (0002:00:02.0) > PMD: net_failsafe: ERROR: sub_dev

[dpdk-dev] [PATCH] net/failsafe: do not probe device if plugged out

2017-07-12 Thread Thomas Monjalon
When probing a sub-devices which does not exist in the system, some errors are logged: EAL: Cannot find unplugged device (0002:00:02.0) PMD: net_failsafe: ERROR: sub_device 0 probe failed It is normal to have these errors when initializing the failsafe device and its sub-devices. B