Re: [Qemu-devel] [PATCH 3/5] scsi-block: extract scsi_block_is_passthrough

2014-07-28 Thread Fam Zheng
On Mon, 07/28 17:08, Paolo Bonzini wrote: > This will be used for both scsi_block_new_request and the scsi-block > implementation of parse_cdb. > > Signed-off-by: Paolo Bonzini > --- > hw/scsi/scsi-disk.c | 38 ++ > 1 file changed, 26 insertions(+), 12 deletio

Re: [Qemu-devel] Dynamic QEMU platform device instantiation in machine files: phone call on Wed July 30

2014-07-28 Thread Markus Armbruster
Eric Auger writes: > Dear all, > > For your information, a phone call will be held this week on Wed July > 30, 17h-18h CET to address the topic of dynamic instantiation of QEMU > platform devices in machine files (using the -device qemu option). > > Related threads are: > http://lists.gnu.org/arc

Re: [Qemu-devel] [PATCH v4 0/5] ACPI fixes for QEMU 2.1

2014-07-28 Thread Paolo Bonzini
Il 29/07/2014 08:16, Markus Armbruster ha scritto: >> > v3->v4: >> >drop all pretense of supporting bridges [me] > Does this non-support need documentation? None visible in diffstat... Yes, it should. It was already broken for 1.7->2.0. Paolo

Re: [Qemu-devel] [PATCH v4 0/5] ACPI fixes for QEMU 2.1

2014-07-28 Thread Markus Armbruster
Paolo Bonzini writes: > v3->v4: > drop all pretense of supporting bridges [me] Does this non-support need documentation? None visible in diffstat... > > v2->v3: > fix tests/acpi-test-data/pc/DSDT [Peter] > track down "make check" failure, fix it [patch 4, me] > split pa

[Qemu-devel] [PATCH for-2.1] po: update Italian translation

2014-07-28 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- po/it.po | 63 --- 1 file changed, 40 insertions(+), 23 deletions(-) diff --git a/po/it.po b/po/it.po index a62665c..e46fb3a 100644 --- a/po/it.po +++ b/po/it.po @@ -1,13 +1,13 @@ # Italian translation

Re: [Qemu-devel] [PATCH 2/3] qemu-char: add -chardev exit-on-eof option

2014-07-28 Thread Markus Armbruster
Eric Blake writes: > On 07/25/2014 02:12 AM, Markus Armbruster wrote: >> Stefan Hajnoczi writes: >> >>> When QEMU is executed as part of a test case or from a script, it is >>> usually desirable to exit if the parent process terminates. This >>> ensures that "leaked" QEMU processes do not cont

Re: [Qemu-devel] [PATCH 0/4] xen:passthrough: introduce a separate machine to igd passthrough

2014-07-28 Thread Chen, Tiejun
Michael, Paolo and Stefano, Any comments? Thanks Tiejun On 2014/7/24 19:30, Tiejun Chen wrote: As we discussed currently we have to introduce a separate machine to work out igd passthrough. Tiejun Chen (4): hw:i386:pc_pii

Re: [Qemu-devel] [PATCH v4 0/5] ACPI fixes for QEMU 2.1

2014-07-28 Thread Paolo Bonzini
Il 28/07/2014 23:27, Michael S. Tsirkin ha scritto: > OK, I applied this, and did some tweaks on top that I think > make it a bit safer. > It's very very late in the release cycle, but also very late in the > day so I don't want to risk sending pull request now. > I did push it out: tag for_upstrea

Re: [Qemu-devel] [PATCH for-2.1] po: Update French translation

2014-07-28 Thread Stefan Weil
Am 28.07.2014 23:44, schrieb Aurelien Jarno: > Add new translations for recently added messages. > > Signed-off-by: Aurelien Jarno > --- > po/fr_FR.po | 54 +++--- > 1 file changed, 39 insertions(+), 15 deletions(-) > Reviewed-by: Stefan Weil

Re: [Qemu-devel] [PATCH] [RFC] Add machine type pc-1.0-qemu-kvm for live migrate compatibility with qemu-kvm

2014-07-28 Thread Serge Hallyn
Quoting Alex Bligh (a...@alex.org.uk): > > On 22 Jul 2014, at 19:43, Alex Bligh wrote: > > > Testing has been light to date (i.e. > > can I migrate it inbound with -S without anything complaining). thanks, Alex! > > I've given this quite a bit more testing today. > > It works fine qemu-kvm 1

Re: [Qemu-devel] [PATCH v2 1/7] bootindex: add modify_boot_device_path function

2014-07-28 Thread Gonglei (Arei)
Hi, > -Original Message- > From: Gonglei (Arei) > Sent: Monday, July 28, 2014 6:16 PM > Subject: RE: [PATCH v2 1/7] bootindex: add modify_boot_device_path function > > > -Original Message- > > From: Gerd Hoffmann [mailto:kra...@redhat.com] > > Sent: Monday, July 28, 2014 6:02 PM >

[Qemu-devel] [PATCH] migrate: auto increase the strength to throttle vcpu

2014-07-28 Thread arei.gonglei
From: ChenLiang Previously, the strength to throttle vcpu is constant that is too arbitrarily. This patch(suggested by Juan) increases the strength according to the times of memory iterator. Signed-off-by: ChenLiang Signed-off-by: Gonglei --- arch_init.c | 4 +++- 1 file changed, 3 insertions

Re: [Qemu-devel] [PATCH v5 2/2] vmdk: Optimize cluster allocation

