> -----Original Message----- > From: Andrew Rybchenko <[email protected]> > Sent: Thursday, October 22, 2020 12:05 AM > To: Guo, Jia <[email protected]>; Lu, Wenzhuo <[email protected]>; > Xing, Beilei <[email protected]>; Iremonger, Bernard > <[email protected]>; Ray Kinsella <[email protected]>; Neil > Horman <[email protected]>; Rasesh Mody <[email protected]>; > Shahed Shaikh <[email protected]>; Andrew Rybchenko > <[email protected]>; Thomas Monjalon > <[email protected]>; Yigit, Ferruh <[email protected]> > Cc: [email protected] > Subject: Re: [PATCH 01/14] ethdev: remove legacy MACVLAN filter type > support > > On 10/21/20 6:31 AM, Guo, Jia wrote: > > > >> -----Original Message----- > >> From: Andrew Rybchenko <[email protected]> > >> Sent: Sunday, October 18, 2020 10:09 PM > >> To: Lu, Wenzhuo <[email protected]>; Xing, Beilei > >> <[email protected]>; Iremonger, Bernard > >> <[email protected]>; Ray Kinsella <[email protected]>; Neil > >> Horman <[email protected]>; Guo, Jia <[email protected]>; > Rasesh > >> Mody <[email protected]>; Shahed Shaikh <[email protected]>; > >> Andrew Rybchenko <[email protected]>; Thomas > Monjalon > >> <[email protected]>; Yigit, Ferruh <[email protected]> > >> Cc: [email protected] > >> Subject: [PATCH 01/14] ethdev: remove legacy MACVLAN filter type > >> support > >> > >> RTE flow API should be used for filtering. > > > > Look like each patch in the patch set remove one specific legacy > > filter, so I think the removing filter info is need to show in the commit > > log > to make it more clear, please add in the next version. > > Sorry, don't understand. Isn't MACVLAN in summary sufficient?
Sorry, maybe I did't say clear. Basically we concentrate the context in title and then detail more in description, for this description, maybe some confuse that if this patch is wholly for all filtering but eventually it just specific for one filtering.
