Re: [dpdk-dev] [PATCH] net/qede: only access sw rx ring index for debug

2019-10-04 Thread Jerin Jacob
On Thu, Oct 3, 2019 at 11:59 PM Rasesh Mody wrote: > > >From: David Marchand > >Sent: Friday, September 27, 2019 4:29 AM > > > >-- > >Caught by clang, this idx value is only used for a debug message when the > >mbufs allocation f

Re: [dpdk-dev] [PATCH] net/qede: only access sw rx ring index for debug

2019-10-03 Thread Rasesh Mody
>From: dev On Behalf Of Jerin Jacob >Sent: Thursday, October 03, 2019 7:40 AM > >On Fri, Sep 27, 2019 at 4:59 PM David Marchand > wrote: >> >> Caught by clang, this idx value is only used for a debug message when >> the mbufs allocation fails. >> No need to use idx as a temporary storage. >> >> Fi

Re: [dpdk-dev] [PATCH] net/qede: only access sw rx ring index for debug

2019-10-03 Thread Rasesh Mody
>From: David Marchand >Sent: Friday, September 27, 2019 4:29 AM > >-- >Caught by clang, this idx value is only used for a debug message when the >mbufs allocation fails. >No need to use idx as a temporary storage. > >Fixes: 8f2312

Re: [dpdk-dev] [PATCH] net/qede: only access sw rx ring index for debug

2019-10-03 Thread Jerin Jacob
On Fri, Sep 27, 2019 at 4:59 PM David Marchand wrote: > > Caught by clang, this idx value is only used for a debug message when > the mbufs allocation fails. > No need to use idx as a temporary storage. > > Fixes: 8f2312474529 ("net/qede: fix performance bottleneck in Rx path") > Cc: sta...@dpdk.o

[dpdk-dev] [PATCH] net/qede: only access sw rx ring index for debug

2019-09-27 Thread David Marchand
Caught by clang, this idx value is only used for a debug message when the mbufs allocation fails. No need to use idx as a temporary storage. Fixes: 8f2312474529 ("net/qede: fix performance bottleneck in Rx path") Cc: sta...@dpdk.org Signed-off-by: David Marchand --- drivers/net/qede/qede_rxtx.c