Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-11-18 Thread Ingo Molnar
* Adam Williamson wrote: > On Fri, 2013-11-01 at 15:02 +0100, Joerg Roedel wrote: > > On Sun, Oct 27, 2013 at 12:06:29AM -0700, Adam Williamson wrote: > > > On Sat, 2013-10-26 at 11:15 +0200, Ingo Molnar wrote: > > > > CONFIG_IRQ_REMAP=y > > > > > > Agh, sorry - I had this down in my mind as

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-11-18 Thread Adam Williamson
On Fri, 2013-11-01 at 15:02 +0100, Joerg Roedel wrote: > On Sun, Oct 27, 2013 at 12:06:29AM -0700, Adam Williamson wrote: > > On Sat, 2013-10-26 at 11:15 +0200, Ingo Molnar wrote: > > > CONFIG_IRQ_REMAP=y > > > > Agh, sorry - I had this down in my mind as a boot time parameter, not a > >

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-11-18 Thread Adam Williamson
On Fri, 2013-11-01 at 15:02 +0100, Joerg Roedel wrote: On Sun, Oct 27, 2013 at 12:06:29AM -0700, Adam Williamson wrote: On Sat, 2013-10-26 at 11:15 +0200, Ingo Molnar wrote: CONFIG_IRQ_REMAP=y Agh, sorry - I had this down in my mind as a boot time parameter, not a compile time

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-11-18 Thread Ingo Molnar
* Adam Williamson awill...@redhat.com wrote: On Fri, 2013-11-01 at 15:02 +0100, Joerg Roedel wrote: On Sun, Oct 27, 2013 at 12:06:29AM -0700, Adam Williamson wrote: On Sat, 2013-10-26 at 11:15 +0200, Ingo Molnar wrote: CONFIG_IRQ_REMAP=y Agh, sorry - I had this down in my mind

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-11-01 Thread Joerg Roedel
On Sun, Oct 27, 2013 at 12:06:29AM -0700, Adam Williamson wrote: > On Sat, 2013-10-26 at 11:15 +0200, Ingo Molnar wrote: > > CONFIG_IRQ_REMAP=y > > Agh, sorry - I had this down in my mind as a boot time parameter, not a > compile time option. I'm off on vacation for a week in the morning, and >

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-11-01 Thread Joerg Roedel
On Sun, Oct 27, 2013 at 12:06:29AM -0700, Adam Williamson wrote: On Sat, 2013-10-26 at 11:15 +0200, Ingo Molnar wrote: CONFIG_IRQ_REMAP=y Agh, sorry - I had this down in my mind as a boot time parameter, not a compile time option. I'm off on vacation for a week in the morning, and it's

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-27 Thread Adam Williamson
On Sat, 2013-10-26 at 11:15 +0200, Ingo Molnar wrote: > So if you are able to test current kernels, it might be an > additional data point to see whether the reboot delay (which appears > to be a reboot hang on other systems) is related to the following > kernel option: > >

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-27 Thread Adam Williamson
On Sat, 2013-10-26 at 11:15 +0200, Ingo Molnar wrote: So if you are able to test current kernels, it might be an additional data point to see whether the reboot delay (which appears to be a reboot hang on other systems) is related to the following kernel option: CONFIG_IRQ_REMAP=y

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-26 Thread Linus Torvalds
On Sat, Oct 26, 2013 at 10:08 AM, Adam Williamson wrote: > > Thanks, I'll give it a shot. Note - I suspect some of the other cases > may really be delays rather than hangs too. It's very easy to look at > the screen sitting there doing absolutely nothing at all for 30-60 > seconds, lose patience,

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-26 Thread Adam Williamson
On Sat, 2013-10-26 at 11:15 +0200, Ingo Molnar wrote: > So, the best range for the breakage we have is v3.3..v3.5. So far, yeah. I will try and narrow it down. > As a blind shot into the dark, that turns out to be a range when a > number of irq-remapping, vt-d changes went upstream: > >

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-26 Thread Lan Tianyu
On 10/25/2013 06:53 PM, Ingo Molnar wrote: * tianyu@intel.com wrote: From: Lan Tianyu Sony Vaio Z1 series require "reboot=pci" for reboot and power off. This patch is to add them machines to quirk table and set pci reboot default. Reference:

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-26 Thread Ingo Molnar
* Adam Williamson wrote: > On Fri, 2013-10-25 at 23:17 -0400, Dave Jones wrote: > > On Fri, Oct 25, 2013 at 07:20:15PM -0700, Adam Williamson wrote: > > > OK, so, findings: F16 and F17 live images reboot quickly. F18 is the > > > first that reboots slowly. > > > > wait, stop. what does

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-26 Thread Adam Williamson
On Fri, 2013-10-25 at 23:17 -0400, Dave Jones wrote: > On Fri, Oct 25, 2013 at 07:20:15PM -0700, Adam Williamson wrote: > > OK, so, findings: F16 and F17 live images reboot quickly. F18 is the > > first that reboots slowly. > > wait, stop. what does "reboots slowly" mean ? > In every one of

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-26 Thread Adam Williamson
On Fri, 2013-10-25 at 23:17 -0400, Dave Jones wrote: On Fri, Oct 25, 2013 at 07:20:15PM -0700, Adam Williamson wrote: OK, so, findings: F16 and F17 live images reboot quickly. F18 is the first that reboots slowly. wait, stop. what does reboots slowly mean ? In every one of the failure

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-26 Thread Ingo Molnar
* Adam Williamson awill...@redhat.com wrote: On Fri, 2013-10-25 at 23:17 -0400, Dave Jones wrote: On Fri, Oct 25, 2013 at 07:20:15PM -0700, Adam Williamson wrote: OK, so, findings: F16 and F17 live images reboot quickly. F18 is the first that reboots slowly. wait, stop. what does

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-26 Thread Lan Tianyu
On 10/25/2013 06:53 PM, Ingo Molnar wrote: * tianyu@intel.com tianyu@intel.com wrote: From: Lan Tianyu tianyu@intel.com Sony Vaio Z1 series require reboot=pci for reboot and power off. This patch is to add them machines to quirk table and set pci reboot default. Reference:

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-26 Thread Adam Williamson
On Sat, 2013-10-26 at 11:15 +0200, Ingo Molnar wrote: So, the best range for the breakage we have is v3.3..v3.5. So far, yeah. I will try and narrow it down. As a blind shot into the dark, that turns out to be a range when a number of irq-remapping, vt-d changes went upstream:

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-26 Thread Linus Torvalds
On Sat, Oct 26, 2013 at 10:08 AM, Adam Williamson awill...@redhat.com wrote: Thanks, I'll give it a shot. Note - I suspect some of the other cases may really be delays rather than hangs too. It's very easy to look at the screen sitting there doing absolutely nothing at all for 30-60 seconds,

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-25 Thread Dave Jones
On Fri, Oct 25, 2013 at 07:20:15PM -0700, Adam Williamson wrote: > OK, so, findings: F16 and F17 live images reboot quickly. F18 is the > first that reboots slowly. wait, stop. what does "reboots slowly" mean ? In every one of the failure I've seen, without a quirk it means reboot doesn't

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-25 Thread Adam Williamson
On Fri, 2013-10-25 at 18:27 -0700, Adam Williamson wrote: > On Fri, 2013-10-25 at 18:16 +0100, Linus Torvalds wrote: > > On Fri, Oct 25, 2013 at 5:47 PM, Adam Williamson > > wrote: > > > > > > I don't, but one comment I could add is that reboot _used_ to work okay > > > on the Z1. And I _think_

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-25 Thread Adam Williamson
On Fri, 2013-10-25 at 18:16 +0100, Linus Torvalds wrote: > On Fri, Oct 25, 2013 at 5:47 PM, Adam Williamson wrote: > > > > I don't, but one comment I could add is that reboot _used_ to work okay > > on the Z1. And I _think_ it's worked OK since the April 2011 commit you > > mention (so the

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-25 Thread Rafael J. Wysocki
On Friday, October 25, 2013 10:43:00 AM Adam Williamson wrote: > On Fri, 2013-10-25 at 18:16 +0100, Linus Torvalds wrote: > > On Fri, Oct 25, 2013 at 5:47 PM, Adam Williamson > > wrote: > > > > > > I don't, but one comment I could add is that reboot _used_ to work okay > > > on the Z1. And I

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-25 Thread Adam Williamson
On Fri, 2013-10-25 at 18:16 +0100, Linus Torvalds wrote: > On Fri, Oct 25, 2013 at 5:47 PM, Adam Williamson wrote: > > > > I don't, but one comment I could add is that reboot _used_ to work okay > > on the Z1. And I _think_ it's worked OK since the April 2011 commit you > > mention (so the

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-25 Thread Linus Torvalds
On Fri, Oct 25, 2013 at 5:47 PM, Adam Williamson wrote: > > I don't, but one comment I could add is that reboot _used_ to work okay > on the Z1. And I _think_ it's worked OK since the April 2011 commit you > mention (so the introduction of that didn't break it), but I can't > absolutely swear to

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-25 Thread Adam Williamson
On Fri, 2013-10-25 at 12:53 +0200, Ingo Molnar wrote: > * tianyu@intel.com wrote: > > > From: Lan Tianyu > > > > Sony Vaio Z1 series require "reboot=pci" for reboot and power off. > > This patch is to add them machines to quirk table and set pci reboot > > default. > > > > Reference:

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-25 Thread Linus Torvalds
On Fri, Oct 25, 2013 at 1:48 PM, H. Peter Anvin wrote: > > The Dell problem I believe is understood... they are invoking the KBC as > their ACPI reboot port, and it is believed that that triggers an SMI > which invokes the BIOS, and the BIOS is broken if you behave like a > non-Windows system.

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-25 Thread H. Peter Anvin
On 10/25/2013 01:44 PM, Dave Jones wrote: > > I have a few machines at home that have the same problem. > There does seem to be a number of vendor wide issues here. Pretty much > every vaio and every Dell seems to need working around. > The Dell problem I believe is understood... they are

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-25 Thread Dave Jones
On Fri, Oct 25, 2013 at 12:53:52PM +0200, Ingo Molnar wrote: > > Sony Vaio Z1 series require "reboot=pci" for reboot and power off. > > This patch is to add them machines to quirk table and set pci reboot > > default. > > > > Reference: https://bugzilla.kernel.org/show_bug.cgi?id=61721 > >

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-25 Thread Ingo Molnar
* tianyu@intel.com wrote: > From: Lan Tianyu > > Sony Vaio Z1 series require "reboot=pci" for reboot and power off. > This patch is to add them machines to quirk table and set pci reboot > default. > > Reference: https://bugzilla.kernel.org/show_bug.cgi?id=61721 > Reported-and-tested-by:

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-25 Thread Ingo Molnar
* tianyu@intel.com tianyu@intel.com wrote: From: Lan Tianyu tianyu@intel.com Sony Vaio Z1 series require reboot=pci for reboot and power off. This patch is to add them machines to quirk table and set pci reboot default. Reference:

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-25 Thread Dave Jones
On Fri, Oct 25, 2013 at 12:53:52PM +0200, Ingo Molnar wrote: Sony Vaio Z1 series require reboot=pci for reboot and power off. This patch is to add them machines to quirk table and set pci reboot default. Reference: https://bugzilla.kernel.org/show_bug.cgi?id=61721

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-25 Thread H. Peter Anvin
On 10/25/2013 01:44 PM, Dave Jones wrote: I have a few machines at home that have the same problem. There does seem to be a number of vendor wide issues here. Pretty much every vaio and every Dell seems to need working around. The Dell problem I believe is understood... they are invoking

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-25 Thread Linus Torvalds
On Fri, Oct 25, 2013 at 1:48 PM, H. Peter Anvin h...@zytor.com wrote: The Dell problem I believe is understood... they are invoking the KBC as their ACPI reboot port, and it is believed that that triggers an SMI which invokes the BIOS, and the BIOS is broken if you behave like a non-Windows

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-25 Thread Adam Williamson
On Fri, 2013-10-25 at 12:53 +0200, Ingo Molnar wrote: * tianyu@intel.com tianyu@intel.com wrote: From: Lan Tianyu tianyu@intel.com Sony Vaio Z1 series require reboot=pci for reboot and power off. This patch is to add them machines to quirk table and set pci reboot

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-25 Thread Linus Torvalds
On Fri, Oct 25, 2013 at 5:47 PM, Adam Williamson awill...@redhat.com wrote: I don't, but one comment I could add is that reboot _used_ to work okay on the Z1. And I _think_ it's worked OK since the April 2011 commit you mention (so the introduction of that didn't break it), but I can't

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-25 Thread Adam Williamson
On Fri, 2013-10-25 at 18:16 +0100, Linus Torvalds wrote: On Fri, Oct 25, 2013 at 5:47 PM, Adam Williamson awill...@redhat.com wrote: I don't, but one comment I could add is that reboot _used_ to work okay on the Z1. And I _think_ it's worked OK since the April 2011 commit you mention (so

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-25 Thread Rafael J. Wysocki
On Friday, October 25, 2013 10:43:00 AM Adam Williamson wrote: On Fri, 2013-10-25 at 18:16 +0100, Linus Torvalds wrote: On Fri, Oct 25, 2013 at 5:47 PM, Adam Williamson awill...@redhat.com wrote: I don't, but one comment I could add is that reboot _used_ to work okay on the Z1. And

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-25 Thread Adam Williamson
On Fri, 2013-10-25 at 18:16 +0100, Linus Torvalds wrote: On Fri, Oct 25, 2013 at 5:47 PM, Adam Williamson awill...@redhat.com wrote: I don't, but one comment I could add is that reboot _used_ to work okay on the Z1. And I _think_ it's worked OK since the April 2011 commit you mention (so

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-25 Thread Adam Williamson
On Fri, 2013-10-25 at 18:27 -0700, Adam Williamson wrote: On Fri, 2013-10-25 at 18:16 +0100, Linus Torvalds wrote: On Fri, Oct 25, 2013 at 5:47 PM, Adam Williamson awill...@redhat.com wrote: I don't, but one comment I could add is that reboot _used_ to work okay on the Z1. And I

Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-25 Thread Dave Jones
On Fri, Oct 25, 2013 at 07:20:15PM -0700, Adam Williamson wrote: OK, so, findings: F16 and F17 live images reboot quickly. F18 is the first that reboots slowly. wait, stop. what does reboots slowly mean ? In every one of the failure I've seen, without a quirk it means reboot doesn't happen

[PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default

2013-10-24 Thread tianyu . lan
From: Lan Tianyu Sony Vaio Z1 series require "reboot=pci" for reboot and power off. This patch is to add them machines to quirk table and set pci reboot default. Reference: https://bugzilla.kernel.org/show_bug.cgi?id=61721 Reported-and-tested-by: Adam Williamson Signed-off-by: Lan Tianyu ---

[PATCH] x86/ACPI: Make Sony Vaio Z1 series to use reboot=pci default

2013-10-24 Thread tianyu . lan
From: Lan Tianyu tianyu@intel.com Sony Vaio Z1 series require reboot=pci for reboot and power off. This patch is to add them machines to quirk table and set pci reboot default. Reference: https://bugzilla.kernel.org/show_bug.cgi?id=61721 Reported-and-tested-by: Adam Williamson