[PATCH v2] net/i40e: support enabling/disabling source pruning
Zhang, Ke1X
ke1x.zhang at intel.com
Tue Jan 31 04:28:32 CET 2023
> -----Original Message-----
> From: David Marchand <david.marchand at redhat.com>
> Sent: Monday, January 30, 2023 4:58 PM
> To: Zhang, Ke1X <ke1x.zhang at intel.com>; mb at smartsharesystems.com;
> thomas at monjalon.net; ferruh.yigit at amd.com
> Cc: dev at dpdk.org; Matz, Olivier <olivier.matz at 6wind.com>; Zhang, Yuying
> <yuying.zhang at intel.com>; Xing, Beilei <beilei.xing at intel.com>
> Subject: Re: [PATCH v2] net/i40e: support enabling/disabling source pruning
>
> On Mon, Jan 30, 2023 at 9:23 AM Ke Zhang <ke1x.zhang at intel.com> wrote:
> >
> > VRRP advertisement packets are dropped on i40e PF devices because
> when
> > a MAC address is added to a device, packets originating from that MAC
> > address are dropped.
> >
> > This patch adds a PMD specific API to enable/disable source pruning to
> > fix above issue.
> >
> > Bugzilla ID: 648
> >
> > Signed-off-by: Ke Zhang <ke1x.zhang at intel.com>
> > ---
> > app/test-pmd/cmdline.c | 84
> +++++++++++++++++++++++++++++++++
> > drivers/net/i40e/i40e_ethdev.c | 43 +++++++++++++++++
> > drivers/net/i40e/i40e_ethdev.h | 1 +
> > drivers/net/i40e/rte_pmd_i40e.c | 20 ++++++++
> > drivers/net/i40e/rte_pmd_i40e.h | 16 +++++++
> > 5 files changed, 164 insertions(+)
> >
> > diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index
> > cb8c174020..76a574affd 100644
> > --- a/app/test-pmd/cmdline.c
> > +++ b/app/test-pmd/cmdline.c
> > @@ -776,6 +776,9 @@ static void cmd_help_long_parsed(void
> > *parsed_result,
> >
> > "port cleanup (port_id) txq (queue_id) (free_cnt)\n"
> > " Cleanup txq mbufs for a specific Tx queue\n\n"
> > +
> > + "port config (port_id|all) src_prune (on|off)\n"
> > + " Set source prune on port_id, or all.\n\n"
> > );
> > }
> >
>
> - This seems i40e specific, please move to drivers/net/i40e/i40e_testpmd.c.
>
> - Besides, I would prefer that something in the command name clearly states
> this is driver (here, i40e) specific.
> Like "port config XX i40e_src_prune" or maybe the other way around, start
> with a "driver i40e" prefix.
>
> Maybe others have an opinion.
>
>
> --
> David Marchand
Thanks for your comments, this command could be used for other Intel NIC like ice,
And It is only finished for i40e in this story: https://jira.devtools.intel.com/browse/DPDK-29564
More information about the dev
mailing list