Re: [Qemu-devel] [PATCH] vhost-net: Move asserts to after check for end < start

2012-03-30 Thread Josh Durgin
On 12/16/2011 12:33 PM, Bruce Rogers wrote: When migrating a vm using vhost-net we hit the following assertion: qemu-kvm: /usr/src/packages/BUILD/qemu-kvm-0.15.1/hw/vhost.c:30: vhost_dev_sync_region: Assertion `start / (0x1000 * (8 * sizeof(vhost_log_chunk_t)))< dev->log_size' failed. I consi

Re: [Qemu-devel] [PATCH] qtest: fix 32-bit build

2012-03-30 Thread Eric Blake
On 03/30/2012 11:54 AM, Anthony Liguori wrote: > time_t appears to be an unsigned long so use %ld. Not portable. POSIX does not guarantee the size of time_t; there are 32-bit platforms where time_t is 64-bit. The only way to print a time_t value is to cast it through a known-width type, and seei

Re: [Qemu-devel] [PATCH v3 4/5] target-unicore32: QOM'ify CPU

2012-03-30 Thread Michael Roth
On Fri, Mar 30, 2012 at 11:38:02PM +0200, Andreas Färber wrote: > Am 30.03.2012 18:15, schrieb Michael Roth: > > On Fri, Mar 30, 2012 at 12:26:09PM +0200, Andreas Färber wrote: > >> +static void unicore_ii_cpu_initfn(Object *obj) > >> +{ > >> +UniCore32CPU *cpu = UNICORE32_CPU(obj); > >> +C

Re: [Qemu-devel] [PATCH v3 4/5] target-unicore32: QOM'ify CPU

2012-03-30 Thread Andreas Färber
Am 30.03.2012 18:15, schrieb Michael Roth: > On Fri, Mar 30, 2012 at 12:26:09PM +0200, Andreas Färber wrote: >> +static void unicore_ii_cpu_initfn(Object *obj) >> +{ >> +UniCore32CPU *cpu = UNICORE32_CPU(obj); >> +CPUUniCore32State *env = &cpu->env; >> + >> +env->cp0.c0_cpuid = 0x400108

Re: [Qemu-devel] [PATCH] kvm: deassign irqs in reset path

2012-03-30 Thread Jan Kiszka
On 2012-03-30 23:09, Alex Williamson wrote: > On Fri, 2012-03-30 at 22:35 +0200, Jan Kiszka wrote: >> On 2012-03-30 22:31, Jason Baron wrote: >>> On Fri, Mar 30, 2012 at 10:18:31PM +0200, Jan Kiszka wrote: >>> The root cause of the problem is that the 'reset_assigned_device()' code >>> firs

[Qemu-devel] Why can not debug a linux kernel thread?

2012-03-30 Thread Parmenides
Hi, It is said that the kernel can be debugged in qemu and I take a try. First, I started the qemu with qemu -m 64M -kernel arch/x86/boot/bzImage -initrd ~/image.cpio.gz -net nic -net tap,ifname=tap0 -s In another console gdb vmlinux (gdb) t

Re: [Qemu-devel] [PATCH] kvm: deassign irqs in reset path

2012-03-30 Thread Alex Williamson
On Fri, 2012-03-30 at 22:35 +0200, Jan Kiszka wrote: > On 2012-03-30 22:31, Jason Baron wrote: > > On Fri, Mar 30, 2012 at 10:18:31PM +0200, Jan Kiszka wrote: > > The root cause of the problem is that the 'reset_assigned_device()' code > > first writes a 0 to the command register. Then, whe

Re: [Qemu-devel] [PATCH] kvm: deassign irqs in reset path

2012-03-30 Thread Jan Kiszka
On 2012-03-30 22:31, Jason Baron wrote: > On Fri, Mar 30, 2012 at 10:18:31PM +0200, Jan Kiszka wrote: > The root cause of the problem is that the 'reset_assigned_device()' code > first writes a 0 to the command register. Then, when qemu subsequently > does > a kvm_deassign_irq() (c

Re: [Qemu-devel] [PATCH] kvm: deassign irqs in reset path

2012-03-30 Thread Jason Baron
On Fri, Mar 30, 2012 at 10:18:31PM +0200, Jan Kiszka wrote: > >>> The root cause of the problem is that the 'reset_assigned_device()' code > >>> first writes a 0 to the command register. Then, when qemu subsequently > >>> does > >>> a kvm_deassign_irq() (called by assign_irq(), in the system_reset

Re: [Qemu-devel] [PATCH] kvm: deassign irqs in reset path

2012-03-30 Thread Jan Kiszka
On 2012-03-30 22:13, Jason Baron wrote: > On Fri, Mar 30, 2012 at 09:29:23PM +0200, Jan Kiszka wrote: >> On 2012-03-30 21:18, Jason Baron wrote: >>> We've hit a kernel host panic, when issuing a 'system_reset' with a 82576 >>> nic >>> assigned and a Windows guest. Host system is a PowerEdge R815.

Re: [Qemu-devel] [PATCH] kvm: deassign irqs in reset path

2012-03-30 Thread Jason Baron
On Fri, Mar 30, 2012 at 09:29:23PM +0200, Jan Kiszka wrote: > On 2012-03-30 21:18, Jason Baron wrote: > > We've hit a kernel host panic, when issuing a 'system_reset' with a 82576 > > nic > > assigned and a Windows guest. Host system is a PowerEdge R815. > > > > [Hardware Error]: Hardware error f

Re: [Qemu-devel] [PATCH v2] qtest: avoid a warning with RTC test

2012-03-30 Thread Blue Swirl
On Fri, Mar 30, 2012 at 19:21, Anthony Liguori wrote: > On 03/30/2012 02:19 PM, Blue Swirl wrote: >> >> Avoid this warning on OpenBSD: >>   CC    tests/rtc-test.o >> /src/qemu/tests/rtc-test.c: In function 'check_time': >> /src/qemu/tests/rtc-test.c:171: warning: format '%ld' expects type >> 'long

Re: [Qemu-devel] [PATCH] qtest: fix 32-bit build

2012-03-30 Thread Blue Swirl
On Fri, Mar 30, 2012 at 19:13, Paolo Bonzini wrote: > Il 30/03/2012 21:06, Anthony Liguori ha scritto: >> On 03/30/2012 01:55 PM, Paolo Bonzini wrote: >>> Il 30/03/2012 20:29, Anthony Liguori ha scritto: > Now it will fail with w64 (which uses 64 bit time_t and > a 32 bit long).

Re: [Qemu-devel] Parallel build broken on master

2012-03-30 Thread Paolo Bonzini
Il 30/03/2012 21:25, Anthony Liguori ha scritto: > On 03/30/2012 02:21 PM, Luiz Capitulino wrote: >> On Fri, 30 Mar 2012 21:16:03 +0200 >> Paolo Bonzini wrote: >> >>> Il 30/03/2012 21:10, Luiz Capitulino ha scritto: Hi, The HEAD is 6b7cff76. Will paste the configure I use because th

Re: [Qemu-devel] [PATCH] kvm: deassign irqs in reset path

2012-03-30 Thread Jan Kiszka
On 2012-03-30 21:18, Jason Baron wrote: > We've hit a kernel host panic, when issuing a 'system_reset' with a 82576 nic > assigned and a Windows guest. Host system is a PowerEdge R815. > > [Hardware Error]: Hardware error from APEI Generic Hardware Error Source: > 32993 > [Hardware Error]: APEI g

Re: [Qemu-devel] [PATCH] qtest: fix 32-bit build

2012-03-30 Thread Anthony Liguori
On 03/30/2012 02:23 PM, Stefan Weil wrote: Am 30.03.2012 21:16, schrieb Anthony Liguori: On 03/30/2012 02:13 PM, Paolo Bonzini wrote: Il 30/03/2012 21:06, Anthony Liguori ha scritto: On 03/30/2012 01:55 PM, Paolo Bonzini wrote: Il 30/03/2012 20:29, Anthony Liguori ha scritto: Now it will fai

Re: [Qemu-devel] [RFC] qxl: don't panic on phys2virt

2012-03-30 Thread Alon Levy
On Fri, Mar 30, 2012 at 10:09:42AM +0200, Gerd Hoffmann wrote: > On 03/29/12 22:56, Alon Levy wrote: > > Issues a qxl_guest_bug from qxl_phys2virt. Everywhere else will fail. > > Nice start (there are more guest-triggerable asserts to be fixed this way). > > > qxl_phys2virt requires an additional

Re: [Qemu-devel] Parallel build broken on master

2012-03-30 Thread Anthony Liguori
On 03/30/2012 02:21 PM, Luiz Capitulino wrote: On Fri, 30 Mar 2012 21:16:03 +0200 Paolo Bonzini wrote: Il 30/03/2012 21:10, Luiz Capitulino ha scritto: Hi, The HEAD is 6b7cff76. Will paste the configure I use because they seem to make the bug more likely: $ ../configure --target-list=x86_64

Re: [Qemu-devel] [PATCH v3] spice_info: add mouse_mode

2012-03-30 Thread Alon Levy
On Fri, Mar 30, 2012 at 10:12:56AM +0200, Gerd Hoffmann wrote: > Hi, > > > +# @SpiceQueryMouseMode > > +# > > +# An enumation of Spice mouse states. > > +# > > +# @client: Mouse cursor position is determined by the client. > > +# > > +# @server: Mouse cursor position is determined by the server.

Re: [Qemu-devel] [PATCH] qtest: fix 32-bit build

2012-03-30 Thread Stefan Weil
Am 30.03.2012 21:16, schrieb Anthony Liguori: On 03/30/2012 02:13 PM, Paolo Bonzini wrote: Il 30/03/2012 21:06, Anthony Liguori ha scritto: On 03/30/2012 01:55 PM, Paolo Bonzini wrote: Il 30/03/2012 20:29, Anthony Liguori ha scritto: Now it will fail with w64 (which uses 64 bit time_t and a 3

Re: [Qemu-devel] [PATCH v2] qtest: avoid a warning with RTC test

2012-03-30 Thread Anthony Liguori
On 03/30/2012 02:19 PM, Blue Swirl wrote: Avoid this warning on OpenBSD: CCtests/rtc-test.o /src/qemu/tests/rtc-test.c: In function 'check_time': /src/qemu/tests/rtc-test.c:171: warning: format '%ld' expects type 'long int', but argument 2 has type 'time_t' /src/qemu/tests/rtc-test.c:173:

Re: [Qemu-devel] Parallel build broken on master

2012-03-30 Thread Luiz Capitulino
On Fri, 30 Mar 2012 21:16:03 +0200 Paolo Bonzini wrote: > Il 30/03/2012 21:10, Luiz Capitulino ha scritto: > > Hi, > > > > The HEAD is 6b7cff76. Will paste the configure I use because they seem to > > make > > the bug more likely: > > > > $ ../configure --target-list=x86_64-softmmu --disable-s

[Qemu-devel] [PATCH v2] qtest: avoid a warning with RTC test

2012-03-30 Thread Blue Swirl
Avoid this warning on OpenBSD: CCtests/rtc-test.o /src/qemu/tests/rtc-test.c: In function 'check_time': /src/qemu/tests/rtc-test.c:171: warning: format '%ld' expects type 'long int', but argument 2 has type 'time_t' /src/qemu/tests/rtc-test.c:173: warning: format '%ld' expects type 'long int'

[Qemu-devel] [PATCH] kvm: deassign irqs in reset path

2012-03-30 Thread Jason Baron
We've hit a kernel host panic, when issuing a 'system_reset' with a 82576 nic assigned and a Windows guest. Host system is a PowerEdge R815. [Hardware Error]: Hardware error from APEI Generic Hardware Error Source: 32993 [Hardware Error]: APEI generic hardware error status [Hardware Error]: severi

Re: [Qemu-devel] [PATCH] qtest: fix 32-bit build

2012-03-30 Thread Anthony Liguori
On 03/30/2012 02:13 PM, Paolo Bonzini wrote: Il 30/03/2012 21:06, Anthony Liguori ha scritto: On 03/30/2012 01:55 PM, Paolo Bonzini wrote: Il 30/03/2012 20:29, Anthony Liguori ha scritto: Now it will fail with w64 (which uses 64 bit time_t and a 32 bit long). That's a bug in w64 (it has a br

Re: [Qemu-devel] Parallel build broken on master

2012-03-30 Thread Paolo Bonzini
Il 30/03/2012 21:10, Luiz Capitulino ha scritto: > Hi, > > The HEAD is 6b7cff76. Will paste the configure I use because they seem to make > the bug more likely: > > $ ../configure --target-list=x86_64-softmmu --disable-strip --enable-debug \ > --disable-xen --disable-brlapi --disable-curses --d

Re: [Qemu-devel] [PATCH] qtest: fix 32-bit build

2012-03-30 Thread Paolo Bonzini
Il 30/03/2012 21:06, Anthony Liguori ha scritto: > On 03/30/2012 01:55 PM, Paolo Bonzini wrote: >> Il 30/03/2012 20:29, Anthony Liguori ha scritto: Now it will fail with w64 (which uses 64 bit time_t and a 32 bit long). >>> >>> That's a bug in w64 (it has a broken ABI). >>> >>> Do we actu

[Qemu-devel] Parallel build broken on master

2012-03-30 Thread Luiz Capitulino
Hi, The HEAD is 6b7cff76. Will paste the configure I use because they seem to make the bug more likely: $ ../configure --target-list=x86_64-softmmu --disable-strip --enable-debug \ --disable-xen --disable-brlapi --disable-curses --disable-curl \ --disable-bluez --disable-system --disable-debu

Re: [Qemu-devel] [PATCH] qtest: avoid a warning with RTC test

2012-03-30 Thread Anthony Liguori
On 03/30/2012 01:59 PM, Blue Swirl wrote: Avoid this warning on OpenBSD: CCtests/rtc-test.o /src/qemu/tests/rtc-test.c: In function 'check_time': /src/qemu/tests/rtc-test.c:171: warning: format '%ld' expects type 'long int', but argument 2 has type 'time_t' /src/qemu/tests/rtc-test.c:173:

Re: [Qemu-devel] [PATCH] qtest: fix 32-bit build

2012-03-30 Thread Anthony Liguori
On 03/30/2012 01:55 PM, Paolo Bonzini wrote: Il 30/03/2012 20:29, Anthony Liguori ha scritto: Now it will fail with w64 (which uses 64 bit time_t and a 32 bit long). That's a bug in w64 (it has a broken ABI). Do we actually build and run on w64?? It's actually sensible and x32 does the same

[Qemu-devel] [PATCH] qtest: use qemu_gettimeofday()

2012-03-30 Thread Anthony Liguori
On linux, qemu_timeval will always be two long ints. On windows, we use our own struct definition. This should fix win64. Signed-off-by: Anthony Liguori --- qtest.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/qtest.c b/qtest.c index cd7186c..daeabb7 10

[Qemu-devel] [PATCH] qtest: avoid a warning with RTC test

2012-03-30 Thread Blue Swirl
Avoid this warning on OpenBSD: CCtests/rtc-test.o /src/qemu/tests/rtc-test.c: In function 'check_time': /src/qemu/tests/rtc-test.c:171: warning: format '%ld' expects type 'long int', but argument 2 has type 'time_t' /src/qemu/tests/rtc-test.c:173: warning: format '%ld' expects type 'long int'

Re: [Qemu-devel] [PATCH] qtest: fix 32-bit build

2012-03-30 Thread Paolo Bonzini
Il 30/03/2012 20:29, Anthony Liguori ha scritto: >> Now it will fail with w64 (which uses 64 bit time_t and >> a 32 bit long). > > That's a bug in w64 (it has a broken ABI). > > Do we actually build and run on w64?? It's actually sensible and x32 does the same. Not too urgent though. Paolo

Re: [Qemu-devel] [PATCH] slirp: Signal free input buffer space to io-thread

2012-03-30 Thread Paolo Bonzini
Il 30/03/2012 19:43, Jan Kiszka ha scritto: > This massively accelerates slirp reception speed: If data arrives > faster than the guest can read it from the input buffer, the file > descriptor for the corresponding socket was taken out of the fdset for > select. However, the event of the guest read

Re: [Qemu-devel] [PATCH v4 2/7] qtest: add test framework

2012-03-30 Thread Paolo Bonzini
Il 30/03/2012 19:53, Anthony Liguori ha scritto: +#define FMT_timeval "%" PRId64 ".%06" PRId64 >>> >>> Does %ld work instead of PRId64? >> >> That works for my 32 bit Debian, but it would fail on 64 bit Linux. > > It works on my 64-bit Linux fwiw. It would fail on x32, but it's a good s

[Qemu-devel] [PATCH 11/14] create_config: remove *dir block

2012-03-30 Thread Eduardo Habkost
Now only the qemu_*dir variables will become #defines. The other directory names aren't used by the C code. That means the following #defines won't be available in C code anymore: - CONFIG_QEMU_BINDIR - CONFIG_QEMU_LIBDIR - CONFIG_QEMU_INCLUDEDIR - CONFIG_QEMU_MANDIR - CONFIG_QEMU_SYSCONFDIR

[Qemu-devel] [PATCH 04/14] create_config: separate section for qemu_*dir variables

2012-03-30 Thread Eduardo Habkost
The generic *dir section will eventually go away and be replaced with qemu_* section. By now, both sections will be kept, while the variables get renamed on config-host.mak. With this patch, a XXXdir variable will become a CONFIG_QEMU_XXXDIR define, and a qemu_XXXdir variable will become CONFIG_QE

[Qemu-devel] [PATCH 13/14] configure: change meaning of --datadir to Autoconf convention

2012-03-30 Thread Eduardo Habkost
Autoconf uses --datadir for the /usr/share directory, not the program-specific subdirectory inside /usr/share. This changes configure to match autoconf behavior. Note that this will break compatibility with existing build scripts. Signed-off-by: Eduardo Habkost --- configure |9 + 1

[Qemu-devel] [PATCH 06/14] config-host.mak: rename confdir to qemu_confdir

2012-03-30 Thread Eduardo Habkost
$confdir is a confusing name, as it's not clear if it's "the system config dir" or "the Qemus-specific directory inside the config dir". $qemu_confdir makes it more clear. The current C code that uses CONFIG_QEMU_CONFDIR should be safe, as now create_config generates the same #define name (CONFIG_

Re: [Qemu-devel] [PATCH] qtest: fix 32-bit build

2012-03-30 Thread Anthony Liguori
On 03/30/2012 01:03 PM, Stefan Weil wrote: Am 30.03.2012 19:59, schrieb Anthony Liguori: Applied. Sorry for the breakage. Regards, Anthony Liguori There is a new breakage: Now it will fail with w64 (which uses 64 bit time_t and a 32 bit long). That's a bug in w64 (it has a broken ABI). D

[Qemu-devel] [PATCH 01/14] configure: rename $datadir to $qemu_datadir

2012-03-30 Thread Eduardo Habkost
The variable is used for the qemu-specific directory and has a different meaning of the autoconf "datadir" variable (that's used for the $prefix/share directory, not for $prefix/share/PACKAGE). This doesn't change behavior or interfaces, it's just an internal variable rename. Signed-off-by: Eduar

[Qemu-devel] [PATCH 05/14] config-host.mak: rename datadir to qemu_datadir

2012-03-30 Thread Eduardo Habkost
Autoconf concept of "datadir" is supposed to be "$prefix/share", not "$prefix/share/PACKAGE", so using datadir for the Qemu-specific directory is confusing. The current C code that uses CONFIG_QEMU_DATADIR should be safe, as now create_config generates the same #define name (CONFIG_QEMU_DATADIR) f

[Qemu-devel] [RFC PATCH v4 5/8] tracetool: Add support for the 'simple' backend

2012-03-30 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova --- scripts/tracetool/backend/simple.py | 60 +++ 1 files changed, 60 insertions(+), 0 deletions(-) create mode 100644 scripts/tracetool/backend/simple.py diff --git a/scripts/tracetool/backend/simple.py b/scripts/tracetool/backen

[Qemu-devel] [PATCH 10/14] config-host.mak: reorder variables a bit

2012-03-30 Thread Eduardo Habkost
Cosmetic change. Signed-off-by: Eduardo Habkost --- configure |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index b02dfa3..5de58cb 100755 --- a/configure +++ b/configure @@ -2973,10 +2973,10 @@ echo "bindir=$bindir" >> $config_host_mak echo "l

[Qemu-devel] [PATCH 08/14] config-host.mak: rename docdir to qemu_docdir

2012-03-30 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- Makefile |4 ++-- configure |2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index bda3680..a308471 100644 --- a/Makefile +++ b/Makefile @@ -267,8 +267,8 @@ BLOBS= endif install-doc: $(DOCS) - $(INSTALL

[Qemu-devel] [PATCH 00/14] configure: --with-confsuffix option

2012-03-30 Thread Eduardo Habkost
This series is the third version of the --confsuffix/--confdir changes I have submitted before. Now it does two interface changes: - Changes --datadir meaning to match Autoconf convention (meaning that it should point to /usr/share, not /usr/share/qemu). NOTE: this breaks compatibility.

Re: [Qemu-devel] [PATCH] qtest: fix 32-bit build

2012-03-30 Thread Stefan Weil
Am 30.03.2012 19:59, schrieb Anthony Liguori: Applied. Sorry for the breakage. Regards, Anthony Liguori There is a new breakage: Now it will fail with w64 (which uses 64 bit time_t and a 32 bit long). Regards, Stefan W.

[Qemu-devel] [PATCH 09/14] config-host.mak: remove CONFIG_QEMU_SHAREDIR

2012-03-30 Thread Eduardo Habkost
There is no code using that variable according to 'git grep', so kill it. Signed-off-by: Eduardo Habkost --- configure |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/configure b/configure index fc206ff..b02dfa3 100755 --- a/configure +++ b/configure @@ -2978,7 +2978,6 @

[Qemu-devel] [RFC PATCH v4 8/8] tracetool: Add MAINTAINERS info

2012-03-30 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova --- MAINTAINERS |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index f83d07c2..0e66dd8 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -548,6 +548,8 @@ Tracing M: Stefan Hajnoczi S: Maintained F: trace/ +F: scrip

Re: [Qemu-devel] [PATCH] qtest: fix 32-bit build

2012-03-30 Thread Anthony Liguori
Applied. Sorry for the breakage. Regards, Anthony Liguori On 03/30/2012 12:54 PM, Anthony Liguori wrote: time_t appears to be an unsigned long so use %ld. Reported-by: Stefan Weil Signed-off-by: Anthony Liguori --- qtest.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff

[Qemu-devel] [PATCH 03/14] configure: rename $confdir to $qemu_confdir

2012-03-30 Thread Eduardo Habkost
There's no "confdir" variable on Autoconf, but it's good to make it clear that it's a variable for the Qemu-specific subdirectory inside $sysconfdir. Signed-off-by: Eduardo Habkost --- configure |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure ind

[Qemu-devel] [RFC PATCH v4 7/8] tracetool: Add support for the 'dtrace' backend

2012-03-30 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova --- scripts/tracetool.py| 32 ++- scripts/tracetool/__init__.py | 11 +++- scripts/tracetool/backend/dtrace.py | 104 +++ scripts/tracetool/format/d.py | 20 +++ scripts/tracetool/format/s

[Qemu-devel] [RFC PATCH v4 6/8] tracetool: Add support for the 'ust' backend

2012-03-30 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova --- scripts/tracetool/backend/ust.py | 102 ++ 1 files changed, 102 insertions(+), 0 deletions(-) create mode 100644 scripts/tracetool/backend/ust.py diff --git a/scripts/tracetool/backend/ust.py b/scripts/tracetool/backend/ust.

[Qemu-devel] [RFC PATCH v4 3/8] tracetool: Add module for the 'h' format

2012-03-30 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova --- scripts/tracetool/format/h.py | 45 + 1 files changed, 45 insertions(+), 0 deletions(-) create mode 100644 scripts/tracetool/format/h.py diff --git a/scripts/tracetool/format/h.py b/scripts/tracetool/format/h.py new fil

[Qemu-devel] [RFC PATCH v4 4/8] tracetool: Add support for the 'stderr' backend

2012-03-30 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova --- scripts/tracetool/backend/stderr.py | 61 +++ 1 files changed, 61 insertions(+), 0 deletions(-) create mode 100644 scripts/tracetool/backend/stderr.py diff --git a/scripts/tracetool/backend/stderr.py b/scripts/tracetool/backen

[Qemu-devel] [RFC PATCH v4 1/8] tracetool: Rewrite infrastructure as python modules

2012-03-30 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova --- Makefile.objs |6 Makefile.target | 13 - configure |4 scripts/tracetool | 648 - scripts/tracetool.py |

[Qemu-devel] [RFC PATCH v4 2/8] tracetool: Add module for the 'c' format

2012-03-30 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova --- scripts/tracetool/format/c.py | 20 1 files changed, 20 insertions(+), 0 deletions(-) create mode 100644 scripts/tracetool/format/c.py diff --git a/scripts/tracetool/format/c.py b/scripts/tracetool/format/c.py new file mode 100644 index 0

[Qemu-devel] [PATCH 02/14] configure: rename $docdir to $qemu_docdir

2012-03-30 Thread Eduardo Habkost
The autoconf "docdir" variable is used for the program-specific subdirectory, but anyway it's better to make it clear that the variable is used for the qemu-specific subdirectory path. Signed-off-by: Eduardo Habkost --- configure |8 1 files changed, 4 insertions(+), 4 deletions(-)

[Qemu-devel] [RFC PATCH v4 0/8] Rewrite tracetool using python modules

2012-03-30 Thread Lluís Vilanova
A full rewrite of the tracetool script using per-format and per-backend modules, so that it's easier to read and extend it in the future. Signed-off-by: Lluís Vilanova --- NOTE: This series applies in current master, ignoring the "Rewrite tracetool using python" series. Changes in v4: * S

[Qemu-devel] [PATCH 07/14] Makefile: use $(qemu_confdir) instead of $(sysconfdir)/qemu

2012-03-30 Thread Eduardo Habkost
Instead of hardcoding the directory suffix, use the qemu-specific directory variable. Signed-off-by: Eduardo Habkost --- Makefile |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index 605f1aa..bda3680 100644 --- a/Makefile +++ b/Makefile @@ -280,8

[Qemu-devel] [PATCH] qtest: fix 32-bit build

2012-03-30 Thread Anthony Liguori
time_t appears to be an unsigned long so use %ld. Reported-by: Stefan Weil Signed-off-by: Anthony Liguori --- qtest.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/qtest.c b/qtest.c index 53e2b79..cd7186c 100644 --- a/qtest.c +++ b/qtest.c @@ -34,7 +34,7 @@ static in

Re: [Qemu-devel] [PATCH v4 7/7] qtest: add rtc-test test-case

2012-03-30 Thread Blue Swirl
On Fri, Mar 30, 2012 at 17:49, Anthony Liguori wrote: > On 03/30/2012 12:22 PM, Blue Swirl wrote: >> >> On Fri, Mar 30, 2012 at 17:15, Blue Swirl  wrote: >>> >>> On Wed, Mar 28, 2012 at 13:42, Paolo Bonzini  wrote: From: Anthony Liguori Signed-off-by: Anthony Liguori Signe

Re: [Qemu-devel] [PATCH v4 2/7] qtest: add test framework

2012-03-30 Thread Anthony Liguori
On 03/30/2012 12:52 PM, Stefan Weil wrote: Am 30.03.2012 19:50, schrieb Anthony Liguori: On 03/30/2012 12:40 PM, Stefan Weil wrote: Hi this commit breaks builds on Linux i386: CC libhw64/qtest.o cc1: warnings being treated as errors /home/stefan/src/qemu/qemu.org/qemu/qtest.c: In function ‘qt

[Qemu-devel] [PATCH 14/14] configure: add --confsuffix option

2012-03-30 Thread Eduardo Habkost
This will allow the user to make Qemu use a different subdirectory name inside $datadir and $sysconfdir, instead of "/qemu". Signed-off-by: Eduardo Habkost --- configure |9 ++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/configure b/configure index b74efe2..e56c569

[Qemu-devel] [PATCH 12/14] create_config: simplify prefix=* block, remove CONFIG_QEMU_PREFIX

2012-03-30 Thread Eduardo Habkost
As now that block handles only the prefix variable, the code can be much simpler. This also removes the CONFIG_QEMU_PREFIX define as it is not used by any C code. Signed-off-by: Eduardo Habkost --- scripts/create_config |7 +-- 1 files changed, 1 insertions(+), 6 deletions(-) diff --git

Re: [Qemu-devel] [PATCH] RFC: options parse in vl.c should be moduled

2012-03-30 Thread Lluís Vilanova
Anthony Liguori writes: >> My plan is to have a -query-capabilities so we can change the help text, then >> use GOptionContext. >> >> It would take a small function that goes through and promotes any option in >> the >> format '-foo' to '--foo'. But that's not too hard. >> >> Then we can complet

Re: [Qemu-devel] [PATCH v4 2/7] qtest: add test framework

2012-03-30 Thread Stefan Weil
Am 30.03.2012 19:50, schrieb Anthony Liguori: On 03/30/2012 12:40 PM, Stefan Weil wrote: Hi this commit breaks builds on Linux i386: CC libhw64/qtest.o cc1: warnings being treated as errors /home/stefan/src/qemu/qemu.org/qemu/qtest.c: In function ‘qtest_send_prefix’: /home/stefan/src/qemu/qem

Re: [Qemu-devel] [PATCH v4 2/7] qtest: add test framework

2012-03-30 Thread Anthony Liguori
On 03/30/2012 12:40 PM, Stefan Weil wrote: Hi this commit breaks builds on Linux i386: CC libhw64/qtest.o cc1: warnings being treated as errors /home/stefan/src/qemu/qemu.org/qemu/qtest.c: In function ‘qtest_send_prefix’: /home/stefan/src/qemu/qemu.org/qemu/qtest.c:156: error: format ‘%lld’ exp

Re: [Qemu-devel] [PATCH v4 7/7] qtest: add rtc-test test-case

2012-03-30 Thread Anthony Liguori
On 03/30/2012 12:22 PM, Blue Swirl wrote: On Fri, Mar 30, 2012 at 17:15, Blue Swirl wrote: On Wed, Mar 28, 2012 at 13:42, Paolo Bonzini wrote: From: Anthony Liguori Signed-off-by: Anthony Liguori Signed-off-by: Paolo Bonzini --- tests/Makefile |5 + tests/rtc-test.c | 263 +

[Qemu-devel] [PATCH] slirp: Signal free input buffer space to io-thread

2012-03-30 Thread Jan Kiszka
This massively accelerates slirp reception speed: If data arrives faster than the guest can read it from the input buffer, the file descriptor for the corresponding socket was taken out of the fdset for select. However, the event of the guest reading enough data from the buffer was not signaled. Th

Re: [Qemu-devel] [PATCH v4 2/7] qtest: add test framework

2012-03-30 Thread Stefan Weil
Hi this commit breaks builds on Linux i386: CC libhw64/qtest.o cc1: warnings being treated as errors /home/stefan/src/qemu/qemu.org/qemu/qtest.c: In function ‘qtest_send_prefix’: /home/stefan/src/qemu/qemu.org/qemu/qtest.c:156: error: format ‘%lld’ expects type ‘long long int’, but argument 3

Re: [Qemu-devel] [PATCH v4 7/7] qtest: add rtc-test test-case

2012-03-30 Thread Blue Swirl
On Fri, Mar 30, 2012 at 17:29, Anthony Liguori wrote: > On 03/30/2012 12:15 PM, Blue Swirl wrote: >> >> On Wed, Mar 28, 2012 at 13:42, Paolo Bonzini  wrote: >>> >>> From: Anthony Liguori >>> >>> Signed-off-by: Anthony Liguori >>> Signed-off-by: Paolo Bonzini >>> --- >>>  tests/Makefile   |    5 +

Re: [Qemu-devel] [PATCH v4 7/7] qtest: add rtc-test test-case

2012-03-30 Thread Anthony Liguori
On 03/30/2012 12:15 PM, Blue Swirl wrote: On Wed, Mar 28, 2012 at 13:42, Paolo Bonzini wrote: From: Anthony Liguori Signed-off-by: Anthony Liguori Signed-off-by: Paolo Bonzini --- tests/Makefile |5 + tests/rtc-test.c | 263 ++ 2 f

[Qemu-devel] [PATCH v2 0/2] target-arm, linux-user: userspace ARM BE8 support

2012-03-30 Thread Peter Maydell
This is an update of Paul Brook's patch adding support for ARM BE8 userspace binaries: http://patchwork.ozlabs.org/patch/143758/ where I've basically just fixed the nits noted in code review: * rebase * use EF_ defines rather than hardcoded constants * make bswap_code a bool for future VMSTAT

[Qemu-devel] [PATCH v2 2/2] Userspace ARM BE8 support

2012-03-30 Thread Peter Maydell
From: Paul Brook Add support for ARM BE8 userspace binaries. i.e. big-endian data and little-endian code. In principle LE8 mode is also possible, but AFAIK has never actually been implemented/used. System emulation doesn't have any useable big-endian board models, but should in principle work on

[Qemu-devel] [PATCH 4/5] target-mips: Use TCG registers for the FPU.

2012-03-30 Thread Richard Henderson
With normal FP, this doesn't have much affect on the generated code, because most of the FP operations are not CONST/PURE, and so we spill registers in about the same frequency as the explicit load/stores. But with Loongson multimedia instructions, which are all integral and whose helpers are in f

Re: [Qemu-devel] [PATCH v4 7/7] qtest: add rtc-test test-case

2012-03-30 Thread Blue Swirl
On Fri, Mar 30, 2012 at 17:15, Blue Swirl wrote: > On Wed, Mar 28, 2012 at 13:42, Paolo Bonzini wrote: >> From: Anthony Liguori >> >> Signed-off-by: Anthony Liguori >> Signed-off-by: Paolo Bonzini >> --- >>  tests/Makefile   |    5 + >>  tests/rtc-test.c |  263 >>

[Qemu-devel] [PATCH 2/2] mips-linux-user: Always support rdhwr.

2012-03-30 Thread Richard Henderson
The kernel will emulate this instruction if it's not supported natively. This insn is used for TLS, among other things, and so is required by modern glibc. Signed-off-by: Richard Henderson Cc: Riku Voipio --- target-mips/translate.c |4 1 files changed, 4 insertions(+), 0 deletions(-)

[Qemu-devel] [PATCH 1/2] target-mips: Streamline indexed cp1 memory addressing.

2012-03-30 Thread Richard Henderson
We've already eliminated both base and index being zero. --- target-mips/translate.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/target-mips/translate.c b/target-mips/translate.c index a663b74..300d95e 100644 --- a/target-mips/translate.c +++ b/target-mips/translate.

[Qemu-devel] [PATCH v2 0/2] misc MIPS improvments

2012-03-30 Thread Richard Henderson
Extracted and rebased from a larger patch set from December. Available as git://repo.or.cz/qemu/rth.git rth/mips/next r~ Richard Henderson (2): target-mips: Streamline indexed cp1 memory addressing. mips-linux-user: Always support rdhwr. target-mips/translate.c |7 +-- 1 fil

Re: [Qemu-devel] [PATCH v4 7/7] qtest: add rtc-test test-case

2012-03-30 Thread Blue Swirl
On Wed, Mar 28, 2012 at 13:42, Paolo Bonzini wrote: > From: Anthony Liguori > > Signed-off-by: Anthony Liguori > Signed-off-by: Paolo Bonzini > --- >  tests/Makefile   |    5 + >  tests/rtc-test.c |  263 > ++ >  2 files changed, 268 insertion

[Qemu-devel] [PATCH v2] target-mips: Implement Loongson Multimedia Instructions

2012-03-30 Thread Richard Henderson
Implements all of the COP2 instructions except for the S family of comparisons. The documentation is unclear for those. Signed-off-by: Richard Henderson --- v2: Rebased vs master. Makefile.target |3 + target-mips/helper.h | 59 target-mips/lmi_helper.c | 744

Re: [Qemu-devel] [RFC PATCH v3 0/8] Rewrite tracetool using python modules

2012-03-30 Thread Lluís Vilanova
Stefan Hajnoczi writes: > Only one point: please don't introduce PUBLIC yet. Let's add it when > it's needed. At the moment nothing uses it. > I have tested this series with all backends and looked at the diff > between the old tracetool and tracetool.py. > I'm happy with this series. We need

[Qemu-devel] [PATCH 3/5] target-mips: Pass DisasContext to fpr32 load/store routines

2012-03-30 Thread Richard Henderson
The large mechanical change in support of a follow-on patch that changes the representation of the fp registers. Signed-off-by: Richard Henderson --- target-mips/translate.c | 312 +++ 1 files changed, 153 insertions(+), 159 deletions(-) diff --git a

[Qemu-devel] [PATCH 2/5] target-mips: Fix MIPS_DEBUG.

2012-03-30 Thread Richard Henderson
The macro uses the DisasContext. Pass it around as needed. Signed-off-by: Richard Henderson --- target-mips/translate.c | 74 --- 1 files changed, 38 insertions(+), 36 deletions(-) diff --git a/target-mips/translate.c b/target-mips/translate.c inde

[Qemu-devel] [PATCH 5/5] target-mips: Add accessors for the two 32-bit halves of a 64-bit FPR

2012-03-30 Thread Richard Henderson
Not much used yet, but more users to come. Signed-off-by: Richard Henderson --- target-mips/translate.c | 46 ++ 1 files changed, 30 insertions(+), 16 deletions(-) diff --git a/target-mips/translate.c b/target-mips/translate.c index 10d8407..2396339

[Qemu-devel] [PATCH v2 0/5] tcg-mips: Use TCG registers for the FPU

2012-03-30 Thread Richard Henderson
This is a re-base of a patch set sent in December. The only real changes are CPUState -> CPUMIPSState. The commit message for patch 4 describe the real intent. This patch set is also available at git://repo.or.cz/qemu/rth.git rth/mips/fpregs r~ Richard Henderson (5): target-mips: Set op

[Qemu-devel] [PATCH 1/5] target-mips: Set opn in gen_ldst_multiple.

2012-03-30 Thread Richard Henderson
Used by MIPS_DEBUG, when enabled. Signed-off-by: Richard Henderson --- target-mips/translate.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/target-mips/translate.c b/target-mips/translate.c index a663b74..85dd65a 100644 --- a/target-mips/translate.c +++ b/target-

Re: [Qemu-devel] [PATCH 14/14] target-arm: Move A9 config_base_address reset value to ARMCPU

2012-03-30 Thread Peter Maydell
On 30 March 2012 18:01, Mark Langsdorf wrote: > Odd that I wasn't actually cc'd by the mailer on this. Oops, sorry, that's my fault. I added the 'cc' to the patch commit message but forgot to tweak my git-send-email command line to honour that. > Do you want me to pull together a patch to make i

[Qemu-devel] [PATCH v2 1/2] elf.h: Update EF_ARM_ constants to newer ABI versions

2012-03-30 Thread Peter Maydell
Update the EF_ARM_* constants (for the ELF header e_flags field) to include the newer flags specified for later versions of the ABI. (This set of constants is from include/elf/arm.h from binutils-2.17 and so licensed under GPL-v2-or-later.) Signed-off-by: Peter Maydell --- elf.h | 21 +

Re: [Qemu-devel] [PATCH 14/14] target-arm: Move A9 config_base_address reset value to ARMCPU

2012-03-30 Thread Mark Langsdorf
Odd that I wasn't actually cc'd by the mailer on this. Do you want me to pull together a patch to make it a QOM property now, or is there some other dependency I should wait on? It may take me a week or so to get time to pull it together and test it. --Mark Langsdorf Calxeda, Inc. On 03/30/2012

Re: [Qemu-devel] [RFC PATCH v3 1/8] tracetool: Rewrite infrastructure as python modules

2012-03-30 Thread Lluís Vilanova
Stefan Hajnoczi writes: >> +kwargs = {} >> + >> +try: >> +tracetool.generate(sys.stdin, arg_format, arg_backend, **kwargs) > If forgot to ask what kwargs is doing here? Can we default to {}? > Otherwise let's drop it for now since there are no options defined yet. It's used late

Re: [Qemu-devel] [PATCH] RFC: options parse in vl.c should be moduled

2012-03-30 Thread Anthony Liguori
On 03/30/2012 11:09 AM, Anthony Liguori wrote: On 03/30/2012 10:59 AM, Lluís Vilanova wrote: Wanpeng Li writes: Consider of the options parse process in main function of vl.c is too long.It should be module into single function to clear ideas, strengthen the source code management, and increas

Re: [Qemu-devel] [PATCH v3 4/5] target-unicore32: QOM'ify CPU

2012-03-30 Thread Michael Roth
On Fri, Mar 30, 2012 at 12:26:09PM +0200, Andreas Färber wrote: > Embed CPUUniCore32State as first member of UniCore32CPU. > > Contributed under GPLv2+. > > Signed-off-by: Andreas Färber > Acked-by: Guan Xuetao > --- > Makefile.target|1 + > target-unicore32/cpu-qom.h | 59 ++

Re: [Qemu-devel] [PATCH] RFC: options parse in vl.c should be moduled

2012-03-30 Thread Anthony Liguori
On 03/30/2012 10:59 AM, Lluís Vilanova wrote: Wanpeng Li writes: Consider of the options parse process in main function of vl.c is too long.It should be module into single function to clear ideas, strengthen the source code management, and increase code readability.So I module the process of op

Re: [Qemu-devel] [PATCHv2] piix: fix up/down races + document

2012-03-30 Thread Igor Mammedov
On 03/29/2012 02:51 PM, Michael S. Tsirkin wrote: piix acpi interface suffers from the following 2 issues: 1. - delete device a - quickly add device b in another slot if we do this before guest reads the down register, the down event is discarded and device will never be deleted. 2. - delete d

Re: [Qemu-devel] [PATCH] RFC: options parse in vl.c should be moduled

2012-03-30 Thread Lluís Vilanova
Wanpeng Li writes: > Consider of the options parse process in main function of vl.c is too > long.It should be module into single function to clear ideas, strengthen > the source code management, and increase code readability.So I module the > process of options parse as function options_parse, a

Re: [Qemu-devel] [PATCH] RFC: options parse in vl.c should be moduled

2012-03-30 Thread Michael Roth
On Fri, Mar 30, 2012 at 09:25:47PM +0800, Wanpeng Li wrote: > On Fri, Mar 30, 2012 at 01:53:14PM +0100, Daniel P. Berrange wrote: > >On Fri, Mar 30, 2012 at 08:36:43PM +0800, Wanpeng Li wrote: > >> Consider of the options parse process in main function of vl.c is too > >> long.It should be module i

[Qemu-devel] [PATCH] qemu/configure: fix CFLAGS handling for i386

2012-03-30 Thread Olaf Hering
configure will generate incorrect CFLAGS which will lead to compile errors due to unknown gcc options, IFF CFLAGS was already in the environment during configure invocation. Add a space before the -march=i486 gcc option. This patch is against the qemu-xen tree, but it should apply also to qemu.g

Re: [Qemu-devel] [PATCH v4 0/7] new test infrastructure + qtest

2012-03-30 Thread Anthony Liguori
On 03/28/2012 08:42 AM, Paolo Bonzini wrote: This is a rebase of qtest. I split the gtester infrastructure into its own patch, and reorganized the tests by moving everything into tests/. Also, libqtest now has bindings for the clock management commands, and I am using them in rtc-test. Finally

  1   2   3   >