Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Amit Shah
repo soon.) and the json dump is here: http://paste.fedoraproject.org/119725/01268914/ Comparing qemu-1.0 to qemu-2.1 -M pc-1.0 gives a few errors: $ ./scripts/vmstate-static-checker.py -s v1.0-1.0.json -d upstream-20140722-v1.0.json Section "xio3130-downstream" Description "xi

[Qemu-devel] [PATCH] tests: set QEMU_AUDIO_DRV=none for pci sound cards

2014-07-22 Thread Gerd Hoffmann
This way the tests run without sound hardware being present on the build machine. Even with sound hardware it IMO isn't very useful to use it in regression testing. Once the sound card tests are advanced enougth that they try to actually play sound we probably want the guests sound output written

[Qemu-devel] [PATCH for-2.1] usb: mtp: tag root property as experimental

2014-07-22 Thread Gerd Hoffmann
Reason: we don't want commit to that interface yet. Possibly the implementation will be switched over to use fsdev. Suggested-by: Paolo Bonzini Signed-off-by: Gerd Hoffmann --- hw/usb/dev-mtp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/usb/dev-mtp.c b/hw/usb/dev-mt

[Qemu-devel] [Bug 1346784] [NEW] qemu internal memory areas visible to a guest via /proc/self/maps

2014-07-22 Thread Mikhail Ilyin
Public bug reported: Qemu internal memory areas are not suppressed in the output and are visible to a guest via /proc/self/maps. $ echo "int main() { return 0; }" > /tmp/test.c $ gcc -m32 -fsanitize=address -fno-common -Wall -g -fPIC -o /tmp/test /tmp/test.c $ qemu-i386-static -R 0 /tmp/test

Re: [Qemu-devel] [PATCH for-2.1] usb: mtp: tag root property as experimental

2014-07-22 Thread Paolo Bonzini
Il 22/07/2014 09:34, Gerd Hoffmann ha scritto: > Reason: we don't want commit to that interface yet. Possibly > the implementation will be switched over to use fsdev. > > Suggested-by: Paolo Bonzini > Signed-off-by: Gerd Hoffmann > --- > hw/usb/dev-mtp.c | 2 +- > 1 file changed, 1 insertion(+

Re: [Qemu-devel] [PATCH for-2.1] exec: fix migration with devices that use address_space_rw

2014-07-22 Thread Gerd Hoffmann
On Mo, 2014-07-21 at 17:06 +0200, Paolo Bonzini wrote: > Devices that use address_space_rw to write large areas to memory > (as opposed to address_space_map/unmap) were broken with respect > to migration since fe680d0 (exec: Limit translation limiting in > address_space_translate to xen, 2014-05-07

[Qemu-devel] [Bug 1346769] Re: /proc/self/maps content returned to 32-bits guest under 64-bits qemu

2014-07-22 Thread Peter Maydell
I think the ifdefs are just accident of history and we can safely remove them. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1346769 Title: /proc/self/maps content returned to 32-bits guest under 6

Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Amit Shah
ons at release-time becomes easy. > > I've backported the -dump-vmstate patches to qemu-1.0, the backport is > here: > > http://paste.fedoraproject.org/119724/14060126/ > > (I'll be publishing these to my git repo soon.) > > and the json dump is here: > >

Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Paolo Bonzini
Il 22/07/2014 11:50, Amit Shah ha scritto: > These are the ones that remain after fixing up the rest: > > Section "xio3130-downstream" Description "xio3130-express-downstream-port": > version error: 2 > 0 > Section "usb-host" Section "usb-host" Description "usb-host": minimum version > error: 0

Re: [Qemu-devel] [RFC PATCH V4 6/6] monitor: Add drift info to 'info jit'

2014-07-22 Thread Sebastian Tanase
- Mail original - > De: "Paolo Bonzini" > À: "Sebastian Tanase" , qemu-devel@nongnu.org > Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell" > , > mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com, > lcapitul...@redhat.com, crobi...@redhat.com, > arm...@

Re: [Qemu-devel] [RFC PATCH V4 6/6] monitor: Add drift info to 'info jit'

