Re: [PATCH 1/2][v2] e1000e: Assign DPM_FLAG_SMART_SUSPEND and DPM_FLAG_MAY_SKIP_RESUME to speed up s2ram

[Chen Yu <yu.c.chen@intel.com>]

On Thu, Nov 26, 2020 at 02:14:40PM +0800, Chen Yu wrote:
> The NIC is put in runtime suspend status when there is no cable connected.
> As a result, it is safe to keep non-wakeup NIC in runtime suspended during
> s2ram because the system does not rely on the NIC plug event nor WoL to wake
> up the system. Besides that, unlike the s2idle, s2ram does not need to
> manipulate S0ix settings during suspend.
> 
> This patch assigns DPM_FLAG_SMART_SUSPEND and DPM_FLAG_MAY_SKIP_RESUME
> to the e1000e driver so that the s2ram could skip the .suspend_late(),
> .suspend_noirq() and .resume_noirq() .resume_early() when possible.
> Also skip .suspend() and .resume() if dev_pm_skip_suspend() and
> dev_pm_skip_resume() return true, so as to speed up the suspend and
> resume during s2ram.
> 
> Signed-off-by: Chen Yu <yu.c.chen@intel.com>
> ---
> v2: Added test data and some commit log revise(Paul Menzel)
>     Only skip the suspend/resume if the NIC is not a wake up device specified
>     by the user(Kai-Heng Feng)
>
Deprecated and replaced by v3.

thanks,
Chenyu