Thanks Hans On 14/12/2020 13:31, Mark Pearson wrote: > > > ------------------------------------------------------------------------ > *From:* Hans de Goede <hdego...@redhat.com> > *Sent:* December 14, 2020 13:24 > *To:* Mario Limonciello <mario.limoncie...@dell.com>; Jeff Kirsher > <jeffrey.t.kirs...@intel.com>; Tony Nguyen <anthony.l.ngu...@intel.com>; > intel-wired-...@lists.osuosl.org <intel-wired-...@lists.osuosl.org>; > David Miller <da...@davemloft.net>; Aaron Ma <aaron...@canonical.com>; > Mark Pearson <mpear...@lenovo.com> > *Cc:* linux-ker...@vger.kernel.org <linux-ker...@vger.kernel.org>; > Netdev <netdev@vger.kernel.org>; Alexander Duyck > <alexander.du...@gmail.com>; Jakub Kicinski <k...@kernel.org>; Sasha > Netfin <sasha.nef...@intel.com>; Aaron Brown <aaron.f.br...@intel.com>; > Stefan Assmann <sassm...@redhat.com>; darc...@redhat.com > <darc...@redhat.com>; yijun.s...@dell.com <yijun.s...@dell.com>; > perry.y...@dell.com <perry.y...@dell.com>; anthony.w...@canonical.com > <anthony.w...@canonical.com> > *Subject:* [External] Re: [PATCH v4 0/4] Improve s0ix flows for systems > i219LM > > Hi All, > <snip> > > ### > > I've added Mark Pearson from Lenovo to the Cc so that Lenovo > can investigate this issue further. > > Mark, this thread is about an issue with enabling S0ix support for > e1000e (i219lm) controllers. This was enabled in the kernel a > while ago, but then got disabled again on vPro / AMT enabled > systems because on some systems (Lenovo X1C7 and now also X1C8) > this lead to suspend/resume issues. > > When AMT is active then there is a handover handshake for the > OS to get access to the ethernet controller from the ME. The > Intel folks have checked and the Windows driver is using a timeout > of 1 second for this handshake, yet on Lenovo systems this is > taking 2 seconds. This likely has something to do with the > ME firmware on these Lenovo models, can you get the firmware > team at Lenovo to investigate this further ? Absolutely - I'll ask them to look into this again.
We did try to make progress with this previously - but it got a bit stuck and hence the need for these patches....but I believe things may have changed a bit so it's worth trying again Mark
