[Qemu-devel] [PATCH v2 2/2] display: stop using DT_NOGRAPHIC, use DT_NONE

2013-07-02 Thread Michael Tokarev
It looks like initially there was -nographic option to turn off display, now there's another option of the same sort, -display none. But code in other places of qemu checks for DT_NOGRAPHIC and does not work well with -display none. Make DT_NOGRAPHIC an internal version which selects DT_NONE, and

[Qemu-devel] [PATCH v2 1/2] trap signals for "-serial mon:stdio"

2013-07-02 Thread Michael Tokarev
From: Paolo Bonzini With mon:stdio you can exit the VM by switching to the monitor and sending the "quit" command. It is then useful to pass Ctrl-C to the VM instead of exiting. This in turn lets us stop tying the default signal handling behavior to -nographic, removing gratuitous differences b

Re: [Qemu-devel] [PATCH 3/7] nbd: use BDS refcount

2013-07-02 Thread Fam Zheng
On Wed, 07/03 07:58, Paolo Bonzini wrote: > Il 03/07/2013 03:10, Fam Zheng ha scritto: > > > This has the important side effect of marking the exported disk as > > > "in_use" (to use the terms before the series). Right now you can serve > > > a disk and, at the same time, stream it or mirror it or

Re: [Qemu-devel] [PATCH v2 5/6] net: defer nested call to BH

2013-07-02 Thread Paolo Bonzini
Il 13/06/2013 11:03, Liu Ping Fan ha scritto: > From: Liu Ping Fan > > Nested call caused by ->receive() will raise issue like deadlock, > so postphone it to BH. When I read this patch, I am completely puzzled. All I see is that a qemu_net_queue_flush is done in a bottom half. I have no clue

Re: [Qemu-devel] Wayland support

2013-07-02 Thread Alexandre DERUMIER
I don't known if it's what you want, but wayland have a beta spice backend http://www.phoronix.com/scan.php?page=news_item&px=MTM1NzQ - Mail original - De: "Max A Yu" À: qemu-devel@nongnu.org Envoyé: Mardi 2 Juillet 2013 05:36:50 Objet: [Qemu-devel] Wayland support Hi, Can QEMU supp

[Qemu-devel] moving CHR_EVENT_OPEN out of BH broke echo for muxed mon

2013-07-02 Thread Michael Tokarev
Before commit bd5c51ee6c4f1c79cae5ad2516d711a27b4ea8ec Author: Michael Roth Date: Fri Jun 7 15:19:53 2013 -0500 qemu-char: don't issue CHR_EVENT_OPEN in a BH we had no echo by default with -nographic, and it gave the prompt when switching to monitor: $ qemu-system-x86_64 -nographic _

Re: [Qemu-devel] [PATCH 04/30] add a header file for atomic operations

2013-07-02 Thread Paolo Bonzini
Il 03/07/2013 04:24, liu ping fan ha scritto: >> > +- atomic operations in Linux are always on a 32-bit int type and > 32-bit? int should be the integer type that the target processor is > most efficient working with. Has Linux ever been ported to a machine where sizeof(int) != 4? (Honest questi

Re: [Qemu-devel] [PATCH 3/7] nbd: use BDS refcount

2013-07-02 Thread Paolo Bonzini
Il 03/07/2013 03:10, Fam Zheng ha scritto: > > This has the important side effect of marking the exported disk as > > "in_use" (to use the terms before the series). Right now you can serve > > a disk and, at the same time, stream it or mirror it or create a live > > snapshot of it. > > > > Do we

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Paolo Bonzini
Il 02/07/2013 22:58, Anthony Liguori ha scritto: > > > We consume the schema in QEMU. No reason for us to consume it in a > > > different format than libvirt. > > > > One reason could be that qapi-schema.json, as written, lacks a schema > > that can be expressed itself using QAPI. > > Yup, but how

Re: [Qemu-devel] [PATCHv2 02/11] iscsi: read unmap info from block limits vpd page

2013-07-02 Thread ronnie sahlberg
max_unmap : If the target does not return an explicit limit for max_unmap it will return 0x which means "no limit". I think you should add a check for max_unmap and clamp it down to something sane. Maybe a maximum of 128M ? Same for bdc, that should also be checked and clamped down to sa

Re: [Qemu-devel] [PATCH v2 3/6] net: make netclient re-entrant with refcnt

2013-07-02 Thread liu ping fan
On Mon, Jul 1, 2013 at 7:50 PM, Stefan Hajnoczi wrote: > On Thu, Jun 20, 2013 at 05:14:56PM +0800, liu ping fan wrote: >> On Tue, Jun 18, 2013 at 8:41 PM, Stefan Hajnoczi wrote: >> > On Thu, Jun 13, 2013 at 05:03:03PM +0800, Liu Ping Fan wrote: >> >> @@ -1109,6 +1146,7 @@ void net_cleanup(void) >

Re: [Qemu-devel] [PATCH 6/9] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-07-02 Thread Asias He
On Fri, Jun 21, 2013 at 10:16:48AM +, Libaiqing wrote: > Hi Asias, > > > -Original Message- > > From: Asias He [mailto:as...@redhat.com] > > Sent: Thursday, June 20, 2013 5:39 PM > > To: Libaiqing > > Cc: Paolo Bonzini; Wenchao Xia; qemu-devel@nongnu.org; > > n...@linux-iscsi.org; Mich

Re: [Qemu-devel] [PATCH 6/9] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-07-02 Thread Asias He
On Wed, Jul 03, 2013 at 03:08:34AM +, Libaiqing wrote: > Hi asias, >I'm testing vhost-blk,for comparimg the performance with virtio-blk. >I got the kernel patch from this mail: https://lkml.org/lkml/2012/12/1/174 You can find the latest vhost-blk kernel bits here: git://github.com/a

[Qemu-devel] [PATCH 6/9] vhost-scsi: new device supporting the tcm_vhost Linux kernel module

2013-07-02 Thread Libaiqing
Hi asias, I'm testing vhost-blk,for comparimg the performance with virtio-blk. I got the kernel patch from this mail: https://lkml.org/lkml/2012/12/1/174 And I got the userspace code for qemu from this git: https://github.com/asias/qemu/tree/blk.vhost-blk; Used vhost-blk device as non-

Re: [Qemu-devel] [PATCH 04/30] add a header file for atomic operations

2013-07-02 Thread liu ping fan
On Sat, Jun 29, 2013 at 2:26 AM, Paolo Bonzini wrote: > We're already using them in several places, but __sync builtins are just > too ugly to type, and do not provide seqcst load/store operations. > > Signed-off-by: Paolo Bonzini > --- > docs/atomics.txt | 345 > +++

[Qemu-devel] 回复: Re: Which part of qemu responds to ACPI control method?

2013-07-02 Thread bobooscar
Take the method “_PTS” for example, how could I know how it access a certain hardware, and what hardware it accesses? I am a newbie in this field, thanks in advance;) 已从三星手机发送 原始邮件 发件人: Laszlo Ersek 日期: 2013-07-02 23:57 (GMT+08:00) 收件人: bobooscar 抄送: qemu-devel@nongnu.

