On 2/21/23 12:22, 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>
> ---
> ofproto/ofproto.c | 4 ++++
> 1 file changed, 4 insertions(+)
Hi, Viacheslav. Thanks for the patch!
>
> diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
> index e4a1bee76..29020c42e 100644
> --- a/ofproto/ofproto.c
> +++ b/ofproto/ofproto.c
> @@ -4827,6 +4827,10 @@ flow_stats_ds(struct ofproto *ofproto, struct rule
> *rule, struct ds *results,
> if (rule->table_id != 0) {
> ds_put_format(results, "table_id=%"PRIu8", ", rule->table_id);
> }
> + if (rule->flow_cookie != 0) {
> + ds_put_format(results, "cookie=0x%"PRIx64", ",
> + ntohll(rule->flow_cookie));
> + }
Should we print this before the table id as it's done in ofctl dump-flows?
> ds_put_format(results, "duration=%llds, ", (time_msec() - created) /
> 1000);
> ds_put_format(results, "n_packets=%"PRIu64", ", stats.n_packets);
> ds_put_format(results, "n_bytes=%"PRIu64", ", stats.n_bytes);
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
