Re: [SeaBIOS] [Qemu-devel] QEMU regression problems

2010-04-18 Thread Gerhard Wiesinger
Kevin O'Connor wrote: On Tue, Apr 13, 2010 at 02:26:10PM +0800, Roy Tam wrote: 2010/4/12 Gerhard Wiesinger : 3.) There is also a problem with the reported base memory under QEMM386 (HIMEM.SYS and EMM386.EXE is correct here). It is 646kB instead of 640kB. Therefore base memory test fails

[Qemu-devel] Re: Problem with QEMU on KVM

2010-04-18 Thread Gerhard Wiesinger
On Tue, 13 Apr 2010, Jan Kiszka wrote: Mulyadi Santosa wrote: Hi Jamie... On Mon, Apr 12, 2010 at 19:07, Jamie Lokier wrote: There are various -no-kvm-XXX options to try: -no-kvm-irqchip disable KVM kernel mode PIC/IOAPIC/LAPIC -no-kvm-pit disable KVM kernel mode PIT -no-kvm-pit-reinjec

[Qemu-devel] Inserting Memory Watch points

2010-04-18 Thread Taimoor Mirza
Hi all, I've developed an application for ARM intergrator and it is running find under QEMU. I am trying to redirect my serial data to my custom emulated device. For that I've inserted memory watch point at Pl011 UART controllers Data Register so that whenever some access is made to this regi

Re: [Qemu-devel] [PATCH][STABLE] Add qerror message if the 'change' target filename can't be opened

2010-04-18 Thread Ryan Harper
* Aurelien Jarno [2010-04-18 15:55]: > On Fri, Apr 09, 2010 at 07:46:38AM -0500, Ryan Harper wrote: > > Currently when using the change command to switch the file in the cd drive > > the command doesn't complain if the file doesn't exit or can't be opened > > and the drive keeps the existing image

Re: [Qemu-devel] wrong behaviour of caps lock