Re: [Qemu-devel] [PATCH V3 0/9] add internal snapshot support at block device level

2013-07-02 Thread Wenchao Xia
Any comments for this version? >This series brings internal snapshot support at block devices level, now we > have two three methods to do block snapshot lively: 1) backing chain, > 2) internal one and 3) drive-back up approach. > > Comparation: > Advantages:

Re: [Qemu-devel] [PATCH v2 02/26] ohci: QOM'ify some more

2013-07-02 Thread Andreas Färber
Am 01.07.2013 12:18, schrieb Hu Tao: > Introduce type constant and avoid DO_UPCAST(). > > Signed-off-by: Hu Tao > --- > hw/usb/hcd-ohci.c | 20 > 1 file changed, 12 insertions(+), 8 deletions(-) > > diff --git a/hw/usb/hcd-ohci.c b/hw/usb/hcd-ohci.c > index 5513924..912255d

Re: [Qemu-devel] help me for nahanni device

2013-07-02 Thread DAI Weibin
Hello Cam, I am honored that receive your reply. 1 the QEMU version is showed as following, the version is 1.0.50 [root@node1 sh-vm]# /opt/qemu-kvm-rt/x86_64-softmmu/qemu-system-x86_64 -version QEMU emulator version 1.0.50 (qemu-kvm-devel), Copyright (c) 2003-2008 Fabrice Bellard [root@node1 sh-v

Re: [Qemu-devel] [PATCH v2 01/26] sysbus: document SysBusDeviceClass about @init

2013-07-02 Thread Hu Tao
On Wed, Jul 03, 2013 at 03:19:53AM +0200, Andreas Färber wrote: > Am 01.07.2013 12:18, schrieb Hu Tao: > > Signed-off-by: Hu Tao > > --- > > include/hw/sysbus.h | 10 ++ > > 1 file changed, 10 insertions(+) > > > > diff --git a/include/hw/sysbus.h b/include/hw/sysbus.h > > index 7c2e316.

Re: [Qemu-devel] [PATCH] qom: Use atomics for object refcounting

2013-07-02 Thread liu ping fan
On Wed, Jul 3, 2013 at 12:36 AM, Anthony Liguori wrote: > Paolo Bonzini writes: > >> Il 02/07/2013 16:47, Anthony Liguori ha scritto: >>> Jan Kiszka writes: >>> Objects can soon be referenced/dereference outside the BQL. So we need to use atomics in object_ref/unref. Based on

Re: [Qemu-devel] [PATCH v2 01/26] sysbus: document SysBusDeviceClass about @init

2013-07-02 Thread Andreas Färber
Am 01.07.2013 12:18, schrieb Hu Tao: > Signed-off-by: Hu Tao > --- > include/hw/sysbus.h | 10 ++ > 1 file changed, 10 insertions(+) > > diff --git a/include/hw/sysbus.h b/include/hw/sysbus.h > index 7c2e316..9614758 100644 > --- a/include/hw/sysbus.h > +++ b/include/hw/sysbus.h > @@ -23

Re: [Qemu-devel] [PATCH 3/7] nbd: use BDS refcount

2013-07-02 Thread Fam Zheng
On Tue, 07/02 12:16, Paolo Bonzini wrote: > Il 02/07/2013 07:59, Fam Zheng ha scritto: > > Previously, nbd call drive_get_ref on the drive of bs. A BDS doesn't > > always have associated dinfo, it's more proper to use bdrv_get_ref(). > > This has the important side effect of marking the exported d

Re: [Qemu-devel] [PATCH 1/7] block: Convert BlockDriverState.in_use to refcount

2013-07-02 Thread Fam Zheng
On Tue, 07/02 13:41, Eric Blake wrote: > On 07/01/2013 11:59 PM, Fam Zheng wrote: > > Use numeric value to replace in_use flag in BDS, this will make > > lifecycle management with ref count possible. This patch only replaces > > existing uses of bdrv_set_in_use, so no logic change here. > > > > Si

[Qemu-devel] [PATCH qom-cpu] target-ppc: Change LOG_MMU_STATE() argument to CPUState

