[ovs-dev] [PATCH v17 7/8] netdev-offload-tc: Add offload support for sFlow
Chris Mi
cmi at nvidia.com
Mon Nov 15 02:39:04 UTC 2021
On 11/5/2021 8:26 PM, Eelco Chaudron wrote:
> Hi Chris,
>
> Only one thing you forgot to fix from the previous review.
>
> If you make no other changes than the below, you can add my ack.
Done, thanks.
>
> //Eelco
>
> On 21 Oct 2021, at 10:01, Chris Mi wrote:
>
<SNIP>
>>
>> +
>> +static int
>> +parse_userspace_action(struct tc_flower *flower, struct tc_action *tc_action,
>> + const struct nlattr *userspace_action,
>> + const struct flow_tnl *tnl, uint32_t *group_id,
>> + const ovs_u128 *ufid)
>> +{
>> + static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(5, 20);
>> + struct dpif_sflow_attr sflow_attr;
>> + int err;
>> +
>> + if (*group_id) {
>> + VLOG_ERR_RL(&error_rl, "Only a single TC_SAMPLE action per flow is"
>> + " supported");
> VLOG_ERR_RL(&error_rl,
> "Only a single TC_SAMPLE action per flow is supported");
>
>
Done.
More information about the dev
mailing list