netdev - Re: [iwl-net][PATCH] Revert "e1000e: move force SMBUS near the end of enable

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]

Date: Tue, 11 Jun 2024 13:45:28 +0800
From: Hui Wang <hui.wang@...onical.com>
To: "Zhang, Rui" <rui.zhang@...el.com>,
 "pmenzel@...gen.mpg.de" <pmenzel@...gen.mpg.de>,
 "davem@...emloft.net" <davem@...emloft.net>,
 "naamax.meir@...ux.intel.com" <naamax.meir@...ux.intel.com>,
 "Ruinskiy, Dima" <dima.ruinskiy@...el.com>,
 "Lifshits, Vitaly" <vitaly.lifshits@...el.com>,
 "kuba@...nel.org" <kuba@...nel.org>,
 "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
 "intel-wired-lan@...ts.osuosl.org" <intel-wired-lan@...ts.osuosl.org>,
 "Nguyen, Anthony L" <anthony.l.nguyen@...el.com>,
 "Neftin, Sasha" <sasha.neftin@...el.com>,
 "edumazet@...gle.com" <edumazet@...gle.com>,
 "pabeni@...hat.com" <pabeni@...hat.com>
Cc: "Keller, Jacob E" <jacob.e.keller@...el.com>,
 "horms@...nel.org" <horms@...nel.org>,
 "Brandt, Todd E" <todd.e.brandt@...el.com>,
 "dmummenschanz@....de" <dmummenschanz@....de>,
 "regressions@...ts.linux.dev" <regressions@...ts.linux.dev>
Subject: Re: [iwl-net][PATCH] Revert "e1000e: move force SMBUS near the end of
 enable_ulp function"


On 6/10/24 22:14, Zhang, Rui wrote:
> On Mon, 2024-06-10 at 08:36 +0200, Paul Menzel wrote:
>> Dear Hui,
>>
>>
>>
>> Naama also added Tested-by lines two both commits in question. Could
>> Intel’s test coverage please extended to the problem at hand?
>>
>> Acked-by: Paul Menzel <pmenzel@...gen.mpg.de>
> Plus that,
> 1. Todd and I can test with upstream + this patch to confirm that
>     a. the regression for Todd is gone.
>     b. the s2idle failure for me is back
> 2. I can test with upstream + this patch + revert of commit
> 861e8086029e (e1000e: move force SMBUS from enable ulp function to
> avoid PHY loss issue) to confirm s2idle is working again.
>
> thanks,
> rui
Thanks.
>
>>
>> Kind regards,
>>
>> Paul