On Sun, May 17, 2020 at 5:47 AM Roman Mashak <m...@mojatatu.com> wrote: > > When a new action is installed, firstuse field of 'tcf_t' is explicitly set > to 0. Value of zero means "new action, not yet used"; as a packet hits the > action, 'firstuse' is stamped with the current jiffies value. > > tcf_tm_dump() should return 0 for firstuse if action has not yet been hit.
Your patch makes sense to me. Just one more thing, how about 'lastuse'? It is initialized with jiffies, not 0, it seems we should initialize it to 0 too, as it is not yet used? Thanks.
