Re: [SeaBIOS] [RFC PATCH v2 20/21] Implement -dimms, -dimmspop command line options

[Avi Kivity]

On 07/11/2012 01:32 PM, Vasilis Liaskovitis wrote:
> Implement batch dimm creation command line options. These could be useful for
> not bloating the command line with a large number of dimms.

IMO this is unneeded.  With a management tool there is no problem
generating a long command line; from the command line -dimm will be a
rarely used option.
> syntax: -dimms pfx=poolid,size=sz,num=n
> Will create numdimms dimms with ids poolid0, ..., poolidn-1. Each dimm has a
> size of sz.
>
> Implement -dimmpop option to populate dimms at bootup
> syntax: -dimmpop pfx=poolid,num=n
> This will populate n dimms with ids poolid0, ..., poolidn-1.
>
> (live-migration could break here without patch 12/21: -dimmspop
> needs to be reworked to support populating of individual dimms with
> same prefix, and not only a range of dimms starting from 0)

[RFC PATCH v2 20/21] Implement -dimms, -dimmspop command line options

[Vasilis Liaskovitis]

Hi,

On Wed, Jul 11, 2012 at 05:55:25PM +0300, Avi Kivity wrote:
> On 07/11/2012 01:32 PM, Vasilis Liaskovitis wrote:
> > Implement batch dimm creation command line options. These could be useful for
> > not bloating the command line with a large number of dimms.
>
> IMO this is unneeded.  With a management tool there is no problem
> generating a long command line; from the command line -dimm will be a
> rarely used option.

ok, I thought so. I guess this patch and the next are unwanted, unless there is a
strong opinion for using them coming from others.

thanks,

- Vasilis