From: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
To: Tom Lanyon <tom@oneshoeco.com>
Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>,
"Linux ACPI" <linux-acpi@vger.kernel.org>,
"Linux PM" <linux-pm@vger.kernel.org>,
"Andy Shevchenko" <andriy.shevchenko@linux.intel.com>,
"Darren Hart" <dvhart@infradead.org>,
LKML <linux-kernel@vger.kernel.org>,
"Mika Westerberg" <mika.westerberg@linux.intel.com>,
"Mario Limonciello" <mario.limonciello@dell.com>,
"Jérôme de Bretagne" <jerome.debretagne@gmail.com>,
"Zheng, Lv" <lv.zheng@intel.com>,
"Linus Torvalds" <torvalds@linux-foundation.org>
Subject: Re: [PATCH] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent systems
Date: Thu, 06 Jul 2017 17:38:53 -0700 [thread overview]
Message-ID: <1499387933.22531.15.camel@linux.intel.com> (raw)
In-Reply-To: <CA+b4Kqd5qO2Q6drnfvhCyj_q33Hjg1nT7vbgm-uoc+wjpX=e3Q@mail.gmail.com>
On Thu, 2017-07-06 at 21:41 +1000, Tom Lanyon wrote:
> Sorry for the slow response - I've just had a chance to run some more
> tests.
>
> I tried to disable the SD card reader in the BIOS as suggested
> earlier
> in the thread, but that didn't seem to make a significant change.
> More inline below.
>
> On Tue, 2017-06-27 at 17:03 +0200, Rafael J. Wysocki wrote:
> >
> > I would carry out s2idle under turbostat to see how much PC10
> > residency is there while suspended. That may be a significant
> > factor.
> >
> > Most likely there is a device preventing the SoC from reaching its
> > deepest low-power states under Linux on your system and it needs to
> > be
> > identified and dealt with.
> I'm not entirely sure how turbostat records metrics so wasn't sure
> how
> to measure correctly. I kept turbostat running while performing a
> s2idle and recorded the output:
>
> https://gist.githubusercontent.com/tomlanyon/3238e742a155e7fa27658aa0
> 960bdee4/raw/98b5f050e5eb2f88af47b2afd17080e7dd85d20f/turbostat
>
> I'm not familiar with the output format - I see some high percentages
> of C10, but nothing in Pkg%pc10. Which is of interest in this
> scenario?
I see that you do have Pkg%pc10 of 86.56 in one interval. The system
was in Pkg%pc8 for multiple intervals before Pkg%pc10. So can you try
this also before your test?
# cd /sys/kernel/debug/pmc_core
# for i in {0..32}; do echo $i > ltr_ignore; done
Thanks,
Srinivas
>
> On 28 June 2017 at 02:14, Srinivas Pandruvada
> <srinivas.pandruvada@linux.intel.com> wrote:
> >
> > Also make sure that you have no FW loading error for i915.
> > #dmesg | grep i915
> > It will display that Guc FW was loaded etc..
> > The latest FW can be downloaded from
> > https://01.org/linuxgraphics/downloads/firmware
> >
> > If you don't see PC10 residency, we can try something more.
> I've confirmed that there's no FW errors for the i915.
next prev parent reply other threads:[~2017-07-07 0:38 UTC|newest]
Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-26 21:21 [PATCH 0/5] PM / sleep: Support power button wakeup from S2I on recent Dell laptops Rafael J. Wysocki
2017-04-26 21:22 ` [PATCH 1/5] PM / wakeup: Integrate mechanism to abort transitions in progress Rafael J. Wysocki
2017-04-26 21:23 ` [PATCH 2/5] ACPI / sleep: Ignore spurious SCI wakeups from suspend-to-idle Rafael J. Wysocki
2017-04-26 21:24 ` [PATCH 3/5] ACPI / sleep: EC-based wakeup from suspend-to-idle on Dell systems Rafael J. Wysocki
2017-04-27 14:47 ` Mario.Limonciello
2017-04-27 14:47 ` Mario.Limonciello
2017-04-27 22:26 ` Rafael J. Wysocki
2017-05-04 7:58 ` Zheng, Lv
2017-05-04 14:23 ` Rafael J. Wysocki
2017-05-04 14:26 ` Rafael J. Wysocki
2017-05-05 0:36 ` Zheng, Lv
2017-04-26 21:24 ` [PATCH 4/5] platform: x86: intel-vbtn: Wake up the system from suspend-to-idle Rafael J. Wysocki
2017-04-26 21:27 ` [PATCH 5/5] platform: x86: intel-hid: " Rafael J. Wysocki
2017-05-04 14:33 ` [PATCH 0/5] PM / sleep: Support power button wakeup from S2I on recent Dell laptops Rafael J. Wysocki
2017-05-31 23:23 ` [PATCH 0/3] ACPI " Rafael J. Wysocki
2017-05-31 23:24 ` [PATCH 1/3] platform: x86: intel-vbtn: Wake up the system from suspend-to-idle Rafael J. Wysocki
2017-05-31 23:26 ` [PATCH 2/3] platform: x86: intel-hid: " Rafael J. Wysocki
2017-05-31 23:27 ` [PATCH 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent Dell systems Rafael J. Wysocki
2017-06-05 15:18 ` Mario.Limonciello
2017-06-05 15:18 ` Mario.Limonciello
2017-06-05 20:51 ` Rafael J. Wysocki
2017-06-01 10:43 ` [PATCH 0/3] ACPI / sleep: Support power button wakeup from S2I on recent Dell laptops Andy Shevchenko
2017-06-01 11:50 ` Tom Lanyon
2017-06-01 14:59 ` Rafael J. Wysocki
2017-06-02 1:06 ` Tom Lanyon
2017-06-02 23:16 ` Rafael J. Wysocki
2017-06-01 15:00 ` Rafael J. Wysocki
2017-06-19 21:48 ` [PATCH v2 " Rafael J. Wysocki
2017-06-19 21:49 ` [PATCH v2 1/3] platform: x86: intel-vbtn: Wake up the system from suspend-to-idle Rafael J. Wysocki
2017-06-19 21:51 ` [PATCH v2 2/3] platform: x86: intel-hid: " Rafael J. Wysocki
2017-06-19 21:53 ` [PATCH v2 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent Dell systems Rafael J. Wysocki
2017-06-19 23:37 ` Zheng, Lv
2017-06-19 23:46 ` Rafael J. Wysocki
2017-06-21 1:13 ` Zheng, Lv
2017-06-20 0:07 ` Linus Torvalds
2017-06-20 1:13 ` Rafael J. Wysocki
2017-06-20 2:00 ` Linus Torvalds
2017-06-20 21:16 ` Rafael J. Wysocki
2017-06-21 1:13 ` Zheng, Lv
2017-06-22 23:56 ` [PATCH] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent systems Rafael J. Wysocki
2017-06-23 2:40 ` Linus Torvalds
2017-06-27 5:50 ` Tom Lanyon
2017-06-27 6:47 ` Andy Shevchenko
2017-06-27 10:54 ` Tom Lanyon
2017-06-27 11:10 ` Mario.Limonciello
2017-06-27 11:10 ` Mario.Limonciello
2017-06-27 15:03 ` Rafael J. Wysocki
2017-06-27 16:14 ` Srinivas Pandruvada
2017-07-06 11:41 ` Tom Lanyon
2017-07-07 0:38 ` Srinivas Pandruvada [this message]
2017-06-23 6:30 ` Zheng, Lv
2017-06-23 12:13 ` Rafael J. Wysocki
2017-06-23 12:27 ` Rafael J. Wysocki
2017-06-23 13:15 ` [PATCH v2] " Rafael J. Wysocki
2017-06-23 15:37 ` Mario.Limonciello
2017-06-23 15:37 ` Mario.Limonciello
2017-06-23 16:06 ` Srinivas Pandruvada
2017-06-23 18:01 ` Mario.Limonciello
2017-06-23 18:01 ` Mario.Limonciello
2017-06-24 0:43 ` Rafael J. Wysocki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1499387933.22531.15.camel@linux.intel.com \
--to=srinivas.pandruvada@linux.intel.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=dvhart@infradead.org \
--cc=jerome.debretagne@gmail.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=lv.zheng@intel.com \
--cc=mario.limonciello@dell.com \
--cc=mika.westerberg@linux.intel.com \
--cc=rjw@rjwysocki.net \
--cc=tom@oneshoeco.com \
--cc=torvalds@linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.