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

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

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

On Thu, 2018-04-19 at 17:30 +0000, Mario.Limonciello@dell.com wrote:

[...]

> > 
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.
> 
> Hmm, Dennis did mention that he had an NVME disk, so it can't be a
> devsleep
> problem.
> 
> Perhaps can you also share your kernel config used with 9365 so
> Dennis
> can compare?
I am using ubuntu kernel config as is. Nothing special.
We need to look at the turbostat data and see what PkgC states the
system can reach. May be the issues is with loading DMC firmware or
some LTR issues.

Thanks,
Srinivas

> 
> I believe we'll need to see some more information from Dennis you
> asked
> in previous email before we can make more progress here though.
> 
> > > 
> > > > 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 --]