Linux-ACPI Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH] ACPI: PM: Revert "ACPI / PM: Blacklist Low Power S0 Idle _DSM for Dell XPS13 9360"
@ 2019-09-26 16:08 Mario Limonciello
  2019-10-07 22:44 ` Paul Menzel
  0 siblings, 1 reply; 4+ messages in thread
From: Mario Limonciello @ 2019-09-26 16:08 UTC (permalink / raw)
  To: Rafael J . Wysocki; +Cc: linux-acpi, LKML, Mario Limonciello, Paul Menzel

This reverts part of
commit 71630b7a832f ("ACPI / PM: Blacklist Low Power S0 Idle _DSM for Dell XPS13 9360")
to remove the S0ix blacklist for the XPS 9360.

The problems with this system occurred in one possible NVME SSD when
putting system into s0ix.  As the NVME sleep behavior has been adjusted
in d916b1be this is expected to be now resolved.

Cc: 'Paul Menzel <pmenzel@molgen.mpg.de>'
BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=196907
Signed-off-by: Mario Limonciello <mario.limonciello@dell.com>
---
The particular failing configuration was reported by only ever failed for Paul Menzel, so hopefully
he can test on his failing system.

 drivers/acpi/sleep.c | 13 -------------
 1 file changed, 13 deletions(-)

diff --git a/drivers/acpi/sleep.c b/drivers/acpi/sleep.c
index 9fa77d7..2af937a 100644
--- a/drivers/acpi/sleep.c
+++ b/drivers/acpi/sleep.c
@@ -362,19 +362,6 @@ static const struct dmi_system_id acpisleep_dmi_table[] __initconst = {
 		},
 	},
 	/*
-	 * https://bugzilla.kernel.org/show_bug.cgi?id=196907
-	 * Some Dell XPS13 9360 cannot do suspend-to-idle using the Low Power
-	 * S0 Idle firmware interface.
-	 */
-	{
-	.callback = init_default_s3,
-	.ident = "Dell XPS13 9360",
-	.matches = {
-		DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
-		DMI_MATCH(DMI_PRODUCT_NAME, "XPS 13 9360"),
-		},
-	},
-	/*
 	 * ThinkPad X1 Tablet(2016) cannot do suspend-to-idle using
 	 * the Low Power S0 Idle firmware interface (see
 	 * https://bugzilla.kernel.org/show_bug.cgi?id=199057).
-- 
2.7.4


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ACPI: PM: Revert "ACPI / PM: Blacklist Low Power S0 Idle _DSM for Dell XPS13 9360"
  2019-09-26 16:08 [PATCH] ACPI: PM: Revert "ACPI / PM: Blacklist Low Power S0 Idle _DSM for Dell XPS13 9360" Mario Limonciello
@ 2019-10-07 22:44 ` Paul Menzel
  2019-10-08  1:44   ` Mario.Limonciello
  0 siblings, 1 reply; 4+ messages in thread
From: Paul Menzel @ 2019-10-07 22:44 UTC (permalink / raw)
  To: Mario Limonciello, Rafael J. Wysocki; +Cc: linux-acpi, LKML

Dear Mario,


On 26.09.19 18:08, Mario Limonciello wrote:
> This reverts part of
> commit 71630b7a832f ("ACPI / PM: Blacklist Low Power S0 Idle _DSM for Dell XPS13 9360")
> to remove the S0ix blacklist for the XPS 9360.
> 
> The problems with this system occurred in one possible NVME SSD when
> putting system into s0ix.  As the NVME sleep behavior has been adjusted
> in d916b1be this is expected to be now resolved.

1.  Please add, that it was the Hynix(?) SSD.
2.  Please add the commit message summary of d916b1be.

     nvme-pci: use host managed power state for suspend

> Cc: 'Paul Menzel <pmenzel@molgen.mpg.de>'
> BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=196907
> Signed-off-by: Mario Limonciello <mario.limonciello@dell.com>

Tag it for the stable series? d916b1be (first tag v5.3-rc1) is not 
tagged for stable.

> ---
> The particular failing configuration was reported by only ever failed for Paul Menzel, so hopefully
> he can test on his failing system.

I successfully tested Linux 5.4-rc1+ with this commit last Friday on the 
Dell XPS 13 9360.

Tested-by: Paul Menzel <pmenzel@molgen.mpg.de>

[…]


Kind regards,

Paul

^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: [PATCH] ACPI: PM: Revert "ACPI / PM: Blacklist Low Power S0 Idle _DSM for Dell XPS13 9360"
  2019-10-07 22:44 ` Paul Menzel
@ 2019-10-08  1:44   ` Mario.Limonciello
  2019-10-10  8:51     ` Rafael J. Wysocki
  0 siblings, 1 reply; 4+ messages in thread
From: Mario.Limonciello @ 2019-10-08  1:44 UTC (permalink / raw)
  To: pmenzel, rjw; +Cc: linux-acpi, linux-kernel

> On 26.09.19 18:08, Mario Limonciello wrote:
> > This reverts part of
> > commit 71630b7a832f ("ACPI / PM: Blacklist Low Power S0 Idle _DSM for
> > Dell XPS13 9360") to remove the S0ix blacklist for the XPS 9360.
> >
> > The problems with this system occurred in one possible NVME SSD when
> > putting system into s0ix.  As the NVME sleep behavior has been
> > adjusted in d916b1be this is expected to be now resolved.
> 
> 1.  Please add, that it was the Hynix(?) SSD.
> 2.  Please add the commit message summary of d916b1be.
> 
>      nvme-pci: use host managed power state for suspend
> 

Rafael, let me know if you want me to adjust the commit message and resubmit
or if you would just handle this task.

> > Cc: 'Paul Menzel <pmenzel@molgen.mpg.de>'
> > BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=196907
> > Signed-off-by: Mario Limonciello <mario.limonciello@dell.com>
> 
> Tag it for the stable series? d916b1be (first tag v5.3-rc1) is not tagged for stable.
> 

Although Dell arranged a lot of testing with partners I don't feel d916b1be is a stable
candidate.  Rafael found a corner case with regards to ASPM configuration last minute
in 5.3rcX, I found a another corner case related to order of events and timing around
PC10 entry that's getting fixed in 5.4.

> > ---
> > The particular failing configuration was reported by only ever failed
> > for Paul Menzel, so hopefully he can test on his failing system.
> 
> I successfully tested Linux 5.4-rc1+ with this commit last Friday on the Dell XPS
> 13 9360.
> 
> Tested-by: Paul Menzel <pmenzel@molgen.mpg.de>
> 

Well that's great, appreciate your testing and confirmation.


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ACPI: PM: Revert "ACPI / PM: Blacklist Low Power S0 Idle _DSM for Dell XPS13 9360"
  2019-10-08  1:44   ` Mario.Limonciello
@ 2019-10-10  8:51     ` Rafael J. Wysocki
  0 siblings, 0 replies; 4+ messages in thread
From: Rafael J. Wysocki @ 2019-10-10  8:51 UTC (permalink / raw)
  To: Mario Limonciello
  Cc: Paul Menzel, Rafael J. Wysocki, ACPI Devel Maling List,
	Linux Kernel Mailing List

On Tue, Oct 8, 2019 at 3:44 AM <Mario.Limonciello@dell.com> wrote:
>
> > On 26.09.19 18:08, Mario Limonciello wrote:
> > > This reverts part of
> > > commit 71630b7a832f ("ACPI / PM: Blacklist Low Power S0 Idle _DSM for
> > > Dell XPS13 9360") to remove the S0ix blacklist for the XPS 9360.
> > >
> > > The problems with this system occurred in one possible NVME SSD when
> > > putting system into s0ix.  As the NVME sleep behavior has been
> > > adjusted in d916b1be this is expected to be now resolved.
> >
> > 1.  Please add, that it was the Hynix(?) SSD.
> > 2.  Please add the commit message summary of d916b1be.
> >
> >      nvme-pci: use host managed power state for suspend
> >
>
> Rafael, let me know if you want me to adjust the commit message and resubmit
> or if you would just handle this task.
>
> > > Cc: 'Paul Menzel <pmenzel@molgen.mpg.de>'
> > > BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=196907
> > > Signed-off-by: Mario Limonciello <mario.limonciello@dell.com>
> >
> > Tag it for the stable series? d916b1be (first tag v5.3-rc1) is not tagged for stable.
> >
>
> Although Dell arranged a lot of testing with partners I don't feel d916b1be is a stable
> candidate.  Rafael found a corner case with regards to ASPM configuration last minute
> in 5.3rcX, I found a another corner case related to order of events and timing around
> PC10 entry that's getting fixed in 5.4.
>
> > > ---
> > > The particular failing configuration was reported by only ever failed
> > > for Paul Menzel, so hopefully he can test on his failing system.
> >
> > I successfully tested Linux 5.4-rc1+ with this commit last Friday on the Dell XPS
> > 13 9360.
> >
> > Tested-by: Paul Menzel <pmenzel@molgen.mpg.de>
> >
>
> Well that's great, appreciate your testing and confirmation.

Applying, thanks!

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, back to index

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-26 16:08 [PATCH] ACPI: PM: Revert "ACPI / PM: Blacklist Low Power S0 Idle _DSM for Dell XPS13 9360" Mario Limonciello
2019-10-07 22:44 ` Paul Menzel
2019-10-08  1:44   ` Mario.Limonciello
2019-10-10  8:51     ` Rafael J. Wysocki

Linux-ACPI Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-acpi/0 linux-acpi/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-acpi linux-acpi/ https://lore.kernel.org/linux-acpi \
		linux-acpi@vger.kernel.org linux-acpi@archiver.kernel.org
	public-inbox-index linux-acpi

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-acpi


AGPL code for this site: git clone https://public-inbox.org/ public-inbox