Re: [dpdk-dev] [PATCH] net/bnxt: allow the mark to use a cfa code of zero

2020-05-27 Thread Ajit Khaparde
On Wed, May 27, 2020 at 8:47 AM Ferruh Yigit wrote: > On 5/23/2020 12:55 AM, Mike Baucom wrote: > > The mark code was too restrictive by disallowing a cfa_code of zero. > > This code loosens the requirement and allows zero. > > I can see "if (cfa_code)" check removed, but can you please give some

Re: [dpdk-dev] [PATCH] net/bnxt: allow the mark to use a cfa code of zero

2020-05-27 Thread Ferruh Yigit
On 5/23/2020 12:55 AM, Mike Baucom wrote: > The mark code was too restrictive by disallowing a cfa_code of zero. > This code loosens the requirement and allows zero. I can see "if (cfa_code)" check removed, but can you please give some details that what it the impact of having that check and why c

Re: [dpdk-dev] [PATCH] net/bnxt: allow the mark to use a cfa code of zero

2020-05-23 Thread Ajit Khaparde
On Fri, May 22, 2020 at 4:55 PM Mike Baucom wrote: > The mark code was too restrictive by disallowing a cfa_code of zero. > This code loosens the requirement and allows zero. > > Fixes: b87abb2e55cb ("net/bnxt: support marking packet") > > Signed-off-by: Mike Baucom > Reviewed-by: Kishore Padman

[dpdk-dev] [PATCH] net/bnxt: allow the mark to use a cfa code of zero

2020-05-22 Thread Mike Baucom
The mark code was too restrictive by disallowing a cfa_code of zero. This code loosens the requirement and allows zero. Fixes: b87abb2e55cb ("net/bnxt: support marking packet") Signed-off-by: Mike Baucom Reviewed-by: Kishore Padmanabha --- drivers/net/bnxt/bnxt_rxr.c | 20 +---