2013-07-02 Thread Andreas Färber
Choose CPUState rather than PowerPCCPU since doing a CPU() cast on the macro argument would hide type mismatches. Signed-off-by: Andreas Färber --- target-ppc/mmu-hash32.c | 4 ++-- target-ppc/mmu-hash64.c | 4 ++-- target-ppc/mmu_helper.c | 6 +++--- 3 files changed, 7 insertions(+), 7 deletion

Re: [Qemu-devel] [PATCH RFC qom-cpu 03/41] gdbstub: Change GDBState::query_cpu to CPUState

2013-07-02 Thread Andreas Färber
Am 01.07.2013 19:05, schrieb Richard Henderson: > On 06/29/2013 01:01 PM, Andreas Färber wrote: >> Since first_cpu/next_cpu are CPUState, CPUArchState is no longer needed. >> >> Signed-off-by: Andreas Färber >> --- >> gdbstub.c | 9 - >> 1 file changed, 4 insertions(+), 5 deletions(-) >

Re: [Qemu-devel] [RFC V8 01/24] qcow2: Add journal specification.

2013-07-02 Thread Benoît Canet
> > 2. Byte-granularity means that read-modify-write is necessary to append > >entries to the journal. Therefore a failure could destroy previously > >committed entries. > > > >Any ideas how existing journals handle this? > > You commit only whole blocks. So in this case we can consi

Re: [Qemu-devel] [PATCH v2 0/9] tcg: remainder and arm runtime detection

2013-07-02 Thread Peter Maydell
On 2 July 2013 21:25, Anthony Liguori wrote: > Peter Maydell writes: >> On 2 July 2013 20:17, Anthony Liguori wrote: >>> Richard Henderson writes: >>> Ping. >>> >>> Peter, could you take a look and bring in through the arm tree? >> >> I can test it and put together a pullrequest, sure, tho

Re: [Qemu-devel] [RFC V8 01/24] qcow2: Add journal specification.

2013-07-02 Thread Benoît Canet
> > +QCOW2 can use one or more instance of a metadata journal. > > s/instance/instances/ > > Is there a reason to use multiple journals rather than a single journal > for all entry types? The single journal area avoids seeks. Here are the main reason for this: For the deduplication some patter

Re: [Qemu-devel] [PATCH RFC qom-cpu 01/41] log: Change log_cpu_state[_mask]() argument to CPUState

2013-07-02 Thread Andreas Färber
Am 02.07.2013 03:26, schrieb Andreas Färber: > Am 29.06.2013 22:01, schrieb Andreas Färber: >> diff --git a/target-microblaze/translate.c b/target-microblaze/translate.c >> index 54f439f..f9acdb1 100644 >> --- a/target-microblaze/translate.c >> +++ b/target-microblaze/translate.c >> @@ -1741,6 +174

Re: [Qemu-devel] [PATCH qom-cpu 2/5] target-lm32: gen_intermediate_code_internal() should be inline

2013-07-02 Thread Michael Walle
Am Dienstag, 2. Juli 2013, 21:31:33 schrieb Andreas Färber: > Reported-by: Richard Henderson > Cc: qemu-sta...@nongnu.org > Signed-off-by: Andreas Färber > --- > target-lm32/translate.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/target-lm32/translate.c b/targe

Re: [Qemu-devel] [PATCH RFC qom-cpu 23/41] target-xtensa: Change gen_intermediate_code_internal() arg to XtensaCPU

2013-07-02 Thread Andreas Färber
Am 01.07.2013 19:16, schrieb Richard Henderson: > On 06/29/2013 01:01 PM, Andreas Färber wrote: >> Also use bool type while at it. >> >> Prepares for moving singlestep_enabled field to CPUState. >> >> Signed-off-by: Andreas Färber >> --- >> target-xtensa/translate.c | 10 ++ >> 1 file cha

Re: [Qemu-devel] [PATCH v2 0/9] tcg: remainder and arm runtime detection

2013-07-02 Thread Andreas Färber
Am 02.07.2013 22:25, schrieb Anthony Liguori: > Peter Maydell writes: >> On 2 July 2013 20:17, Anthony Liguori wrote: >>> Peter, could you take a look and bring in through the arm tree? >> >> I can test it and put together a pullrequest, sure, though >> tcg/arm isn't really covered by either arm-

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Anthony Liguori
Paolo Bonzini writes: > Il 02/07/2013 20:21, Anthony Liguori ha scritto: >>> > >>> > Indeed, I wouldn't mind a one-time pass over qapi-schema.json to make it >>> > follow a more rigid format if that made it easier to use it as-is with >>> > less post-processing. It won't be very nice to backport

Re: [Qemu-devel] [PATCH 01/17] pseries: move interrupt controllers to hw/intc/

2013-07-02 Thread Andreas Färber
Am 27.06.2013 08:45, schrieb Alexey Kardashevskiy: > Signed-off-by: Alexey Kardashevskiy > --- > default-configs/ppc64-softmmu.mak |1 + > hw/intc/Makefile.objs |1 + > hw/{ppc => intc}/xics.c |0 > hw/ppc/Makefile.objs |2 +- > 4 files changed,

Re: [Qemu-devel] [PATCH] savevm: Fix potential memory leak

2013-07-02 Thread Stefan Weil
Am 02.07.2013 22:33, schrieb Stefan Weil: > Am 16.06.2013 13:33, schrieb Stefan Weil: >> The leak was reported by cppcheck. Fix it by moving the g_malloc0 after >> the argument validity check. >> >> Signed-off-by: Stefan Weil >> --- >> savevm.c |3 ++- >> 1 file changed, 2 insertions(+), 1 de

Re: [Qemu-devel] [PATCH] fsdev: Fix potential memory leak