2014-07-28 Thread Fam Zheng
On Mon, 07/28 16:11, Stefan Hajnoczi wrote: > On Tue, May 27, 2014 at 04:49:22PM +0800, Fam Zheng wrote: > > +if (!bs->backing_hd) { > > +memset(whole_grain, 0, skip_start_sector << BDRV_SECTOR_BITS); > > +memset(whole_grain + (skip_end_sector << BDRV_SECTOR_BITS), 0, > > +

Re: [Qemu-devel] [PATCH v2 7/7] spapr: fix possible memory leak

2014-07-28 Thread Gonglei (Arei)
Hi, > -Original Message- > From: Alexander Graf [mailto:ag...@suse.de] > Sent: Monday, July 28, 2014 6:45 PM > Subject: Re: [PATCH v2 7/7] spapr: fix possible memory leak > > > On 25.07.14 08:52, arei.gong...@huawei.com wrote: > > From: Gonglei > > > > get_boot_devices_list() will mallo

Re: [Qemu-devel] [PATCH 2/3] qemu-char: add -chardev exit-on-eof option

2014-07-28 Thread Eric Blake
On 07/25/2014 02:12 AM, Markus Armbruster wrote: > Stefan Hajnoczi writes: > >> When QEMU is executed as part of a test case or from a script, it is >> usually desirable to exit if the parent process terminates. This >> ensures that "leaked" QEMU processes do not continue consuming resources >>

Re: [Qemu-devel] [PATCH 2/2] target-mips/translate.c: Add judgement for msb and lsb

2014-07-28 Thread Aurelien Jarno
On Mon, Jul 28, 2014 at 11:34:30PM +0100, Peter Maydell wrote: > On 28 July 2014 23:32, Aurelien Jarno wrote: > > On Mon, Jul 28, 2014 at 11:01:02PM +0100, Peter Maydell wrote: > >> This may be true, but the TCG README doesn't define negative > >> lengths as being "unspecified behaviour" (ie guara

Re: [Qemu-devel] [PATCH 2/2] target-mips/translate.c: Add judgement for msb and lsb

2014-07-28 Thread Peter Maydell
On 28 July 2014 23:32, Aurelien Jarno wrote: > On Mon, Jul 28, 2014 at 11:01:02PM +0100, Peter Maydell wrote: >> This may be true, but the TCG README doesn't define negative >> lengths as being "unspecified behaviour" (ie guaranteed to at >> least not crash even if the result isn't specified), and

Re: [Qemu-devel] [PATCH 2/2] target-mips/translate.c: Add judgement for msb and lsb

2014-07-28 Thread Aurelien Jarno
On Mon, Jul 28, 2014 at 11:01:02PM +0100, Peter Maydell wrote: > On 28 July 2014 22:42, Aurelien Jarno wrote: > > On Mon, Jul 28, 2014 at 11:58:22PM +0800, Dongxue Zhang wrote: > >> Use 'if' to make sure the real msb greater than the lsb. As the compiler > >> may > >> not do this. > > > > What ar

Re: [Qemu-devel] [PATCH 2/2] target-mips/translate.c: Add judgement for msb and lsb

2014-07-28 Thread Peter Maydell
On 28 July 2014 22:42, Aurelien Jarno wrote: > On Mon, Jul 28, 2014 at 11:58:22PM +0800, Dongxue Zhang wrote: >> Use 'if' to make sure the real msb greater than the lsb. As the compiler may >> not do this. > > What are you trying to fix exactly? These cases are defined as > "unpredictable" in the

Re: [Qemu-devel] [PATCH RFC v2 05/12] VMstate test: basic VMState testing mechanism

2014-07-28 Thread Eric Blake
On 07/25/2014 09:39 AM, Sanidhya Kashyap wrote: > In this patch, I have made the following changes: > > * changed the DPRINT statement. > * renamed the variables. > * added noqdev variable which decides which option to use for resetting. > * added devices option which can help in resetting one or

Re: [Qemu-devel] [PATCH RFC v2 03/12] VMState test: query command to extract the qdevified device names

2014-07-28 Thread Eric Blake
On 07/25/2014 09:39 AM, Sanidhya Kashyap wrote: > I have provided a qmp interface for getting the list of qdevified devices > that have been registered with SaveVMHandlers. > > Signed-off-by: Sanidhya Kashyap > --- > qapi-schema.json | 22 ++ > qmp-commands.hx | 25 +

[Qemu-devel] [PATCH for-2.1] po: Update French translation

2014-07-28 Thread Aurelien Jarno
Add new translations for recently added messages. Signed-off-by: Aurelien Jarno --- po/fr_FR.po | 54 +++--- 1 file changed, 39 insertions(+), 15 deletions(-) diff --git a/po/fr_FR.po b/po/fr_FR.po index ec54eb9..bbb5ef8 100644 --- a/po/fr_FR.po

Re: [Qemu-devel] [PATCH v2 for-2.1] po: Update German translation

2014-07-28 Thread Aurelien Jarno
On Fri, Jul 18, 2014 at 04:44:21PM +0200, Stefan Weil wrote: > Line numbers changed, and some translations were missing after commit > 3d914488aee3dc1bf495e461aedf8fb4e5bb2270. > > Update also "Show Tabs" to a more common translation, and remove some > old unused lines at the end. > > Signed-off-

Re: [Qemu-devel] [PATCH 2/2] target-mips/translate.c: Add judgement for msb and lsb

2014-07-28 Thread Aurelien Jarno
On Mon, Jul 28, 2014 at 11:58:22PM +0800, Dongxue Zhang wrote: > Use 'if' to make sure the real msb greater than the lsb. As the compiler may > not do this. What are you trying to fix exactly? These cases are defined as "unpredictable" in the MIPS ISA manual, which is what is implemented in QEMU.

Re: [Qemu-devel] [PATCH 1/2] target-mips/translate.c: Free TCG in OPC_DINSV

2014-07-28 Thread Aurelien Jarno
On Mon, Jul 28, 2014 at 11:58:21PM +0800, Dongxue Zhang wrote: > Free t0 and t1 in opcode OPC_DINSV. > > Signed-off-by: Dongxue Zhang > --- > target-mips/translate.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/target-mips/translate.c b/target-mips/translate.c > index d7b8c4d..c3

Re: [Qemu-devel] [PATCH] target-mips: Ignore unassigned accesses with KVM

2014-07-28 Thread Aurelien Jarno
On Mon, Jul 28, 2014 at 12:37:50PM +0100, James Hogan wrote: > MIPS registers an unassigned access handler which raises a guest bus > error exception. However this causes QEMU to crash when KVM is enabled > as it isn't called from the main execution loop so longjmp() gets called > without a corresp

Re: [Qemu-devel] [PATCH RFC v2 01/12] QEMUSizedBuffer/QEMUFile

2014-07-28 Thread Eric Blake
On 07/25/2014 09:39 AM, Sanidhya Kashyap wrote: > From: "Dr. David Alan Gilbert" > > Stefan Berger's to create a QEMUFile that goes to a memory buffer; Missing something. Maybe you meant: This is based on Stefan Berger's patch to create ... > from: > > http://lists.gnu.org/archive/html/qemu-

Re: [Qemu-devel] [PATCH v2 3/4] qemu-char: add -chardev exit-on-eof option

2014-07-28 Thread Eric Blake
On 07/25/2014 07:16 AM, Stefan Hajnoczi wrote: > When QEMU is executed as part of a test case or from a script, it is > usually desirable to exit if the parent process terminates. This > ensures that "leaked" QEMU processes do not continue consuming resources > after their parent has died. > > Th

Re: [Qemu-devel] [PATCH v4 0/5] ACPI fixes for QEMU 2.1

2014-07-28 Thread Michael S. Tsirkin
On Mon, Jul 28, 2014 at 05:34:13PM +0200, Paolo Bonzini wrote: > v3->v4: > drop all pretense of supporting bridges [me] > > v2->v3: > fix tests/acpi-test-data/pc/DSDT [Peter] > track down "make check" failure, fix it [patch 4, me] > split patch 2 in two parts [mst] >

Re: [Qemu-devel] [PATCH 3/5] pc: future-proof migration-compatibility of ACPI tables

2014-07-28 Thread Michael S. Tsirkin
On Mon, Jul 28, 2014 at 06:08:53PM +0200, Paolo Bonzini wrote: > Il 28/07/2014 17:59, Michael S. Tsirkin ha scritto: > > On Mon, Jul 28, 2014 at 05:34:16PM +0200, Paolo Bonzini wrote: > >> This patch avoids that similar changes break QEMU again in the future. > >> QEMU will now hard-code 64k as the

[Qemu-devel] [PATCH] acpi-build: tweak acpi migration limits

2014-07-28 Thread Michael S. Tsirkin
- Tweak error message for legacy machine type: Basically if table size exceeds the limits we set all bets are off for migration: e.g. it can start failing even within given qemu minor version simply because of a bugfix. - Increase table size to 128k. - Make sure we notice it long before we st

[Qemu-devel] [PATCH] piix: set legacy table size for 1.7

2014-07-28 Thread Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin --- Paolo, so the following is needed on top of your patch? hw/i386/pc_piix.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c index 4524e6b..9694f88 100644 --- a/hw/i386/pc_piix.c +++ b/hw/i386/pc_piix.c @@ -326,6 +326,

Re: [Qemu-devel] [PATCH for-2.2 1/2] rename parse_enum_option to qapi_enum_parse and make it public

2014-07-28 Thread Peter Lieven
Am 28.07.2014 um 21:58 schrieb Eric Blake : > On 07/28/2014 01:43 PM, Peter Lieven wrote: >> Suggested-by: Markus Armbruster >> Signed-off-by: Hu Tao >> Signed-off-by: Peter Lieven >> --- >> blockdev.c | 30 ++ >> include/qapi/util.h | 17 +++

Re: [Qemu-devel] [PATCH for-2.1] qemu-options: add missing -drive discard option to cmdline help

2014-07-28 Thread Eric Blake
On 07/28/2014 01:53 PM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > qemu-options.hx |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Eric Blake > > diff --git a/qemu-options.hx b/qemu-options.hx > index 1549625..251535f 100644 > --- a/qemu-options.hx > +++

Re: [Qemu-devel] [PATCH for-2.2 2/2] qemu-nbd: add option to set detect-zeroes mode

2014-07-28 Thread Eric Blake
On 07/28/2014 01:43 PM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > qemu-nbd.c | 81 > +++- > 1 file changed, 53 insertions(+), 28 deletions(-) > > @@ -57,45 +60,47 @@ static void usage(const char *name) > "Usage: %s [OPT

Re: [Qemu-devel] [PATCH for-2.2 1/2] rename parse_enum_option to qapi_enum_parse and make it public

2014-07-28 Thread Eric Blake
On 07/28/2014 01:43 PM, Peter Lieven wrote: > Suggested-by: Markus Armbruster > Signed-off-by: Hu Tao > Signed-off-by: Peter Lieven > --- > blockdev.c | 30 ++ > include/qapi/util.h | 17 + > qapi/Makefile.objs |2 +- > qapi/qapi-uti

[Qemu-devel] [PATCH for-2.1] qemu-options: add missing -drive discard option to cmdline help

2014-07-28 Thread Peter Lieven
Signed-off-by: Peter Lieven --- qemu-options.hx |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qemu-options.hx b/qemu-options.hx index 1549625..251535f 100644 --- a/qemu-options.hx +++ b/qemu-options.hx @@ -427,7 +427,7 @@ DEF("drive", HAS_ARG, QEMU_OPTION_drive, "

[Qemu-devel] [PATCH for-2.2 2/2] qemu-nbd: add option to set detect-zeroes mode

2014-07-28 Thread Peter Lieven
Signed-off-by: Peter Lieven --- qemu-nbd.c | 81 +++- 1 file changed, 53 insertions(+), 28 deletions(-) diff --git a/qemu-nbd.c b/qemu-nbd.c index 626e584..75cf54f 100644 --- a/qemu-nbd.c +++ b/qemu-nbd.c @@ -18,11 +18,13 @@ #include "

[Qemu-devel] [PATCH for-2.2 1/2] rename parse_enum_option to qapi_enum_parse and make it public

2014-07-28 Thread Peter Lieven
Suggested-by: Markus Armbruster Signed-off-by: Hu Tao Signed-off-by: Peter Lieven --- blockdev.c | 30 ++ include/qapi/util.h | 17 + qapi/Makefile.objs |2 +- qapi/qapi-util.c| 34 ++ 4 files ch

[Qemu-devel] [PATCH for-2.2 0/2] qemu-nbd: add option to set detect-zeroes mode

2014-07-28 Thread Peter Lieven
Peter Lieven (2): rename parse_enum_option to qapi_enum_parse and make it public qemu-nbd: add option to set detect-zeroes mode blockdev.c | 30 --- include/qapi/util.h | 17 +++ qapi/Makefile.objs |2 +- qapi/qapi-util.c| 34 +++

Re: [Qemu-devel] [PATCH 1/3] loader: add support for resizeable blobs

2014-07-28 Thread Paolo Bonzini
Il 28/07/2014 21:07, Michael S. Tsirkin ha scritto: > On Mon, Jul 28, 2014 at 05:52:27PM +0100, Dr. David Alan Gilbert wrote: >> * Michael S. Tsirkin (m...@redhat.com) wrote: >>> Support resizeable blobs: we allocate more memory than currently >>> available in the blob, which can later be filled in

Re: [Qemu-devel] [PATCH 1/3] loader: add support for resizeable blobs

2014-07-28 Thread Michael S. Tsirkin
On Mon, Jul 28, 2014 at 05:52:27PM +0100, Dr. David Alan Gilbert wrote: > * Michael S. Tsirkin (m...@redhat.com) wrote: > > Support resizeable blobs: we allocate more memory than currently > > available in the blob, which can later be filled in. > > > > Signed-off-by: Michael S. Tsirkin > > --- >

Re: [Qemu-devel] [PATCH for 2.1 V3] qemu-img info: show nocow info

2014-07-28 Thread Eric Blake
On 07/28/2014 09:19 AM, Stefan Hajnoczi wrote: > On Wed, Jul 09, 2014 at 10:43:13AM +0800, Chunyan Liu wrote: >> Add nocow info in 'qemu-img info' output to show whether the file >> currently has NOCOW flag set or not. >> >> Signed-off-by: Chunyan Liu >> --- >> Changes: >> - update output info t

Re: [Qemu-devel] [PATCH 1/3] loader: add support for resizeable blobs

2014-07-28 Thread Dr. David Alan Gilbert
* Michael S. Tsirkin (m...@redhat.com) wrote: > Support resizeable blobs: we allocate more memory than currently > available in the blob, which can later be filled in. > > Signed-off-by: Michael S. Tsirkin > --- > include/hw/loader.h | 14 +++-- > include/hw/nvram/fw_cfg.h | 2 +- > h

[Qemu-devel] [Bug 1347387] Re: while i was created the new virtual machine using qemu the following error was shown in fedora version 20

2014-07-28 Thread Cole Robinson
** Changed in: qemu Status: New => Invalid ** Bug watch added: Red Hat Bugzilla #1096654 https://bugzilla.redhat.com/show_bug.cgi?id=1096654 -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/13

Re: [Qemu-devel] [PATCH/RFC 4/5] s390x/kvm: test whether a cpu is STOPPED when checking "has_work"

2014-07-28 Thread Alexander Graf
On 28.07.14 17:03, David Hildenbrand wrote: On 28.07.2014, at 16:16, David Hildenbrand wrote: On 10.07.14 15:10, Christian Borntraeger wrote: From: David Hildenbrand If a cpu is stopped, it must never be allowed to run and no interrupt may wake it up. A cpu also has to be unhalted if it i

Re: [Qemu-devel] [Bug 1347387] [NEW] while i was created the new virtual machine using qemu the following error was shown in fedora version 20

2014-07-28 Thread Cole Robinson
On 07/28/2014 12:09 PM, Stefan Hajnoczi wrote: > On Wed, Jul 23, 2014 at 04:43:27AM -, selvakumar wrote: >> Public bug reported: >> >> [root@localhost pkgs]# qemu-img create virtualdisk.img 100M >> qemu-img: symbol lookup error: qemu-img: undefined symbol: glfs_discard_async > You need to upd

[Qemu-devel] [PATCH v2 4/4] parallels: 2TB+ parallels images support

2014-07-28 Thread Denis V. Lunev
Parallels has released in the recent updates of Parallels Server 5/6 new addition to his image format. Images with signature WithouFreSpacExt have offsets in the catalog coded not as offsets in sectors (multiple of 512 bytes) but offsets coded in blocks (i.e. header->tracks * 512) In this case all

[Qemu-devel] [PATCH v2 0/4] block/parallels: 2TB+ parallels images support

2014-07-28 Thread Denis V. Lunev
Parallels has released in the recent updates of Parallels Server 5/6 new addition to his image format. Images with signature WithouFreSpacExt have offsets in the catalog coded not as offsets in sectors (multiple of 512 bytes) but offsets coded in blocks (i.e. header->tracks * 512) In this case to

[Qemu-devel] [PATCH v2 3/4] parallels: split check for parallels format in parallels_open

2014-07-28 Thread Denis V. Lunev
and rework error path a bit. There is no difference at the moment, but the code will be definitely shorter when additional processing will be required for WithouFreSpacExt Signed-off-by: Denis V. Lunev CC: Jeff Cody CC: Kevin Wolf CC: Stefan Hajnoczi --- block/parallels.c | 13 -

[Qemu-devel] [PATCH v2 2/4] parallels: replace tabs with spaces in block/parallels.c

2014-07-28 Thread Denis V. Lunev
Signed-off-by: Denis V. Lunev Reviewed-by: Jeff Cody CC: Kevin Wolf CC: Stefan Hajnoczi --- block/parallels.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/block/parallels.c b/block/parallels.c index e39..16d14ad 100644 --- a/block/parallels.c +++ b/block/p

[Qemu-devel] [PATCH v2 1/4] parallels: extend parallels format header with actual data values

2014-07-28 Thread Denis V. Lunev
Parallels image format has several additional fields inside: - nb_sectors is actually 64 bit wide. Upper 32bits are not used for images with signature "WithoutFreeSpace" and must be explicitly zeroed according to Parallels. They will be used for images with signature "WithouFreSpacExt" - inus

Re: [Qemu-devel] [PATCH for-2.1] qemu-options: fix another allows-to for -net l2tpv3

2014-07-28 Thread Stefan Hajnoczi
On Thu, Jul 24, 2014 at 08:11:26PM +0400, Michael Tokarev wrote: > Signed-off-by: Michael Tokarev > --- > qemu-options.hx |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/qemu-options.hx b/qemu-options.hx > index 9e54686..1549625 100644 > --- a/qemu-options.hx > +++ b/

Re: [Qemu-devel] [PATCH v3] docs/multiple-iothreads.txt: add documentation on IOThread programming

2014-07-28 Thread Stefan Hajnoczi
On Wed, Jul 23, 2014 at 12:55:32PM +0100, Stefan Hajnoczi wrote: > This document explains how IOThreads and the main loop are related, > especially how to write code that can run in an IOThread. Currently > only virtio-blk-data-plane uses these techniques. The next obvious > target is virtio-scsi

[Qemu-devel] Dynamic QEMU platform device instantiation in machine files: phone call on Wed July 30

2014-07-28 Thread Eric Auger
Dear all, For your information, a phone call will be held this week on Wed July 30, 17h-18h CET to address the topic of dynamic instantiation of QEMU platform devices in machine files (using the -device qemu option). Related threads are: http://lists.gnu.org/archive/html/qemu-ppc/2014-07/msg00047

Re: [Qemu-devel] [Bug 1347387] [NEW] while i was created the new virtual machine using qemu the following error was shown in fedora version 20

2014-07-28 Thread Stefan Hajnoczi
On Wed, Jul 23, 2014 at 04:43:27AM -, selvakumar wrote: > Public bug reported: > > [root@localhost pkgs]# qemu-img create virtualdisk.img 100M > qemu-img: symbol lookup error: qemu-img: undefined symbol: glfs_discard_async CCing Fedora qemu-kvm package maintainer. Stefan > [root@localhost p

Re: [Qemu-devel] [PATCH 3/5] pc: future-proof migration-compatibility of ACPI tables

2014-07-28 Thread Paolo Bonzini
Il 28/07/2014 17:59, Michael S. Tsirkin ha scritto: > On Mon, Jul 28, 2014 at 05:34:16PM +0200, Paolo Bonzini wrote: >> This patch avoids that similar changes break QEMU again in the future. >> QEMU will now hard-code 64k as the maximum ACPI table size, which >> (despite being an order of magnitude

Re: [Qemu-devel] nic is lost, in the virtual machine running

2014-07-28 Thread Stefan Hajnoczi
On Sat, Jul 19, 2014 at 03:31:00PM +0800, melolinux wrote: > 1. /usr/bin/qemu-system-x86_64 -machine accel=kvm -name > a58970a2-10aa-4973-9261-7384b2f53221 -S -machine pc-0.14,accel=kvm,usb=off -m > 4096 -smp 4,sockets=1,cores=4,threads=1 -uuid > f3a2217f-ae5c-461b-a922-d37a3a98edc6 -no-user-con

Re: [Qemu-devel] [PATCH 3/5] pc: future-proof migration-compatibility of ACPI tables

2014-07-28 Thread Michael S. Tsirkin
On Mon, Jul 28, 2014 at 05:34:16PM +0200, Paolo Bonzini wrote: > This patch avoids that similar changes break QEMU again in the future. > QEMU will now hard-code 64k as the maximum ACPI table size, which > (despite being an order of magnitude smaller than 640k) should be enough > for everyone. Fam

[Qemu-devel] [PATCH 1/2] target-mips/translate.c: Free TCG in OPC_DINSV

2014-07-28 Thread Dongxue Zhang
Free t0 and t1 in opcode OPC_DINSV. Signed-off-by: Dongxue Zhang --- target-mips/translate.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/target-mips/translate.c b/target-mips/translate.c index d7b8c4d..c381366 100644 --- a/target-mips/translate.c +++ b/target-mips/translate.c @@ -1530

[Qemu-devel] [PATCH 2/2] target-mips/translate.c: Add judgement for msb and lsb

2014-07-28 Thread Dongxue Zhang
Use 'if' to make sure the real msb greater than the lsb. As the compiler may not do this. Signed-off-by: Dongxue Zhang --- target-mips/translate.c | 9 + 1 file changed, 9 insertions(+) diff --git a/target-mips/translate.c b/target-mips/translate.c index c381366..e2cce31 100644 --- a/ta

Re: [Qemu-devel] [PATCH/RFC 4/5] s390x/kvm: test whether a cpu is STOPPED when checking "has_work"

2014-07-28 Thread David Hildenbrand
> > > > On 28.07.2014, at 16:16, David Hildenbrand wrote: > > > > >> > > >> On 10.07.14 15:10, Christian Borntraeger wrote: > > >>> From: David Hildenbrand > > >>> > > >>> If a cpu is stopped, it must never be allowed to run and no interrupt > > >>> may wake it > > >>> up. A cpu also has to

Re: [Qemu-devel] [PATCH v2] Tap: fix vcpu long time io blocking on tap

2014-07-28 Thread Stefan Hajnoczi
On Fri, Jul 18, 2014 at 09:33:42AM +, Wangkai (Kevin,C) wrote: > fix vcpu long time io blocking on tap, when too many packets was delivered > to the guest os via tap interface. > > -- > Signed-off-by: Wangkai Thanks, applied to my net-next tree: https://github.com/stefanha/qemu/commits/net-

[Qemu-devel] [PATCH 3/3] loader: mark MR for resizeable blobs as extendable

2014-07-28 Thread Michael S. Tsirkin
This makes migration from older QEMU versions more robust. Signed-off-by: Michael S. Tsirkin --- hw/core/loader.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/hw/core/loader.c b/hw/core/loader.c index ad6ec67..fc00a87 100644 --- a/hw/core/loader.c +++ b/hw/core/loader.c @@ -745,6 +

[Qemu-devel] [PATCH 2/3] migration: load smaller RAMBlock to a bigger one if permitted

2014-07-28 Thread Michael S. Tsirkin
From: Igor Mammedov Add API to mark memory region as extend-able on migration, to allow migration code to load smaller RAMBlock into a bigger one on destination QEMU instance. This will allow to fix broken migration from QEMU 1.7/2.0 to QEMU 2.1 due to ACPI tables size changes across 1.7/2.0/2.

[Qemu-devel] [PATCH 1/3] loader: add support for resizeable blobs

2014-07-28 Thread Michael S. Tsirkin
Support resizeable blobs: we allocate more memory than currently available in the blob, which can later be filled in. Signed-off-by: Michael S. Tsirkin --- include/hw/loader.h | 14 +++-- include/hw/nvram/fw_cfg.h | 2 +- hw/core/loader.c | 15 + hw/nvram/fw_cfg.c

[Qemu-devel] [PATCH 2/5] pc: hack for migration compatibility from QEMU 2.0

2014-07-28 Thread Paolo Bonzini
Changing the ACPI table size causes migration to break, and the memory hotplug work opened our eyes on how horribly we were breaking things in 2.0 already. The ACPI table size is rounded to the next 4k, which one would think gives some headroom. In practice this is not the case, because the user

[Qemu-devel] [PATCH 5/5] pc: acpi: generate AML only for PCI0 devices if PCI bridge hotplug is disabled

2014-07-28 Thread Paolo Bonzini
From: Igor Mammedov Fixes migration regression from QEMU-1.7 to a newer QEMUs. SSDT table size in QEMU-1.7 doesn't change regardless of a number of PCI bridge devices present at startup. However in QEMU-2.0 since addition of hotplug on PCI bridges, each PCI bridge adds ~1875 bytes to SSDT table,

[Qemu-devel] [PATCH 3/5] pc: future-proof migration-compatibility of ACPI tables

2014-07-28 Thread Paolo Bonzini
This patch avoids that similar changes break QEMU again in the future. QEMU will now hard-code 64k as the maximum ACPI table size, which (despite being an order of magnitude smaller than 640k) should be enough for everyone. Reviewed-by: Laszlo Ersek Tested-by: Igor Mammedov Signed-off-by: Paolo

[Qemu-devel] [PATCH 4/5] bios-tables-test: fix ASL normalization false positive

2014-07-28 Thread Paolo Bonzini
My version of IASL (from RHEL7) puts two newlines between the head comment and the DefinitionBlock property. Kill all newlines after the comment, so that normalize_asl works properly. Signed-off-by: Paolo Bonzini --- tests/bios-tables-test.c | 6 +- 1 file changed, 5 insertions(+), 1 deleti

[Qemu-devel] [PATCH 1/5] acpi-dsdt: procedurally generate _PRT

2014-07-28 Thread Paolo Bonzini
This replaces the _PRT constant with a method that computes it. The problem is that the DSDT+SSDT have grown from 2.0 to 2.1, enough to cross the 8k barrier (we align the ACPI tables to 4k before putting them in fw_cfg). This causes problems with migration and the pc-i440fx-2.0 machine type. The

[Qemu-devel] [PATCH v4 0/5] ACPI fixes for QEMU 2.1

2014-07-28 Thread Paolo Bonzini
v3->v4: drop all pretense of supporting bridges [me] v2->v3: fix tests/acpi-test-data/pc/DSDT [Peter] track down "make check" failure, fix it [patch 4, me] split patch 2 in two parts [mst] do not make bsel_alloc global [mst] include Igor's bridge pat

Re: [Qemu-devel] [PATCH for 2.1 V3] qemu-img info: show nocow info

2014-07-28 Thread Stefan Hajnoczi
On Wed, Jul 09, 2014 at 10:43:13AM +0800, Chunyan Liu wrote: > Add nocow info in 'qemu-img info' output to show whether the file > currently has NOCOW flag set or not. > > Signed-off-by: Chunyan Liu > --- > Changes: > - update output info to "NOCOW flag: set" > > block/qapi.c | 25 +++

Re: [Qemu-devel] [PATCH v5 2/2] vmdk: Optimize cluster allocation

2014-07-28 Thread Stefan Hajnoczi
On Tue, May 27, 2014 at 04:49:22PM +0800, Fam Zheng wrote: > +if (!bs->backing_hd) { > +memset(whole_grain, 0, skip_start_sector << BDRV_SECTOR_BITS); > +memset(whole_grain + (skip_end_sector << BDRV_SECTOR_BITS), 0, > + cluster_bytes - (skip_end_sector << BDRV_SE

[Qemu-devel] [PATCH 5/5] virtio-scsi: implement parse_cdb

2014-07-28 Thread Paolo Bonzini
Enable passthrough of vendor-specific commands. Signed-off-by: Paolo Bonzini --- hw/scsi/virtio-scsi.c | 25 + 1 file changed, 25 insertions(+) diff --git a/hw/scsi/virtio-scsi.c b/hw/scsi/virtio-scsi.c index 0eb069a..2dd9255 100644 --- a/hw/scsi/virtio-scsi.c +++ b/hw/s

[Qemu-devel] [PATCH 4/5] scsi-block, scsi-generic: implement parse_cdb

2014-07-28 Thread Paolo Bonzini
The callback lets the bus provide the direction and transfer count for passthrough commands, enabling passthrough of vendor-specific commands. Signed-off-by: Paolo Bonzini --- hw/scsi/scsi-bus.c | 3 +-- hw/scsi/scsi-disk.c| 14 ++ hw/scsi/scsi-generic.c | 7 +++ includ

[Qemu-devel] [PATCH 2/5] scsi-bus: introduce parse_cdb in SCSIDeviceClass and SCSIBusInfo

2014-07-28 Thread Paolo Bonzini
These callbacks will let devices do their own request parsing, or defer it to the bus. If the bus does not provide an implementation, in turn, fall back to the default parsing routine. Swap the first two arguments to scsi_req_parse, and rename it to scsi_req_parse_cdb, for consistency. Signed-of

[Qemu-devel] [PATCH 3/5] scsi-block: extract scsi_block_is_passthrough

2014-07-28 Thread Paolo Bonzini
This will be used for both scsi_block_new_request and the scsi-block implementation of parse_cdb. Signed-off-by: Paolo Bonzini --- hw/scsi/scsi-disk.c | 38 ++ 1 file changed, 26 insertions(+), 12 deletions(-) diff --git a/hw/scsi/scsi-disk.c b/hw/scsi/scsi-d

[Qemu-devel] [PATCH for-2.2 v2 0/5] scsi: enable passthrough of vendor-specific commands

2014-07-28 Thread Paolo Bonzini
Right now scsi-generic is parsing the CDB, in order to compute the expected number of bytes to be transferred. This is necessary if DMA is done by the HBA via scsi_req_data, but it prevents executing vendor-specific commands via scsi-generic because we don't know how to parse them. If DMA is dele

[Qemu-devel] [PATCH 1/5] scsi-bus: prepare scsi_req_new for introduction of parse_cdb

2014-07-28 Thread Paolo Bonzini
The per-SCSIDevice parse_cdb callback must not be called if the request will go through special SCSIReqOps, so detect the special cases early enough. Signed-off-by: Paolo Bonzini --- hw/scsi/scsi-bus.c | 51 ++- 1 file changed, 30 insertions(+), 21

Re: [Qemu-devel] [PATCH 2/5] pc: hack for migration compatibility from QEMU 2.0

2014-07-28 Thread Paolo Bonzini
Il 28/07/2014 17:05, Michael S. Tsirkin ha scritto: >> > +/* These are used to size the ACPI tables for -M pc-i440fx-1.7 and >> > + * -M pc-i440fx-2.0. Even if the actual amount of AML generated grows >> > + * a little bit, there should be plenty of free space since the DSDT >> > + * shrunk by ~1.

Re: [Qemu-devel] [PATCH 2/5] pc: hack for migration compatibility from QEMU 2.0

2014-07-28 Thread Michael S. Tsirkin
On Mon, Jul 28, 2014 at 04:02:12PM +0200, Paolo Bonzini wrote: > Changing the ACPI table size causes migration to break, and the memory > hotplug work opened our eyes on how horribly we were breaking things in > 2.0 already. > > The ACPI table size is rounded to the next 4k, which one would think

Re: [Qemu-devel] [PATCH/RFC 4/5] s390x/kvm: test whether a cpu is STOPPED when checking "has_work"

2014-07-28 Thread David Hildenbrand
> > On 28.07.2014, at 16:16, David Hildenbrand wrote: > > >> > >> On 10.07.14 15:10, Christian Borntraeger wrote: > >>> From: David Hildenbrand > >>> > >>> If a cpu is stopped, it must never be allowed to run and no interrupt may > >>> wake it > >>> up. A cpu also has to be unhalted if it is

Re: [Qemu-devel] [PATCH/RFC 4/5] s390x/kvm: test whether a cpu is STOPPED when checking "has_work"

2014-07-28 Thread Alexander Graf
On 28.07.2014, at 16:16, David Hildenbrand wrote: >> >> On 10.07.14 15:10, Christian Borntraeger wrote: >>> From: David Hildenbrand >>> >>> If a cpu is stopped, it must never be allowed to run and no interrupt may >>> wake it >>> up. A cpu also has to be unhalted if it is halted and has work

Re: [Qemu-devel] [PATCH/RFC 4/5] s390x/kvm: test whether a cpu is STOPPED when checking "has_work"

2014-07-28 Thread Paolo Bonzini
Il 28/07/2014 16:16, David Hildenbrand ha scritto: > Later, another vcpu might decide to turn that vcpu back on (by e.g. sending a > SIGP START to that vcpu). > > I am not sure if such a mechanism/scenario is applicable to any other arch. > They > all seem to reset the cs->halted flag if they kno

Re: [Qemu-devel] [PATCH/RFC 4/5] s390x/kvm: test whether a cpu is STOPPED when checking "has_work"

2014-07-28 Thread David Hildenbrand
> > On 10.07.14 15:10, Christian Borntraeger wrote: > > From: David Hildenbrand > > > > If a cpu is stopped, it must never be allowed to run and no interrupt may > > wake it > > up. A cpu also has to be unhalted if it is halted and has work to do - this > > scenario wasn't hit in kvm case yet, a

Re: [Qemu-devel] [PATCH] e1000: Delay LSC until mask is active

2014-07-28 Thread Alexander Graf
On 03.07.14 22:18, Michael S. Tsirkin wrote: On Thu, Jul 03, 2014 at 08:00:04PM +0200, Alexander Graf wrote: On 03.07.14 19:57, Peter Maydell wrote: On 3 July 2014 18:39, Alexander Graf wrote: Mac OS X reads ICR on every interrupt. When the IRQ line is shared, this may result in a race where

[Qemu-devel] [PATCH 5/5] pc: acpi: generate AML only for PCI0 devices if PCI bridge hotplug is disabled

2014-07-28 Thread Paolo Bonzini
From: Igor Mammedov Fixes migration regression from QEMU-1.7 to a newer QEMUs. SSDT table size in QEMU-1.7 doesn't change regardless of a number of PCI bridge devices present at startup. However in QEMU-2.0 since addition of hotplug on PCI bridges, each PCI bridge adds ~1875 bytes to SSDT table,

[Qemu-devel] [PATCH 4/5] bios-tables-test: fix ASL normalization false positive

2014-07-28 Thread Paolo Bonzini
My version of IASL (from RHEL7) puts two newlines between the head comment and the DefinitionBlock property. One was already removed because the test uses sizeof instead of strlen, but the extra one breaks the detection of DefinitionBlock. Killing all newlines after the comment drops the warning.

[Qemu-devel] [PATCH 3/5] pc: future-proof migration-compatibility of ACPI tables

2014-07-28 Thread Paolo Bonzini
This patch avoids that similar changes break QEMU again in the future. QEMU will now hard-code 64k as the maximum ACPI table size, which (despite being an order of magnitude smaller than 640k) should be enough for everyone. Reviewed-by: Laszlo Ersek Tested-by: Igor Mammedov Signed-off-by: Paolo

[Qemu-devel] [PATCH 2/5] pc: hack for migration compatibility from QEMU 2.0

2014-07-28 Thread Paolo Bonzini
Changing the ACPI table size causes migration to break, and the memory hotplug work opened our eyes on how horribly we were breaking things in 2.0 already. The ACPI table size is rounded to the next 4k, which one would think gives some headroom. In practice this is not the case, because the user

[Qemu-devel] [PATCH 1/5] acpi-dsdt: procedurally generate _PRT

2014-07-28 Thread Paolo Bonzini
This replaces the _PRT constant with a method that computes it. The problem is that the DSDT+SSDT have grown from 2.0 to 2.1, enough to cross the 8k barrier (we align the ACPI tables to 4k before putting them in fw_cfg). This causes problems with migration and the pc-i440fx-2.0 machine type. The

[Qemu-devel] [PATCH v3 0/5] ACPI fixes for QEMU 2.1

2014-07-28 Thread Paolo Bonzini
v2->v3: fix tests/acpi-test-data/pc/DSDT [Peter] track down "make check" failure, fix it [patch 4, me] split patch 2 in two parts [mst] do not make bsel_alloc global [mst] include Igor's bridge patch [mst, as discussed on IRC] Igor Mammedov (1): pc: acpi:

Re: [Qemu-devel] [PATCH v12 1/4] spapr_pci: Make find_phb()/find_dev() public

2014-07-28 Thread Alexey Kardashevskiy
On 07/28/2014 10:49 PM, Alexander Graf wrote: > > On 16.07.14 02:20, Gavin Shan wrote: >> From: Alexey Kardashevskiy >> >> This makes find_phb()/find_dev() public and changed its names >> to spapr_pci_find_phb()/spapr_pci_find_dev() as they are going to >> be used from other parts of QEMU such as

Re: [Qemu-devel] [PATCH/RFC 4/5] s390x/kvm: test whether a cpu is STOPPED when checking "has_work"

2014-07-28 Thread Alexander Graf
On 10.07.14 15:10, Christian Borntraeger wrote: From: David Hildenbrand If a cpu is stopped, it must never be allowed to run and no interrupt may wake it up. A cpu also has to be unhalted if it is halted and has work to do - this scenario wasn't hit in kvm case yet, as only "disabled wait" is

Re: [Qemu-devel] [PATCH/RFC 0/5] s390x/kvm: track the logical cpu state in QEMU and propagate it to kvm

2014-07-28 Thread Alexander Graf
On 28.07.2014, at 15:43, Alexander Graf wrote: > > On 10.07.14 15:27, David Hildenbrand wrote: This is the qemu part of kernel series "Let user space control the cpu states" Christian Borntraeger (1): update linux headers with with cpustate changes David H

Re: [Qemu-devel] [PATCH/RFC 0/5] s390x/kvm: track the logical cpu state in QEMU and propagate it to kvm

2014-07-28 Thread Alexander Graf
On 10.07.14 15:27, David Hildenbrand wrote: This is the qemu part of kernel series "Let user space control the cpu states" Christian Borntraeger (1): update linux headers with with cpustate changes David Hildenbrand (4): s390x/kvm: introduce proper states for s390 cpus s390x/kvm: prop

Re: [Qemu-devel] [PATCH 3/3] ppc/spapr: Fix MAX_CPUS to 255

2014-07-28 Thread Alexander Graf
On 27.06.14 08:47, Nikunj A Dadhania wrote: MAX_CPUS 256 is inconsistent with qemu supporting upto 255 cpus. This MAX_CPUS number was percolated back to "virsh capabilities" with wrong max_cpus. Signed-off-by: Nikunj A Dadhania Nice one :). Thanks, applied to ppc-next-2.2. Alex

Re: [Qemu-devel] [PATCH 0/4 v8] ppc: Add debug stub support

2014-07-28 Thread Alexander Graf
On 14.07.14 11:15, Bharat Bhushan wrote: This patchset add support for - software breakpoint - h/w breakpoint - h/w watchpoint Please find description in individual patch. Thanks, applied (with minor edits in comments) to ppc-next-2.2. Alex

Re: [Qemu-devel] [PULL for-2.1 0/3] Last minute patches

2014-07-28 Thread Igor Mammedov
On Mon, 28 Jul 2014 14:59:26 +0200 Juan Quintela wrote: > Peter Maydell wrote: > > On 25 July 2014 15:22, Paolo Bonzini wrote: > >> Since Igor hasn't sent his patches, and I'm leaving the office, I pushed > >> this to > >> > >>git://github.com/bonzini/qemu.git tags/for-upstream-full > >> >

  1   2   >