On 2/21/23 14:02, Viacheslav Galaktionov via dev wrote:
> Cookies are an important part of flow descriptions and must be available
> to the end user.
>
> Signed-off-by: Viacheslav Galaktionov <viacheslav.galaktio...@arknetworks.am>
> ---
> v2: print cookies before table ids
>
> ofproto/ofproto.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
> index e4a1bee76..863b34d25 100644
> --- a/ofproto/ofproto.c
> +++ b/ofproto/ofproto.c
> @@ -4824,6 +4824,10 @@ flow_stats_ds(struct ofproto *ofproto, struct rule
> *rule, struct ds *results,
> created = rule->created;
> ovs_mutex_unlock(&rule->mutex);
>
> + if (rule->flow_cookie != 0) {
> + ds_put_format(results, "cookie=0x%"PRIx64", ",
> + ntohll(rule->flow_cookie));
> + }
> if (rule->table_id != 0) {
> ds_put_format(results, "table_id=%"PRIu8", ", rule->table_id);
> }
Applied. Thanks!
nit: Don't send new versions in reply to old ones. Only dpdk-dev requires that.
In most other places it's frowned upon. :)
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