2013-07-02 Thread Stefan Weil
Am 16.06.2013 12:02, schrieb Stefan Weil: > This leak was reported by cppcheck. > > Signed-off-by: Stefan Weil > --- > fsdev/qemu-fsdev.c |2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/fsdev/qemu-fsdev.c b/fsdev/qemu-fsdev.c > index 6eaf36d..ccfec13 100644 > --- a/fsdev/qemu-fsdev.

Re: [Qemu-devel] [PATCH] hw/9pfs: Fix potential memory leak and avoid reuse of freed memory

2013-07-02 Thread Stefan Weil
Am 16.06.2013 12:14, schrieb Stefan Weil: > The leak was reported by cppcheck. > > Function proxy_init also calls g_free for ctx->fs_root. > Avoid reuse of this memory by setting ctx->fs_root to NULL. > > Signed-off-by: Stefan Weil > --- > > Hi, > > I'm not sure whether ctx->fs_root should also be

Re: [Qemu-devel] [PATCH] savevm: Fix potential memory leak

2013-07-02 Thread Stefan Weil
Am 16.06.2013 13:33, schrieb Stefan Weil: > The leak was reported by cppcheck. Fix it by moving the g_malloc0 after > the argument validity check. > > Signed-off-by: Stefan Weil > --- > savevm.c |3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/savevm.c b/savevm.c > in

Re: [Qemu-devel] [PATCH 3/3] spapr: Fix compiler warning for some versions of gcc (spapr_io_read)

2013-07-02 Thread Alexander Graf
Am 02.07.2013 um 22:22 schrieb Stefan Weil : > Am 02.07.2013 14:45, schrieb Alexander Graf: >> On 06/24/2013 07:48 PM, Stefan Weil wrote: >>> i686-w64-mingw32-gcc (GCC) 4.6.3 from Debian wheezy reports this >>> warning: >>> >>> hw/ppc/spapr_pci.c:454:1: warning: >>> control reaches end of non-

Re: [Qemu-devel] [PATCH v2 0/9] tcg: remainder and arm runtime detection

2013-07-02 Thread Anthony Liguori
Peter Maydell writes: > On 2 July 2013 20:17, Anthony Liguori wrote: >> Richard Henderson writes: >> >>> Ping. >> >> Peter, could you take a look and bring in through the arm tree? > > I can test it and put together a pullrequest, sure, though > tcg/arm isn't really covered by either arm-devs o

Re: [Qemu-devel] [PATCH 3/3] spapr: Fix compiler warning for some versions of gcc (spapr_io_read)

2013-07-02 Thread Stefan Weil
Am 02.07.2013 14:45, schrieb Alexander Graf: > On 06/24/2013 07:48 PM, Stefan Weil wrote: >> i686-w64-mingw32-gcc (GCC) 4.6.3 from Debian wheezy reports this >> warning: >> >> hw/ppc/spapr_pci.c:454:1: warning: >> control reaches end of non-void function [-Wreturn-type] >> >> Adding a default cas

Re: [Qemu-devel] [PATCH qom-cpu 0/5] TCG code generation performance fix

2013-07-02 Thread Andreas Färber
Am 02.07.2013 21:59, schrieb Richard Henderson: > On 07/02/2013 12:31 PM, Andreas Färber wrote: >> Hello, >> >> As Richard explained, the purpose of having separate gen_intermediate_code() >> and gen_intermediate_code_pc() functions per target is to compile-optimize >> gen_intermediate_code_interna

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Eric Blake
On 07/02/2013 02:00 PM, Paolo Bonzini wrote: > Il 02/07/2013 20:21, Anthony Liguori ha scritto: Indeed, I wouldn't mind a one-time pass over qapi-schema.json to make it follow a more rigid format if that made it easier to use it as-is with less post-processing. It won't be very

Re: [Qemu-devel] [PATCH 6/7] block: add target-id option to drive-backup QMP command

2013-07-02 Thread Eric Blake
On 07/01/2013 11:59 PM, Fam Zheng wrote: > Add target-id (optional) to drive-backup command, to make the target bs > a named drive so that we can operate on it (e.g. export with NBD). > > Signed-off-by: Fam Zheng > --- > blockdev.c | 4 +++- > qapi-schema.json | 7 +-- > qmp-commands.h

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Paolo Bonzini
Il 02/07/2013 20:21, Anthony Liguori ha scritto: >> > >> > Indeed, I wouldn't mind a one-time pass over qapi-schema.json to make it >> > follow a more rigid format if that made it easier to use it as-is with >> > less post-processing. It won't be very nice to backport such a >> > conversion, but I

[Qemu-devel] [PATCH qom-cpu 5/5] target-xtensa: gen_intermediate_code_internal() should be inline

2013-07-02 Thread Andreas Färber
Reported-by: Richard Henderson Cc: qemu-sta...@nongnu.org Signed-off-by: Andreas Färber --- target-xtensa/translate.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/target-xtensa/translate.c b/target-xtensa/translate.c index dcb90a5..e384812 100644 --- a/target-xtensa/t

Re: [Qemu-devel] [PATCH qom-cpu 0/5] TCG code generation performance fix

2013-07-02 Thread Richard Henderson
On 07/02/2013 12:31 PM, Andreas Färber wrote: > Hello, > > As Richard explained, the purpose of having separate gen_intermediate_code() > and gen_intermediate_code_pc() functions per target is to compile-optimize > gen_intermediate_code_internal() for the non-_pc case. > > Multiple targets were u

Re: [Qemu-devel] [PATCH v3 3/3] qapi-schema: Use BlockdevSnapshot type for blockdev-snapshot-sync

