Re: [PATCH net-next v3 1/2] net: sched: add empty status flag for NOLOCK qdisc

[Eric Dumazet]

On 03/22/2019 08:01 AM, Paolo Abeni wrote:
> The queue is marked not empty after acquiring the seqlock,
> and it's up to the NOLOCK qdisc clearing such flag on dequeue.
> Since the empty status lays on the same cache-line of the
> seqlock, it's always hot on cache during the updates.
> 
> This makes the empty flag update a little bit loosy. Given
> the lack of synchronization between enqueue and dequeue, this
> is unavoidable.
> 

Reviewed-by: Eric Dumazet <eduma...@google.com>