Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1737941 > unrolled thread

[PATCH 0/3] fix reuseaddr regression

Started byJosef Bacik <josef@toxicpanda.com>
First post2017-09-23 02:30 +0200
Last post2017-09-23 05:50 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 0/3] fix reuseaddr regression  Josef Bacik <josef@toxicpanda.com> - 2017-09-23 02:30 +0200
    Re: [PATCH 0/3] fix reuseaddr regression  David Miller <davem@davemloft.net> - 2017-09-23 05:50 +0200

#1737941 — [PATCH 0/3] fix reuseaddr regression

FromJosef Bacik <josef@toxicpanda.com>
Date2017-09-23 02:30 +0200
Subject[PATCH 0/3] fix reuseaddr regression
Message-ID<usGut-4u3-9@gated-at.bofh.it>
I introduced a regression when reworking the fastreuse port stuff that allows
bind conflicts to occur once a reuseaddr successfully opens on an existing tb.
The root cause is I reversed an if statement which caused us to set the tb as if
there were no owners on the socket if there were, which obviously is not
correct.

Dave could you please queue these changes up for -stable, I've run them through
the net tests and added another test to check for this problem specifically.
Thanks,

Josef

[toc] | [next] | [standalone]


#1737959

FromDavid Miller <davem@davemloft.net>
Date2017-09-23 05:50 +0200
Message-ID<usJC1-6nT-5@gated-at.bofh.it>
In reply to#1737941
From: Josef Bacik <josef@toxicpanda.com>
Date: Fri, 22 Sep 2017 20:20:05 -0400

> I introduced a regression when reworking the fastreuse port stuff that allows
> bind conflicts to occur once a reuseaddr successfully opens on an existing tb.
> The root cause is I reversed an if statement which caused us to set the tb as if
> there were no owners on the socket if there were, which obviously is not
> correct.
> 
> Dave could you please queue these changes up for -stable, I've run them through
> the net tests and added another test to check for this problem specifically.

Series applied and queued up for -stable, thanks.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web