| Message ID | 1357944049-29620-18-git-send-email-yinghai@kernel.org |
|---|---|
| State | Superseded |
| Headers | show |
On Friday, January 11, 2013 02:40:44 PM Yinghai Lu wrote: > Will use it with acpiphp and pci_root_hp events handling I think it would be better if you folded patches [17-20/22] together. Thanks, Rafael > Signed-off-by: Yinghai Lu <yinghai@kernel.org> > Cc: Len Brown <lenb@kernel.org> > Cc: linux-acpi@vger.kernel.org > --- > drivers/acpi/osl.c | 21 +++++++++++++++++++++ > include/acpi/acpiosxf.h | 9 +++++++++ > 2 files changed, 30 insertions(+) > > diff --git a/drivers/acpi/osl.c b/drivers/acpi/osl.c > index 3ff2678..e6539a3 100644 > --- a/drivers/acpi/osl.c > +++ b/drivers/acpi/osl.c > @@ -1778,3 +1778,24 @@ void acpi_os_set_prepare_sleep(int (*func)(u8 sleep_state, > { > __acpi_os_prepare_sleep = func; > } > + > +void alloc_acpi_hp_work(acpi_handle handle, u32 type, void *context, > + void (*func)(struct work_struct *work)) > +{ > + struct acpi_hp_work *hp_work; > + int ret; > + > + hp_work = kmalloc(sizeof(*hp_work), GFP_KERNEL); > + if (!hp_work) > + return; > + > + hp_work->handle = handle; > + hp_work->type = type; > + hp_work->context = context; > + > + INIT_WORK(&hp_work->work, func); > + ret = queue_work(kacpi_hotplug_wq, &hp_work->work); > + if (!ret) > + kfree(hp_work); > +} > +EXPORT_SYMBOL(alloc_acpi_hp_work); > diff --git a/include/acpi/acpiosxf.h b/include/acpi/acpiosxf.h > index 4315274..2a64da5 100644 > --- a/include/acpi/acpiosxf.h > +++ b/include/acpi/acpiosxf.h > @@ -195,6 +195,15 @@ void acpi_os_fixed_event_count(u32 fixed_event_number); > */ > extern struct workqueue_struct *kacpi_hotplug_wq; > > +struct acpi_hp_work { > + struct work_struct work; > + acpi_handle handle; > + u32 type; > + void *context; > +}; > +void alloc_acpi_hp_work(acpi_handle handle, u32 type, void *context, > + void (*func)(struct work_struct *work)); > + > acpi_thread_id acpi_os_get_thread_id(void); > > acpi_status >
On Sat, Jan 12, 2013 at 3:45 PM, Rafael J. Wysocki <rjw@sisk.pl> wrote: > On Friday, January 11, 2013 02:40:44 PM Yinghai Lu wrote: >> Will use it with acpiphp and pci_root_hp events handling > > I think it would be better if you folded patches [17-20/22] together. just want to make move/split more simple. so could separate out old code in acpiphp is moved at first, then apply change etc. -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Monday, January 14, 2013 10:59:28 PM Yinghai Lu wrote: > On Sat, Jan 12, 2013 at 3:45 PM, Rafael J. Wysocki <rjw@sisk.pl> wrote: > > On Friday, January 11, 2013 02:40:44 PM Yinghai Lu wrote: > >> Will use it with acpiphp and pci_root_hp events handling > > > > I think it would be better if you folded patches [17-20/22] together. > > just want to make move/split more simple. so could separate out old > code in acpiphp is > moved at first, then apply change etc. Well, OK. Thanks, Rafael
diff --git a/drivers/acpi/osl.c b/drivers/acpi/osl.c index 3ff2678..e6539a3 100644 --- a/drivers/acpi/osl.c +++ b/drivers/acpi/osl.c @@ -1778,3 +1778,24 @@ void acpi_os_set_prepare_sleep(int (*func)(u8 sleep_state, { __acpi_os_prepare_sleep = func; } + +void alloc_acpi_hp_work(acpi_handle handle, u32 type, void *context, + void (*func)(struct work_struct *work)) +{ + struct acpi_hp_work *hp_work; + int ret; + + hp_work = kmalloc(sizeof(*hp_work), GFP_KERNEL); + if (!hp_work) + return; + + hp_work->handle = handle; + hp_work->type = type; + hp_work->context = context; + + INIT_WORK(&hp_work->work, func); + ret = queue_work(kacpi_hotplug_wq, &hp_work->work); + if (!ret) + kfree(hp_work); +} +EXPORT_SYMBOL(alloc_acpi_hp_work); diff --git a/include/acpi/acpiosxf.h b/include/acpi/acpiosxf.h index 4315274..2a64da5 100644 --- a/include/acpi/acpiosxf.h +++ b/include/acpi/acpiosxf.h @@ -195,6 +195,15 @@ void acpi_os_fixed_event_count(u32 fixed_event_number); */ extern struct workqueue_struct *kacpi_hotplug_wq; +struct acpi_hp_work { + struct work_struct work; + acpi_handle handle; + u32 type; + void *context; +}; +void alloc_acpi_hp_work(acpi_handle handle, u32 type, void *context, + void (*func)(struct work_struct *work)); + acpi_thread_id acpi_os_get_thread_id(void); acpi_status
Will use it with acpiphp and pci_root_hp events handling Signed-off-by: Yinghai Lu <yinghai@kernel.org> Cc: Len Brown <lenb@kernel.org> Cc: linux-acpi@vger.kernel.org --- drivers/acpi/osl.c | 21 +++++++++++++++++++++ include/acpi/acpiosxf.h | 9 +++++++++ 2 files changed, 30 insertions(+)