2013-07-02 Thread Eric Blake
On 07/01/2013 08:31 AM, Kevin Wolf wrote: > We don't have to duplicate the definition any more now that we may refer > to a type instead. > > Signed-off-by: Kevin Wolf > --- > qapi-schema.json | 14 ++ > 1 file changed, 2 insertions(+), 12 deletions(-) Since this patch is not in yet

Re: [Qemu-devel] [PATCH v2 0/9] tcg: remainder and arm runtime detection

2013-07-02 Thread Peter Maydell
On 2 July 2013 20:17, Anthony Liguori wrote: > Richard Henderson writes: > >> Ping. > > Peter, could you take a look and bring in through the arm tree? I can test it and put together a pullrequest, sure, though tcg/arm isn't really covered by either arm-devs or target-arm. -- PMM

[Qemu-devel] [PATCH qom-cpu 3/5] target-microblaze: gen_intermediate_code_internal() should be inline

2013-07-02 Thread Andreas Färber
Reported-by: Richard Henderson Cc: qemu-sta...@nongnu.org Signed-off-by: Andreas Färber --- target-microblaze/translate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target-microblaze/translate.c b/target-microblaze/translate.c index 54f439f..27da4bf 100644 --- a/target-m

[Qemu-devel] [PATCH qom-cpu 4/5] target-moxie: gen_intermediate_code_internal() should be inline

2013-07-02 Thread Andreas Färber
Reported-by: Richard Henderson Cc: qemu-sta...@nongnu.org Signed-off-by: Andreas Färber --- target-moxie/translate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target-moxie/translate.c b/target-moxie/translate.c index b0ae38a..664d359 100644 --- a/target-moxie/translate.

[Qemu-devel] [PATCH qom-cpu 2/5] target-lm32: gen_intermediate_code_internal() should be inline

2013-07-02 Thread Andreas Färber
Reported-by: Richard Henderson Cc: qemu-sta...@nongnu.org Signed-off-by: Andreas Färber --- target-lm32/translate.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/target-lm32/translate.c b/target-lm32/translate.c index 227a801..7d82dc7 100644 --- a/target-lm32/translate

Re: [Qemu-devel] [PATCH v2 0/9] tcg: remainder and arm runtime detection

2013-07-02 Thread Andreas Färber
Am 02.07.2013 21:17, schrieb Anthony Liguori: > Richard Henderson writes: > >> Ping. > > Peter, could you take a look and bring in through the arm tree? The ppc bits look pretty obvious and the rem concept fine to me. Didn't look too close at the arm parts. Andreas > > Regards, > > Anthony

[Qemu-devel] [PATCH qom-cpu 1/5] target-cris: gen_intermediate_code_internal() should be inlined

2013-07-02 Thread Andreas Färber
Reported-by: Richard Henderson Cc: qemu-sta...@nongnu.org Signed-off-by: Andreas Färber --- target-cris/translate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target-cris/translate.c b/target-cris/translate.c index 09d0d2b..ee9ae22 100644 --- a/target-cris/translate.c ++

[Qemu-devel] [PATCH qom-cpu 0/5] TCG code generation performance fix

2013-07-02 Thread Andreas Färber
Hello, As Richard explained, the purpose of having separate gen_intermediate_code() and gen_intermediate_code_pc() functions per target is to compile-optimize gen_intermediate_code_internal() for the non-_pc case. Multiple targets were using static rather than static inline though, fix this. I'v

Re: [Qemu-devel] [PATCH 1/7] block: Convert BlockDriverState.in_use to refcount

2013-07-02 Thread Eric Blake
On 07/01/2013 11:59 PM, Fam Zheng wrote: > Use numeric value to replace in_use flag in BDS, this will make > lifecycle management with ref count possible. This patch only replaces > existing uses of bdrv_set_in_use, so no logic change here. > > Signed-off-by: Fam Zheng > --- > block-migration.c

Re: [Qemu-devel] [PATCH v2 4/9] tcg-ppc64: Don't implement rem

2013-07-02 Thread Andreas Färber
Am 26.06.2013 22:52, schrieb Richard Henderson: > Signed-off-by: Richard Henderson Reviewed-by: Andreas Färber Andreas > --- > tcg/ppc64/tcg-target.c | 26 -- > tcg/ppc64/tcg-target.h | 4 ++-- > 2 files changed, 2 insertions(+), 28 deletions(-) > > diff --git a/tcg/

Re: [Qemu-devel] [PATCH v2 3/9] tcg-ppc: Don't implement rem

2013-07-02 Thread Andreas Färber
Am 26.06.2013 22:52, schrieb Richard Henderson: > Signed-off-by: Richard Henderson Reviewed-by: Andreas Färber Andreas > --- > tcg/ppc/tcg-target.c | 14 -- > tcg/ppc/tcg-target.h | 2 +- > 2 files changed, 1 insertion(+), 15 deletions(-) > > diff --git a/tcg/ppc/tcg-target.c b/

Re: [Qemu-devel] [PATCH v2 0/9] tcg: remainder and arm runtime detection

2013-07-02 Thread Anthony Liguori
Richard Henderson writes: > Ping. Peter, could you take a look and bring in through the arm tree? Regards, Anthony Liguori > > On 06/26/2013 01:52 PM, Richard Henderson wrote: >> This patch set includes both the remainder series and arm runtime >> detection series that I've previouslyt posted

Re: [Qemu-devel] [Xen-devel] [PATCH] Citrix PV Bus device [V3]

