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