Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <1569514137-2307-1-git-send-email-mario.limonciello@dell.com>
 <97e66464-8407-fd58-21a6-aeb7736dec7b@molgen.mpg.de> <541d9bfa3ccf45b5b07e35e3eebb72ed@AUSX13MPC105.AMER.DELL.COM>
In-Reply-To: <541d9bfa3ccf45b5b07e35e3eebb72ed@AUSX13MPC105.AMER.DELL.COM>
From:   "Rafael J. Wysocki" <rafael@kernel.org>
Date:   Thu, 10 Oct 2019 10:51:23 +0200
Message-ID: <CAJZ5v0i6tApHQ-QUS3PNo8BZ7CHaqpZea+z=Layyzt0yhGjVeQ@mail.gmail.com>
Subject: Re: [PATCH] ACPI: PM: Revert "ACPI / PM: Blacklist Low Power S0 Idle
 _DSM for Dell XPS13 9360"
To:     Mario Limonciello <Mario.Limonciello@dell.com>
Cc:     Paul Menzel <pmenzel@molgen.mpg.de>,
        "Rafael J. Wysocki" <rjw@rjwysocki.net>,
        ACPI Devel Maling List <linux-acpi@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

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!