2013-07-02 Thread Anthony Liguori
Stefano Stabellini writes: > On Tue, 2 Jul 2013, Stefano Stabellini wrote: >> - the new device should have configurable vendor and device ids, so that >> host admins can select which vendor's PV drivers are going to be >> automatically installed on all your Windows guests. This should probably >>

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Anthony Liguori
Eric Blake writes: > On 07/02/2013 11:06 AM, Anthony Liguori wrote: >>> Because qapi-schema.json requires further parsing. For example, how is >>> a client supposed to know that '*foo':'int' means that there is an >>> argument named 'foo' but it is optional? The rule of thumb with QMP is >>> th

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Anthony Liguori
Eric Blake writes: > On 07/02/2013 11:01 AM, Paolo Bonzini wrote: Arguably that rule of thumb would apply equally to the QEMU build scripts which already parse qapi-schema.json. It could be possible to normalize qapi-schema.json somewhat to remove this 2-stage parsing if we we

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Anthony Liguori
Eric Blake writes: > On 07/02/2013 09:39 AM, Daniel P. Berrange wrote: Maybe I'm being too meta here, but why not just return qapi-schema.json as a string and call it as day? >>> >>> Because qapi-schema.json requires further parsing. For example, how is >>> a client supposed to know th

Re: [Qemu-devel] [Xen-devel] [PATCH] Citrix PV Bus device [V3]

2013-07-02 Thread Stefano Stabellini
On Tue, 2 Jul 2013, Stefano Stabellini wrote: > - the new device should have configurable vendor and device ids, so that > host admins can select which vendor's PV drivers are going to be > automatically installed on all your Windows guests. This should probably > be a VM config option (pvdevice=,

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Eric Blake
On 07/02/2013 11:06 AM, Anthony Liguori wrote: >> Because qapi-schema.json requires further parsing. For example, how is >> a client supposed to know that '*foo':'int' means that there is an >> argument named 'foo' but it is optional? The rule of thumb with QMP is >> that if you have to post-proc

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Eric Blake
On 07/02/2013 11:01 AM, Paolo Bonzini wrote: >>> Arguably that rule of thumb would apply equally to the QEMU >>> build scripts which already parse qapi-schema.json. It could >>> be possible to normalize qapi-schema.json somewhat to remove >>> this 2-stage parsing if we went down this route. >> >> I

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Anthony Liguori
Eric Blake writes: > On 07/02/2013 08:51 AM, Anthony Liguori wrote: >> Amos Kong writes: >> >>> Introduces new monitor command to query QMP schema information, >>> the return data is a nested dict/list, it contains the useful >>> metadata. >>> >>> we can add events definations to qapi-schema.js

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Paolo Bonzini
> > Arguably that rule of thumb would apply equally to the QEMU > > build scripts which already parse qapi-schema.json. It could > > be possible to normalize qapi-schema.json somewhat to remove > > this 2-stage parsing if we went down this route. > > Indeed, I wouldn't mind a one-time pass over qa

[Qemu-devel] [PATCH v2] semaphore: fix a hangup problem under load on NetBSD hosts.

2013-07-02 Thread Izumi Tsutsui
Fix following bugs in "fallback implementation of counting semaphores with mutex+condvar" added in c166cb72f1676855816340666c3b618beef4b976: - waiting threads are not restarted properly if more than one threads are waiting unblock signals in qemu_sem_timedwait() - possible missing pthread_cond

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Eric Blake
On 07/02/2013 09:39 AM, Daniel P. Berrange wrote: >>> Maybe I'm being too meta here, but why not just return qapi-schema.json >>> as a string and call it as day? >> >> Because qapi-schema.json requires further parsing. For example, how is >> a client supposed to know that '*foo':'int' means that t

Re: [Qemu-devel] [Xen-devel] [PATCH] Citrix PV Bus device [V3]

2013-07-02 Thread Stefano Stabellini
On Tue, 2 Jul 2013, Paul Durrant wrote: > This patch introduces a new PCI device which will act as the binding point > for Citrix branded PV drivers for Xen. > The intention is that Citrix Windows PV drivers will be available on Windows > Update and thus using the existing Xen platform PCI device a

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Eric Blake
On 06/19/2013 06:49 AM, Amos Kong wrote: > On Wed, Jun 19, 2013 at 08:24:37PM +0800, Amos Kong wrote: >> Introduces new monitor command to query QMP schema information, >> the return data is a nested dict/list, it contains the useful >> metadata. >> >> we can add events definations to qapi-schema.j

Re: [Qemu-devel] [PATCH] qom: Use atomics for object refcounting

2013-07-02 Thread Anthony Liguori
Paolo Bonzini writes: > Il 02/07/2013 16:47, Anthony Liguori ha scritto: >> Jan Kiszka writes: >> >>> Objects can soon be referenced/dereference outside the BQL. So we need >>> to use atomics in object_ref/unref. >>> >>> Based on patch by Liu Ping Fan. >>> >>> Signed-off-by: Jan Kiszka >>> ---

Re: [Qemu-devel] [PATCH v3] linux-user: improve target_to_host_sock_type conversion

2013-07-02 Thread Peter Maydell
On 2 July 2013 15:13, Petar Jovanovic wrote: > Yet, I ended up changing 80% of the header file with this version, so I > was under impression it makes sense to change the remaining 20%. Yeah, that's fine -- but once you get to that scale of change it's nicer to have patch 1/2 fix formatting, patc

Re: [Qemu-devel] help me for nahanni device

2013-07-02 Thread Cam Macdonell
Hello Dai, Which qemu version you are using? When you say QEMU/KVM "doesn't work", do you mean it does not start? Cam On Mon, Jul 1, 2013 at 12:39 AM, DAI Weibin wrote: > Hi all, > When use nahanni device as following, The QEMU/KVM doesn’t work. > I attach two ivshmem devices to VM, QEMU/KV

