Skip to contentSkip to conversation

README: add information about memory usage

PATCH In review Luiz Capitulino 2 participants 2 messages Feb 25, 2014 – Feb 26, 2014 KVM

lore
Share & tools
lore

Luiz Capitulino

author
#0 lore raw

PATCH 1/1 README: add information about memory usage

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 <[email protected]>
---
 README | 4 ++++
 1 file changed, 4 insertions(+)
README +4 -0
index db525e3..0f5d810 100644
@@ -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:
Signature · 1 line
1.8.1.4
maintainer
#1 lore raw

Il 25/02/2014 04:29, Luiz Capitulino ha scritto:

Quoted 18 lines

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 <[email protected]>
---
 README | 4 ++++
 1 file changed, 4 insertions(+)
README +0 -0
index db525e3..0f5d810 100644

README This invocation runs the msr test case. The test outputs to stdio. · line 27

Quoted 10 lines
README +4 -0
@@ -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