Re: [PATCH net-next 3/5] net/sched: taprio: add netlink reporting for offload statistics counters

From: Vladimir Oltean
Date: Wed May 31 2023 - 09:34:04 EST

On Tue, May 30, 2023 at 03:52:17PM -0700, Vinicius Costa Gomes wrote:
> > + memset(stats, 0xff, sizeof(*stats));
> The only part that I didn't like, at first, was this, that the
> initialization of the offload struct is divided into two parts: one to
> set the command/tc, and one to set the "invalid/not set" value to all
> stats fields.
> I was thinking of adding a macro to do initialization of the stats
> fields, but it has a problem that it won't complain when a new field is
> added. Your solution should always work. I don't have better
> suggestions.

Right, it's no coincidence that it's where it is and the way it is.
Again, I drew inspiration from stats_prepare_data() in ethtool.

> Acked-by: Vinicius Costa Gomes <vinicius.gomes@xxxxxxxxx>

Thanks for the review.