[Qemu-devel] [Bug 688085] Re: Guest kernel hang during boot when KVM is active on i386 host

2011-01-09 Thread Bug Watch Updater
Launchpad has imported 7 comments from the remote bug at http://bugs.meego.com/show_bug.cgi?id=10075. If you reply to an imported comment from within Launchpad, your comment will be sent to the remote bug automatically. Read more about Launchpad's inter-bugtracker facilities at https://help.launch

Re: [Qemu-devel] [PATCH 3/7] linux-user: Implement FS_IOC_FIEMAP ioctl

2011-01-09 Thread Peter Maydell
On 9 January 2011 19:08, Blue Swirl wrote: > On Sun, Jan 9, 2011 at 5:47 PM, riku voipio wrote: >> On 01/09/2011 04:47 PM, Blue Swirl wrote: >>> This fails if the file doesn't exist: >>>   CC    i386-linux-user/syscall.o >>> /src/qemu/linux-user/syscall.c:86:26: error: linux/fiemap.h: No such >>>

Re: [Qemu-devel] Re: phys_page_find bug?

2011-01-09 Thread Bob Breuer
Blue Swirl wrote: > On Mon, Nov 8, 2010 at 6:55 PM, Artyom Tarasenko wrote: > >> On Fri, May 7, 2010 at 6:26 PM, Artyom Tarasenko >> wrote: >> >>> phys_page_find (exec.c) returns sometimes a page for addresses where >>> nothing is connected. >>> >>> One example, done with qemu-system-spar

Re: [Qemu-devel] [PATCH 3/3] tcg/arm: improve constant loading

2011-01-09 Thread Peter Maydell
On 9 January 2011 23:33, andrzej zaborowski wrote: > On 9 January 2011 23:40, Aurelien Jarno wrote: >> Note that binutils is not able to disassemble such an instruction and >> outputs in qemu.log something like: >> | 0x0108:  e3aa50ff  undefined instruction 0xe3aa50ff >> >> However what worri

[Qemu-devel] Re: [rfc] rewrite tcg-ia64 for auto-bundling

2011-01-09 Thread Aurelien Jarno
On Sun, Jan 09, 2011 at 03:24:44PM -0800, Richard Henderson wrote: > Something I noticed while working on that deposit patch for ia64 > was how annoying it is to bundle instructions by hand. And the > fact that it's been done incorrectly at least once. E.g. > > static inline void tcg_out_bswap64

Re: [Qemu-devel] Re: [PATCH 5/7] tcg-i386: Implement deposit operation.

2011-01-09 Thread Richard Henderson
On 01/09/2011 04:16 PM, Aurelien Jarno wrote: > The code being written now or latter doesn't change the question to know > if it is always possible to allocate one scratch register here on i386. Yes. Here there's only one register that needs to remain live. In the worst case, we'll spill one li

Re: [Qemu-devel] Re: [PATCH 5/7] tcg-i386: Implement deposit operation.

