linux-kernel - Re: [PATCH] r8169: Disable multicast filter for RTL_GIGA_MAC_VER

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <5caf123b-f626-fb68-476a-5b5cf9a7f31d@gmail.com>
Date:   Fri, 1 Sep 2023 14:48:39 +0200
From:   Heiner Kallweit <hkallweit1@...il.com>
To:     Patrick Thompson <ptf@...gle.com>, Jakub Kicinski <kuba@...nel.org>
Cc:     LKML <linux-kernel@...r.kernel.org>,
        "David S. Miller" <davem@...emloft.net>,
        Eric Dumazet <edumazet@...gle.com>,
        Paolo Abeni <pabeni@...hat.com>, netdev@...r.kernel.org,
        nic_swsd@...ltek.com, Chun-Hao Lin <hau@...ltek.com>
Subject: Re: [PATCH] r8169: Disable multicast filter for RTL_GIGA_MAC_VER_46

On 01.09.2023 14:28, Patrick Thompson wrote:
> Hello,
> 
> I was wondering if this should be extended to all RTL_GIGA_MAC_VERs
> greater than 35 as well.
> 
I *think* the mc filtering issue with version 35 is different from the
one you're seeing. So not every chip version may be affected.
As there's no public errata information let's wait for a statement
from Realtek.

> Realtek responded to me but I was slow to get them packet captures
> that they needed. I am hoping to restart things and get back to this
> over the finish line if it's a valid patch.
> 
> I will add the appropriate tags and annotations once I hear back.
> 
> On Tue, Jun 6, 2023 at 5:29 PM Jakub Kicinski <kuba@...nel.org> wrote:
>>
>> On Tue, 6 Jun 2023 17:11:27 +0200 Heiner Kallweit wrote:
>>> Thanks for the report and the patch. I just asked a contact in Realtek
>>> whether more chip versions may be affected. Then the patch should be
>>> extended accordingly. Let's wait few days for a response.
>>>
>>> I think we should make this a fix. Add the following as Fixes tag
>>> and annotate the patch as "net" (see netdev FAQ).
>>>
>>> 6e1d0b898818 ("r8169:add support for RTL8168H and RTL8107E")
>>
>> Perhaps it's best if you repost with the Fixes tag included once
>> Realtek responded.