On 01/07/2020 21:58, Cong Wang wrote:
On Wed, Jul 1, 2020 at 9:05 AM Cong Wang <xiyou.wangc...@gmail.com> wrote:
On Tue, Jun 30, 2020 at 2:08 PM Josh Hunt <joh...@akamai.com> wrote:
Do either of you know if there's been any development on a fix for this
issue? If not we can propose something.
If you have a reproducer, I can look into this.
Does the attached patch fix this bug completely?
It's easier to comment if you inline the patch, but after taking a quick
look it seems too simplistic.
i) Are you sure you haven't got the return values on qdisc_run reversed?
ii) There's a "bypass" path that skips the enqueue/dequeue operation if
the queue is empty; that needs a similar treatment: after releasing
seqlock it needs to ensure that another packet hasn't been enqueued
since it last checked.