[dpdk-dev] [PATCH 01/14] ethdev: remove legacy MACVLAN filter type support

[Guo, Jia]

> -----Original Message-----
> From: Andrew Rybchenko <arybchenko at solarflare.com>
> Sent: Thursday, October 22, 2020 12:05 AM
> To: Guo, Jia <jia.guo at intel.com>; Lu, Wenzhuo <wenzhuo.lu at intel.com>;
> Xing, Beilei <beilei.xing at intel.com>; Iremonger, Bernard
> <bernard.iremonger at intel.com>; Ray Kinsella <mdr at ashroe.eu>; Neil
> Horman <nhorman at tuxdriver.com>; Rasesh Mody <rmody at marvell.com>;
> Shahed Shaikh <shshaikh at marvell.com>; Andrew Rybchenko
> <andrew.rybchenko at oktetlabs.ru>; Thomas Monjalon
> <thomas at monjalon.net>; Yigit, Ferruh <ferruh.yigit at intel.com>
> Cc: dev at dpdk.org
> 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 <arybchenko at solarflare.com>
> >> Sent: Sunday, October 18, 2020 10:09 PM
> >> To: Lu, Wenzhuo <wenzhuo.lu at intel.com>; Xing, Beilei
> >> <beilei.xing at intel.com>; Iremonger, Bernard
> >> <bernard.iremonger at intel.com>; Ray Kinsella <mdr at ashroe.eu>; Neil
> >> Horman <nhorman at tuxdriver.com>; Guo, Jia <jia.guo at intel.com>;
> Rasesh
> >> Mody <rmody at marvell.com>; Shahed Shaikh <shshaikh at marvell.com>;
> >> Andrew Rybchenko <andrew.rybchenko at oktetlabs.ru>; Thomas
> Monjalon
> >> <thomas at monjalon.net>; Yigit, Ferruh <ferruh.yigit at intel.com>
> >> Cc: dev at dpdk.org
> >> 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.