diff mbox

README: add information about memory usage

Message ID 20140224222937.46a1b511@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Luiz Capitulino Feb. 25, 2014, 3:29 a.m. UTC
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(+)

Comments

Paolo Bonzini Feb. 26, 2014, 11:05 a.m. UTC | #1
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
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

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: