[Qemu-devel] Re: PC keyboard emulation (was: Regression: more 0.12 regression (SeaBIOS related?))

2010-03-14 Thread Kevin O'Connor
On Mon, Mar 15, 2010 at 02:47:16AM +, Jamie Lokier wrote: > Kevin O'Connor wrote: > > On my epia-cn, a read of port 0x60 immiediately clears the OBF flag of > > the status register. > > Ok. As in like this, in rapid succession? > >inb(0x64) -> OBF is set >inb(0x60) -> keyboard byte

[Qemu-devel] Re: PC keyboard emulation (was: Regression: more 0.12 regression (SeaBIOS related?))

2010-03-14 Thread Jamie Lokier
Kevin O'Connor wrote: > On Fri, Mar 12, 2010 at 11:24:27PM +, Jamie Lokier wrote: > > The DOS coding method brought up in this thread, resulting > > in two reads of port 0x60, is quite common. It works on all real PCs, > > and correct emulation must handle it. > > > > I'm not sure if reading

Re: [Qemu-devel] qemu-system-sh4 broken again.

2010-03-14 Thread Jamie Lokier
Rob Landley wrote: > I found out that "-serial stdio" is apparently trying to open /dev/stdio, > which Ubuntu 9.04 hasn't got. If I say -serial /dev/tty it works from the > command line (but not in scripts). I don't think any Linux systems have /dev/stdio. /dev/stdin, /dev/stdout, /dev/stderr,

[Qemu-devel] [PATCH] [PATCH] usb: class specific interface requests

2010-03-14 Thread Max Reitz
Mass Storage Reset and Get Max LUN are class specific requests, but they were not marked as such in hw/usb-msd.c, moved therefore ClassInterfaceRequest and ClassInterfaceOutRequest from hw/usb-net.c to hw/usb.h. Furthermore there was a problem in hw/usb-ohci.c when using DEBUG concerning systems wh

[Qemu-devel] Stellaris lm3s811 board init

2010-03-14 Thread Benjamin Bagland
Hi, I'm having an issue while trying to use the lm3s811 board emulation with qemu 0.12.3 or git. It works fine however with qemu-0.11.1. The problem comes from hw/armv7m.c, around line 231, the initial PC and SP are set by reading from physical memory, however the rom data is not copied in physic

Re: [Qemu-devel] qemu-system-sh4 broken again.

2010-03-14 Thread Rob Landley
On Sunday 14 March 2010 16:28:32 Aurelien Jarno wrote: > On Sat, Mar 13, 2010 at 05:11:43PM -0600, Rob Landley wrote: > > I found out that "-serial stdio" is apparently trying to open /dev/stdio, > > which Ubuntu 9.04 hasn't got. If I say -serial /dev/tty it works from > > the command line (but no

Re: [Qemu-devel] Re: linux-user issues

2010-03-14 Thread malc
On Sun, 14 Mar 2010, malc wrote: > On Sun, 14 Mar 2010, Paul Brook wrote: > > > > On Sat, 13 Mar 2010, Paul Brook wrote: > > > > > a) elfload.c:859 > > > > > > > > > >#define TARGET_ELF_PAGESTART(_v) ((_v) & ~(unsigned > > > > > long)(TARGET_ELF_EXEC_PAGESIZE-1)) > > > > > > > > > >This

Re: [Qemu-devel] [PATCH] load_elf: replace the address addend by a translation function

2010-03-14 Thread Aurelien Jarno
On Sun, Mar 14, 2010 at 10:55:48PM +, Paul Brook wrote: > > Thanks for the review. > > > > On Sun, Mar 14, 2010 at 09:59:44PM +, Paul Brook wrote: > > > > The patch also convert all machines that have an addend, simplify the > > > > PowerPC kernel loading > > > > > > The new PPC code looks

Re: [Qemu-devel] [PATCH] qemu-img rebase: Document -f option

2010-03-14 Thread Aurelien Jarno
On Fri, Mar 12, 2010 at 02:03:49PM +0100, Kevin Wolf wrote: > The option was implemented in e53dbee0, but I forgot documenting it. > > Signed-off-by: Kevin Wolf Thanks, applied. > --- > qemu-img-cmds.hx |4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/qemu-img-

Re: [Qemu-devel] [PATCH] load_elf: replace the address addend by a translation function

2010-03-14 Thread Paul Brook
> Thanks for the review. > > On Sun, Mar 14, 2010 at 09:59:44PM +, Paul Brook wrote: > > > The patch also convert all machines that have an addend, simplify the > > > PowerPC kernel loading > > > > The new PPC code looks like it may break images > 16M in size. > > The idea was to get rid of t

[Qemu-devel] [PATCH v2] load_elf: replace the address addend by a translation function

2010-03-14 Thread Aurelien Jarno
A few machines need to translate the ELF header addresses into physical addresses. Currently the only possibility is to add a value to the addresses. This patch replaces the addend argument by and a translation function and an opaque passed to the function. A NULL function does not translate the a

Re: [Qemu-devel] [PATCH] load_elf: replace the address addend by a translation function

2010-03-14 Thread Aurelien Jarno
Thanks for the review. On Sun, Mar 14, 2010 at 09:59:44PM +, Paul Brook wrote: > > The patch also convert all machines that have an addend, simplify the > > PowerPC kernel loading > > The new PPC code looks like it may break images > 16M in size. The idea was to get rid of the most signific

Re: [Qemu-devel] [PATCH] load_elf: replace the address addend by a translation function

2010-03-14 Thread Paul Brook
> The patch also convert all machines that have an addend, simplify the > PowerPC kernel loading The new PPC code looks like it may break images > 16M in size. > and fix the MIPS kernel loading using this new > feature. I'm fairly sure the MIPS routines are not board specific, so it'd be nice i

Re: [Qemu-devel] Re: [RESEND][PATCH][STABLE] Don't set default monitor when there is a mux'ed one

2010-03-14 Thread Aurelien Jarno
On Sat, Mar 13, 2010 at 04:58:36PM +0100, Jan Kiszka wrote: > Aurelien Jarno wrote: > > On Mon, Mar 08, 2010 at 10:44:18AM -0600, Anthony Liguori wrote: > >> On 03/07/2010 04:28 AM, Jan Kiszka wrote: > >>> This fixes eg. "-nographic -serial mon:stdio [-serial ...]". > >>> > >>> Signed-off-by: Jan K

Re: [Qemu-devel] qemu-system-sh4 broken again.

2010-03-14 Thread Aurelien Jarno
On Sat, Mar 13, 2010 at 05:11:43PM -0600, Rob Landley wrote: > On Saturday 13 March 2010 15:11:03 Rob Landley wrote: > > On Saturday 13 March 2010 04:26:47 Aurelien Jarno wrote: > > > On Sat, Mar 13, 2010 at 01:51:35AM -0600, Rob Landley wrote: > > > > Using qemu-system-sh4, this commit: > > > > >

[Qemu-devel] [PATCH] load_elf: replace the address addend by a translation function

2010-03-14 Thread Aurelien Jarno
A few machines need to translate the ELF header addresses into physical addresses. Currently the only possibility is to add a value to the addresses. This patch replaces the addend argument by and a translation function and an opaque passed to the function. A NULL function does not translate the a

[Qemu-devel] [PATCH] Restore terminal monitor attributes - addition

2010-03-14 Thread Shahar Havivi
Patch 2d753894c7553d6a05e8fdbed5f4704398919a35 was missing this check, when running monitor as /dev/tty and other serial device, i.e: qemu -monitor /dev/tty -serial /dev/pts/1 Without this patch any serial device will override the monitor stored attributes. (monitor is called in main() before an

Re: [Qemu-devel] [PATCH] target-mips: Fix 32 bit mode (wrong size of physical addresses)

2010-03-14 Thread Aurelien Jarno
On Sun, Mar 14, 2010 at 05:28:51PM +0100, Stefan Weil wrote: > Paul Brook schrieb: > >> Commit 30724e758a21ba9f807efafe268626bd479db9de breaks > >> malta (and other) mips 32 bit emulation. > >> > >> Fixing the physical address size for 32 bit machines > >> makes it work again. > > > >> -#define TAR

Re: [Qemu-devel] [PATCH 5/6] Implement multi-level page tables.

2010-03-14 Thread Aurelien Jarno
On Sun, Mar 14, 2010 at 03:02:11PM +, Paul Brook wrote: > > Richard Henderson schrieb: > > > Define L1_MAP_ADDR_SPACE_BITS to be either the virtual address size > > > (in user mode) or physical address size (in system mode), and use > > > that to size l1_map. This rewrites page_find_alloc, pag

Re: [Qemu-devel] [PATCH] target-mips: Fix 32 bit mode (wrong size of physical addresses)

2010-03-14 Thread Stefan Weil
Paul Brook schrieb: >> Commit 30724e758a21ba9f807efafe268626bd479db9de breaks >> malta (and other) mips 32 bit emulation. >> >> Fixing the physical address size for 32 bit machines >> makes it work again. > >> -#define TARGET_PHYS_ADDR_SPACE_BITS 36 >> +#define TARGET_PHYS_ADDR_SPACE_BITS 32 > > Th

[Qemu-devel] Re: linux-user issues

2010-03-14 Thread malc
On Sun, 14 Mar 2010, Paul Brook wrote: > > On Sat, 13 Mar 2010, Paul Brook wrote: > > > > a) elfload.c:859 > > > > > > > >#define TARGET_ELF_PAGESTART(_v) ((_v) & ~(unsigned > > > > long)(TARGET_ELF_EXEC_PAGESIZE-1)) > > > > > > > >This means that for 64bit guest on a 32bit host the _v's

[Qemu-devel] Re: linux-user issues

2010-03-14 Thread Paul Brook
> On Sat, 13 Mar 2010, Paul Brook wrote: > > > a) elfload.c:859 > > > > > >#define TARGET_ELF_PAGESTART(_v) ((_v) & ~(unsigned > > > long)(TARGET_ELF_EXEC_PAGESIZE-1)) > > > > > >This means that for 64bit guest on a 32bit host the _v's value is > > >silently reduced to 32bit, the cast

Re: [Qemu-devel] [PATCH 5/6] Implement multi-level page tables.

2010-03-14 Thread Paul Brook
> Richard Henderson schrieb: > > Define L1_MAP_ADDR_SPACE_BITS to be either the virtual address size > > (in user mode) or physical address size (in system mode), and use > > that to size l1_map. This rewrites page_find_alloc, page_flush_tb, > > and walk_memory_regions. > > > > Use TARGET_PHYS_ADD

[Qemu-devel] Cortex m3 initialisation

2010-03-14 Thread Benjamin Bagland
Hi, I'm having an issue while trying to use the lm3s811 board emulation with qemu 0.12.3 or git. It works fine however with qemu-0.11.1. The problem comes from hw/armv7m.c, around line 231, the initial PC and SP are set by reading from physical memory, however the rom data is not copied in physic

Re: [Qemu-devel] [PATCH] target-mips: Fix 32 bit mode (wrong size of physical addresses)

2010-03-14 Thread Paul Brook
> Commit 30724e758a21ba9f807efafe268626bd479db9de breaks > malta (and other) mips 32 bit emulation. > > Fixing the physical address size for 32 bit machines > makes it work again. >-#define TARGET_PHYS_ADDR_SPACE_BITS 36 >+#define TARGET_PHYS_ADDR_SPACE_BITS 32 This is almost certainly the wrong

[Qemu-devel] [PATCH v2 1/2] read-only: minor cleanup

2010-03-14 Thread Naphtali Sprei
Really use read-only flags for opening the file when asked for read-only Signed-off-by: Naphtali Sprei --- qemu-nbd.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/qemu-nbd.c b/qemu-nbd.c index eac0c21..a393583 100644 --- a/qemu-nbd.c +++ b/qemu-nbd.c @@ -258,6 +258,7

[Qemu-devel] Re: manpage errors

2010-03-14 Thread Blue Swirl
On 3/13/10, Vagrant Cascadian wrote: > On Sat, Mar 13, 2010 at 01:05:03PM +0200, Blue Swirl wrote: > > On 3/12/10, Vagrant Cascadian wrote: > > > i found this spelling typo and the previous one by running lintian on > the qemu > > > packages i work on for debian: > > > > > > > http://li

Re: [Qemu-devel] spelling typo (compatibilty) in hw/fw_cfg.c

2010-03-14 Thread Blue Swirl
On 3/13/10, Vagrant Cascadian wrote: > On Sat, Mar 13, 2010 at 01:05:03PM +0200, Blue Swirl wrote: > > On 3/12/10, Vagrant Cascadian wrote: > > > here's a trivial patch to fix the spelling of "compatibility": > > > > Please add a Signed-off-by: line. > > > hope this is what you're looking for