Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <a624ec85-ea21-c72e-f997-06273d9b9f9e@valvesoftware.com>
 <20190621214139.GA31034@kroah.com> <CAHk-=wgXoBMWdBahuQR9e75ri6oeVBBjoVEnk0rN1QXfSKK2Eg@mail.gmail.com>
 <CANn89iL5+x3n9H9v4O6y39W=jvQs=uuXbzOvN5mBbcj0t+wdeg@mail.gmail.com> <5a012e7c-cbed-c3a9-8d84-851de34630d8@valvesoftware.com>
In-Reply-To: <5a012e7c-cbed-c3a9-8d84-851de34630d8@valvesoftware.com>
From:   Josh Hunt <joshhunt00@gmail.com>
Date:   Fri, 21 Jun 2019 16:42:59 -0700
Message-ID: <CAKA=qzaJETw4nzcmM0QJipcCjNJ51aNFSJiqA8tc++T3tP2_-g@mail.gmail.com>
Subject: Re: Steam is broken on new kernels
To:     "Pierre-Loup A. Griffais" <pgriffais@valvesoftware.com>
Cc:     Eric Dumazet <edumazet@google.com>,
        Linus Torvalds <torvalds@linux-foundation.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        lkml <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Fri, Jun 21, 2019 at 4:07 PM Pierre-Loup A. Griffais
<pgriffais@valvesoftware.com> wrote:
>
>
>
> On 6/21/19 3:38 PM, Eric Dumazet wrote:
> > Please look at my recent patch.
> >   Sorry I am travelling....
> >
> > On Fri, Jun 21, 2019, 6:19 PM Linus Torvalds
> > <torvalds@linux-foundation.org <mailto:torvalds@linux-foundation.org>>
> > wrote:
> >
> >     On Fri, Jun 21, 2019 at 2:41 PM Greg Kroah-Hartman
> >     <gregkh@linuxfoundation.org <mailto:gregkh@linuxfoundation.org>> wrote:
> >      >
> >      > What specific commit caused the breakage?
> >
> >     Both on reddit and on github there seems to be confusion about whether
> >     it's a problem or not. Some people have it working with the exact same
> >     kernel that breaks for others.
> >
> >     And then some people seem to say it works intermittently for them,
> >     which seems to indicate a timing issue.
> >
> >     Looking at the SACK patches (assuming it's one of them), I'd suspect
> >     the "tcp: tcp_fragment() should apply sane memory limits".
> >
> >     Eric, that one does
> >
> >             if (unlikely((sk->sk_wmem_queued >> 1) > sk->sk_sndbuf)) {
> >                     NET_INC_STATS(sock_net(sk), LINUX_MIB_TCPWQUEUETOOBIG);
> >                     return -ENOMEM;
> >             }
> >
> >     but I think it's *normal* for "sk_wmem_queued >> 1" to be around the
> >     same size as sk_sndbuf. So if there is some fragmentation, and we add
> >     more skb's to it, that would seem to trigger fairly easily.
> >     Particularly since this is all in "truesize" units, which can be a lot
> >     bigger than the packets themselves.
> >
> >     I don't know the code, so I may be out to lunch and barking up
> >     completely the wrong tree, but that particular check does seem like it
> >     might trigger much more easily than I think the code _intended_ it to
> >     trigger?
> >
> >     Pierre-Loup - do you guys have a test-case inside of valve? Or is this
> >     purely "we see some people with problems"?
>
> Definitely the latter, although the volume of complaints clearly points
> to a real problem from our experience. Reproducing locally, bisecting
> and testing possible fixes is just now starting on our end.
>
> I agree not all users seem affected; most affected people report success
> by using -tcp to launch Steam, which makes it use direct TCP instead of
> WebSockets, our current default connection method for Linux.
>
> Thanks,
>   - Pierre-Loup
>
> >
> >                     Linus
> >
>
I asked on the github thread if users seeing the problem could check
the new wqueue too big counter:
https://github.com/ValveSoftware/steam-for-linux/issues/6326

So far one person is seeing the counter increase when they see the
problem, and another that doesn't see the problem has the counter at
0. Obviously not a great sample size, but hopefully more will report.
If nothing else, someone is seeing the counter increase while trying
to connect to steam.
-- 
Josh