2011-01-09 Thread Aurelien Jarno
On Sun, Jan 09, 2011 at 02:55:13PM -0800, Richard Henderson wrote: > On 01/09/2011 01:53 PM, Aurelien Jarno wrote: > >> +if (inout == val) { > >> +TCGType type = rexw ? TCG_TYPE_I64 : TCG_TYPE_I32; > >> +TCGRegSet inuse = s->reserved_regs; > >> + > >> +tcg_regset_set_reg

[Qemu-devel] [Bug 700774] Re: sh7750.c:672: sh7750_mmct_writel: Assertion `0' failed.

2011-01-09 Thread Khem Raj
yes I backported these fixes on top of 0.12.5 and now I can boot the image successfully. thanks for quick turnaround. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/700774 Title: sh7750.c:672: sh775

Re: [Qemu-devel] [PATCH 3/3] tcg/arm: improve constant loading

2011-01-09 Thread andrzej zaborowski
On 9 January 2011 23:40, Aurelien Jarno wrote: > On Fri, Jan 07, 2011 at 04:56:32PM +0100, andrzej zaborowski wrote: >> On 7 January 2011 15:40, Aurelien Jarno wrote: >> > On Fri, Jan 07, 2011 at 01:52:25PM +0100, andrzej zaborowski wrote: >> >> On 6 January 2011 22:54, Aurelien Jarno wrote: >>

Re: [Qemu-devel] [RFC][PATCH] lsi53c895a: Update dnad when skipping MSGOUT bytes

2011-01-09 Thread Nicholas A. Bellinger
On Sat, 2011-01-08 at 20:20 +, Stefan Hajnoczi wrote: > Update not only dbc but also dnad when skipping bytes during the MSGOUT > phase. Previously only dbc was updated which is probably wrong and > could lead to bogus message codes being read. > > Signed-off-by: Stefan Hajnoczi > --- > I do

[Qemu-devel] [Bug 700774] Re: sh7750.c:672: sh7750_mmct_writel: Assertion `0' failed.

2011-01-09 Thread Aurelien Jarno
The problem there is that for some unknown reasons your kernel is flushing the iTLB through the memory-mmaped interface (which is not yet implemented) instead of using the MMUCR.TI bit. I was actually playing with the SH4 MMU code this week-end and wrote this missing part of the code, but I was un

Re: [Qemu-devel] Re: [PATCH 5/7] tcg-i386: Implement deposit operation.

2011-01-09 Thread Richard Henderson
On 01/09/2011 01:53 PM, Aurelien Jarno wrote: >> +if (inout == val) { >> +TCGType type = rexw ? TCG_TYPE_I64 : TCG_TYPE_I32; >> +TCGRegSet inuse = s->reserved_regs; >> + >> +tcg_regset_set_reg(inuse, inout); >> +val = tcg_reg_alloc(s, tcg_target_available_regs[ty

Re: [Qemu-devel] [PATCH 1/7] tcg: Define "deposit" as an optional operation.

2011-01-09 Thread Richard Henderson
On 01/09/2011 01:38 PM, Aurelien Jarno wrote: > The encoding of the constant part actually doesn't match the one in the > C function where two arguments are separated and not encoded. What about > adding a tcg_gen_op5ii_i32/64 function and having the two arguments > separated? I certainly didn't w

Re: [Qemu-devel] [PATCH 3/3] tcg/arm: improve constant loading

2011-01-09 Thread Aurelien Jarno
On Fri, Jan 07, 2011 at 04:56:32PM +0100, andrzej zaborowski wrote: > On 7 January 2011 15:40, Aurelien Jarno wrote: > > On Fri, Jan 07, 2011 at 01:52:25PM +0100, andrzej zaborowski wrote: > >> Hi, > >> > >> On 6 January 2011 22:54, Aurelien Jarno wrote: > >> > Improve constant loading in two way

[Qemu-devel] Re: [PATCH 4/7] tcg-ia64: Implement deposit operation.

2011-01-09 Thread Aurelien Jarno
On Fri, Jan 07, 2011 at 02:43:00PM -0800, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > tcg/ia64/tcg-target.c | 92 > + > tcg/ia64/tcg-target.h |2 + > 2 files changed, 94 insertions(+), 0 deletions(-) > > diff --git a/

[Qemu-devel] Re: [PATCH 5/7] tcg-i386: Implement deposit operation.

2011-01-09 Thread Aurelien Jarno
On Fri, Jan 07, 2011 at 02:43:01PM -0800, Richard Henderson wrote: > Special case deposits that are implementable with byte and word stores. > Otherwise implement with double-word shift plus rotates. > > Expose tcg_reg_alloc to the backend for allocation of scratch registers. > There's an edge con

Re: [Qemu-devel] [PATCH 1/7] tcg: Define "deposit" as an optional operation.

2011-01-09 Thread Aurelien Jarno
On Fri, Jan 07, 2011 at 02:42:57PM -0800, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > tcg/README| 14 ++ > tcg/tcg-op.h | 40 > tcg/tcg-opc.h |6 ++ > tcg/tcg.c | 13 + > 4 files change

Re: [Qemu-devel] [PATCH] tcg: fix typo in readme

2011-01-09 Thread Blue Swirl
Thanks, applied. On Sun, Jan 9, 2011 at 8:45 AM, Mike Frysinger wrote: > Signed-off-by: Mike Frysinger > --- >  tcg/README |    2 +- >  1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/tcg/README b/tcg/README > index 68d27ff..8ec6845 100644 > --- a/tcg/README > +++ b/tcg/README

Re: [Qemu-devel] [PATCH] tcg/README: Spelling fixes

2011-01-09 Thread Blue Swirl
Thanks, applied. On Fri, Jan 7, 2011 at 8:34 PM, Stefan Weil wrote: > Signed-off-by: Stefan Weil > --- >  tcg/README |    8 >  1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/tcg/README b/tcg/README > index 68d27ff..a2b69dd 100644 > --- a/tcg/README > +++ b/tcg/README

Re: [Qemu-devel] [PATCH] qemu-tech: Spelling fixes

2011-01-09 Thread Blue Swirl
Thanks, applied. On Fri, Jan 7, 2011 at 8:31 PM, Stefan Weil wrote: > Signed-off-by: Stefan Weil > --- >  qemu-tech.texi |    4 ++-- >  1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/qemu-tech.texi b/qemu-tech.texi > index 2e2a081..138e3ce 100644 > --- a/qemu-tech.texi > +++ b

Re: [Qemu-devel] [PATCH 1/4] qemu-doc: Clean whitespace

2011-01-09 Thread Blue Swirl
Thanks, applied all. On Fri, Jan 7, 2011 at 5:59 PM, Stefan Weil wrote: > Remove blanks at line endings. > > Signed-off-by: Stefan Weil > --- >  qemu-doc.texi |    4 ++-- >  1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/qemu-doc.texi b/qemu-doc.texi > index 7ce8999..21d8a82 1

Re: [Qemu-devel] [PATCH 1/2] Add checkpatch.pl from Linux kernel

2011-01-09 Thread Stuart Brady
On Sun, Jan 09, 2011 at 06:46:31PM +0100, Stefan Weil wrote: > I appreciate that it will be possible to use scripts like this one > for QEMU, too. Let me just add a small remark. > > QEMU's root directory is already crowded with too many files > (at least that's my personal opinion). > > Why not

Re: [Qemu-devel] [PATCH 3/7] linux-user: Implement FS_IOC_FIEMAP ioctl

2011-01-09 Thread Martin Mohring
On 01/09/2011 06:47 PM, riku voipio wrote: > On 01/09/2011 04:47 PM, Blue Swirl wrote: >> This fails if the file doesn't exist: >>CCi386-linux-user/syscall.o >> /src/qemu/linux-user/syscall.c:86:26: error: linux/fiemap.h: No such >> file or directory > >> The fix is to introduce a feature c

[Qemu-devel] [Bug 700774] [NEW] sh7750.c:672: sh7750_mmct_writel: Assertion `0' failed.

2011-01-09 Thread Khem Raj
Public bug reported: qemu 0.12.5 on ubuntu ends up with this error when booting a SH4 machine usb 1-1: Manufacturer: 1 INIT: version 2.86 booting Error, no support currently for 8 bpp frame buffers Trying to change pixel format... Please wait: booting... Switched to a 32 bpp 8,8,8 frame buffer St

[Qemu-devel] Re: [PATCH] Avoid divide by zero when there is no block device to migrate

2011-01-09 Thread Pierre Riteau
On 25 déc. 2010, at 21:52, Pierre Riteau wrote: > When block migration is requested and no read-write block device is > present, a divide by zero exception is triggered because > total_sector_sum equals zero. > > Signed-off-by: Pierre Riteau > --- > block-migration.c |6 +- > 1 files chan

Re: [Qemu-devel] [PATCH 3/7] linux-user: Implement FS_IOC_FIEMAP ioctl

2011-01-09 Thread Blue Swirl
On Sun, Jan 9, 2011 at 5:47 PM, riku voipio wrote: > On 01/09/2011 04:47 PM, Blue Swirl wrote: >> >> This fails if the file doesn't exist: >>   CC    i386-linux-user/syscall.o >> /src/qemu/linux-user/syscall.c:86:26: error: linux/fiemap.h: No such >> file or directory > >> The fix is to introduce

[Qemu-devel] Re: [PATCH 04/35] Add "broadcast" option for mce command

2011-01-09 Thread Jan Kiszka
Am 06.01.2011 18:56, Marcelo Tosatti wrote: > From: Jin Dongming > > When the following test case is injected with mce command, maybe user could > not > get the expected result. > DATA >command cpu bank status mcg_status addr misc > (qemu) mce 1 1

Re: [Qemu-devel] [RFC/PATCH] elfload: add FDPIC support

2011-01-09 Thread Mike Frysinger
On Sun, Jan 9, 2011 at 03:48, Mike Frysinger wrote: > This is a PoC at this point, but it seems to be working for me.  At > least, all the current crashes I'm seeing are due to my Blackfin port > being incomplete.  All of the FDPIC table parsing seems to be OK ... > > If someone with a more functio

Re: [Qemu-devel] [PATCH 1/2] Add checkpatch.pl from Linux kernel

2011-01-09 Thread Stefan Weil
Am 09.01.2011 12:45, schrieb Blue Swirl: Unchanged import from http://www.kernel.org/pub/linux/kernel/people/apw/checkpatch/checkpatch.pl-0.31 Signed-off-by: Blue Swirl --- checkpatch.pl-0.31 | 2937 1 files changed, 2937 insertions(+), 0 de

Re: [Qemu-devel] [PATCH 3/7] linux-user: Implement FS_IOC_FIEMAP ioctl

2011-01-09 Thread riku voipio
On 01/09/2011 04:47 PM, Blue Swirl wrote: This fails if the file doesn't exist: CCi386-linux-user/syscall.o /src/qemu/linux-user/syscall.c:86:26: error: linux/fiemap.h: No such file or directory The fix is to introduce a feature check in configure. Perhaps we can do without configure

[Qemu-devel] [Bug 686613] Re: USB MSD are not marked as removable

2011-01-09 Thread Stefan Hajnoczi
** Changed in: qemu Status: New => Confirmed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/686613 Title: USB MSD are not marked as removable Status in QEMU: Confirmed Bug description:

[Qemu-devel] [PATCH] cris: remove a write-only variable

2011-01-09 Thread Blue Swirl
Avoid a warning with GCC 4.6.0: /src/qemu/target-cris/translate.c: In function 'gen_intermediate_code_internal': /src/qemu/target-cris/translate.c:3185:25: error: variable 'orig_flags' set but not used [-Werror=unused-but-set-variable] Signed-off-by: Blue Swirl --- target-cris/translate.c |6

[Qemu-devel] Re: [PATCH 2/2] checkpatch: adjust to QEMUisms

2011-01-09 Thread Jan Kiszka
Am 09.01.2011 14:24, Blue Swirl wrote: > On Sun, Jan 9, 2011 at 1:01 PM, Stuart Brady wrote: >> On Sun, Jan 09, 2011 at 11:46:33AM +, Blue Swirl wrote: >>> Change checkpatch.pl for QEMU use: >>> - Root directory detection >>> - Forbid tabs >>> - Indent at 4 spaces >>> - Allow typedefs >>>

Re: [Qemu-devel] [PATCH 3/7] linux-user: Implement FS_IOC_FIEMAP ioctl

2011-01-09 Thread Blue Swirl
On Fri, Jan 7, 2011 at 8:52 PM, Riku Voipio wrote: > From: Peter Maydell > > Implement the FS_IOC_FIEMAP ioctl using the new support for > custom handling of ioctls; this is needed because the struct > that is passed includes a variable-length array. > > Signed-off-by: Peter Maydell > Signed-off

[Qemu-devel] [Bug 612901] Re: Qemu doesn't implement SCSI READ DISC INFORMATION command (0x51) Qemu reports: SK=5h/ASC=20h/ACQ=00h

2011-01-09 Thread Stefan Hajnoczi
Unable to reproduce on qemu.git and RHEL 6 x86_64 install CD. /lib/udev/cdrom_id completes successfully and does not print the info_scsi_cmd_err. qemu.git d66bddd7a4535cca3fc9e98c3195a7411779693c $ x86_64-softmmu/qemu-system-x86_64 -enable-kvm -m 512 -cdrom rhel6.iso Please check that this issue

Re: [Qemu-devel] [PATCH 2/2] checkpatch: adjust to QEMUisms

2011-01-09 Thread Blue Swirl
On Sun, Jan 9, 2011 at 1:01 PM, Stuart Brady wrote: > On Sun, Jan 09, 2011 at 11:46:33AM +, Blue Swirl wrote: >> Change checkpatch.pl for QEMU use: >>  - Root directory detection >>  - Forbid tabs >>  - Indent at 4 spaces >>  - Allow typedefs >>  - Enforce brace use even for single statement b

[Qemu-devel] [PATCH] add bepo (french dvorak) keyboard layout

2011-01-09 Thread Fred Boiteux
Hello, It's my first message here, and I'm not a git user, so I hope the patch format will be good (I've done a "git diff -cached" in the qemu tree). I'm using the Qemu program with VNC I/O, and I had some problems with my keyboard layout, so I've prepared a definition to be included i

Re: [Qemu-devel] [PATCH 2/2] checkpatch: adjust to QEMUisms

2011-01-09 Thread Stuart Brady
On Sun, Jan 09, 2011 at 11:46:33AM +, Blue Swirl wrote: > Change checkpatch.pl for QEMU use: > - Root directory detection > - Forbid tabs > - Indent at 4 spaces > - Allow typedefs > - Enforce brace use even for single statement blocks > - Don't suggest nonexistent cleanup tools Could you

Re: [Qemu-devel] Re: [PATCH v3] savevm: Fix no_migrate

2011-01-09 Thread Blue Swirl
On Sun, Jan 9, 2011 at 9:57 AM, Michael S. Tsirkin wrote: > On Fri, Jan 07, 2011 at 07:47:34PM +0100, Jan Kiszka wrote: >> (Is there really no one bored out there and wants to write a check-patch >> script?) > > Might be better to use an existing tool: > http://www.kernel.org/pub/linux/kernel/peop

[Qemu-devel] Lembrete sobre o convite de Alexandre Felipe Muller de Souza

2011-01-09 Thread Alexandre Felipe Muller de Souza (LinkedIn Invitations)
LinkedIn Este convite está aguardando sua resposta: De Alexandre Felipe Muller de Souza -- (c) 2010, LinkedIn Corporation

[Qemu-devel] Re: [PATCH v4] savevm: Fix no_migrate

2011-01-09 Thread Michael S. Tsirkin
On Fri, Jan 07, 2011 at 03:13:25PM -0700, Alex Williamson wrote: > The no_migrate save state flag is currently only checked in the > last phase of migration. This means that we potentially waste > a lot of time and bandwidth with the live state handlers before > we ever check the no_migrate flags.

[Qemu-devel] Re: [PATCH v4] savevm: Fix no_migrate

2011-01-09 Thread Michael S. Tsirkin
On Fri, Jan 07, 2011 at 03:13:25PM -0700, Alex Williamson wrote: > The no_migrate save state flag is currently only checked in the > last phase of migration. This means that we potentially waste > a lot of time and bandwidth with the live state handlers before > we ever check the no_migrate flags.

[Qemu-devel] Re: [PATCH v3] savevm: Fix no_migrate

2011-01-09 Thread Michael S. Tsirkin
On Fri, Jan 07, 2011 at 07:47:34PM +0100, Jan Kiszka wrote: > (Is there really no one bored out there and wants to write a check-patch > script?) Might be better to use an existing tool: http://www.kernel.org/pub/linux/kernel/people/apw/checkpatch/ -- MST

[Qemu-devel] [RFC/PATCH] elfload: add FDPIC support

2011-01-09 Thread Mike Frysinger
This is a PoC at this point, but it seems to be working for me. At least, all the current crashes I'm seeing are due to my Blackfin port being incomplete. All of the FDPIC table parsing seems to be OK ... If someone with a more functional target would like to try this, that'd be cool. Or if peo

[Qemu-devel] [PATCH] tcg: fix typo in readme

2011-01-09 Thread Mike Frysinger
Signed-off-by: Mike Frysinger --- tcg/README |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tcg/README b/tcg/README index 68d27ff..8ec6845 100644 --- a/tcg/README +++ b/tcg/README @@ -364,7 +364,7 @@ formed from two 32-bit arguments. The result is a 32-bit value. *