Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-19 Thread Rafael J. Wysocki
On Wednesday, June 19, 2013 05:22:04 PM Toshi Kani wrote: > On Thu, 2013-06-20 at 00:24 +0200, Rafael J. Wysocki wrote: > > On Wednesday, June 19, 2013 10:37:27 AM Tony Luck wrote: > > > > Can you please apply the appended patch on top of it and see if the > > > > system > > > > still works then?

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-19 Thread Toshi Kani
On Thu, 2013-06-20 at 00:24 +0200, Rafael J. Wysocki wrote: > On Wednesday, June 19, 2013 10:37:27 AM Tony Luck wrote: > > > Can you please apply the appended patch on top of it and see if the system > > > still works then? > > > > Still works with this patch. > > Cool, thanks! :-) > > If you

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-19 Thread Rafael J. Wysocki
On Wednesday, June 19, 2013 03:40:21 PM Tony Luck wrote: > > If you don't mind, I'll queue up > > https://patchwork.kernel.org/patch/2712741/ and > > this for 3.11. > > Mark them > > Tested-by: Tony Luck > > if you like. I will, thank you! Rafael -- To unsubscribe from this list: send the

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-19 Thread Tony Luck
> If you don't mind, I'll queue up https://patchwork.kernel.org/patch/2712741/ > and > this for 3.11. Mark them Tested-by: Tony Luck if you like. -Tony -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-19 Thread Rafael J. Wysocki
On Wednesday, June 19, 2013 10:37:27 AM Tony Luck wrote: > > Can you please apply the appended patch on top of it and see if the system > > still works then? > > Still works with this patch. Cool, thanks! :-) If you don't mind, I'll queue up https://patchwork.kernel.org/patch/2712741/ and this

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-19 Thread Tony Luck
> Can you please apply the appended patch on top of it and see if the system > still works then? Still works with this patch. -Tony > --- > drivers/acpi/scan.c |3 +++ > drivers/acpi/video.c |3 --- > 2 files changed, 3 insertions(+), 3 deletions(-) > > Index:

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-19 Thread Tony Luck
Can you please apply the appended patch on top of it and see if the system still works then? Still works with this patch. -Tony --- drivers/acpi/scan.c |3 +++ drivers/acpi/video.c |3 --- 2 files changed, 3 insertions(+), 3 deletions(-) Index: linux-pm/drivers/acpi/scan.c

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-19 Thread Rafael J. Wysocki
On Wednesday, June 19, 2013 10:37:27 AM Tony Luck wrote: Can you please apply the appended patch on top of it and see if the system still works then? Still works with this patch. Cool, thanks! :-) If you don't mind, I'll queue up https://patchwork.kernel.org/patch/2712741/ and this for

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-19 Thread Tony Luck
If you don't mind, I'll queue up https://patchwork.kernel.org/patch/2712741/ and this for 3.11. Mark them Tested-by: Tony Luck tony.l...@intel.com if you like. -Tony -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-19 Thread Rafael J. Wysocki
On Wednesday, June 19, 2013 03:40:21 PM Tony Luck wrote: If you don't mind, I'll queue up https://patchwork.kernel.org/patch/2712741/ and this for 3.11. Mark them Tested-by: Tony Luck tony.l...@intel.com if you like. I will, thank you! Rafael -- To unsubscribe from this list:

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-19 Thread Toshi Kani
On Thu, 2013-06-20 at 00:24 +0200, Rafael J. Wysocki wrote: On Wednesday, June 19, 2013 10:37:27 AM Tony Luck wrote: Can you please apply the appended patch on top of it and see if the system still works then? Still works with this patch. Cool, thanks! :-) If you don't mind, I'll

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-19 Thread Rafael J. Wysocki
On Wednesday, June 19, 2013 05:22:04 PM Toshi Kani wrote: On Thu, 2013-06-20 at 00:24 +0200, Rafael J. Wysocki wrote: On Wednesday, June 19, 2013 10:37:27 AM Tony Luck wrote: Can you please apply the appended patch on top of it and see if the system still works then? Still

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-14 Thread Rafael J. Wysocki
On Friday, June 14, 2013 03:32:42 PM Tony Luck wrote: > On Fri, Jun 14, 2013 at 3:23 PM, Rafael J. Wysocki wrote: > > Can you please just test patch [5/5] alone without patches [1-4/5]? We > > believe > > that this should work too and if that's the case, we'll only need that patch > > and a

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-14 Thread Tony Luck
On Fri, Jun 14, 2013 at 3:23 PM, Rafael J. Wysocki wrote: > Can you please just test patch [5/5] alone without patches [1-4/5]? We > believe > that this should work too and if that's the case, we'll only need that patch > and a reworked [1/5]. Your belief is sound - I popped all five patches

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-14 Thread Rafael J. Wysocki
On Friday, June 14, 2013 06:01:41 PM Luck, Tony wrote: > >> Tony promised me to test those patches on his box, so we'll know for sure > >> in a while. > > Tested this series - and the box boots just fine with no unexpected messages. Thanks! > But I should note that this box doesn't have

