netdev - Re: [v2,2/5] net: phy: realtek: simplify C22 reg access via MDIO_MMD

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: <94604894-16b9-44dc-a2a5-bf2106b027ab@meta.com>
Date: Sat, 17 Jan 2026 19:35:06 -0500
From: Chris Mason <clm@...a.com>
To: Jakub Kicinski <kuba@...nel.org>
Cc: Daniel Golle <daniel@...rotopia.org>, hkallweit1@...il.com,
        linux-kernel@...r.kernel.org, michael@...sekall.de,
        linux@...linux.org.uk, edumazet@...gle.com, andrew@...n.ch,
        olek2@...pl, davem@...emloft.net, vladimir.oltean@....com,
        netdev@...r.kernel.org, pabeni@...hat.com
Subject: Re: [v2,2/5] net: phy: realtek: simplify C22 reg access via
 MDIO_MMD_VEND2

On 1/17/26 7:17 PM, Jakub Kicinski wrote:
> On Sat, 17 Jan 2026 19:10:15 -0500 Chris Mason wrote:
>>>> Yeah. Just that this is not part of the series submitted.
>>>> It's rather a (halucinated) partial revert of
>>>> [v2,4/5] net: phy: realtek: demystify PHYSR register location  
>>>
>>> Oh wow, that's a first. No idea how this happened. Is the chunk if
>>> hallucinated from another WIP patch set?
>>>
>>> Chris, FWIW this is before we added lore indexing so I don't think 
>>> it got it from the list. Is it possible that semcode index is polluted
>>> by previous submissions? Still, even if, it's weird that it'd
>>> hallucinate a chunk of a patch.  
>>
>> We've definitely had it mix up hunks from other commits, but not since 
>> I changed the prompts to make it re-read the files before writing
>> review-inline.txt.
> 
> To be clear as Daniel mentioned the chunk in patch 4 is the other way,
> so it "reverted" the direction too. At least we have a chance to use
> the "mark as false positive" in the system :)
> 
> Daniel, series applied, thanks! The pw-bot is down, I think K is
> repacking repos so expect a delay in the official "applied" msg.

Yeah, it looked forward in the series and noticed the later patch
reverting those hunks.  If you read the logs, it goes back and forth
trying to decide if an issue it found was worth reporting given that it
was changed in the later commit.

And then, I think it just forgot which hunks were which.  I'll try to
beef up the section double checking diff contents.

-chris