2010-04-18 Thread Jamie Lokier
Benjamin Drung wrote: > -/* SDL does not send the key up event, so we generate it */ Was the original comment just plain wrong? > -kbd_put_keycode(keycode); > -kbd_put_keycode(keycode | 0x80); > +if (ev->type == SDL_KEYUP) { > +kbd_put_keycode(keycode |

Re: [Qemu-devel] [PATCH, resend] [STABLE] Sync OSS_GETVERSION handling with head

2010-04-18 Thread Andreas Färber
Am 18.04.2010 um 22:52 schrieb Aurelien Jarno: On Sun, Apr 11, 2010 at 06:17:32PM +0200, Juergen Lock wrote: As suggested by Andreas Färber, here is a cumulative patch that syncs OSS_GETVERSION handling with head by merging the following commits: 1. oss: issue OSS_GETVERSION ioctl only when ne

Re: [Qemu-devel] Re: [PATCH][STABLE] fix CPUID vendor override

2010-04-18 Thread Andre Przywara
Aurelien Jarno wrote: On Sun, Apr 11, 2010 at 09:49:40PM +0200, Andre Przywara wrote: Avi Kivity wrote: On 04/11/2010 10:21 PM, Andre Przywara wrote: the meaning of vendor_override is actually the opposite of how it is currently used :-( Fix it to allow KVM to export the non-native CPUID vendo

Re: [Qemu-devel] [PATCH 05/21] tcg-i386: Tidy bswap operations.

2010-04-18 Thread Aurelien Jarno
On Tue, Apr 13, 2010 at 04:33:59PM -0700, Richard Henderson wrote: > Define OPC_BSWAP. Factor opcode emission to separate functions. > Use bswap+shift to implement 16-bit swap instead of a rolw; this > gets the proper zero-extension required by INDEX_op_bswap16_i32. This is not required by INDEX_

Re: [Qemu-devel] [PATCH, resend] [STABLE] Sync OSS_GETVERSION handling with head

2010-04-18 Thread Juergen Lock
On Sun, Apr 18, 2010 at 11:38:34PM +0200, Aurelien Jarno wrote: > On Sun, Apr 18, 2010 at 11:16:46PM +0200, Juergen Lock wrote: > > On Sun, Apr 18, 2010 at 10:52:22PM +0200, Aurelien Jarno wrote: > > > On Sun, Apr 11, 2010 at 06:17:32PM +0200, Juergen Lock wrote: > > > > As suggested by Andreas Fär

Re: [Qemu-devel] [PATCH 00/21][PULL]: QMP/Monitor queue

2010-04-18 Thread Aurelien Jarno
On Mon, Apr 05, 2010 at 05:33:41PM -0300, Luiz Capitulino wrote: > Hi Anthony, > > I was going to nag you to apply patches from the list, but as I'm maintaining > a QMP/Monitor queue to test them, I thought it would be a good idea to nag you > in the form of a git pull :) > > All the patches i

Re: [Qemu-devel] Re: [PATCH][STABLE] fix CPUID vendor override

2010-04-18 Thread Aurelien Jarno
On Sun, Apr 11, 2010 at 09:49:40PM +0200, Andre Przywara wrote: > Avi Kivity wrote: >> On 04/11/2010 10:21 PM, Andre Przywara wrote: >>> the meaning of vendor_override is actually the opposite of how it >>> is currently used :-( >>> Fix it to allow KVM to export the non-native CPUID vendor if >>> e

Re: [Qemu-devel] [PATCH, resend] [STABLE] Sync OSS_GETVERSION handling with head

2010-04-18 Thread Aurelien Jarno
On Sun, Apr 18, 2010 at 11:16:46PM +0200, Juergen Lock wrote: > On Sun, Apr 18, 2010 at 10:52:22PM +0200, Aurelien Jarno wrote: > > On Sun, Apr 11, 2010 at 06:17:32PM +0200, Juergen Lock wrote: > > > As suggested by Andreas Färber, here is a cumulative patch that syncs > > > OSS_GETVERSION handling

Re: [Qemu-devel] [PATCH, resend] [STABLE] Sync OSS_GETVERSION handling with head

2010-04-18 Thread Juergen Lock
On Sun, Apr 18, 2010 at 10:52:22PM +0200, Aurelien Jarno wrote: > On Sun, Apr 11, 2010 at 06:17:32PM +0200, Juergen Lock wrote: > > As suggested by Andreas Färber, here is a cumulative patch that syncs > > OSS_GETVERSION handling with head by merging the following commits: > > > > 1. oss: issue OS

[Qemu-devel] [PATCH] [PPC] [KVM] Make iothread work

2010-04-18 Thread Alexander Graf
When running with --enable-io-thread the timer we have doesn't help, because it doesn't wake up the CPU thread. So instead we need to actually kick it. While at it I refined the logic a bit to not dumbly trigger a timer every 500ms, but rather do it more often after an interrupt got injected. If t

Re: [Qemu-devel] [PATCH, resend] [STABLE] Sync OSS_GETVERSION handling with head

2010-04-18 Thread Aurelien Jarno
On Sun, Apr 11, 2010 at 06:17:32PM +0200, Juergen Lock wrote: > As suggested by Andreas Färber, here is a cumulative patch that syncs > OSS_GETVERSION handling with head by merging the following commits: > > 1. oss: issue OSS_GETVERSION ioctl only when needed >6d246526ce3c145b2831285def6983f5d

Re: [Qemu-devel] [PATCH][STABLE] Add qerror message if the 'change' target filename can't be opened

2010-04-18 Thread Aurelien Jarno
On Fri, Apr 09, 2010 at 07:46:38AM -0500, Ryan Harper wrote: > Currently when using the change command to switch the file in the cd drive > the command doesn't complain if the file doesn't exit or can't be opened > and the drive keeps the existing image. This patch adds a qerror_report > call to p

[Qemu-devel] [PATCH 2/2 v4] Two new monitor commands: 'info keyboard', 'keyboard_set'

2010-04-18 Thread Shahar Havivi
Adding ability to handle which keyboard qemu will use and to see which keyboard are currently available. $ info keyboard $ keyboard_set Signed-off-by: Shahar Havivi --- console.h |5 ++ hw/usb-hid.c|6 +++ input.c | 111 +++

[Qemu-devel] [PATCH 1/2 v4] Support for multiple keyboard devices

2010-04-18 Thread Shahar Havivi
Patch add QEMUPutKbdEntry structure - handling each keyboard entry, the structure handled by qemu tail queue. Adding a new keyboard add to the list and select it, removing keyboard select the previous keyboard in list. Signed-off-by: Shahar Havivi --- console.h| 14 -

[Qemu-devel] [PATCH 0/2 v4] Qemu support for multiple keyboard devices

2010-04-18 Thread Shahar Havivi
v4: Patch #1 Xen keyboard, added QEMUPutKbdEntry to XenInput in order to properly remove when disconnected (Juan Quintela comment). Rename mouse_index to index (Markus Armbruster comment). Patch #2 Remove un-used variable (Juan Quintela comment). Split do_keyboard_set()

[Qemu-devel] Re: Guest latency issues due to bdrv_check_byte_request

2010-04-18 Thread Jan Kiszka
Stefan Hajnoczi wrote: > Thanks Christoph. > > Cached getlength with pread/pwrite: > % time seconds usecs/call callserrors syscall > -- --- --- - - > 96.971.760111 11893 148 4 futex > 1.610.029209

Re: [Qemu-devel] Guest latency issues due to bdrv_check_byte_request

2010-04-18 Thread Christoph Hellwig
You should split this up into two patches - one for the the compat AIO implementation and one for the getlength caching.

Re: [Qemu-devel] Re: [PATCH 2/3] target-sparc: Free instruction temporaries.

2010-04-18 Thread Richard Henderson
On 04/17/2010 01:41 PM, Blue Swirl wrote: > Yes, but we would still gain the small optimizations for add by 0, and > with 0x etc. in tcg-op.h. Sparc QEMU target generates a lot of > those because of poor constant formation choices made by the guest > compilers. Another thing that gets fixe

[Qemu-devel] Re: [patch 02/02] sparc32 use empty_slot for missing RAM v1

2010-04-18 Thread Blue Swirl
Thanks, applied both. On 4/18/10, Artyom Tarasenko wrote: > use empty_slot device for the RAM which is not installed > > Models without ECC don't trap when missing ram is accessed. > > v0->v1 compile only once and fix indentation > > Signed-off-by: Artyom Tarasenko > --- > diff --git a/Make

[Qemu-devel] [RFC, PATCH 2/2] bt-sdp: Fix if statement with empty body, spotted by clang

2010-04-18 Thread Blue Swirl
Fix clang error: CCbt-sdp.o /src/qemu/hw/bt-sdp.c:174:17: error: if statement has empty body [-Wempty-body] if (len > 1); However, fixing this means that some code that was previously ignored by the compiler now gets compiled, resulting in this error: CCbt-sdp.o cc1: warnings being

[Qemu-devel] [RFC, PATCH 1/2] bt-l2cap: fix if statement with empty body, spotted by clang

2010-04-18 Thread Blue Swirl
Fix clang error: CCbt-l2cap.o /src/qemu/hw/bt-l2cap.c:1000:41: error: if statement has empty body [-Wempty-body] /* TODO: Signal an error? */; This means that l2cap_sframe_in() may now get called. Signed-off-by: Blue Swirl --- hw/bt-l2cap.c |6 +++--- 1 files changed, 3 in