[OSSTEST PATCH] ts-windows-install: Bump RAM from 512M to 1700M

[OSSTEST PATCH] ts-windows-install: Bump RAM from 512M to 1700M

[Ian Jackson]

Sadly we lack a proper infrastructure for handing guest memory size
and fitting it into the host.  We don't currently even have in our
database the amount of RAM the host has.

The recommended minimum for Windows is 2Gby.  But we have (in the
Cambridge instance) some hosts which only have 4Gby.  We allow 512 for
dom0.  So set this to 1700 which should fit.

Later we may want to do something more sophisticated, like adjusting
this to 2048 if the host has enough, or only running the test on
bigger hosts, or something.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
CC: Wei Liu <wei.liu2@citrix.com>
---
 ts-windows-install |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ts-windows-install b/ts-windows-install
index 7740ee6..bdedf2b 100755
--- a/ts-windows-install
+++ b/ts-windows-install
@@ -28,7 +28,7 @@ our $ho= selecthost($whhost);
 
 our $gho;
 
-our $ram_mb=    512;
+our $ram_mb=   1700;
 our $disk_mb= 10000;
 
 our $guesthost= 'win.guest.osstest';
-- 
1.7.10.4

Re: [OSSTEST PATCH] ts-windows-install: Bump RAM from 512M to 1700M

[Andrew Cooper]

On 16/07/15 11:31, Ian Jackson wrote:
> Sadly we lack a proper infrastructure for handing guest memory size
> and fitting it into the host.  We don't currently even have in our
> database the amount of RAM the host has.
>
> The recommended minimum for Windows is 2Gby.  But we have (in the
> Cambridge instance) some hosts which only have 4Gby.  We allow 512 for
> dom0.  So set this to 1700 which should fit.
>
> Later we may want to do something more sophisticated, like adjusting
> this to 2048 if the host has enough, or only running the test on
> bigger hosts, or something.
>
> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
> CC: Wei Liu <wei.liu2@citrix.com>

1.7 + 1.7 + 0.5 = 3.9

That is cutting it very narrow for Xen.  How about 1.5 for windows instead?

~Andrew

Re: [OSSTEST PATCH] ts-windows-install: Bump RAM from 512M to 1700M

[Ian Campbell]

On Thu, 2015-07-16 at 11:36 +0100, Andrew Cooper wrote:
> On 16/07/15 11:31, Ian Jackson wrote:
> > Sadly we lack a proper infrastructure for handing guest memory size
> > and fitting it into the host.  We don't currently even have in our
> > database the amount of RAM the host has.
> >
> > The recommended minimum for Windows is 2Gby.  But we have (in the
> > Cambridge instance) some hosts which only have 4Gby.  We allow 512 for
> > dom0.  So set this to 1700 which should fit.
> >
> > Later we may want to do something more sophisticated, like adjusting
> > this to 2048 if the host has enough, or only running the test on
> > bigger hosts, or something.
> >
> > Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
> > CC: Wei Liu <wei.liu2@citrix.com>
> 
> 1.7 + 1.7 + 0.5 = 3.9

Where 2x1.7 is for localhost migration. right?

> 
> That is cutting it very narrow for Xen.  How about 1.5 for windows instead?
> 
> ~Andrew

Re: [OSSTEST PATCH] ts-windows-install: Bump RAM from 512M to 1700M

[Andrew Cooper]

On 16/07/15 11:54, Ian Campbell wrote:
> On Thu, 2015-07-16 at 11:36 +0100, Andrew Cooper wrote:
>> On 16/07/15 11:31, Ian Jackson wrote:
>>> Sadly we lack a proper infrastructure for handing guest memory size
>>> and fitting it into the host.  We don't currently even have in our
>>> database the amount of RAM the host has.
>>>
>>> The recommended minimum for Windows is 2Gby.  But we have (in the
>>> Cambridge instance) some hosts which only have 4Gby.  We allow 512 for
>>> dom0.  So set this to 1700 which should fit.
>>>
>>> Later we may want to do something more sophisticated, like adjusting
>>> this to 2048 if the host has enough, or only running the test on
>>> bigger hosts, or something.
>>>
>>> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
>>> CC: Wei Liu <wei.liu2@citrix.com>
>> 1.7 + 1.7 + 0.5 = 3.9
> Where 2x1.7 is for localhost migration. right?

Yes.  Sorry - I should have stated that outright.

~Andrew

[OSSTEST PATCH v2] ts-windows-install: Bump RAM from 512M to 1536M

[Ian Jackson]

Sadly we lack a proper infrastructure for handing guest memory size
and fitting it into the host.  We don't currently even have in our
database the amount of RAM the host has.