Re: [Qemu-devel] [Xen-devel] [PATCH] Citrix PV Bus device

2013-07-02 Thread Alex Bligh
Ian, --On 2 July 2013 14:42:41 +0100 Ian Campbell wrote: Let's say we do this, then Xirtic also produce PV drivers, and get assigned their own device ID. Will both of them be visible with a different device ID for the same device? Different device ID for the same device isn't possible, a d

Re: [Qemu-devel] Which part of qemu responds to ACPI control method?

2013-07-02 Thread Laszlo Ersek
On 07/02/13 15:05, bobooscar wrote: > > Hi,all: > When a guest domain excutes a control method, such as “_PTS”, which > part of qemu would respond and handle the request? thank you in advance. IMHO, like any other method, _PTS (Prepare To Sleep) is executed by the guest's ACPI interpreter,

Re: [Qemu-devel] [PATCH 25/30] exec: put memory map in AddressSpaceDispatch

2013-07-02 Thread Jan Kiszka
On 2013-07-02 17:08, Paolo Bonzini wrote: > Il 02/07/2013 16:42, Jan Kiszka ha scritto: +typedef PhysPageEntry Node[L2_SIZE]; + struct AddressSpaceDispatch { /* This is a multi-level map on the physical address space. * The bottom level has pointers to MemoryReg

Re: [Qemu-devel] [PATCH] semaphore: fix a hangup problem under loadon NetBSD hosts.

2013-07-02 Thread Laszlo Ersek
On 07/02/13 17:27, Izumi Tsutsui wrote: > Laszlo Ersek wrote: >> Conversely, the only time we need to send a signal is the 0->1 count >> transition (*). > > Per comments from Soda, signals could be required even on count >0, > if more than one threads are sleeping in qemu_cond_timedwait(), > and

[Qemu-devel] Which part of qemu responds to ACPI control method?

2013-07-02 Thread bobooscar
Hi,all:     When a guest domain excutes a control method, such as  “_PTS”, which part of qemu would respond and handle the request? thank you in advance.  已从三星手机发送

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Daniel P. Berrange
On Tue, Jul 02, 2013 at 09:28:51AM -0600, Eric Blake wrote: > On 07/02/2013 08:51 AM, Anthony Liguori wrote: > > Amos Kong writes: > > > >> Introduces new monitor command to query QMP schema information, > >> the return data is a nested dict/list, it contains the useful > >> metadata. > >> > >> w

[Qemu-devel] [PATCH] Citrix PV Bus device [V3]

2013-07-02 Thread Paul Durrant
This patch introduces a new PCI device which will act as the binding point for Citrix branded PV drivers for Xen. The intention is that Citrix Windows PV drivers will be available on Windows Update and thus using the existing Xen platform PCI device as an anchor point is not desirable as that devic

Re: [Qemu-devel] [PATCH] qom: Use atomics for object refcounting

2013-07-02 Thread Paolo Bonzini
Il 02/07/2013 16:47, Anthony Liguori ha scritto: > Jan Kiszka writes: > >> Objects can soon be referenced/dereference outside the BQL. So we need >> to use atomics in object_ref/unref. >> >> Based on patch by Liu Ping Fan. >> >> Signed-off-by: Jan Kiszka >> --- >> qom/object.c |5 ++--- >>

Re: [Qemu-devel] [PATCH] full introspection support for QMP

2013-07-02 Thread Eric Blake
On 07/02/2013 08:51 AM, Anthony Liguori wrote: > Amos Kong writes: > >> Introduces new monitor command to query QMP schema information, >> the return data is a nested dict/list, it contains the useful >> metadata. >> >> we can add events definations to qapi-schema.json, then it can >> also be que

Re: [Qemu-devel] [PATCH v3] configure: Simplify alternate .text segment

2013-07-02 Thread Richard Henderson
Ping. On 06/21/2013 07:10 PM, Richard Henderson wrote: > For bsd-user and linux-user emulation modes QEMU needs to be linked at an > alternate .text segment address, so that it's out of the way of the guest > executable. Instead of including modified linker scripts for each arch, > just set the a

Re: [Qemu-devel] [PATCH] semaphore: fix a hangup problem under loadon NetBSD hosts.

2013-07-02 Thread Izumi Tsutsui
Laszlo Ersek wrote: > On 06/29/13 12:22, Izumi Tsutsui wrote: > > Fix following bugs in "fallback implementation of counting semaphores > > with mutex+condvar" added in c166cb72f1676855816340666c3b618beef4b976: > > - waiting threads are not restarted properly if more than one threads > >are w

Re: [Qemu-devel] [PATCH v4 08/10] qemu-ga: call Windows VSS requester in fsfreeze command handler

2013-07-02 Thread Laszlo Ersek
On 07/02/13 17:16, Luiz Capitulino wrote: > On Tue, 02 Jul 2013 16:44:55 +0200 > Laszlo Ersek wrote: > >> On 07/02/13 16:09, Luiz Capitulino wrote: >>> On Tue, 02 Jul 2013 08:36:11 +0200 >>> Laszlo Ersek wrote: >>> On 07/01/13 19:59, Tomoki Sekiyama wrote: > On 7/1/13 9:29 , "Laszlo Ers

Re: [Qemu-devel] [PATCH] configure: detect endian via compile test

2013-07-02 Thread Richard Henderson
On 06/30/2013 08:30 PM, Mike Frysinger wrote: > This avoids needing to execute a program and keeping an (incomplete) > list when cross-compiling. > > Signed-off-by: Mike Frysinger For the benefit of the list, this is a porting of the autoconf test. Reviewed-by: Richard Henderson r~

Re: [Qemu-devel] [PATCH v4 08/10] qemu-ga: call Windows VSS requester in fsfreeze command handler

2013-07-02 Thread Luiz Capitulino
On Tue, 02 Jul 2013 16:44:55 +0200 Laszlo Ersek wrote: > On 07/02/13 16:09, Luiz Capitulino wrote: > > On Tue, 02 Jul 2013 08:36:11 +0200 > > Laszlo Ersek wrote: > > > >> On 07/01/13 19:59, Tomoki Sekiyama wrote: > >>> On 7/1/13 9:29 , "Laszlo Ersek" wrote: > >>> > > +error: > > +q

Re: [Qemu-devel] [PATCH v2 0/9] tcg: remainder and arm runtime detection

2013-07-02 Thread Richard Henderson
Ping. On 06/26/2013 01:52 PM, Richard Henderson wrote: > This patch set includes both the remainder series and arm runtime > detection series that I've previouslyt posted separately, as there > are small conflicts between the two series. > > Aside from rebasing vs master, the only other change is

Re: [Qemu-devel] [PATCH v2 0/4] tcg-arm: Implement tcg_register_jit

2013-07-02 Thread Richard Henderson
Ping 4. Should I just send a PULL for the only partially reviewed series? r~ On 06/24/2013 08:44 PM, Richard Henderson wrote: > Ping 3. > > On 06/17/2013 08:54 AM, Richard Henderson wrote: >> Ping. >> >> >> On 06/10/2013 11:41 AM, Richard Henderson wrote: >>> Ping. >>> >>> >>> On 06/05/2013 10:

Re: [Qemu-devel] [PATCH 25/30] exec: put memory map in AddressSpaceDispatch

2013-07-02 Thread Paolo Bonzini
Il 02/07/2013 16:42, Jan Kiszka ha scritto: >> > +typedef PhysPageEntry Node[L2_SIZE]; >> > + >> > struct AddressSpaceDispatch { >> > /* This is a multi-level map on the physical address space. >> > * The bottom level has pointers to MemoryRegionSections. >> > */ >> > PhysPag

Re: [Qemu-devel] [PATCH v3] linux-user: improve target_to_host_sock_type conversion

2013-07-02 Thread Petar Jovanovic
From: Peter Maydell [peter.mayd...@linaro.org] Sent: Tuesday, July 02, 2013 3:44 PM To: Petar Jovanovic Cc: qemu-devel@nongnu.org; Petar Jovanovic; aurel...@aurel32.net; riku.voi...@linaro.org; r...@twiddle.net Subject: Re: [PATCH v3] linux-user: improve t

Re: [Qemu-devel] [PATCH v4 08/10] qemu-ga: call Windows VSS requester in fsfreeze command handler

2013-07-02 Thread Michael Roth
On Tue, Jul 2, 2013 at 1:36 AM, Laszlo Ersek wrote: > On 07/01/13 19:59, Tomoki Sekiyama wrote: >> On 7/1/13 9:29 , "Laszlo Ersek" wrote: >> +error: +qmp_guest_fsfreeze_thaw(NULL); >>> >>> Passing NULL here as "errp" concerns me slightly. I've been assuming >>> that "errp" is never

Re: [Qemu-devel] [PATCH] Citrix PV Bus device

2013-07-02 Thread Paul Durrant
> -Original Message- > From: Peter Maydell [mailto:peter.mayd...@linaro.org] > Sent: 02 July 2013 15:58 > To: Paul Durrant > Cc: qemu-devel@nongnu.org; xen-de...@lists.xen.org > Subject: Re: [Qemu-devel] [PATCH] Citrix PV Bus device > > On 2 July 2013 15:03, Paul Durrant wrote: > > This p

Re: [Qemu-devel] [PATCH] Citrix PV Bus device

2013-07-02 Thread Paul Durrant
> -Original Message- > From: Anthony Liguori [mailto:anth...@codemonkey.ws] > Sent: 02 July 2013 15:43 > To: Paul Durrant; qemu-devel@nongnu.org; xen-de...@lists.xen.org > Cc: Paul Durrant > Subject: Re: [Qemu-devel] [PATCH] Citrix PV Bus device > > Paul Durrant writes: > > > This patch

Re: [Qemu-devel] [PATCH] Citrix PV Bus device

2013-07-02 Thread Peter Maydell
On 2 July 2013 15:03, Paul Durrant wrote: > This patch introduces a new PCI device which will act as the binding point > for Citrix branded PV drivers for Xen. > The intention is that Citrix Windows PV drivers will be available on Windows > Update and thus using the existing Xen platform PCI devic

Re: [Qemu-devel] [PATCH 27/30] exec: change some APIs to take AddressSpaceDispatch

2013-07-02 Thread Jan Kiszka
On 2013-06-28 20:26, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini > --- > exec.c | 11 +-- > 1 file changed, 5 insertions(+), 6 deletions(-) > > diff --git a/exec.c b/exec.c > index 7f87e16..528c4d7 100644 > --- a/exec.c > +++ b/exec.c > @@ -236,11 +236,10 @@ bool memory_region_is

Re: [Qemu-devel] [RFC V8 01/24] qcow2: Add journal specification.

2013-07-02 Thread Kevin Wolf
Am 02.07.2013 um 16:42 hat Stefan Hajnoczi geschrieben: > On Thu, Jun 20, 2013 at 04:26:09PM +0200, Benoît Canet wrote: > > --- > > docs/specs/qcow2.txt | 42 ++ > > 1 file changed, 42 insertions(+) > > > > diff --git a/docs/specs/qcow2.txt b/docs/specs/q

  1   2   3   >