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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <3c730d38c3b345289ec27434d79349cb@ausx13mpc120.AMER.DELL.COM>
Date:   Thu, 19 Apr 2018 19:53:18 +0000
From:   <Mario.Limonciello@...l.com>
To:     <srinivas.pandruvada@...el.com>, <linux-acpi@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, <dgilmore@...hat.com>
Subject: RE: issues with suspend on Dell XPS 13 2-in-1

> -----Original Message-----
> From: Pandruvada, Srinivas [mailto:srinivas.pandruvada@...el.com]
> Sent: Thursday, April 19, 2018 2:46 PM
> To: linux-acpi@...r.kernel.org; linux-kernel@...r.kernel.org;
> dgilmore@...hat.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@...l.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