On Mon, 20 Nov 2023 17:22:59 +0100 Robin Jarry <rja...@redhat.com> wrote:
> diff --git a/usertools/dpdk-rss-flows.py b/usertools/dpdk-rss-flows.py > index fd225a697f08..be9b3d760c03 100755 > --- a/usertools/dpdk-rss-flows.py > +++ b/usertools/dpdk-rss-flows.py > @@ -226,6 +226,23 @@ def reta_size(self, num_queues: int) -> int: > ), > reta_size=512, > ), > + "cnxk": DriverInfo( > + key=bytes( > + ( > + # fmt: off > + # roc_nix_rss_key_default_fill, see > drivers/common/cnxk/roc_nix_rss.c > + # Marvell's cnxk NICs take 48 bytes keys > + 0xfe, 0xed, 0x0b, 0xad, 0xfe, 0xed, 0x0b, 0xad, > + 0xfe, 0xed, 0x0b, 0xad, 0xfe, 0xed, 0x0b, 0xad, > + 0xfe, 0xed, 0x0b, 0xad, 0xfe, 0xed, 0x0b, 0xad, > + 0xfe, 0xed, 0x0b, 0xad, 0xfe, 0xed, 0x0b, 0xad, > + 0xfe, 0xed, 0x0b, 0xad, 0xfe, 0xed, 0x0b, 0xad, > + 0xfe, 0xed, 0x0b, 0xad, 0xfe, 0xed, 0x0b, 0xad, > + # fmt: on > + ) > + ), > + reta_size=64, > + ), > } Maybe an enhancement in future to expose the default key better. Having it live in two places means adding new drivers has another thing to worry about (and not documented).