Re: [SeaBIOS] [RFC PATCH v3 00/19] ACPI memory hotplug

[Avi Kivity]

On 10/31/2012 12:58 PM, Stefan Hajnoczi wrote:
> On Fri, Sep 21, 2012 at 1:17 PM, Vasilis Liaskovitis
>  wrote:
> > This is v3 of the ACPI memory hotplug functionality. Only x86_64 target is supported
> > for now.
>
> Hi Vasilis,
> Regarding the hot unplug issue we've been discussing, it's possible to
> progress this patch series without fully solving that problem upfront.
>
> Karen Noel suggested that the series could be rolled without the hot
> unplug command, so that it's not possible to hit the unsafe case.
> This would allow users to hot plug additional memory.  They would have
> to use virtio-balloon to reduce the memory footprint again.  Later,
> when the memory region referencing issue has been solved the hot
> unplug command can be added.
>
> Just wanted to mention Karen's idea in case you feel stuck right now.

We could introduce hotunplug as an experimental feature so people can
test and play with it, and later graduate it to a fully supported feature.

[RFC PATCH v3 00/19] ACPI memory hotplug

[Vasilis Liaskovitis]

On Wed, Oct 31, 2012 at 01:16:56PM +0200, Avi Kivity wrote:
> On 10/31/2012 12:58 PM, Stefan Hajnoczi wrote:
> > On Fri, Sep 21, 2012 at 1:17 PM, Vasilis Liaskovitis
> >  wrote:
> > > This is v3 of the ACPI memory hotplug functionality. Only x86_64 target is supported
> > > for now.
> >
> > Hi Vasilis,
> > Regarding the hot unplug issue we've been discussing, it's possible to
> > progress this patch series without fully solving that problem upfront.
> >
> > Karen Noel suggested that the series could be rolled without the hot
> > unplug command, so that it's not possible to hit the unsafe case.
> > This would allow users to hot plug additional memory.  They would have
> > to use virtio-balloon to reduce the memory footprint again.  Later,
> > when the memory region referencing issue has been solved the hot
> > unplug command can be added.
> >
> > Just wanted to mention Karen's idea in case you feel stuck right now.
>
> We could introduce hotunplug as an experimental feature so people can
> test and play with it, and later graduate it to a fully supported feature.

ok, I 'll separate hotplug and hotunplug patches for next version of the
patchseries (maybe even offer hotunplug in a separate series)

thanks,

- Vasilis