Re: issues with suspend on Dell XPS 13 2-in-1

["Pandruvada, Srinivas" <srinivas.pandruvada@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 3963 bytes --]

On Wed, 2018-04-18 at 21:01 +0000, Mario.Limonciello@dell.com wrote:

> > 
> > > [...]

> > > Srinivas,
> > > 
> > > Do you know why Runtime PM is defaulting to disabled for all of
> > > these
> > > devices?  Is that a default kernel policy problem or a distro
> > > policy
> > > problem?
> > 
> > This is default kernel policy.
> 
> Why is policy set this way by default in kernel?  Could we discuss to
> change
> Default kernel policy so that users can get better power consumption
> by default
Good question. This was discussed before, the problem is that it can
cause some issues with legacy devices. It should be better done in user
 space by some policy in gnome power manager, which user can
enable/disable.

> 
> I think it will be especially important as more machines continue to
> adopt
> suspend-to-idle.
Those should impact opportunistic idle since for S2I regular suspend
callbacks are called. In my 9365, I can get PC10 during S2I path
without adjusting them. Only problematic is systems with SATA, where we
need to enable devsleep.

> 
> > I suggest run
> > #turbostat
> > then suspend and wake
> > when wake up let the turbostat collect data for next sampling
> > interval.
> > We have to see what are PkgC% residencies are?
> 
> Assuming you mean for Dan to run this after he adjusts all the PM
> settings
> for those options power top called out, right?
It is better.
May be some device is not behaving correctly. So he can try with
powertop --auto-tune and check turbostat and see status of PkgC states.

Thanks,
Srinivas

> 
> > 
> > Thanks,
> > Srinivas
> > 
> > > 
> > > 
> > > >    Good          Bluetooth device interface status
> > > >    Good          Enable Audio codec power management
> > > >    Good          Runtime PM for I2C Adapter i2c-8 (Synopsys
> > > > DesignWare
> > > > I2C adapter)
> > > >    Good          Autosuspend for USB device
> > > > Integrated_Webcam_HD
> > > > [CNFGE16N092020028362]
> > > >    Good          Autosuspend for USB device xHCI Host
> > > > Controller
> > > > [usb1]
> > > >    Good          Autosuspend for USB device xHCI Host
> > > > Controller
> > > > [usb2]
> > > >    Good          Runtime PM for I2C Adapter i2c-7 (SMBus I801
> > > > adapter
> > > > at efa0)
> > > >    Good          Autosuspend for unknown USB device 1-2
> > > > (8087:0a2b)
> > > >    Good          Runtime PM for I2C Adapter i2c-6 (Synopsys
> > > > DesignWare
> > > > I2C adapter)
> > > >    Good          I2C Device i2c-DLL077A:01 has no runtime power
> > > > management
> > > >    Good          I2C Device i2c-WCOM482F:00 has no runtime
> > > > power
> > > > management
> > > >    Good          Runtime PM for PCI Device Intel Corporation
> > > > Sunrise
> > > > Point-LP PCI Express Root Port #10
> > > >    Good          Runtime PM for PCI Device Intel Corporation
> > > > Sunrise
> > > > Point-LP CSME HECI #1
> > > >    Good          Runtime PM for PCI Device Intel Corporation
> > > > Sunrise
> > > > Point-LP Serial IO I2C Controller #1
> > > >    Good          Runtime PM for PCI Device Intel Corporation
> > > > Sunrise
> > > > Point-LP SMBus
> > > >    Good          Runtime PM for PCI Device Intel Corporation
> > > > Sunrise
> > > > Point-LP PCI Express Root Port #5
> > > >    Good          Runtime PM for PCI Device Intel Corporation
> > > > Sunrise
> > > > Point-LP Serial IO I2C Controller #0
> > > >    Good          Wake-on-lan status for device virbr0-nic
> > > >    Good          Wake-on-lan status for device virbr0
> > > >    Good          Wake-on-lan status for device wlp60s0
> > > > 
> > > > Regards
> > > > 
> > > > Dennis

[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 3290 bytes --]