Default RSS for outer src/dst IP address field in iavf is not supported before, 
so it does not cause any error.
However, if it can be dropped, I suggest to do so. It seems to be safer to add 
IPID field here.

> -----Original Message-----
> From: David Marchand <david.march...@redhat.com>
> Sent: Tuesday, September 7, 2021 3:10 PM
> To: Wu, Wenjun1 <wenjun1...@intel.com>; Yigit, Ferruh
> <ferruh.yi...@intel.com>
> Cc: dev <dev@dpdk.org>; Yang, Qiming <qiming.y...@intel.com>; Zhang, Qi
> Z <qi.z.zh...@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v1] net/ice: revert removing IPID from
> default hash field
> 
> On Tue, Sep 7, 2021 at 9:05 AM Wenjun Wu <wenjun1...@intel.com> wrote:
> >
> > We try to refine default RSS for IP fragment packets. However, the
> > change will lead to more serious errors. The scenario that there is
> > overlap/conflict between the new characteristics and the existing ones
> > has not been supported, so non-fragment packets and fragment packets
> > cannot share the same hash fields, or all related profiles will be
> > removed.
> >
> > Therefore, IPID field is necessary for fragment packets.
> >
> > Fixes: cf37e1e5e9d2 ("net/ice: fix default RSS hash for IP fragment
> > packets")
> >
> > Signed-off-by: Wenjun Wu <wenjun1...@intel.com>
> 
> - If this is a revert of cf37e1e5e9d2, maybe it is simpler to drop the 
> original
> change in next-net before it gets pulled in the main repo.
> - A similar change has been applied to net/iavf? Is it still relevant?
> 
> 
> --
> David Marchand

Reply via email to