netdev - Re: RTL8169 question

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <20191009142041.1e8d9cb0d79ed91a38d03b0a@skyboo.net>
Date:   Wed, 9 Oct 2019 14:20:41 +0200
From:   Mariusz Bialonczyk <manio@...boo.net>
To:     Heiner Kallweit <hkallweit1@...il.com>
Cc:     "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        Francois Romieu <romieu@...zoreil.com>
Subject: Re: RTL8169 question

On Tue, 8 Oct 2019 22:04:45 +0200
Heiner Kallweit <hkallweit1@...il.com> wrote:

> Thanks for the comprehensive analysis! Comparing the chip registers before
> and after suspend your BIOS seems to change registers on resume from suspend,
> and the driver doesn't configure jumbo when resuming. This may explain the
> issue. The combination jumbo + suspend + BIOS bug seems to be quite rare,
> else I think we should have seen such a report years ago already.
I see.

> Could you please check whether the following patch fixes the issue for you?
I've tested your patch with minor change when applying on top of the current
master tree.

The result is: it is working great!
No more problems, that I was observing before.
Thank you very much, Heiner!

You can add:
Reported-by: Mariusz Bialonczyk <manio@...boo.net>
Tested-by: Mariusz Bialonczyk <manio@...boo.net>

regards,
-- 
Mariusz Białończyk | xmpp/e-mail: manio@...boo.net
http://manio.skyboo.net | https://github.com/manio