Re: [libvirt] [PATCH] docs: correct setmem text plus expand setmaxmem text

2011-01-26 Thread Matthias Bolte
2011/1/26 Justin Clift : > On 20/01/2011, at 11:30 PM, Matthias Bolte wrote: >> 2011/1/19 Justin Clift : >>> This completes the man page updates required for BZ # 622534: >>> >>>  https://bugzilla.redhat.com/show_bug.cgi?id=622534 >> >>> -Change the maximum memory allocation limit in the guest doma

Re: [libvirt] [PATCH] docs: correct setmem text plus expand setmaxmem text

2011-01-25 Thread Justin Clift
On 20/01/2011, at 11:30 PM, Matthias Bolte wrote: > 2011/1/19 Justin Clift : >> This completes the man page updates required for BZ # 622534: >> >> https://bugzilla.redhat.com/show_bug.cgi?id=622534 > >> -Change the maximum memory allocation limit in the guest domain. This should >> -not change

Re: [libvirt] [PATCH] docs: correct setmem text plus expand setmaxmem text

2011-01-25 Thread Justin Clift
On 20/01/2011, at 2:05 AM, Daniel P. Berrange wrote: > On Wed, Jan 19, 2011 at 07:54:52AM -0700, Eric Blake wrote: >> On 01/18/2011 05:28 PM, Justin Clift wrote: >>> This completes the man page updates required for BZ # 622534: >>> >>> https://bugzilla.redhat.com/show_bug.cgi?id=622534 >>> =item

Re: [libvirt] [PATCH] docs: correct setmem text plus expand setmaxmem text

2011-01-20 Thread Justin Clift
On 20/01/2011, at 11:30 PM, Matthias Bolte wrote: > 2011/1/19 Justin Clift : >> This completes the man page updates required for BZ # 622534: >> >> https://bugzilla.redhat.com/show_bug.cgi?id=622534 > >> -Change the maximum memory allocation limit in the guest domain. This should >> -not change

Re: [libvirt] [PATCH] docs: correct setmem text plus expand setmaxmem text

2011-01-20 Thread Matthias Bolte
2011/1/19 Justin Clift : > This completes the man page updates required for BZ # 622534: > >  https://bugzilla.redhat.com/show_bug.cgi?id=622534 > -Change the maximum memory allocation limit in the guest domain. This should > -not change the current memory use. The memory limit is specified in > -

Re: [libvirt] [PATCH] docs: correct setmem text plus expand setmaxmem text

2011-01-19 Thread Eric Blake
On 01/19/2011 12:40 PM, Matthias Bolte wrote: > I wonder in which direction to go in order to unify this: being more > strict and do "value % 1024 == 0" checks all over the place, or being > less strict and just divide by 1024. See also this bug report, where the request was made to round up to th

Re: [libvirt] [PATCH] docs: correct setmem text plus expand setmaxmem text

2011-01-19 Thread Daniel P. Berrange
On Wed, Jan 19, 2011 at 08:40:33PM +0100, Matthias Bolte wrote: > 2011/1/19 Eric Blake : > > On 01/18/2011 05:28 PM, Justin Clift wrote: > >> This completes the man page updates required for BZ # 622534: > > >> + > >> +Some hypervisors require a larger granularity than kilobytes, and requests > >>

Re: [libvirt] [PATCH] docs: correct setmem text plus expand setmaxmem text

2011-01-19 Thread Matthias Bolte
2011/1/19 Eric Blake : > On 01/18/2011 05:28 PM, Justin Clift wrote: >> This completes the man page updates required for BZ # 622534: >> + >> +Some hypervisors require a larger granularity than kilobytes, and requests >> +that are not an even multiple will either be rounded down or rejected.  For

Re: [libvirt] [PATCH] docs: correct setmem text plus expand setmaxmem text

2011-01-19 Thread Daniel P. Berrange
On Wed, Jan 19, 2011 at 07:54:52AM -0700, Eric Blake wrote: > On 01/18/2011 05:28 PM, Justin Clift wrote: > > This completes the man page updates required for BZ # 622534: > > > > https://bugzilla.redhat.com/show_bug.cgi?id=622534 > > =item B I B > > > > -Change the maximum memory allocation

Re: [libvirt] [PATCH] docs: correct setmem text plus expand setmaxmem text

2011-01-19 Thread Eric Blake
On 01/18/2011 05:28 PM, Justin Clift wrote: > This completes the man page updates required for BZ # 622534: > > https://bugzilla.redhat.com/show_bug.cgi?id=622534 > =item B I B > > -Change the maximum memory allocation limit in the guest domain. This should > -not change the current memory us

[libvirt] [PATCH] docs: correct setmem text plus expand setmaxmem text

2011-01-18 Thread Justin Clift
This completes the man page updates required for BZ # 622534: https://bugzilla.redhat.com/show_bug.cgi?id=622534 --- tools/virsh.pod | 15 +++ 1 files changed, 11 insertions(+), 4 deletions(-) diff --git a/tools/virsh.pod b/tools/virsh.pod index 1f15fef..c0cda79 100644 --- a/tool