[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1524112616.31134.43.camel@intel.com>
Date: Thu, 19 Apr 2018 04:36:59 +0000
From: "Pandruvada, Srinivas" <srinivas.pandruvada@...el.com>
To: "dgilmore@...hat.com" <dgilmore@...hat.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-acpi@...r.kernel.org" <linux-acpi@...r.kernel.org>,
"Mario.Limonciello@...l.com" <Mario.Limonciello@...l.com>
Subject: Re: issues with suspend on Dell XPS 13 2-in-1
On Wed, 2018-04-18 at 21:01 +0000, Mario.Limonciello@...l.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
Download attachment "smime.p7s" of type "application/x-pkcs7-signature" (3290 bytes)
Powered by blists - more mailing lists