mbox series

[0/3] fix error path of multi-process probe

Message ID 20190302024253.15594-1-thomas@monjalon.net (mailing list archive)
Headers
Series fix error path of multi-process probe |

Message

Thomas Monjalon March 2, 2019, 2:42 a.m. UTC 
  While working on multi-process support of failsafe with Raslan,
some issues have been discovered on probing failures.

Thomas Monjalon (3):
  eal: remove useless checks for already probed device
  eal: remove error logs for already probed device
  eal: fix multi-process probe failure handling

 lib/librte_eal/common/eal_common_dev.c | 31 ++++++++++----------------
 lib/librte_eal/common/eal_private.h    |  2 +-
 lib/librte_eal/common/hotplug_mp.c     | 14 +++++++-----
 3 files changed, 21 insertions(+), 26 deletions(-)

Comments

Stephen Hemminger June 14, 2023, 7:39 p.m. UTC | #1 
On Sat,  2 Mar 2019 03:42:50 +0100
Thomas Monjalon <thomas@monjalon.net> wrote:

> While working on multi-process support of failsafe with Raslan,
> some issues have been discovered on probing failures.
> 
> Thomas Monjalon (3):
>   eal: remove useless checks for already probed device
>   eal: remove error logs for already probed device
>   eal: fix multi-process probe failure handling
> 
>  lib/librte_eal/common/eal_common_dev.c | 31 ++++++++++----------------
>  lib/librte_eal/common/eal_private.h    |  2 +-
>  lib/librte_eal/common/hotplug_mp.c     | 14 +++++++-----
>  3 files changed, 21 insertions(+), 26 deletions(-)
> 

Is this old patch set relevant. Seems like it addressing a
"this should never happen but if it does don't be stupid" kind of scenario.

Marking it as "Changes Requested" since rebase and retest would be needed.