Re: [PATCH][RESEND] Revert "PM: ACPI: reboot: Use S5 for reboot"

2021-03-19 Thread Kai-Heng Feng
On Fri, Mar 19, 2021 at 3:02 AM Josef Bacik wrote: > > On 3/18/21 1:42 AM, Kai-Heng Feng wrote: > > On Thu, Mar 18, 2021 at 1:25 AM Josef Bacik wrote: > > [snipped] > >> "shutdown now" works fine with and without your patch. Thanks, > > > > Rafael, > > Please revert the patch while we are workin

Re: [PATCH][RESEND] Revert "PM: ACPI: reboot: Use S5 for reboot"

2021-03-18 Thread Josef Bacik
On 3/18/21 1:42 AM, Kai-Heng Feng wrote: On Thu, Mar 18, 2021 at 1:25 AM Josef Bacik wrote: [snipped] "shutdown now" works fine with and without your patch. Thanks, Rafael, Please revert the patch while we are working on it. Josef, Can you please test the following patch: That made it wo

Re: [PATCH][RESEND] Revert "PM: ACPI: reboot: Use S5 for reboot"

2021-03-18 Thread Rafael J. Wysocki
On 3/18/2021 6:42 AM, Kai-Heng Feng wrote: On Thu, Mar 18, 2021 at 1:25 AM Josef Bacik wrote: [snipped] "shutdown now" works fine with and without your patch. Thanks, Rafael, Please revert the patch while we are working on it. Done, thanks! Josef, Can you please test the following patch:

Re: [PATCH][RESEND] Revert "PM: ACPI: reboot: Use S5 for reboot"

2021-03-17 Thread Kai-Heng Feng
On Thu, Mar 18, 2021 at 1:25 AM Josef Bacik wrote: [snipped] > "shutdown now" works fine with and without your patch. Thanks, Rafael, Please revert the patch while we are working on it. Josef, Can you please test the following patch: diff --git a/kernel/reboot.c b/kernel/reboot.c index eb1b158

Re: [PATCH][RESEND] Revert "PM: ACPI: reboot: Use S5 for reboot"

2021-03-17 Thread Josef Bacik
On 3/17/21 12:14 PM, Kai-Heng Feng wrote: On Wed, Mar 17, 2021 at 11:19 PM Josef Bacik wrote: On 3/16/21 10:50 PM, Kai-Heng Feng wrote: Hi, On Wed, Mar 17, 2021 at 10:17 AM Josef Bacik wrote: This reverts commit d60cd06331a3566d3305b3c7b566e79edf4e2095. This patch causes a panic when reb

Re: [PATCH][RESEND] Revert "PM: ACPI: reboot: Use S5 for reboot"

2021-03-17 Thread Josef Bacik
On 3/16/21 10:50 PM, Kai-Heng Feng wrote: Hi, On Wed, Mar 17, 2021 at 10:17 AM Josef Bacik wrote: This reverts commit d60cd06331a3566d3305b3c7b566e79edf4e2095. This patch causes a panic when rebooting my Dell Poweredge r440. I do not have the full panic log as it's lost at that stage of the

Re: [PATCH][RESEND] Revert "PM: ACPI: reboot: Use S5 for reboot"

2021-03-17 Thread Kai-Heng Feng
On Wed, Mar 17, 2021 at 11:19 PM Josef Bacik wrote: > > On 3/16/21 10:50 PM, Kai-Heng Feng wrote: > > Hi, > > > > On Wed, Mar 17, 2021 at 10:17 AM Josef Bacik wrote: > >> > >> This reverts commit d60cd06331a3566d3305b3c7b566e79edf4e2095. > >> > >> This patch causes a panic when rebooting my Dell

Re: [PATCH][RESEND] Revert "PM: ACPI: reboot: Use S5 for reboot"

2021-03-16 Thread Kai-Heng Feng
Hi, On Wed, Mar 17, 2021 at 10:17 AM Josef Bacik wrote: > > This reverts commit d60cd06331a3566d3305b3c7b566e79edf4e2095. > > This patch causes a panic when rebooting my Dell Poweredge r440. I do > not have the full panic log as it's lost at that stage of the reboot and > I do not have a serial

[PATCH][RESEND] Revert "PM: ACPI: reboot: Use S5 for reboot"

2021-03-16 Thread Josef Bacik
This reverts commit d60cd06331a3566d3305b3c7b566e79edf4e2095. This patch causes a panic when rebooting my Dell Poweredge r440. I do not have the full panic log as it's lost at that stage of the reboot and I do not have a serial console. Reverting this patch makes my system able to reboot again.