Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-26 Thread Toshi Kani
On Tue, 2013-03-26 at 13:22 +0100, Rafael J. Wysocki wrote: > On Monday, March 25, 2013 04:57:11 PM Toshi Kani wrote: > > On Mon, 2013-03-25 at 23:29 +0100, Rafael J. Wysocki wrote: > > > On Monday, March 25, 2013 02:45:36 PM Toshi Kani wrote: > > > > On Fri, 2013-03-15 at 11:47 +0100, Vasilis

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-26 Thread Rafael J. Wysocki
On Monday, March 25, 2013 04:57:11 PM Toshi Kani wrote: > On Mon, 2013-03-25 at 23:29 +0100, Rafael J. Wysocki wrote: > > On Monday, March 25, 2013 02:45:36 PM Toshi Kani wrote: > > > On Fri, 2013-03-15 at 11:47 +0100, Vasilis Liaskovitis wrote: > > > > Hi, > > > > > > > > On Thu, Mar 14, 2013 at

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-26 Thread Rafael J. Wysocki
On Monday, March 25, 2013 04:57:11 PM Toshi Kani wrote: On Mon, 2013-03-25 at 23:29 +0100, Rafael J. Wysocki wrote: On Monday, March 25, 2013 02:45:36 PM Toshi Kani wrote: On Fri, 2013-03-15 at 11:47 +0100, Vasilis Liaskovitis wrote: Hi, On Thu, Mar 14, 2013 at 06:16:30PM +0100,

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-26 Thread Toshi Kani
On Tue, 2013-03-26 at 13:22 +0100, Rafael J. Wysocki wrote: On Monday, March 25, 2013 04:57:11 PM Toshi Kani wrote: On Mon, 2013-03-25 at 23:29 +0100, Rafael J. Wysocki wrote: On Monday, March 25, 2013 02:45:36 PM Toshi Kani wrote: On Fri, 2013-03-15 at 11:47 +0100, Vasilis Liaskovitis

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-25 Thread Toshi Kani
On Mon, 2013-03-25 at 23:29 +0100, Rafael J. Wysocki wrote: > On Monday, March 25, 2013 02:45:36 PM Toshi Kani wrote: > > On Fri, 2013-03-15 at 11:47 +0100, Vasilis Liaskovitis wrote: > > > Hi, > > > > > > On Thu, Mar 14, 2013 at 06:16:30PM +0100, Rafael J. Wysocki wrote: > > > > Sorry for the

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-25 Thread Rafael J. Wysocki
On Monday, March 25, 2013 02:45:36 PM Toshi Kani wrote: > On Fri, 2013-03-15 at 11:47 +0100, Vasilis Liaskovitis wrote: > > Hi, > > > > On Thu, Mar 14, 2013 at 06:16:30PM +0100, Rafael J. Wysocki wrote: > > > Sorry for the sluggish response, I've been travelling recently. -> > > [...] > > > > > >

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-25 Thread Toshi Kani
On Fri, 2013-03-15 at 11:47 +0100, Vasilis Liaskovitis wrote: > Hi, > > On Thu, Mar 14, 2013 at 06:16:30PM +0100, Rafael J. Wysocki wrote: > > Sorry for the sluggish response, I've been travelling recently. -> > [...] > > > > > So, I'd suggest the following changes. > > > > > - Remove the

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-25 Thread Toshi Kani
On Fri, 2013-03-15 at 11:47 +0100, Vasilis Liaskovitis wrote: Hi, On Thu, Mar 14, 2013 at 06:16:30PM +0100, Rafael J. Wysocki wrote: Sorry for the sluggish response, I've been travelling recently. - [...] So, I'd suggest the following changes. - Remove the uevents attribute.

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-25 Thread Rafael J. Wysocki
On Monday, March 25, 2013 02:45:36 PM Toshi Kani wrote: On Fri, 2013-03-15 at 11:47 +0100, Vasilis Liaskovitis wrote: Hi, On Thu, Mar 14, 2013 at 06:16:30PM +0100, Rafael J. Wysocki wrote: Sorry for the sluggish response, I've been travelling recently. - [...] So, I'd suggest

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-25 Thread Toshi Kani
On Mon, 2013-03-25 at 23:29 +0100, Rafael J. Wysocki wrote: On Monday, March 25, 2013 02:45:36 PM Toshi Kani wrote: On Fri, 2013-03-15 at 11:47 +0100, Vasilis Liaskovitis wrote: Hi, On Thu, Mar 14, 2013 at 06:16:30PM +0100, Rafael J. Wysocki wrote: Sorry for the sluggish response,

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-15 Thread Vasilis Liaskovitis
Hi, On Thu, Mar 14, 2013 at 06:16:30PM +0100, Rafael J. Wysocki wrote: > Sorry for the sluggish response, I've been travelling recently. -> [...] > > > > So, I'd suggest the following changes. > > > > - Remove the "uevents" attribute. KOBJ_ONLINE/OFFLINE are not used for > > > > ACPI device

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-15 Thread Vasilis Liaskovitis
Hi, On Thu, Mar 14, 2013 at 06:16:30PM +0100, Rafael J. Wysocki wrote: Sorry for the sluggish response, I've been travelling recently. - [...] So, I'd suggest the following changes. - Remove the uevents attribute. KOBJ_ONLINE/OFFLINE are not used for ACPI device objects. -

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-14 Thread Rafael J. Wysocki
Sorry for the sluggish response, I've been travelling recently. -> On Monday, March 04, 2013 02:10:23 PM Vasilis Liaskovitis wrote: > Hi, > > On Tue, Feb 26, 2013 at 12:39:50AM +0100, Rafael J. Wysocki wrote: > > On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: > > > On Sat, 2013-02-23

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-14 Thread Rafael J. Wysocki
Sorry for the sluggish response, I've been travelling recently. - On Monday, March 04, 2013 02:10:23 PM Vasilis Liaskovitis wrote: Hi, On Tue, Feb 26, 2013 at 12:39:50AM +0100, Rafael J. Wysocki wrote: On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: On Sat, 2013-02-23 at 22:38

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-04 Thread Vasilis Liaskovitis
Hi, On Tue, Feb 26, 2013 at 12:39:50AM +0100, Rafael J. Wysocki wrote: > On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: > > On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: > > > From: Rafael J. Wysocki > > > > > > Multiple drivers handling hotplug-capable ACPI device

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-03-04 Thread Vasilis Liaskovitis
Hi, On Tue, Feb 26, 2013 at 12:39:50AM +0100, Rafael J. Wysocki wrote: On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: From: Rafael J. Wysocki rafael.j.wyso...@intel.com Multiple drivers handling hotplug-capable

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Rafael J. Wysocki
On Tuesday, February 26, 2013 12:40:22 PM Yasuaki Ishimatsu wrote: > 2013/02/26 12:11, Rafael J. Wysocki wrote: > > On Tuesday, February 26, 2013 11:02:56 AM Yasuaki Ishimatsu wrote: > >> 2013/02/26 10:09, Toshi Kani wrote: > >>> On Tue, 2013-02-26 at 09:40 +0900, Yasuaki Ishimatsu wrote: >

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Yasuaki Ishimatsu
2013/02/26 12:11, Rafael J. Wysocki wrote: On Tuesday, February 26, 2013 11:02:56 AM Yasuaki Ishimatsu wrote: 2013/02/26 10:09, Toshi Kani wrote: On Tue, 2013-02-26 at 09:40 +0900, Yasuaki Ishimatsu wrote: 2013/02/26 8:39, Rafael J. Wysocki wrote: On Monday, February 25, 2013 11:07:52 AM

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Rafael J. Wysocki
On Tuesday, February 26, 2013 11:02:56 AM Yasuaki Ishimatsu wrote: > 2013/02/26 10:09, Toshi Kani wrote: > > On Tue, 2013-02-26 at 09:40 +0900, Yasuaki Ishimatsu wrote: > >> 2013/02/26 8:39, Rafael J. Wysocki wrote: > >>> On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: > On Sat,

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Yasuaki Ishimatsu
2013/02/26 10:09, Toshi Kani wrote: On Tue, 2013-02-26 at 09:40 +0900, Yasuaki Ishimatsu wrote: 2013/02/26 8:39, Rafael J. Wysocki wrote: On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: From: Rafael J. Wysocki Multiple

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Toshi Kani
On Tue, 2013-02-26 at 09:40 +0900, Yasuaki Ishimatsu wrote: > 2013/02/26 8:39, Rafael J. Wysocki wrote: > > On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: > >> On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: > >>> From: Rafael J. Wysocki > >>> > >>> Multiple drivers

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Yasuaki Ishimatsu
2013/02/26 8:39, Rafael J. Wysocki wrote: On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: From: Rafael J. Wysocki Multiple drivers handling hotplug-capable ACPI device nodes install notify handlers covering the same types

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Toshi Kani
On Tue, 2013-02-26 at 00:39 +0100, Rafael J. Wysocki wrote: > On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: > > On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: > > > From: Rafael J. Wysocki > > > > > > Multiple drivers handling hotplug-capable ACPI device nodes install >

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Rafael J. Wysocki
On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: > On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: > > From: Rafael J. Wysocki > > > > Multiple drivers handling hotplug-capable ACPI device nodes install > > notify handlers covering the same types of events in a very similar

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Toshi Kani
On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > Multiple drivers handling hotplug-capable ACPI device nodes install > notify handlers covering the same types of events in a very similar > way. Moreover, those handlers are installed in separate namespace

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Yasuaki Ishimatsu
2013/02/26 12:11, Rafael J. Wysocki wrote: On Tuesday, February 26, 2013 11:02:56 AM Yasuaki Ishimatsu wrote: 2013/02/26 10:09, Toshi Kani wrote: On Tue, 2013-02-26 at 09:40 +0900, Yasuaki Ishimatsu wrote: 2013/02/26 8:39, Rafael J. Wysocki wrote: On Monday, February 25, 2013 11:07:52 AM

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Rafael J. Wysocki
On Tuesday, February 26, 2013 12:40:22 PM Yasuaki Ishimatsu wrote: 2013/02/26 12:11, Rafael J. Wysocki wrote: On Tuesday, February 26, 2013 11:02:56 AM Yasuaki Ishimatsu wrote: 2013/02/26 10:09, Toshi Kani wrote: On Tue, 2013-02-26 at 09:40 +0900, Yasuaki Ishimatsu wrote: 2013/02/26 8:39,

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Toshi Kani
On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: From: Rafael J. Wysocki rafael.j.wyso...@intel.com Multiple drivers handling hotplug-capable ACPI device nodes install notify handlers covering the same types of events in a very similar way. Moreover, those handlers are installed

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Rafael J. Wysocki
On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: From: Rafael J. Wysocki rafael.j.wyso...@intel.com Multiple drivers handling hotplug-capable ACPI device nodes install notify handlers covering the same types of events

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Toshi Kani
On Tue, 2013-02-26 at 00:39 +0100, Rafael J. Wysocki wrote: On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: From: Rafael J. Wysocki rafael.j.wyso...@intel.com Multiple drivers handling hotplug-capable ACPI device

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Yasuaki Ishimatsu
2013/02/26 8:39, Rafael J. Wysocki wrote: On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: From: Rafael J. Wysocki rafael.j.wyso...@intel.com Multiple drivers handling hotplug-capable ACPI device nodes install notify

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Toshi Kani
On Tue, 2013-02-26 at 09:40 +0900, Yasuaki Ishimatsu wrote: 2013/02/26 8:39, Rafael J. Wysocki wrote: On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: From: Rafael J. Wysocki rafael.j.wyso...@intel.com Multiple

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Yasuaki Ishimatsu
2013/02/26 10:09, Toshi Kani wrote: On Tue, 2013-02-26 at 09:40 +0900, Yasuaki Ishimatsu wrote: 2013/02/26 8:39, Rafael J. Wysocki wrote: On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: On Sat, 2013-02-23 at 22:38 +, Rafael J. Wysocki wrote: From: Rafael J. Wysocki

Re: [Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-25 Thread Rafael J. Wysocki
On Tuesday, February 26, 2013 11:02:56 AM Yasuaki Ishimatsu wrote: 2013/02/26 10:09, Toshi Kani wrote: On Tue, 2013-02-26 at 09:40 +0900, Yasuaki Ishimatsu wrote: 2013/02/26 8:39, Rafael J. Wysocki wrote: On Monday, February 25, 2013 11:07:52 AM Toshi Kani wrote: On Sat, 2013-02-23 at

[Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-23 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Multiple drivers handling hotplug-capable ACPI device nodes install notify handlers covering the same types of events in a very similar way. Moreover, those handlers are installed in separate namespace walks, although that really should be done during namespace scans

[Update 4][PATCH 2/7] ACPI / scan: Introduce common code for ACPI-based device hotplug

2013-02-23 Thread Rafael J. Wysocki
From: Rafael J. Wysocki rafael.j.wyso...@intel.com Multiple drivers handling hotplug-capable ACPI device nodes install notify handlers covering the same types of events in a very similar way. Moreover, those handlers are installed in separate namespace walks, although that really should be done