[Qemu-devel] [PATCH] Remove code duplication from savevm instance_id creation code.

2010-11-07 Thread Gleb Natapov
Code in register_savevm_live() and vmstate_register_with_alias_id() looks identical except idstr used. Move into separate function. Signed-off-by: Gleb Natapov diff --git a/savevm.c b/savevm.c index cf56121..6ca6920 100644 --- a/savevm.c +++ b/savevm.c @@ -1080,6 +1080,33 @@ static int calculate_

[Qemu-devel] [Bug 668799] Re: qemu-arm segfaults executing msgmerge (gettext)

2010-11-07 Thread Jan-Simon Möller
We always see this in : exec.c1662: void cpu_exit(CPUState *env) { cpu_unlink_tb(env); env->exit_request = 1; } A quick test with the statement cpu_unlink_tb(env) removed passed the test. -- qemu-arm segfaults executing msgmerge (gettext) https://bugs.launchpad.net/bugs/668799 You rec

[Qemu-devel] [Bug 668799] Re: qemu-arm segfaults executing msgmerge (gettext)

2010-11-07 Thread Jan-Simon Möller
Alternative testcase: compile and "export OMP_NUM_THREADS=6" before running. /** * FILE: omp_mm.c * DESCRIPTION: * OpenMp Example - Matrix Multiply - C Version * Demonstrates a matrix multiply using OpenMP. Threads

Re: [Qemu-devel] [PATCH v2 5/6] backdoor: [i386] provide and implement intruction-based backdoor interface

2010-11-07 Thread Gleb Natapov
On Thu, Nov 04, 2010 at 11:36:15PM +0100, Lluís wrote: > Take the unused CPUID 0x40001xxx range as the backdoor instruction. > In KVM (and it fits the spec nicely) cpuid is defined in terms of tables. There is no callback that is called when particular cpuid is queried, so such backdoor interface

Re: [Qemu-devel] [RESEND][PATCH] Remove 16-character limit on process title

2010-11-07 Thread Andreas Färber
Hello, Am 05.11.2010 um 17:35 schrieb John Morrissey: qemu uses prctl() to set its process title. I bumped up against prctl()'s 16-character limit recently, when adding process title support to libvirt[1][2]. The attached patch overwrites argv instead. Linux seems to maintain the length o

[Qemu-devel] Re: [sparc] Unassigned mem read access on writing?

2010-11-07 Thread Blue Swirl
On Fri, Nov 5, 2010 at 5:48 PM, Artyom Tarasenko wrote: > Trying to boot Solaris 2.1 I get the following error message: > > Unassigned mem read access of 1 byte to 0768 from f0014768 > qemu: fatal: Trap 0x29 while interrupts disabled, Error state > pc: f0014768  npc: f001476c > > Looks

Re: [Qemu-devel] [PATCH v2 RESEND] pc: e820 qemu_cfg tables need to be packed

2010-11-07 Thread Blue Swirl
On Fri, Nov 5, 2010 at 9:40 PM, Alex Williamson wrote: > We can't let the compiler define the alignment for qemu_cfg data. Actually, whole e820_table implementation seems to be buggy. The structure may not be passed directly to fw_cfg interface without endianness conversions, otherwise the emulat

[Qemu-devel] [PATCH] HDA: Honor WAKEEN bits when deciding to raise an interrupt on codec status

2010-11-07 Thread François Revol
From 2e6c61248c76d3f0c511658b9f34660d034703a4 Mon Sep 17 00:00:00 2001 HDA: Honor WAKEEN bits when deciding to raise an interrupt on codec status change. This prevents an interrupt storm with the Haiku HDA driver which does not handle codec status changes in the irq handler. Signed-off-by: Fra

[Qemu-devel] [PATCH] Overwrite argv to set process title, eliminating 16-character prctl() limit.

2010-11-07 Thread John Morrissey
Linux seems to maintain the length of the original args, even when the new args are shorter and NULL-terminated, so the trailing whitespace in ps(1) output is probably unavoidable. I've seen the same result with other daemons that overwrite argv. Signed-off-by: John Morrissey --- os-posix.c

[Qemu-devel] [PATCH] Initial implementation of a mpeg1 layer2 streaming audio driver.

2010-11-07 Thread François Revol
Initial implementation of a mpeg1 layer2 streaming audio driver. It is based on the twolame library . It allows one to listen to the audio produced by a VM from an mp3 http streaming client. I just noticed esdaudio.c which I used as template on was under BSD licence, whic

[Qemu-devel] Re: [SeaBIOS] [PATCH v3 0/2] pciinit: fix overflow when bar allocation

2010-11-07 Thread Kevin O'Connor
On Thu, Oct 28, 2010 at 03:54:34PM +0900, Isaku Yamahata wrote: > Changes v2 -> v3: > - use [first, last] instead of [start, end) > > Changes v1 -> v2: > - add comment. > > Patch description: > This patch set fixes PCI bar allocation when bar overflow occured. > I checked if pmm_alloc facility ca

Re: [Qemu-devel] [PATCH] HDA: Honor WAKEEN bits when deciding to raise an interrupt on codec status

2010-11-07 Thread malc
On Sun, 7 Nov 2010, Fran?ois Revol wrote: > From 2e6c61248c76d3f0c511658b9f34660d034703a4 Mon Sep 17 00:00:00 2001 > > HDA: Honor WAKEEN bits when deciding to raise an interrupt on codec > status change. This prevents an interrupt storm with the Haiku HDA > driver which does not handle codec st

[Qemu-devel] sparc32 "bneg,a" bug?

2010-11-07 Thread Artyom Tarasenko
Can it be that bneg,a branches unconditionally, or annuls unconditionally? 0xf0071520: subcc %g3, %g2, %g3 => 0xf0071524: bneg,a 0xf007152c 0xf0071528: clr %g3 0xf007152c: st %g3, [ %i0 + 0x58 ] (gdb) info registers g3 psr g3 0x18 24 psr0x4000ae7

[Qemu-devel] Re: [sparc] Unassigned mem read access on writing?

2010-11-07 Thread Artyom Tarasenko
On Sun, Nov 7, 2010 at 3:56 PM, Blue Swirl wrote: > On Fri, Nov 5, 2010 at 5:48 PM, Artyom Tarasenko wrote: >> Trying to boot Solaris 2.1 I get the following error message: >> >> Unassigned mem read access of 1 byte to 0768 from f0014768 >> qemu: fatal: Trap 0x29 while interrupts disa

Re: [Qemu-devel] [PATCH] Initial implementation of a mpeg1 layer2 streaming audio driver.

2010-11-07 Thread malc
On Sun, 7 Nov 2010, Fran?ois Revol wrote: Please CC audio related stuff to audio maintainer. > Initial implementation of a mpeg1 layer2 streaming audio driver. > It is based on the twolame library . > It allows one to listen to the audio produced by a VM from an mp3 http

Re: [Qemu-devel] [PATCH] Initial implementation of a mpeg1 layer2 streaming audio driver.

2010-11-07 Thread François Revol
Le 7 nov. 2010 à 19:09, malc a écrit : > On Sun, 7 Nov 2010, Fran?ois Revol wrote: > > Please CC audio related stuff to audio maintainer. And that'd be you according to MAINTAINERS ? >> +static const char http_header[] = "HTTP/1.1 200 OK\r\nServer: >> QEMU\r\nContent-Type: audio/mpeg\r\n\r\n"

Re: [Qemu-devel] [PATCH] Overwrite argv to set process title, eliminating 16-character prctl() limit.

2010-11-07 Thread Andreas Färber
Am 07.11.2010 um 16:44 schrieb John Morrissey: Linux seems to maintain the length of the original args, even when the new args are shorter and NULL-terminated, so the trailing whitespace in ps(1) output is probably unavoidable. I've seen the same result with other daemons that overwrite ar

Re: [Qemu-devel] [PATCH] Initial implementation of a mpeg1 layer2 streaming audio driver.

2010-11-07 Thread malc
On Sun, 7 Nov 2010, Fran?ois Revol wrote: > > Le 7 nov. 2010 ? 19:09, malc a ?crit : > > > On Sun, 7 Nov 2010, Fran?ois Revol wrote: > > > > Please CC audio related stuff to audio maintainer. > > And that'd be you according to MAINTAINERS ? > > >> +static const char http_header[] = "HTTP/1.1

[Qemu-devel] Re: [PATCH] Overwrite argv to set process title, eliminating 16-character prctl() limit.

2010-11-07 Thread John Morrissey
On Sun, Nov 07, 2010 at 07:27:22PM +0100, Andreas Färber wrote: > Am 07.11.2010 um 16:44 schrieb John Morrissey: > >@@ -149,20 +145,24 @@ char *os_find_datadir(const char *argv0) > >#undef SHARE_SUFFIX > >#undef BUILD_SUFFIX > > > >-void os_set_proc_name(const char *s) > >+void os_set_proc_name(int

[Qemu-devel] Re: sparc32 "bneg,a" bug?

2010-11-07 Thread Blue Swirl
On Sun, Nov 7, 2010 at 5:22 PM, Artyom Tarasenko wrote: > Can it be that bneg,a branches unconditionally, or annuls unconditionally? > >   0xf0071520:  subcc  %g3, %g2, %g3 > => 0xf0071524:  bneg,a   0xf007152c >   0xf0071528:  clr  %g3 >   0xf007152c:  st  %g3, [ %i0 + 0x58 ] > (gdb) info registe

Re: [Qemu-devel] [PATCH] Overwrite argv to set process title, eliminating 16-character prctl() limit.

2010-11-07 Thread Torsten Förtsch
Hi, On Sunday, November 07, 2010 16:44:12 John Morrissey wrote: > -if (prctl(PR_SET_NAME, name)) { > -perror("unable to change process name"); > -exit(1); > -} > + > +last_argv_byte = argv[argc - 1] + strlen(argv[argc - 1]); > + > +len = snprintf(argv[0], last_argv_

[Qemu-devel] Re: sparc32 "bneg,a" bug?

2010-11-07 Thread Artyom Tarasenko
On Sun, Nov 7, 2010 at 9:32 PM, Blue Swirl wrote: > On Sun, Nov 7, 2010 at 5:22 PM, Artyom Tarasenko wrote: >> Can it be that bneg,a branches unconditionally, or annuls unconditionally? >> >>   0xf0071520:  subcc  %g3, %g2, %g3 >> => 0xf0071524:  bneg,a   0xf007152c >>   0xf0071528:  clr  %g3 >>

[Qemu-devel] [PATCH 1/2] rtl8139: add vlan tag insertion

2010-11-07 Thread Benjamin Poirier
Add support to the emulated hardware to add vlan tags in packets going from the guest to the network. Signed-off-by: Benjamin Poirier Cc: Igor V. Kovalenko --- hw/rtl8139.c | 46 +++--- 1 files changed, 35 insertions(+), 11 deletions(-) diff --git a/hw

[Qemu-devel] [PATCH 2/2] rtl8139: add vlan tag extraction

2010-11-07 Thread Benjamin Poirier
Add support to the emulated hardware to remove vlan tags in packets going from the network to the guest. Signed-off-by: Benjamin Poirier Cc: Igor V. Kovalenko -- AFAIK, extraction is optional to get vlans working. The driver requests rx detagging but should not assume that it was done. Under L

[Qemu-devel] [Try2][PATCH] Initial implementation of a mpeg1 layer2 streaming audio driver.

2010-11-07 Thread François Revol
Initial implementation of a mpeg1 layer2 streaming audio driver. 2nd try: Style & other fixes from malc's comments. It is based on the twolame library . Since twolame is very similar to lame (on purpose), one might easily create a lame version from it for better quality. I

[Qemu-devel] Re: AIX emulated on x86 host

2010-11-07 Thread Andreas Färber
Am 30.10.2010 um 14:49 schrieb Andreas Schwab: Andreas Färber writes: * Any advice on how to get GCC 4.3+ powerpc64-linux cross-compilers built [2] would be appreciated. We could then better rule out compiler bugs. You don't need any of the target libraries, just the compiler, since y

[Qemu-devel] Re: AIX emulated on x86 host

2010-11-07 Thread Andreas Schwab
Andreas Färber writes: > You wouldn't happen to know what to do about an "undefined reference to > _GLOBAL_OFFSET_TABLE_'"? That symbol is generated by the linker, so it depends on how you call it. > Output format is binary and supposed to be relocatable (tried -fpic and > -fpie; for an RTAS bl

[Qemu-devel] Re: [Try2][PATCH] Initial implementation of a mpeg1 layer2 streaming audio driver.

2010-11-07 Thread François Revol
Le 8 nov. 2010 à 01:02, malc a écrit : >> + >> +static void qtwolame_listen_read(void *opaque) > > No space here. ? >> +if (csock != -1) { >> +twolame->sock = csock; >> +dolog ("Accepted peer\n"); >> +if (write (twolame->sock, http_header, sizeof(http_header) - 1) <

Re: [Qemu-devel] [PATCH 0/3] v4 Decouple block device removal from device removal

2010-11-07 Thread Ryan Harper
* Markus Armbruster [2010-11-06 04:19]: > Ryan Harper writes: > > > * Markus Armbruster [2010-11-05 11:11]: > >> Ryan Harper writes: > >> > >> > * Markus Armbruster [2010-11-05 08:28]: > >> >> I'd be fine with any of these: > >> >> > >> >> 1. A new command "device_disconnet ID" (or similar

[Qemu-devel] Re: [PATCH] [Seabios] Over 4GB address ranges for 64bit PCI BARs

2010-11-07 Thread Alexey Korolev
Hi, > Hi. > The current BAR allocation doesn't check overflow and some patches > are floating around which aren't merged yet. > There are several issues. > > - overflow check > This should be fixed. > Some patches are proposed. None hasn't been merged yet. > Your patch also addresses this is

[Qemu-devel] [PATCH] pc: Fix e820 fw_cfg for big endian

2010-11-07 Thread Alex Williamson
Signed-off-by: Alex Williamson --- Compile tested only. Only current user is kvm, no cross-arch users. hw/pc.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/hw/pc.c b/hw/pc.c index 0264e3d..cc8ec14 100644 --- a/hw/pc.c +++ b/hw/pc.c @@ -467,19 +467,19 @

[Qemu-devel] Re: [Try2][PATCH] Initial implementation of a mpeg1 layer2 streaming audio driver.

2010-11-07 Thread malc
On Mon, 8 Nov 2010, Fran?ois Revol wrote: > > Le 8 nov. 2010 ? 01:02, malc a ?crit : > [..snip..] > > > > And can you, please, elaborate some more on usage scenarios of this thing? > > > cf. > http://dev.haiku-os.org/browser/haiku/trunk/3rdparty/mmu_man/onlinedemo/haiku.php Sorry my PHP sk

[Qemu-devel] Re: [PATCH] [Seabios] Over 4GB address ranges for 64bit PCI BARs

2010-11-07 Thread Isaku Yamahata
On Mon, Nov 08, 2010 at 04:35:38PM +1300, Alexey Korolev wrote: > > - >4GB 64bit bar allocation > > Your patche tries to address this issue. But it breaks PCI-to-PCI > > bridge filtering support. > Hmm, it is quite possible, as we don't know a lot about seabios PCI-to-PCI > bridge filtering su

[Qemu-devel] Re: [Try2][PATCH] Initial implementation of a mpeg1 layer2 streaming audio driver.

2010-11-07 Thread François Revol
Le 8 nov. 2010 à 04:57, malc a écrit : >>> And can you, please, elaborate some more on usage scenarios of this thing? >> >> cf. >> http://dev.haiku-os.org/browser/haiku/trunk/3rdparty/mmu_man/onlinedemo/haiku.php > > Sorry my PHP skills have lapsed aeons ago. Sorry I don't have it installed on

Re: [Qemu-devel] [PATCH] virtio-9p: fix build on !CONFIG_UTIMENSAT v2

2010-11-07 Thread M. Mohan Kumar
> This patch introduce a fallback mechanism for old systems that do not > support utimensat. This will fix build failure with following warnings: > > hw/virtio-9p-local.c: In function 'local_utimensat': > hw/virtio-9p-local.c:479: warning: implicit declaration of function > 'utimensat' hw/virtio-