The recommended minimum for Windows is 2Gby.  But we have (in the
Cambridge instance) some hosts which only have 4Gby.  We allow 512 for
dom0.  We need to leave some spare for Xen and other overheads.  So
set this to 1536 (1.5Gby) which should fit even during localhost
migration.

Later we may want to do something more sophisticated, like adjusting
this to 2048 if the host has enough, or only running the test on
bigger hosts, or something.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
CC: Wei Liu <wei.liu2@citrix.com>
CC: Andrew Cooper <andrew.cooper3@citrix.com>
---
v2: 1536 not 1700.
    Mention migration in the commit message.
---
 ts-windows-install |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ts-windows-install b/ts-windows-install
index 7740ee6..bdedf2b 100755
--- a/ts-windows-install
+++ b/ts-windows-install
@@ -28,7 +28,7 @@ our $ho= selecthost($whhost);
 
 our $gho;
 
-our $ram_mb=    512;
+our $ram_mb=   1700;
 our $disk_mb= 10000;
 
 our $guesthost= 'win.guest.osstest';
-- 
1.7.10.4

Re: [OSSTEST PATCH v2] ts-windows-install: Bump RAM from 512M to 1536M

[Wei Liu]

On Thu, Jul 16, 2015 at 12:21:12PM +0100, Ian Jackson wrote:
> Sadly we lack a proper infrastructure for handing guest memory size
> and fitting it into the host.  We don't currently even have in our
> database the amount of RAM the host has.
> 
> The recommended minimum for Windows is 2Gby.  But we have (in the
> Cambridge instance) some hosts which only have 4Gby.  We allow 512 for
> dom0.  We need to leave some spare for Xen and other overheads.  So
> set this to 1536 (1.5Gby) which should fit even during localhost
> migration.
> 
> Later we may want to do something more sophisticated, like adjusting
> this to 2048 if the host has enough, or only running the test on
> bigger hosts, or something.
> 
> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
> CC: Wei Liu <wei.liu2@citrix.com>
> CC: Andrew Cooper <andrew.cooper3@citrix.com>
> ---
> v2: 1536 not 1700.
>     Mention migration in the commit message.
> ---
>  ts-windows-install |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/ts-windows-install b/ts-windows-install
> index 7740ee6..bdedf2b 100755
> --- a/ts-windows-install
> +++ b/ts-windows-install
> @@ -28,7 +28,7 @@ our $ho= selecthost($whhost);
>  
>  our $gho;
>  
> -our $ram_mb=    512;
> +our $ram_mb=   1700;

This disagrees with your commit message.

>  our $disk_mb= 10000;
>  
>  our $guesthost= 'win.guest.osstest';
> -- 
> 1.7.10.4

Re: [OSSTEST PATCH v2] ts-windows-install: Bump RAM from 512M to 1536M

[Ian Campbell]

On Thu, 2015-07-16 at 12:21 +0100, Ian Jackson wrote:
> Sadly we lack a proper infrastructure for handing guest memory size
> and fitting it into the host.  We don't currently even have in our
> database the amount of RAM the host has.
> 
> The recommended minimum for Windows is 2Gby.  But we have (in the
> Cambridge instance) some hosts which only have 4Gby.  We allow 512 for
> dom0.  We need to leave some spare for Xen and other overheads.  So
> set this to 1536 (1.5Gby) which should fit even during localhost
> migration.
> 
> Later we may want to do something more sophisticated, like adjusting
> this to 2048 if the host has enough, or only running the test on
> bigger hosts, or something.
> 
> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
> CC: Wei Liu <wei.liu2@citrix.com>
> CC: Andrew Cooper <andrew.cooper3@citrix.com>
> ---
> v2: 1536 not 1700.

The patch still says 1700..

A patch which said 1536 would be:
Acked-by: Ian Campbell <ian.campbell@citrix.com>

>     Mention migration in the commit message.
> ---
>  ts-windows-install |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/ts-windows-install b/ts-windows-install
> index 7740ee6..bdedf2b 100755
> --- a/ts-windows-install
> +++ b/ts-windows-install
> @@ -28,7 +28,7 @@ our $ho= selecthost($whhost);
>  
>  our $gho;
>  
> -our $ram_mb=    512;
> +our $ram_mb=   1700;
>  our $disk_mb= 10000;
>  
>  our $guesthost= 'win.guest.osstest';

Re: [OSSTEST PATCH v2] ts-windows-install: Bump RAM from 512M to 1536M

[Ian Jackson]

Ian Campbell writes ("Re: [OSSTEST PATCH v2] ts-windows-install: Bump RAM from 512M to 1536M"):
> The patch still says 1700..
> 
> A patch which said 1536 would be:
> Acked-by: Ian Campbell <ian.campbell@citrix.com>

Earlier today I was very sleepy and slow-moving.  Now I have had
coffee.  I'm not sure this is an improvement.

Thanks,
Ian.