Cong Wang <xiyou.wangc...@gmail.com> writes: > 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?
Yes, exactly. I was planning to send a separate patch for this. Thanks for review, Cong.
