Hi Stephen Thanks your comments. We already change it in the following series and please take a look: https://patchwork.dpdk.org/project/dpdk/list/?series=16524 + /** + * Color the packet to reflect the meter color result. + * Set the meter color in the mbuf to the selected color. + * + * See struct rte_flow_action_meter_color. + */ + RTE_FLOW_ACTION_TYPE_METER_COLOR,
Regards, Li Zhang > -----Original Message----- > From: Stephen Hemminger <[email protected]> > Sent: Wednesday, April 21, 2021 1:56 AM > To: Li Zhang <[email protected]> > Cc: [email protected]; Ori Kam <[email protected]>; Slava Ovsiienko > <[email protected]>; Matan Azrad <[email protected]>; Shahaf > Shuler <[email protected]>; [email protected]; > [email protected]; NBU-Contact-Thomas Monjalon > <[email protected]>; Ferruh Yigit <[email protected]>; Andrew > Rybchenko <[email protected]>; Cristian Dumitrescu > <[email protected]>; [email protected]; Raslan Darawsheh > <[email protected]>; Roni Bar Yanai <[email protected]> > Subject: Re: [dpdk-dev] [PATCH 1/2] [RFC]: ethdev: add pre-defined meter > policy API > > External email: Use caution opening links or attachments > > > On Thu, 18 Mar 2021 10:58:14 +0200 > Li Zhang <[email protected]> wrote: > > > + > > + /** > > + * Color the packet to reflect the meter color result. > > + * > > + * See struct rte_flow_action_color. > > + */ > > + RTE_FLOW_ACTION_TYPE_COlOR, > > Why the odd use of lower case here? Shouldn't it be > RTE_FLOW_ACTION_TYPE_COLOR
