Re: [kvm-devel] [PATCH] kvm: testsuite: silence warnings on x86_64

2008-01-31 Thread Carlo Marcelo Arenas Belon
On Thu, Jan 24, 2008 at 06:21:56PM +0200, Avi Kivity wrote: Bernhard Kaindl wrote: I did not test this patch as I did not find documentation on how to run the test cases and I could not find a make target to run them from make. make -C user test_cases user/kvmctl

Re: [kvm-devel] [PATCH] kvm: testsuite: silence warnings on x86_64

2008-01-31 Thread Avi Kivity
Carlo Marcelo Arenas Belon wrote: [I forgot to reply to the this:] --- kvm-60/user/test/x86/access.c +++ kvm-60/user/test/x86/access.c 2008/01/24 15:14:16 @@ -1,6 +1,7 @@ #include smp.h #include printf.h +#include string.h #define true 1 #define false 0 @@ -569,7 +570,7 @@

Re: [kvm-devel] [PATCH] kvm: testsuite: silence warnings on x86_64

2008-01-31 Thread Avi Kivity
Carlo Marcelo Arenas Belon wrote: On Thu, Jan 24, 2008 at 06:21:56PM +0200, Avi Kivity wrote: Bernhard Kaindl wrote: I did not test this patch as I did not find documentation on how to run the test cases and I could not find a make target to run them from make. make

Re: [kvm-devel] [PATCH] kvm: testsuite: silence warnings on x86_64

2008-01-26 Thread Avi Kivity
Bernhard Kaindl wrote: This patch fixes user/test/x86/emulator.flat on kvm-59 with x86_64/E6850 and three other warnings: Applied, thanks. Please send separate patches for separate problems in the future. -- error compiling committee.c: too many arguments to function

[kvm-devel] [PATCH] kvm: testsuite: silence warnings on x86_64

2008-01-24 Thread Bernhard Kaindl
Hi, please find the following trivial warning fix patch for kvm-60/user/test/x86/* below. Feel free to merge to upstream kvm. It's diffed against kvm-60. The patch does not change things, just shuts up the warnings. The deal here is that we check for warnings and are

Re: [kvm-devel] [PATCH] kvm: testsuite: silence warnings on x86_64

2008-01-24 Thread Avi Kivity
Bernhard Kaindl wrote: I did not test this patch as I did not find documentation on how to run the test cases and I could not find a make target to run them from make. make -C user test_cases user/kvmctl user/test/x86/bootstrap user/test/x86/access.flat (we should rename user - test)

Re: [kvm-devel] [PATCH] kvm: testsuite: silence warnings on x86_64

2008-01-24 Thread Bernhard Kaindl
On Thu, 24 Jan 2008, Avi Kivity wrote: make -C user test_cases user/kvmctl user/test/x86/bootstrap user/test/x86/access.flat Ah, thanks! -smp_init(ac_test_run); +smp_init((void (*)(void))ac_test_run); Better to add a wrapper that conforms to the expected signature, and makes