2014-07-22 Thread Paolo Bonzini
Il 22/07/2014 11:58, Sebastian Tanase ha scritto: > > -timers_state.cpu_clock_offset contains the offset between the real and > virtual clocks. > However, when using the value of the virtual clock > (qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL)), > qemu_icount_bias already includes this offset because,

Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Amit Shah
On (Tue) 22 Jul 2014 [11:55:29], Paolo Bonzini wrote: > Il 22/07/2014 11:50, Amit Shah ha scritto: > > These are the ones that remain after fixing up the rest: > > > > Section "xio3130-downstream" Description "xio3130-express-downstream-port": > > version error: 2 > 0 > > Section "usb-host" Secti

Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Alex Bligh
On 22 Jul 2014, at 11:22, Amit Shah wrote: > Also interesting is qemu 1.0 -> qemu-kvm-1.0: > > Section "isa-pit", Description "i8254": expected field "channels", got > "flags"; skipping rest This one, together with the PIIX4 one (which for some reason doesn't show up) where the two I hit, afte

[Qemu-devel] [PULL 2/2] kvm-all: Use 'tmpcpu' instead of 'cpu' in sub-looping to avoid 'cpu' be NULL

2014-07-22 Thread Paolo Bonzini
From: Chen Gang If kvm_arch_remove_sw_breakpoint() in CPU_FOREACH() always be fail, it will let 'cpu' NULL. And the next kvm_arch_remove_sw_breakpoint() in QTAILQ_FOREACH_SAFE() will get NULL parameter for 'cpu'. And kvm_arch_remove_sw_breakpoint() can assumes 'cpu' must never be NULL, so need d

[Qemu-devel] [PULL for-2.1 0/2] Two fixes for KVM and memory

2014-07-22 Thread Paolo Bonzini
The following changes since commit fa666c10f2f3e15685ff88abd3bc433ddce012d6: Revert "kvmclock: Ensure time in migration never goes backward" (2014-07-18 15:28:03 +0200) are available in the git repository at: git://github.com/bonzini/qemu.git tags/for-upstream for you to fetch changes up t

[Qemu-devel] [PULL 1/2] exec: fix migration with devices that use address_space_rw

2014-07-22 Thread Paolo Bonzini
Devices that use address_space_rw to write large areas to memory (as opposed to address_space_map/unmap) were broken with respect to migration since fe680d0 (exec: Limit translation limiting in address_space_translate to xen, 2014-05-07). Such devices include IDE CD-ROMs. The reason is that inval

Re: [Qemu-devel] ipv6 slirp network

2014-07-22 Thread Vasiliy Tolstov
2014-07-22 0:49 GMT+04:00 Samuel Thibault : > Alternatively, you can track the parameters branch, which I don't > regenerate. Thanks. Now all works fine. -- Vasiliy Tolstov, e-mail: v.tols...@selfip.ru jabber: v...@selfip.ru

Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Amit Shah
On (Tue) 22 Jul 2014 [11:32:41], Alex Bligh wrote: > > On 22 Jul 2014, at 11:22, Amit Shah wrote: > > > Also interesting is qemu 1.0 -> qemu-kvm-1.0: > > > > Section "isa-pit", Description "i8254": expected field "channels", got > > "flags"; skipping rest > > This one, together with the PIIX4

Re: [Qemu-devel] [PATCH v3] virtio-rng: Add human-readable error message for negative max-bytes parameter

2014-07-22 Thread Markus Armbruster
John Snow writes: > If a negative integer is used for the max_bytes parameter, QEMU currently > calls abort() and leaves behind a core dump. This patch adds a simple > error message to make the reason for the termination clearer. It also avoids the abort, doesn't it? > There is an underlying in

Re: [Qemu-devel] [PATCH for 2.1] ppc: fix -mem-path failure

2014-07-22 Thread Alexander Graf
On 07/21/2014 11:30 AM, Hu Tao wrote: commit e938ba0c tried to enable -mem-path for ppc but breaked some ppc boards. The problems are: 1. it fails when allocating memory for rom, sram whose sizes are less than huge page size: ./ppc-softmmu/qemu-system-ppc -m 512 -mem-path /hugepages/

Re: [Qemu-devel] [PATCH v3] virtio-rng: Add human-readable error message for negative max-bytes parameter

2014-07-22 Thread Amit Shah
On (Mon) 21 Jul 2014 [17:44:37], John Snow wrote: > If a negative integer is used for the max_bytes parameter, QEMU currently > calls abort() and leaves behind a core dump. This patch adds a simple > error message to make the reason for the termination clearer. It avoids the abort(), which in the

Re: [Qemu-devel] [PATCH v3] virtio-rng: Add human-readable error message for negative max-bytes parameter

2014-07-22 Thread Amit Shah
On (Mon) 21 Jul 2014 [17:44:37], John Snow wrote: > If a negative integer is used for the max_bytes parameter, QEMU currently > calls abort() and leaves behind a core dump. This patch adds a simple > error message to make the reason for the termination clearer. > > There is an underlying insuffici

Re: [Qemu-devel] [PATCH] tests: set QEMU_AUDIO_DRV=none for pci sound cards

2014-07-22 Thread Markus Armbruster
Gerd Hoffmann writes: > This way the tests run without sound hardware being present > on the build machine. Even with sound hardware it IMO isn't > very useful to use it in regression testing. Once the sound > card tests are advanced enougth that they try to actually > play sound we probably wa

Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Alex Bligh
On 22 Jul 2014, at 11:54, Amit Shah wrote: >> >> This one, together with the PIIX4 one (which for some reason doesn't >> show up) where the two I hit, after manually fixing the rom sizes >> stuff on the command line. >> >> Apparently "flags" and "channels" are pseudonyms. > > No, they're not;

Re: [Qemu-devel] [PATCH v3] virtio-rng: Add human-readable error message for negative max-bytes parameter

2014-07-22 Thread Markus Armbruster
Amit Shah writes: > On (Mon) 21 Jul 2014 [17:44:37], John Snow wrote: >> If a negative integer is used for the max_bytes parameter, QEMU currently >> calls abort() and leaves behind a core dump. This patch adds a simple >> error message to make the reason for the termination clearer. > > It avoid

Re: [Qemu-devel] [PATCH v3] virtio-rng: Add human-readable error message for negative max-bytes parameter

2014-07-22 Thread Markus Armbruster
Amit Shah writes: > On (Mon) 21 Jul 2014 [17:44:37], John Snow wrote: >> If a negative integer is used for the max_bytes parameter, QEMU currently >> calls abort() and leaves behind a core dump. This patch adds a simple >> error message to make the reason for the termination clearer. >> >> There

[Qemu-devel] [PATCH 1/2] vmstate static checker: whitelist additions

2014-07-22 Thread Amit Shah
Comparing json outputs from qemu-1.0 with qemu-2.1 turned up a few description name changes; whitelist them here. Signed-off-by: Amit Shah --- scripts/vmstate-static-checker.py | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/scripts/vmstate-static-checker.py

[Qemu-devel] [PATCH 0/2] vmstate static checker: handle unused fields,

2014-07-22 Thread Amit Shah
Hi, This patchset updates the whitelist and adds handling for 'unused' fields. These were found when comparing qemu-1.0 json output with qemu-2.1 -M pc-1.0. Amit Shah (2): vmstate static checker: whitelist additions checker: ignore fields marked unused scripts/vmstate-static-checker.py | 7

[Qemu-devel] [PATCH 2/2] checker: ignore fields marked unused

2014-07-22 Thread Amit Shah
While comparing qemu-1.0 json output with qemu-2.1, a few fields got marked unused. These need to be skipped over, and not flagged as mismatches. For handling unused fields, the exact number of bytes need to be skipped over as the size of the unused field. Signed-off-by: Amit Shah --- scripts/

Re: [Qemu-devel] [PATCH v2 2/3] tap-bsd: implement a FreeBSD only version of tap_open

2014-07-22 Thread Roger Pau Monné
On 27/05/14 15:29, Stefan Hajnoczi wrote: > On Fri, May 23, 2014 at 05:57:48PM +0200, Roger Pau Monne wrote: >> The current behaviour of tap_open for BSD systems differ greatly from >> it's Linux counterpart. Since FreeBSD supports interface renaming and >> tap device cloning by opening /dev/tap, i

Re: [Qemu-devel] [PATCH v3] virtio-rng: Add human-readable error message for negative max-bytes parameter

2014-07-22 Thread Amit Shah
On (Tue) 22 Jul 2014 [13:41:43], Markus Armbruster wrote: > Amit Shah writes: > > > On (Mon) 21 Jul 2014 [17:44:37], John Snow wrote: > >> If a negative integer is used for the max_bytes parameter, QEMU currently > >> calls abort() and leaves behind a core dump. This patch adds a simple > >> erro

Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Amit Shah
On (Tue) 22 Jul 2014 [12:38:14], Alex Bligh wrote: > > On 22 Jul 2014, at 11:54, Amit Shah wrote: > >> > >> This one, together with the PIIX4 one (which for some reason doesn't > >> show up) where the two I hit, after manually fixing the rom sizes > >> stuff on the command line. > >> > >> Appar

[Qemu-devel] [PULL v2 for-2.1] virtio-rng: Fix abort on invalid input

2014-07-22 Thread Amit Shah
The following changes since commit 35858955e6c6f9ef41c199d15457c13426ac6434: Merge remote-tracking branch 'remotes/afaerber/tags/qom-devices-for-2.1' into staging (2014-07-21 18:06:12 +0100) are available in the git repository at: git://git.kernel.org/pub/scm/virt/qemu/amit/virtio-rng.git

[Qemu-devel] [PATCH 1/1] virtio-rng: Add human-readable error message for negative max-bytes parameter

2014-07-22 Thread Amit Shah
From: John Snow If a negative integer is used for the max_bytes parameter, QEMU currently calls abort() and leaves behind a core dump. This patch replaces the abort with a simple error message to make the reason for the termination clearer. This also ensures device-hotplug with invalid input does

Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Paolo Bonzini
Il 22/07/2014 13:54, Amit Shah ha scritto: >> > a) because I'm using qemu-kvm-1.0, not qemu-1.0. You looked at qemu-1.0 >> >in the previous message; that's not directly relevant for what I'm >> >looking at as my source is qemu-kvm-1.0 (qemu vs qemu-kvm). > Sigh; please read both my replies.

Re: [Qemu-devel] [PULL for-2.1 0/2] Two fixes for KVM and memory

2014-07-22 Thread Peter Maydell
On 22 July 2014 11:51, Paolo Bonzini wrote: > The following changes since commit fa666c10f2f3e15685ff88abd3bc433ddce012d6: > > Revert "kvmclock: Ensure time in migration never goes backward" (2014-07-18 > 15:28:03 +0200) > > are available in the git repository at: > > git://github.com/bonzini

Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Alex Bligh
On 22 Jul 2014, at 12:54, Amit Shah wrote: > Sigh; please read both my replies. > > The only difference between qemu-kvm-1.0 and qemu-1.0 is the presence > of the pci-assign section in qemu-kvm-1.0. All other output from > qemu-1.0 -> qemu-2.1 is equally applicable to qemu-kvm-1.0 -> > qemu-2.

Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Alex Bligh
On 22 Jul 2014, at 13:12, Paolo Bonzini wrote: > There's something similar going on with PIIX4_PM but I don't remember > the details. From memory: * qemu-1.0 uses the v2 format * qemu-kvm-1.0 uses the v3 format but advertises itself as v2 * qemu-2.1 uses the v3 format I don't quite understand

Re: [Qemu-devel] [PATCH v2 2/3] tap-bsd: implement a FreeBSD only version of tap_open

2014-07-22 Thread Stefano Stabellini
On Tue, 22 Jul 2014, Roger Pau Monné wrote: > On 27/05/14 15:29, Stefan Hajnoczi wrote: > > On Fri, May 23, 2014 at 05:57:48PM +0200, Roger Pau Monne wrote: > >> The current behaviour of tap_open for BSD systems differ greatly from > >> it's Linux counterpart. Since FreeBSD supports interface renam

Re: [Qemu-devel] [PATCH v6 1/5] block: Support Archipelago as a QEMU block backend

2014-07-22 Thread Stefan Hajnoczi
On Fri, Jun 27, 2014 at 11:24:08AM +0300, Chrysostomos Nanakos wrote: > +xseg_set_req_data(s->xseg, req, reqdata); > +if (op == ARCHIP_OP_WRITE) { > +data = xseg_get_data(s->xseg, req); > +if (!data) { > +archipelagolog("Cannot get XSEG data\n"); > +g

Re: [Qemu-devel] [PATCH v6 1/5] block: Support Archipelago as a QEMU block backend

2014-07-22 Thread Stefan Hajnoczi
On Thu, Jul 10, 2014 at 01:04:54PM +0300, Chrysostomos Nanakos wrote: > On 07/10/2014 03:23 AM, Jeff Cody wrote: > >On Fri, Jun 27, 2014 at 11:24:08AM +0300, Chrysostomos Nanakos wrote: > >>+err_exit: > >>+__sync_add_and_fetch(&segreq->failed, 1); > >>+if (segments_nr == 1) { > >>+i

Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Amit Shah
On (Tue) 22 Jul 2014 [14:12:00], Paolo Bonzini wrote: > Il 22/07/2014 13:54, Amit Shah ha scritto: > >> > a) because I'm using qemu-kvm-1.0, not qemu-1.0. You looked at qemu-1.0 > >> >in the previous message; that's not directly relevant for what I'm > >> >looking at as my source is qemu-kv

Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Amit Shah
On (Tue) 22 Jul 2014 [13:15:41], Alex Bligh wrote: > > On 22 Jul 2014, at 12:54, Amit Shah wrote: > > > Sigh; please read both my replies. > > > > The only difference between qemu-kvm-1.0 and qemu-1.0 is the presence > > of the pci-assign section in qemu-kvm-1.0. All other output from > > qemu

Re: [Qemu-devel] Live migrate, inconsistent machine types - new machine type to fix?

2014-07-22 Thread Amit Shah
On (Tue) 22 Jul 2014 [13:19:43], Alex Bligh wrote: > > On 22 Jul 2014, at 13:12, Paolo Bonzini wrote: > > > There's something similar going on with PIIX4_PM but I don't remember > > the details. > > From memory: > > * qemu-1.0 uses the v2 format > * qemu-kvm-1.0 uses the v3 format but advertis

Re: [Qemu-devel] [PATCH for-2.1] exec: fix migration with devices that use address_space_rw

2014-07-22 Thread Juan Quintela
Paolo Bonzini wrote: > Devices that use address_space_rw to write large areas to memory > (as opposed to address_space_map/unmap) were broken with respect > to migration since fe680d0 (exec: Limit translation limiting in > address_space_translate to xen, 2014-05-07). Such devices include > IDE CD

Re: [Qemu-devel] KVM call for agenda for 2014-07-22

2014-07-22 Thread Juan Quintela
Juan Quintela wrote: > Hi > > Please, send any topic that you are interested in covering. > > Thanks, Juan. > > Call details: > > 15:00 CEST > 13:00 UTC > 09:00 EDT > > Every two weeks > > If you need phone number details, contact me privately No topis, no call. See you in 2 weeks.

Re: [Qemu-devel] [PULL v2 for-2.1] virtio-rng: Fix abort on invalid input

2014-07-22 Thread Peter Maydell
On 22 July 2014 13:03, Amit Shah wrote: > The following changes since commit 35858955e6c6f9ef41c199d15457c13426ac6434: > > Merge remote-tracking branch 'remotes/afaerber/tags/qom-devices-for-2.1' > into staging (2014-07-21 18:06:12 +0100) > > are available in the git repository at: > > > git:

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

2014-07-22 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: Kevin Wolf CC: Stefan Hajnoczi --- block/parallels.c | 13 +++-- 1 file changed,

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

2014-07-22 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 2/4] block/parallels: replace tabs with spaces in block/parallels.c

2014-07-22 Thread Denis V. Lunev
Signed-off-by: Denis V. Lunev 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 c44df87..8f9ec8a 100644 --- a/block/parallels.c +++ b/block/parallels.c @@ -61,11 +61

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

2014-07-22 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 explicitely zeroed according to Parallels. They will be used for images with signature "WithouFreSpacExt" - inu

Re: [Qemu-devel] [RFC PATCH V4 6/6] monitor: Add drift info to 'info jit'

2014-07-22 Thread Sebastian Tanase
- Mail original - > De: "Paolo Bonzini" > À: "Sebastian Tanase" > Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell" > , > mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com, > lcapitul...@redhat.com, crobi...@redhat.com, > arm...@redhat.com, wenchaoq...

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

2014-07-22 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

Re: [Qemu-devel] [RFC PATCH V4 0/6] icount: Implement delay algorithm between guest and host clocks

2014-07-22 Thread Sebastian Tanase
- Mail original - > De: "Paolo Bonzini" > À: "Sebastian Tanase" , qemu-devel@nongnu.org > Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell" > , > mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com, > lcapitul...@redhat.com, crobi...@redhat.com, > arm...@

Re: [Qemu-devel] [RFC PATCH V4 0/6] icount: Implement delay algorithm between guest and host clocks

2014-07-22 Thread Paolo Bonzini
Il 22/07/2014 16:02, Sebastian Tanase ha scritto: > Yes, QEMU_CLOCK_VIRTUAL counts up from qemu_clock_get_ns(QEMU_CLOCK_REALTIME) > on ARM (I have only tested with the versatilepb and vexpress boards). That's a bug to fix indeed, then---it should count up from 0 without icount, and icount shouldn'

Re: [Qemu-devel] [RFC PATCH V4 0/6] icount: Implement delay algorithm between guest and host clocks

2014-07-22 Thread Sebastian Tanase
- Mail original - > De: "Paolo Bonzini" > À: "Sebastian Tanase" > Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell" > , > mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com, > lcapitul...@redhat.com, crobi...@redhat.com, > arm...@redhat.com, wenchaoq...

Re: [Qemu-devel] [RFC PATCH V4 0/6] icount: Implement delay algorithm between guest and host clocks

2014-07-22 Thread Sebastian Tanase
- Mail original - > De: "Paolo Bonzini" > À: "Sebastian Tanase" > Cc: aligu...@amazon.com, afaer...@suse.de, r...@twiddle.net, "peter maydell" > , > mich...@walle.cc, a...@alex.org.uk, stefa...@redhat.com, > lcapitul...@redhat.com, crobi...@redhat.com, > arm...@redhat.com, wenchaoq...

Re: [Qemu-devel] [RFC PATCH V4 0/6] icount: Implement delay algorithm between guest and host clocks

2014-07-22 Thread Paolo Bonzini
Il 22/07/2014 17:17, Sebastian Tanase ha scritto: >> > That's a bug to fix indeed, then---it should count up from 0 without >> > icount, and icount shouldn't affect this. Thanks for investigating >> > it. > Just to be sure I don't missunderstand, when you say "without icount" > you refer to qemu_i

[Qemu-devel] [PATCH for-2.1 for-stable] vmstate_xhci_event: fix unterminated field list

2014-07-22 Thread Laszlo Ersek
"vmstate_xhci_event" was introduced in commit 37352df3 ("xhci: add live migration support"), and first released in v1.6.0. The field list in this VMSD is not terminated with the VMSTATE_END_OF_LIST() macro. During normal use (ie. migration), the issue is practically invisible, because the "vmstate

Re: [Qemu-devel] [PATCH for-2.1 for-stable] vmstate_xhci_event: fix unterminated field list

2014-07-22 Thread Amit Shah
On (Tue) 22 Jul 2014 [17:26:41], Laszlo Ersek wrote: > "vmstate_xhci_event" was introduced in commit 37352df3 ("xhci: add live > migration support"), and first released in v1.6.0. The field list in this > VMSD is not terminated with the VMSTATE_END_OF_LIST() macro. > > During normal use (ie. migra

[Qemu-devel] [PULL 1/1] ppc: fix -mem-path failure

2014-07-22 Thread Alexander Graf
From: Hu Tao commit e938ba0c tried to enable -mem-path for ppc but breaked some ppc boards. The problems are: 1. it fails when allocating memory for rom, sram whose sizes are less than huge page size: ./ppc-softmmu/qemu-system-ppc -m 512 -mem-path /hugepages/ \ -kernel /home/hutao/Do

Re: [Qemu-devel] [PATCH v3] virtio-rng: Add human-readable error message for negative max-bytes parameter

2014-07-22 Thread John Snow
On 07/22/2014 07:48 AM, Amit Shah wrote: On (Tue) 22 Jul 2014 [13:41:43], Markus Armbruster wrote: Amit Shah writes: On (Mon) 21 Jul 2014 [17:44:37], John Snow wrote: If a negative integer is used for the max_bytes parameter, QEMU currently calls abort() and leaves behind a core dump. This

[Qemu-devel] [PULL 2.1 0/1] ppc patch queue 2014-07-22

2014-07-22 Thread Alexander Graf
Hi Peter, This is my current patch queue for ppc. Please pull. Alex The following changes since commit 35858955e6c6f9ef41c199d15457c13426ac6434: Merge remote-tracking branch 'remotes/afaerber/tags/qom-devices-for-2.1' into staging (2014-07-21 18:06:12 +0100) are available in the git repos

Re: [Qemu-devel] [PATCH for-2.1 for-stable] vmstate_xhci_event: fix unterminated field list

2014-07-22 Thread Paolo Bonzini
Il 22/07/2014 17:26, Laszlo Ersek ha scritto: > "vmstate_xhci_event" was introduced in commit 37352df3 ("xhci: add live > migration support"), and first released in v1.6.0. The field list in this > VMSD is not terminated with the VMSTATE_END_OF_LIST() macro. > > During normal use (ie. migration),

Re: [Qemu-devel] [RFC PATCH V4 0/6] icount: Implement delay algorithm between guest and host clocks

2014-07-22 Thread Paolo Bonzini
Il 22/07/2014 17:28, Sebastian Tanase ha scritto: > When not using "-icount" everything is fine because QEMU_CLOCK_VIRTUAL > is based on the real clock and the offset; qemu_icount_bias > doesn't come into play. So the vm_clock_warp_start fix is only for the case > where we use "-icount". I mea

Re: [Qemu-devel] [PATCH for-2.1 for-stable] vmstate_xhci_event: fix unterminated field list

2014-07-22 Thread Laszlo Ersek
On 07/22/14 17:42, Paolo Bonzini wrote: > Il 22/07/2014 17:26, Laszlo Ersek ha scritto: >> "vmstate_xhci_event" was introduced in commit 37352df3 ("xhci: add live >> migration support"), and first released in v1.6.0. The field list in this >> VMSD is not terminated with the VMSTATE_END_OF_LIST() ma

[Qemu-devel] [PATCH 2/3] intel-iommu: add DMAR table to ACPI tables

2014-07-22 Thread Le Tan
Expose Intel IOMMU to the BIOS. If object of TYPE_INTEL_IOMMU_DEVICE exists, add DMAR table to ACPI RSDT table. For now the DMAR table indicates that there is only one hardware unit without INTR_REMAP capability on the platform. Signed-off-by: Le Tan --- hw/i386/acpi-build.c | 41 +++

[Qemu-devel] [PATCH 1/3] intel-iommu: introduce Intel IOMMU (VT-d) emulation

2014-07-22 Thread Le Tan
Add support for emulating Intel IOMMU according to the VT-d specification for the q35 chipset machine. Implement the logic for DMAR (DMA remapping) without PASID support. Use register-based invalidation for context-cache invalidation and IOTLB invalidation. Basic fault reporting and caching are not

[Qemu-devel] [PATCH 0/3] intel-iommu: introduce Intel IOMMU (VT-d) emulation to q35 chipset

2014-07-22 Thread Le Tan
Hi, These patches are intended to introduce Intel IOMMU (VT-d) emulation to q35 chipset. The major job in these patches is to add support for emulating Intel IOMMU according to the VT-d specification, including basic responses to CSRs accesses, the logic of DMAR (DMA remapping) and DMA memory addr

[Qemu-devel] [PATCH 3/3] intel-iommu: add Intel IOMMU emulation to q35 and add a machine option "vtd" as a switch

2014-07-22 Thread Le Tan
Add Intel IOMMU emulation to q35 chipset and expose it to the guest. 1. Add a machine option. Users can use "-machine vtd=on|off" in the command line to enable/disable Intel IOMMU. The default is off. 2. Accroding to the machine option, q35 will initialize the Intel IOMMU and use pci_setup_iommu()

Re: [Qemu-devel] [PATCH v3] virtio-rng: Add human-readable error message for negative max-bytes parameter

2014-07-22 Thread Amit Shah
On (Tue) 22 Jul 2014 [11:30:28], John Snow wrote: > > On 07/22/2014 07:48 AM, Amit Shah wrote: > -assert(vrng->conf.max_bytes <= INT64_MAX); > +/* Workaround: Property parsing does not enforce unsigned integers, > + * So this is a hack to reject such numbers. */ > +

Re: [Qemu-devel] [PATCH for-2.1 for-stable] vmstate_xhci_event: fix unterminated field list

2014-07-22 Thread Peter Maydell
On 22 July 2014 16:48, Laszlo Ersek wrote: > On 07/22/14 17:42, Paolo Bonzini wrote: >> Cc: qemu-sta...@nongnu.org > > As far as I can see, this address was present in my original To: list. > :) (Admittedly, not with CC.) Including the line in the commit message means it can get pulled back out o

[Qemu-devel] [PATCH] hw/misc/imx_ccm.c: Add missing VMState list terminator

2014-07-22 Thread Peter Maydell
The VMStateDescription for the imx_ccm device was missing its terminator. Found by static search of the codebase using a regex based on one suggested by Ian Jackson: pcregrep -rMi '(?s)VMStateField(?:(?!END_OF_LIST).)*?;' $(git grep -l 'VMStateField\[\]') Signed-off-by: Peter Maydell Cc: qemu-

Re: [Qemu-devel] [PULL 2.1 0/1] ppc patch queue 2014-07-22

2014-07-22 Thread Peter Maydell
On 22 July 2014 16:38, Alexander Graf wrote: > Hi Peter, > > This is my current patch queue for ppc. Please pull. > > Alex > > > The following changes since commit 35858955e6c6f9ef41c199d15457c13426ac6434: > > Merge remote-tracking branch 'remotes/afaerber/tags/qom-devices-for-2.1' > into stag

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

2014-07-22 Thread Alex Bligh
Add a machine type pc-1.0-qemu-kvm for live migrate compatibility with qemu-kvm version 1.0. Signed-off-by: Alex Bligh --- hw/acpi/piix4.c | 49 -- hw/i386/pc_piix.c| 31 + hw/timer/i8254_common.c | 4

Re: [Qemu-devel] [Bug 1344320] Re: qemu-aarch64 cannot execute glibc

2014-07-22 Thread Richard Henderson
On 07/21/2014 10:37 AM, Peter Maydell wrote: >> It's trying to measure clock cycles required to perform the startup >> relocations. > > That's a neat trick, given that the generic timers are not cycle > counters! They're a fixed frequency counter which is generally > unrelated and rather slower th

Re: [Qemu-devel] [PATCH] Add ACPI tables for TPM

2014-07-22 Thread Michael S. Tsirkin
On Tue, Jul 22, 2014 at 03:55:38PM -0400, Stefan Berger wrote: > From: Stefan Berger > > Add an SSDT ACPI table for the TPM device. > Add a TCPA table for BIOS logging area when a TPM is being used. > > The latter follows this spec here: > > http://www.trustedcomputinggroup.org/files/static_pag

Re: [Qemu-devel] [PATCH] pci: Don't deliver MSI/MSI-X messages if bus master support is off

2014-07-22 Thread Michael S. Tsirkin
On Mon, Jul 21, 2014 at 12:04:22AM +0200, Jan Kiszka wrote: > On 2014-07-20 23:03, Michael S. Tsirkin wrote: > > On Sun, Jul 20, 2014 at 11:45:10PM +0200, Jan Kiszka wrote: > >> On 2014-07-20 21:48, Michael S. Tsirkin wrote: > >>> On Sat, Jul 19, 2014 at 06:55:48PM +0200, Jan Kiszka wrote: > F

Re: [Qemu-devel] [PATCH 1/2] qemu-img: Allow source cache mode specification

2014-07-22 Thread Max Reitz
On 21.07.2014 17:52, Eric Blake wrote: On 07/19/2014 02:35 PM, Max Reitz wrote: Many qemu-img subcommands only read the source file(s) once. For these use cases, a full write-back cache is unnecessary and mainly clutters host cache memory. Though this is generally no concern as cache memory is f

Re: [Qemu-devel] [PATCH 2/2] qemu-img: Allow cache mode specification for amend

2014-07-22 Thread Max Reitz
On 21.07.2014 17:57, Eric Blake wrote: On 07/19/2014 02:35 PM, Max Reitz wrote: qemu-img amend may extensively modify the target image, depending on the options to be amended (e.g. conversion to qcow2 compat level 0.10 from 1.1 for an image with many unallocated zero clusters). Therefore it make

Re: [Qemu-devel] [PATCH v2 2/5] block: vdi - use block layer ops in vdi_create, instead of posix calls

2014-07-22 Thread Max Reitz
On 21.07.2014 21:52, Jeff Cody wrote: Use the block layer to create, and write to, the image file in the VDI .bdrv_create() operation. This has a couple of benefits: Images can now be created over protocols, and hacks such as NOCOW are not needed in the image format driver, and the underlying fi

Re: [Qemu-devel] [PATCH v2 2/5] block: vdi - use block layer ops in vdi_create, instead of posix calls

2014-07-22 Thread Jeff Cody
On Tue, Jul 22, 2014 at 10:14:58PM +0200, Max Reitz wrote: > On 21.07.2014 21:52, Jeff Cody wrote: > >Use the block layer to create, and write to, the image file in the > >VDI .bdrv_create() operation. > > > >This has a couple of benefits: Images can now be created over protocols, > >and hacks such

Re: [Qemu-devel] [PATCH v2 2/5] block: vdi - use block layer ops in vdi_create, instead of posix calls

2014-07-22 Thread Max Reitz
On 22.07.2014 22:19, Jeff Cody wrote: On Tue, Jul 22, 2014 at 10:14:58PM +0200, Max Reitz wrote: On 21.07.2014 21:52, Jeff Cody wrote: Use the block layer to create, and write to, the image file in the VDI .bdrv_create() operation. This has a couple of benefits: Images can now be created over

Re: [Qemu-devel] [PATCH v2 5/5] block: iotest - update 084 to test static VDI image creation

2014-07-22 Thread Max Reitz
On 21.07.2014 21:52, Jeff Cody wrote: This updates the VDI corruption test to also test static VDI image creation, as well as the default dynamic image creation. Signed-off-by: Jeff Cody --- tests/qemu-iotests/084 | 16 ++-- tests/qemu-iotests/084.out | 15 +++ 2

Re: [Qemu-devel] [PATCH v2 5/5] block: iotest - update 084 to test static VDI image creation

2014-07-22 Thread Jeff Cody
On Tue, Jul 22, 2014 at 10:21:48PM +0200, Max Reitz wrote: > On 21.07.2014 21:52, Jeff Cody wrote: > >This updates the VDI corruption test to also test static VDI image > >creation, as well as the default dynamic image creation. > > > >Signed-off-by: Jeff Cody > >--- > > tests/qemu-iotests/084

Re: [Qemu-devel] [PATCH] Add ACPI tables for TPM

2014-07-22 Thread Michael S. Tsirkin
On Tue, Jul 22, 2014 at 04:13:01PM -0400, Stefan Berger wrote: > On 07/22/2014 03:05 PM, Michael S. Tsirkin wrote: > >On Tue, Jul 22, 2014 at 03:55:38PM -0400, Stefan Berger wrote: > >>From: Stefan Berger > >> > >>Add an SSDT ACPI table for the TPM device. > >>Add a TCPA table for BIOS logging are

Re: [Qemu-devel] [PATCH] Add ACPI tables for TPM

2014-07-22 Thread Stefan Berger
On 07/22/2014 03:05 PM, Michael S. Tsirkin wrote: On Tue, Jul 22, 2014 at 03:55:38PM -0400, Stefan Berger wrote: From: Stefan Berger Add an SSDT ACPI table for the TPM device. Add a TCPA table for BIOS logging area when a TPM is being used. The latter follows this spec here: http://www.trust

[Qemu-devel] [PATCH v2 2/2] qemu-img: Allow cache mode specification for amend

2014-07-22 Thread Max Reitz
qemu-img amend may extensively modify the target image, depending on the options to be amended (e.g. conversion to qcow2 compat level 0.10 from 1.1 for an image with many unallocated zero clusters). Therefore it makes sense to allow the user to specify the cache mode to be used. Signed-off-by: Max

[Qemu-devel] [PATCH v2 0/2] qemu-img: Allow source cache mode specification

2014-07-22 Thread Max Reitz
Currently, qemu-img does not allow setting the cache mode for source images. However, it reads images generally only once, therefore a full writeback cache unnecessarily clutters the host cache. In case the user finds this undesirable, there has to be a way of disabling that cache. This series firs

[Qemu-devel] [PATCH v2 1/2] qemu-img: Allow source cache mode specification

2014-07-22 Thread Max Reitz
Many qemu-img subcommands only read the source file(s) once. For these use cases, a full write-back cache is unnecessary and mainly clutters host cache memory. Though this is generally no concern as cache memory is freely available and can be scaled by the host OS, it may become a concern with thin

[Qemu-devel] [PATCH for-2.1] docs: document remaining QMP events

2014-07-22 Thread Eric Blake
Commit dfab4892 restored this file, but did not address any of the grammar problems that had been fixed in passing when moving events out of this file. There are also a couple events that were undocumented since introduction, and one that had been added only in the time that this file was temporar

Re: [Qemu-devel] [PATCH 1/3] intel-iommu: introduce Intel IOMMU (VT-d) emulation

2014-07-22 Thread Michael S. Tsirkin
On Tue, Jul 22, 2014 at 11:47:48PM +0800, Le Tan wrote: > Add support for emulating Intel IOMMU according to the VT-d specification for > the q35 chipset machine. Implement the logic for DMAR (DMA remapping) without > PASID support. Use register-based invalidation for context-cache invalidation > a

Re: [Qemu-devel] [PATCH v2 1/2] qemu-img: Allow source cache mode specification

2014-07-22 Thread Eric Blake
On 07/22/2014 02:58 PM, Max Reitz wrote: > Many qemu-img subcommands only read the source file(s) once. For these > use cases, a full write-back cache is unnecessary and mainly clutters > host cache memory. Though this is generally no concern as cache memory > is freely available and can be scaled

[Qemu-devel] [PATCH] Add ACPI tables for TPM

2014-07-22 Thread Stefan Berger
From: Stefan Berger Add an SSDT ACPI table for the TPM device. Add a TCPA table for BIOS logging area when a TPM is being used. The latter follows this spec here: http://www.trustedcomputinggroup.org/files/static_page_files/DCD4188E-1A4B-B294-D050A155FB6F7385/TCG_ACPIGeneralSpecification_Public

[Qemu-devel] [PATCH v10 00/14] qemu-img: Implement commit like QMP

2014-07-22 Thread Max Reitz
qemu-img should use QMP commands whenever possible in order to ensure feature completeness of both online and offline image operations. For the "commit" command, this is relatively easy, so implement it first (in the hope that indeed others will follow). As qemu-img does not have access to QMP (du

[Qemu-devel] [PATCH v10 01/14] qcow2: Allow "full" discard

2014-07-22 Thread Max Reitz
Normally, discarded sectors should read back as zero. However, there are cases in which a sector (or rather cluster) should be discarded as if they were never written in the first place, that is, reading them should fall through to the backing file again. Signed-off-by: Max Reitz Reviewed-by: Eri

[Qemu-devel] [PATCH v10 05/14] blockjob: Add "ready" field

2014-07-22 Thread Max Reitz
When a block job signals readiness, this is currently reported only through QMP. If qemu wants to use block jobs for internal tasks, there needs to be another way to correctly detect when a block job may be completed. For this reason, introduce a bool "ready" which is set when the block job may be

[Qemu-devel] [PATCH v10 03/14] qcow2: Optimize bdrv_make_empty()

2014-07-22 Thread Max Reitz
bdrv_make_empty() is currently only called if the current image represents an external snapshot that has been committed to its base image; it is therefore unlikely to have internal snapshots. In this case, bdrv_make_empty() can be greatly sped up by creating an empty L1 table and dropping all data

[Qemu-devel] [PATCH v10 02/14] qcow2: Implement bdrv_make_empty()

2014-07-22 Thread Max Reitz
Implement this function by making all clusters in the image file fall through to the backing file (by using the recently extended discard). Signed-off-by: Max Reitz Reviewed-by: Eric Blake Reviewed-by: Kevin Wolf --- block/qcow2.c | 27 +++ 1 file changed, 27 insertions

[Qemu-devel] [PATCH v10 09/14] qemu-img: Enable progress output for commit

2014-07-22 Thread Max Reitz
Implement progress output for the commit command by querying the progress of the block job. Signed-off-by: Max Reitz Reviewed-by: Eric Blake --- qemu-img-cmds.hx | 4 ++-- qemu-img.c | 24 ++-- qemu-img.texi| 2 +- 3 files changed, 25 insertions(+), 5 deletions(-

  1   2   >