On 9/22/2018 5:55 AM, Ajit Khaparde wrote: > We need to set HWRM_CFA_L2_FILTER_ALLOC_INPUT_FLAGS_OUTERMOST bit in > L2_FILTER_ALLOC for filtering non-tunnel packets based on outermost MAC.
What happens if that bit is not set, filtering is not working as expected? If so can you please convert this patch to a fix patch? > Signed-off-by: Ajit Khaparde <ajit.khapa...@broadcom.com> > --- > drivers/net/bnxt/bnxt_hwrm.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c > index 51fe3a4c2..b605659ed 100644 > --- a/drivers/net/bnxt/bnxt_hwrm.c > +++ b/drivers/net/bnxt/bnxt_hwrm.c > @@ -383,6 +383,8 @@ int bnxt_hwrm_set_l2_filter(struct bnxt *bp, > HWRM_PREP(req, CFA_L2_FILTER_ALLOC, BNXT_USE_CHIMP_MB); > > req.flags = rte_cpu_to_le_32(filter->flags); > + req.flags |= > + rte_cpu_to_le_32(HWRM_CFA_L2_FILTER_ALLOC_INPUT_FLAGS_OUTERMOST); > > enables = filter->enables | > HWRM_CFA_L2_FILTER_ALLOC_INPUT_ENABLES_DST_ID; >
