Re: [PATCH net 1/1] net sched: fix reporting the first-time use timestamp

2020-05-18 Thread David Miller
From: Roman Mashak Date: Sun, 17 May 2020 08:46:31 -0400 > 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() sh

Re: [PATCH net 1/1] net sched: fix reporting the first-time use timestamp

2020-05-18 Thread Cong Wang
On Mon, May 18, 2020 at 5:43 AM Jamal Hadi Salim wrote: > > On 2020-05-17 9:10 p.m., Roman Mashak wrote: > > Cong Wang writes: > > > >> On Sun, May 17, 2020 at 5:47 AM Roman Mashak wrote: > >>> > >>> When a new action is installed, firstuse field of 'tcf_t' is explicitly > >>> set > >>> to 0. V

Re: [PATCH net 1/1] net sched: fix reporting the first-time use timestamp

2020-05-18 Thread Jamal Hadi Salim
On 2020-05-17 9:10 p.m., Roman Mashak wrote: Cong Wang writes: On Sun, May 17, 2020 at 5:47 AM Roman Mashak 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 sta

Re: [PATCH net 1/1] net sched: fix reporting the first-time use timestamp

2020-05-17 Thread Roman Mashak
Cong Wang writes: > On Sun, May 17, 2020 at 5:47 AM Roman Mashak 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.

Re: [PATCH net 1/1] net sched: fix reporting the first-time use timestamp

2020-05-17 Thread Cong Wang
On Sun, May 17, 2020 at 5:47 AM Roman Mashak 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

[PATCH net 1/1] net sched: fix reporting the first-time use timestamp

2020-05-17 Thread Roman Mashak
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. Fixes: 48