netdev - Re: [PATCH net-next 2/3] iavf: fix err handling for MAC replace

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 for Android: free password hash cracker in your pocket

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

Message-ID: <d5b2b152-4325-a32d-3daf-e4629ad4818d@intel.com>
Date: Fri, 16 Jun 2023 09:09:36 +0200
From: Przemek Kitszel <przemyslaw.kitszel@...el.com>
To: "Fijalkowski, Maciej" <maciej.fijalkowski@...el.com>, "Nguyen, Anthony L"
	<anthony.l.nguyen@...el.com>
CC: Michal Swiatkowski <michal.swiatkowski@...ux.intel.com>, "Romanowski,
 Rafal" <rafal.romanowski@...el.com>, "netdev@...r.kernel.org"
	<netdev@...r.kernel.org>, "Gardocki, PiotrX" <piotrx.gardocki@...el.com>
Subject: Re: [PATCH net-next 2/3] iavf: fix err handling for MAC replace

On 6/7/23 21:08, Fijalkowski, Maciej wrote:
>>
>> On 6/6/23 12:14, Przemek Kitszel wrote:
>>> On 6/5/23 21:17, Maciej Fijalkowski wrote:
>>>> On Fri, Jun 02, 2023 at 10:13:01AM -0700, Tony Nguyen wrote:
>>>>> From: Przemek Kitszel <przemyslaw.kitszel@...el.com>
>>>>>
>>>>> Defer removal of current primary MAC until a replacement is
>>>>> successfully added.
>>>>> Previous implementation would left filter list with no primary MAC.
>>>>
>>
>> [...]
>>
>> Tony, without Piotr's patch for short-cutting new Mac == old Mac case,
>> supposedly my patch would not work (we have to either re-test via our
>> VAL or just wait for Piotr's next version).
> 
> Would be good to share a link to patch you refer to + short explanation
> why this would not work (I know which patch you had on mind but not
> every other reader would do so).

Right :)

> 
>>
>> Przemek


Final version of Piotr's patch [1] has been merged into net-next,
so @Tony, you could re-apply this patch (Subject line here) of mine to 
your queue.

[1] 
https://lore.kernel.org/netdev/20230614145302.902301-2-piotrx.gardocki@intel.com/