[PATCH] README: add information about memory usage

[PATCH] README: add information about memory usage

[Luiz Capitulino]

I got a report of someone trying to run tests with a large amount of
RAM (4GB), which broke the guest as free_memory() function (called
by setup_vm()) will override the PCI hole.

Let's document memory constraints so that people don't do that.

Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
---
 README | 4 ++++
 1 file changed, 4 insertions(+)


diff --git a/README b/README
index db525e3..0f5d810 100644
--- a/README
+++ b/README
@@ -18,6 +18,10 @@ This invocation runs the msr test case. The test outputs to stdio.
 Using qemu (supported since qemu 1.3):
 qemu-system-x86_64 -enable-kvm -device pc-testdev -serial stdio -device isa-debug-exit,iobase=0xf4,iosize=0x4 -kernel ./x86/msr.flat
 
+Note that it's not necessary to specify the "-m" option to qemu. The default
+memory size is enough. Actually, the tests infrastructure doesn't support too
+much RAM anyway, so specifying a large amount of RAM may break it.
+
 Or use a runner script to detect the correct invocation:
 ./x86-run ./x86/msr.flat
 To select a specific qemu binary, specify the QEMU=<path> environment:
-- 
1.8.1.4

Re: [PATCH] README: add information about memory usage

[Paolo Bonzini]

Il 25/02/2014 04:29, Luiz Capitulino ha scritto:
> I got a report of someone trying to run tests with a large amount of
> RAM (4GB), which broke the guest as free_memory() function (called
> by setup_vm()) will override the PCI hole.
>
> Let's document memory constraints so that people don't do that.
>
> Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
> ---
>  README | 4 ++++
>  1 file changed, 4 insertions(+)
>
>
> diff --git a/README b/README
> index db525e3..0f5d810 100644
> --- a/README
> +++ b/README
> @@ -18,6 +18,10 @@ This invocation runs the msr test case. The test outputs to stdio.
>  Using qemu (supported since qemu 1.3):
>  qemu-system-x86_64 -enable-kvm -device pc-testdev -serial stdio -device isa-debug-exit,iobase=0xf4,iosize=0x4 -kernel ./x86/msr.flat
>
> +Note that it's not necessary to specify the "-m" option to qemu. The default
> +memory size is enough. Actually, the tests infrastructure doesn't support too
> +much RAM anyway, so specifying a large amount of RAM may break it.
> +
>  Or use a runner script to detect the correct invocation:
>  ./x86-run ./x86/msr.flat
>  To select a specific qemu binary, specify the QEMU=<path> environment:
>

Applying to kvm-unit-tests.

Paolo