Hi Cong,

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.



Reply via email to