Re: [PATCH] app: fix silent enqueue fail in test_mbuf test_refcnt_iter

2023-08-30 Thread Thomas Monjalon
Hello, 22/08/2023 08:34, jhascoet: > From: Julien Hascoet > > In case of ring full state, we retry the enqueue > operation in order to avoid mbuf loss. > > Fixes: af75078fece ("first public release") > > Signed-off-by: Julien Hascoet Please could you give more information about when this is

Re: [PATCH] app: fix silent enqueue fail in test_mbuf test_refcnt_iter

2023-08-24 Thread Hascoet Julien
Hello Oliver, thanks, your response helped a lot, I managed to find the root cause of the instability which is on our side. It was due to other internal developments. I'll still add an error check on the enqueue ops to catch eventual problems earlier, if that suits you. Best regards, Julien On

Re: [PATCH] app: fix silent enqueue fail in test_mbuf test_refcnt_iter

2023-08-22 Thread Olivier Matz
Hello Julien, On Tue, Aug 22, 2023 at 08:34:53AM +0200, jhascoet wrote: > From: Julien Hascoet > > In case of ring full state, we retry the enqueue > operation in order to avoid mbuf loss. > > Fixes: af75078fece ("first public release") > > Signed-off-by: Julien Hascoet > --- > app/test/test

Re: [PATCH] app: fix silent enqueue fail in test_mbuf test_refcnt_iter

2023-08-18 Thread Hascoet Julien
Any news on that ? Thanks Julien On Thu, Aug 10, 2023 at 6:09 PM jhascoet wrote: > From: Julien Hascoet > > In case of ring full state, we retry the enqueue > operation in order to avoid mbuf loss. > > Fixes: af75078fece ("first public release") > > Signed-off-by: Julien Hascoet > --- > app/

Re: [PATCH] app: fix silent enqueue fail in test_mbuf test_refcnt_iter

2023-08-17 Thread David Marchand
On Fri, Aug 18, 2023 at 7:17 AM Hascoet Julien wrote: > > Any news on that ? Thanks Don't forget to copy maintainers. Let's leave some time to Olivier to have a look. Thanks. -- David Marchand

RE: [PUB] Re: [PATCH] app: fix silent enqueue fail in test_mbuf test_refcnt_iter

2023-08-10 Thread Julien Hascoet
Yes, just did it. Thanks ! De : Stephen Hemminger Envoyé : jeudi 10 août 2023 17:33 À : jhascoet Cc : david.march...@redhat.com ; dev@dpdk.org Objet : [PUB] Re: [PATCH] app: fix silent enqueue fail in test_mbuf test_refcnt_iter On Thu, 10 Aug 2023 08:00:30

Re: [PATCH] app: fix silent enqueue fail in test_mbuf test_refcnt_iter

2023-08-10 Thread Stephen Hemminger
On Thu, 10 Aug 2023 08:00:30 +0200 jhascoet wrote: > diff --git a/app/test/test_mbuf.c b/app/test/test_mbuf.c > index efac01806b..be114e3302 100644 > --- a/app/test/test_mbuf.c > +++ b/app/test/test_mbuf.c > @@ -1033,12 +1033,17 @@ test_refcnt_iter(unsigned int lcore, unsigned int > iter, >

Re: [PATCH] app: fix silent enqueue fail in test_mbuf test_refcnt_iter

2023-08-09 Thread David Marchand
On Wed, Aug 9, 2023 at 7:39 AM jhascoet wrote: > > In case of ring full state, we retry the enqueue > operation in order to avoid mbuf loss. > > Fixes: af75078fece ("first public release") Not sure we need to backport, but in doubt, I would mark it Cc: sta...@dpdk.org > > Signed-off-by: Julien