Re: [PATCH v3 0/3] ACPI: container hot remove support.

2012-11-27 Thread Hanjun Guo
On 2012/11/27 10:38, Tang Chen wrote: > On 11/27/2012 09:08 AM, Hanjun Guo wrote: >> On 2012/11/26 14:06, Tang Chen wrote: >>> On 11/26/2012 01:42 PM, Hanjun Guo wrote: Hi all, I think Yasuaki mentioned the key point for the container device remove, that is dependency.

Re: [PATCH v3 0/3] ACPI: container hot remove support.

2012-11-26 Thread Tang Chen
On 11/27/2012 09:08 AM, Hanjun Guo wrote: On 2012/11/26 14:06, Tang Chen wrote: On 11/26/2012 01:42 PM, Hanjun Guo wrote: Hi all, I think Yasuaki mentioned the key point for the container device remove, that is dependency. Currently, container, processor, and memory hotpulg are managed by dif

Re: [PATCH v3 0/3] ACPI: container hot remove support.

2012-11-26 Thread Hanjun Guo
On 2012/11/26 14:06, Tang Chen wrote: > On 11/26/2012 01:42 PM, Hanjun Guo wrote: >> >> Hi all, >> I think Yasuaki mentioned the key point for the container device remove, >> that is dependency. >> >> Currently, container, processor, and memory hotpulg are managed by different >> ACPI >> hotplug d

Re: [PATCH v3 0/3] ACPI: container hot remove support.

2012-11-26 Thread Hanjun Guo
On 2012/11/26 14:06, Tang Chen wrote: > On 11/26/2012 01:42 PM, Hanjun Guo wrote: >> >> Hi all, >> I think Yasuaki mentioned the key point for the container device remove, >> that is dependency. >> >> Currently, container, processor, and memory hotpulg are managed by different >> ACPI >> hotplug d

Re: [PATCH v3 0/3] ACPI: container hot remove support.

2012-11-25 Thread Tang Chen
On 11/26/2012 01:42 PM, Hanjun Guo wrote: Hi all, I think Yasuaki mentioned the key point for the container device remove, that is dependency. Currently, container, processor, and memory hotpulg are managed by different ACPI hotplug drivers, the driver works when handle device hotplug individu

Re: [PATCH v3 0/3] ACPI: container hot remove support.

2012-11-25 Thread Hanjun Guo
On 2012/10/31 19:09, Yasuaki Ishimatsu wrote: > Hi Tang, > > 2012/10/31 16:27, Tang Chen wrote: >> Hi, >> >> The container hotplug handler container_notify_cb() didn't implement >> the hot-remove functionality. So, these 3 patches implement it like >> the following way: >> >> patch 1. Do not use k

Re: [PATCH v3 0/3] ACPI: container hot remove support.

2012-11-04 Thread Jiang Liu
On 11/01/2012 12:48 AM, Yinghai Lu wrote: > On Wed, Oct 31, 2012 at 4:09 AM, Yasuaki Ishimatsu > wrote: >>> patch 2. Introduce a new function container_device_remove() to handle >>> ACPI_NOTIFY_EJECT_REQUEST event for container. >> >> If container device contains memory device, the funct

Re: [PATCH v3 0/3] ACPI: container hot remove support.

2012-10-31 Thread Tang Chen
Hi Yinghai, How do you think the 1st patch ? Is the idea OK with you ? And about the memory hotplug thing, so far as I know, we are trying to limit kernel memory in some nodes, and only support to hot-remove the nodes with out kernel memory. This functionality is called online_movable. And some

Re: [PATCH v3 0/3] ACPI: container hot remove support.

2012-10-31 Thread Tang Chen
On 10/31/2012 07:09 PM, Yasuaki Ishimatsu wrote: > Hi Tang, > > If container device contains memory device, the function is > very danger. As you know, we are developing a memory hotplug. > If memory has kernel memory, memory hot remove operations fails. > But container_device_remove() cannot real

Re: [PATCH v3 0/3] ACPI: container hot remove support.

2012-10-31 Thread Yinghai Lu
On Wed, Oct 31, 2012 at 4:09 AM, Yasuaki Ishimatsu wrote: >> patch 2. Introduce a new function container_device_remove() to handle >> ACPI_NOTIFY_EJECT_REQUEST event for container. > > If container device contains memory device, the function is > very danger. As you know, we are developi

Re: [PATCH v3 0/3] ACPI: container hot remove support.

2012-10-31 Thread Yasuaki Ishimatsu
Hi Tang, 2012/10/31 16:27, Tang Chen wrote: > Hi, > > The container hotplug handler container_notify_cb() didn't implement > the hot-remove functionality. So, these 3 patches implement it like > the following way: > > patch 1. Do not use kacpid_wq/kacpid_notify_wq to handle container hotplug >

[PATCH v3 0/3] ACPI: container hot remove support.

2012-10-31 Thread Tang Chen
Hi, The container hotplug handler container_notify_cb() didn't implement the hot-remove functionality. So, these 3 patches implement it like the following way: patch 1. Do not use kacpid_wq/kacpid_notify_wq to handle container hotplug event, use kacpi_hotplug_wq instead to avoid deadloc