RE: [PATCH V2 2/2] Xen ACPI memory hotplug

2013-01-13 Thread Liu, Jinsong
Agree, w/ minor comments below. Konrad Rzeszutek Wilk wrote: >> + >> +/* >> + * Early boot code has recognized memory area by EFI/E820. >> + * If DSDT shows these memory devices on boot, hotplug is not >> necessary + * for them. So, it just returns until completion of >> this driver

Re: [PATCH V2 2/2] Xen ACPI memory hotplug

2013-01-11 Thread Konrad Rzeszutek Wilk
On Wed, Jan 09, 2013 at 07:42:52AM +, Liu, Jinsong wrote: > This patch implements real Xen acpi memory hotplug driver as module. > When loaded, it replaces Xen stub driver. > > When an acpi memory device hotadd event occurs, it notifies OS and > invokes notification callback, adding related me