RE: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-14 Thread Luck, Tony
>> Tony promised me to test those patches on his box, so we'll know for sure >> in a while. Tested this series - and the box boots just fine with no unexpected messages. But I should note that this box doesn't have anything that is hot pluggable, so I couldn't test hotplug (which seems to be

RE: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-14 Thread Luck, Tony
Tony promised me to test those patches on his box, so we'll know for sure in a while. Tested this series - and the box boots just fine with no unexpected messages. But I should note that this box doesn't have anything that is hot pluggable, so I couldn't test hotplug (which seems to be deeply

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-14 Thread Rafael J. Wysocki
On Friday, June 14, 2013 06:01:41 PM Luck, Tony wrote: Tony promised me to test those patches on his box, so we'll know for sure in a while. Tested this series - and the box boots just fine with no unexpected messages. Thanks! But I should note that this box doesn't have anything that is

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-14 Thread Tony Luck
On Fri, Jun 14, 2013 at 3:23 PM, Rafael J. Wysocki r...@sisk.pl wrote: Can you please just test patch [5/5] alone without patches [1-4/5]? We believe that this should work too and if that's the case, we'll only need that patch and a reworked [1/5]. Your belief is sound - I popped all five

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-14 Thread Rafael J. Wysocki
On Friday, June 14, 2013 03:32:42 PM Tony Luck wrote: On Fri, Jun 14, 2013 at 3:23 PM, Rafael J. Wysocki r...@sisk.pl wrote: Can you please just test patch [5/5] alone without patches [1-4/5]? We believe that this should work too and if that's the case, we'll only need that patch and a

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-13 Thread Toshi Kani
On Fri, 2013-06-14 at 00:13 +0200, Rafael J. Wysocki wrote: > On Thursday, June 13, 2013 03:28:59 PM Toshi Kani wrote: > > On Thu, 2013-06-13 at 01:23 +0200, Rafael J. Wysocki wrote: > > > Hi All, > > > > > > It turns out that some BIOSes add container device IDs as _CIDs to device > > > object

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-13 Thread Rafael J. Wysocki
On Thursday, June 13, 2013 03:28:59 PM Toshi Kani wrote: > On Thu, 2013-06-13 at 01:23 +0200, Rafael J. Wysocki wrote: > > Hi All, > > > > It turns out that some BIOSes add container device IDs as _CIDs to device > > object that in principle may be matched against the other scan handlers (or > >

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-13 Thread Toshi Kani
On Thu, 2013-06-13 at 01:23 +0200, Rafael J. Wysocki wrote: > Hi All, > > It turns out that some BIOSes add container device IDs as _CIDs to device > object that in principle may be matched against the other scan handlers (or > ACPI drivers, but that's not a problem, because the container scan

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-13 Thread Toshi Kani
On Thu, 2013-06-13 at 01:23 +0200, Rafael J. Wysocki wrote: Hi All, It turns out that some BIOSes add container device IDs as _CIDs to device object that in principle may be matched against the other scan handlers (or ACPI drivers, but that's not a problem, because the container scan handler

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-13 Thread Rafael J. Wysocki
On Thursday, June 13, 2013 03:28:59 PM Toshi Kani wrote: On Thu, 2013-06-13 at 01:23 +0200, Rafael J. Wysocki wrote: Hi All, It turns out that some BIOSes add container device IDs as _CIDs to device object that in principle may be matched against the other scan handlers (or ACPI

Re: [PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-13 Thread Toshi Kani
On Fri, 2013-06-14 at 00:13 +0200, Rafael J. Wysocki wrote: On Thursday, June 13, 2013 03:28:59 PM Toshi Kani wrote: On Thu, 2013-06-13 at 01:23 +0200, Rafael J. Wysocki wrote: Hi All, It turns out that some BIOSes add container device IDs as _CIDs to device object that in

[PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-12 Thread Rafael J. Wysocki
Hi All, It turns out that some BIOSes add container device IDs as _CIDs to device object that in principle may be matched against the other scan handlers (or ACPI drivers, but that's not a problem, because the container scan handler can co-exist with an ACPI driver). That's why our recent fix

[PATCH 0/5] ACPI / scan: Make it possible to use the container hotplug with other scan handlers

2013-06-12 Thread Rafael J. Wysocki
Hi All, It turns out that some BIOSes add container device IDs as _CIDs to device object that in principle may be matched against the other scan handlers (or ACPI drivers, but that's not a problem, because the container scan handler can co-exist with an ACPI driver). That's why our recent fix