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

[<Mario.Limonciello@dell.com>]

> -----Original Message-----
> From: Pandruvada, Srinivas [mailto:srinivas.pandruvada@intel.com]
> Sent: Thursday, April 19, 2018 2:46 PM
> To: linux-acpi@vger.kernel.org; linux-kernel@vger.kernel.org;
> dgilmore@redhat.com; Limonciello, Mario
> Subject: Re: issues with suspend on Dell XPS 13 2-in-1
> 
> 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.

It could be potentially needing to update to new DMC firmware than
Fedora installs by default too then.

Eg due to:
https://github.com/torvalds/linux/commit/39ccc9852e2b46964c9c44eba52db57413ba6d27
https://github.com/torvalds/linux/commit/4f0aa1fa3e3849caee450ee5d14fcc289cf16703

> 
> 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