18/10/2023 09:26, Sunil Kumar Kori: > From: Robin Jarry <rja...@redhat.com> > > From: Sunil Kumar Kori <sk...@marvell.com> > > > > > > This patch adds RSS key for CNXK platforms. CNXK platform uses > > > 48 bytes long key for hash calculations. > > > > > > For the same patch also updates help mesaages to provide range > > > information for supporting NICs/platforms. > > > > > > Also CNXK uses reta size as 64 so to get correct offset to retrieve > > > queue index, user must pass reta_size option as 64 i.e. -t 64. > > > > I think we should add some driver abstraction that contains the required key > > length and default reta size. Instead of requiring the user to guess the > > correct > > values. Is that something you could do? > > > Okay but in either case i.e. -t option or driver abstraction, user must know > the reta size and key size before configuring. > So I am not sure that how adding driver abstraction will help to solve this > issue unless/until its documented somewhere.
You can start with an option to get the size printed, depending on driver name. > So for current release, I am planning to go this version as it is because we > are close. > Later on we can think of it and add required support. > Please provide input on it. Please provide a more user friendly experience in this release.
