[Qemu-devel] [PATCH] spice: fix "info spice"

2014-04-15 Thread Gerd Hoffmann
In case no listening address was specified, "info spice" reports "0.0.0.0" as address. Which is incorrect in case spice is listening on ipv6. Replace it by a wildcard "*" to indicate it is not limited to a specific address. Note: Being more specific is not possible without extending the spice-s

[Qemu-devel] [qmu] VNC sharePolicy not working as expected

2014-04-15 Thread Kekane, Abhishek
Hi All, Greetings!!! We are using KVM hypervisor driver for running OpenStack IaaS. Couple of months back we have reported one security issue [1] in OS. Basically we want to limit on the number of vnc client connections that can be opened by users for a given VM. >From libvirt 1.0.6 version onw

Re: [Qemu-devel] [qmu] VNC sharePolicy not working as expected

2014-04-15 Thread Gerd Hoffmann
Hi, > But while testing I am not able to get expected results for > allow-exclusive and ignore sharePolicy. > For allow-exclusive sharePolicy previous connections are not getting > dropped and console contents are getting shared among all open > consoles. The vnc protocol has an option to speci

Re: [Qemu-devel] Unresponsive linux guest once migrated

2014-04-15 Thread Dr. David Alan Gilbert
* Marcus (shadow...@gmail.com) wrote: > Dang, I was hoping some ground was being made on this. Can you answer the same questions I asked Marcin? What's the latest version of QEMU you've seen this on, what CPU are you using, what guest OS and what's your QEMU command line? Dave > > On Wed, Apr 2

Re: [Qemu-devel] segfault while booting from saved snapshot

2014-04-15 Thread Dr. David Alan Gilbert
* Shehbaz Jaffer (shehbazjaffer...@gmail.com) wrote: > Hi, > > I am running a Ubuntu 12.04 guest VM that I created using vm-builder on a > Ubuntu 12.04 Operating System. I am using qemu-2.0.0-rc0 version. I want to > boot from a saved snapshot instance of the guest VM, so I created a > snapshot us

Re: [Qemu-devel] [QEMU v6 PATCH 00/17] SMBIOS: build full tables in QEMU

2014-04-15 Thread Gerd Hoffmann
On Mo, 2014-04-14 at 16:54 -0400, Gabriel L. Somlo wrote: > New in version 6 of the patch set: > > - down to 17 patches (squashed adding spec v2.4 fields > in together with adding v2.8 fields further down). > > - switching to monolithic aggregate tables plus entry point > in patch 11/

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

2014-04-15 Thread Roger Pau Monne
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, implement a FreeBSD specific version of tap_open that behaves like it's Linux counterpart. This is specially important

[Qemu-devel] [PATCH 0/2] qemu-freebsd: fixes for running Xen guests

2014-04-15 Thread Roger Pau Monne
This two patches allow FreeBSD Xen Dom0 to use Qemu (i.e., launch HVM guests). First patch fixes the usage of ENODATA, which doesn't exist on FreeBSD and is replaced with ENOENT instead. The second patch is more controversial probably, since it introduces a FreeBSD specific version of tap_open wh

[Qemu-devel] [PATCH 1/2] xen: fix usage of ENODATA

2014-04-15 Thread Roger Pau Monne
ENODATA doesn't exist on FreeBSD, so ENODATA errors returned by the hypervisor are translated to ENOENT. Also, the error code is returned in errno if the call returns -1, so compare the error code with the value in errno instead of the value returned by the function. Signed-off-by: Roger Pau Monn

Re: [Qemu-devel] [SeaBIOS v3 PATCH] SMBIOS: Check for full tables & entry point in fw_cfg

2014-04-15 Thread Markus Armbruster
"Gabriel L. Somlo" writes: > Check fw_cfg for the presence of a complete set of smbios > tables (etc/smbios/smbios-tables) and an entry point structure > (etc/smbios/smbios-anchor), and, if found, use them instead of > generating our own copies locally. > > We ensure the presence of a type 0 (bio

Re: [Qemu-devel] [PATCH v7 1/8] virtio: endian-ambivalent targets using legacy virtio

2014-04-15 Thread Greg Kurz
On Mon, 14 Apr 2014 15:08:23 +0200 Alexander Graf wrote: > > On 14.04.14 14:55, Michael S. Tsirkin wrote: > > On Mon, Apr 14, 2014 at 02:40:04PM +0200, Alexander Graf wrote: > >> On 14.04.14 14:37, Michael S. Tsirkin wrote: > >>> On Mon, Apr 14, 2014 at 02:29:20PM +0200, Alexander Graf wrote: >

[Qemu-devel] [PATCH] timer: fix qemu_poll_ns early timeout on windows

2014-04-15 Thread Stanislav Vorobiov
From: Sangho Park g_poll has a problem on windows when using timeouts < 10ms, in glib/gpoll.c: /* If not, and we have a significant timeout, poll again with * timeout then. Note that this will return indication for only * one event, or only for messages. We ignore timeouts less than * ten mil

Re: [Qemu-devel] [SeaBIOS] [SeaBIOS v3 PATCH] SMBIOS: Check for full tables & entry point in fw_cfg

2014-04-15 Thread Gerd Hoffmann
> If I read your discussion correctly, you're proposing to change "from > SeaBIOS defaults" to "from QEMU defaults". Correct? That is correct, where "qemu defaults" will be "seabios defaults of the current stable release aka 1.7.4". cheers, Gerd

Re: [Qemu-devel] [PATCH trivial 2/3] vl: remove redundant local variable 'res'

2014-04-15 Thread Markus Armbruster
Peter Crosthwaite writes: > On Tue, Apr 8, 2014 at 10:02 PM, Chen Gang wrote: >> In function, if no additional resources to free before quit, commonly, >> need not use additional local variable 'res' to notice about it. So >> remove it to simplify code. >> > > Styling wise, there is a school of

Re: [Qemu-devel] [SeaBIOS] [SeaBIOS v3 PATCH] SMBIOS: Check for full tables & entry point in fw_cfg

2014-04-15 Thread Gerd Hoffmann
On Mo, 2014-04-14 at 15:30 -0400, Gabriel L. Somlo wrote: > If QEMU adds its (still optional) type 0 structure, SeaBIOS just uses > everything as-is. If no type 0 structure is found, we generate and > prepend our own, and update the entry point fields accordingly. > > I expect the latter to be the

Re: [Qemu-devel] qemu-kvm-1.0.1 cdrom device with iso hotplug issue

2014-04-15 Thread Kevin Wolf
Am 15.04.2014 um 05:18 hat xuanmao_001 geschrieben: > Hi, there: > > I found an issue that cdrom device hotplug iso image. > 1. if I startup qemu with an iso image, then iso can easily be replaced and it > worked. > 2. but if I startup qemu with null image, when I change iso image use qemu > moni

Re: [Qemu-devel] [PATCH trivial 0/3] vl: simplify code for main() and get_boot_device()

2014-04-15 Thread Markus Armbruster
Chen Gang writes: > In "vl.c", at least, we can simplify the code below, so can let readers > read professional C code (especially for new readers, which often start > reading code at main function). > > - remove useless 'continue' in main(). > > - remove redundant local variable 'res' in get_b

Re: [Qemu-devel] [PATCH trivial 1/3] vl: remove useless 'continue'

2014-04-15 Thread Markus Armbruster
Chen Gang writes: > Normal "if (...) {...} else {...}" is enough in "while(...) {...}", not > need additional useless 'continue'. > > Signed-off-by: Chen Gang > --- > vl.c |1 - > 1 file changed, 1 deletion(-) > > diff --git a/vl.c b/vl.c > index 9975e5a..7505002 100644 > --- a/vl.c > +++ b

Re: [Qemu-devel] [PATCH trivial 3/3] vl: remove local variable 'args' in the middle of code block

2014-04-15 Thread Markus Armbruster
Chen Gang writes: > For C code, it does not recommend to define a local variable in the > middle of code block without "{...}". The original author may want to > zero members not mentioned in structure assignment. > > So recommend to use structure initializing block "{...}" for structure > assign

[Qemu-devel] [Bug 1223467] Re: Unable to use USB as hda in Windows

2014-04-15 Thread Kevin Wolf
I think this has been fixed by commit 68dc0364, which was included in qemu 1.7.0 and also backported to 1.6.1. Can you please try upgrading and confirm whether it fixes the problem? -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. http

Re: [Qemu-devel] [Qemu-trivial] [PATCH] arch_init.c: remove duplicate function

2014-04-15 Thread Laszlo Ersek
On 04/15/14 01:55, Michael R. Hines wrote: > On 04/14/2014 05:19 PM, Laszlo Ersek wrote: >> On 04/14/14 04:27, Amos Kong wrote: >>> We already have a function buffer_is_zero() in util/cutils.c >>> >>> Signed-off-by: Amos Kong >>> --- >>> arch_init.c | 9 ++--- >>> 1 file changed, 2 insertio

Re: [Qemu-devel] qemu + rbd block driver with cache=writeback, is live migration safe ?

2014-04-15 Thread Kevin Wolf
Am 12.04.2014 um 17:01 hat Alexandre DERUMIER geschrieben: > Hello, > > I known that qemu live migration with disk with cache=writeback are not safe > with storage like nfs,iscsi... > > Is it also true with rbd ? First of all, in order to avoid misunderstandings, let's be clear that there are t

Re: [Qemu-devel] qapi-commands.py generates code that uses uninitialized variables

2014-04-15 Thread Kevin Wolf
Am 11.04.2014 um 15:11 hat Eric Blake geschrieben: > On 04/11/2014 01:27 AM, Peter Maydell wrote: > > On 11 April 2014 02:40, Eric Blake wrote: > >> We uncovered a real bug that would be fixed by this patch: > >> https://lists.gnu.org/archive/html/qemu-devel/2014-04/msg01745.html > > > > No, that

[Qemu-devel] Global overview

2014-04-15 Thread Kevin Grandemange
Hi, I was wondering if a document describing how the differents parts of Qemu worked together existed and if someone could pinpoint it to me. (or explain it to me ;) ) This project is so big that I don't know where to begin. thanks in advance !

Re: [Qemu-devel] qemu-kvm-1.0.1 cdrom device with iso hotplug issue

2014-04-15 Thread Kevin Wolf
Am 15.04.2014 um 11:21 hat xuanmao_001 geschrieben: > Hi, Kevin: > The lastest version of qemu is ok. > But I want to know what is going wrong with qemu-kvm-1.0.1. There have been a lot of changes between 1.0.1 and now. To be honest, as long as it works now, I don't feel like investing a lot of ti

Re: [Qemu-devel] Global overview

2014-04-15 Thread Dr. David Alan Gilbert
* Kevin Grandemange (grandemange.ke...@gmail.com) wrote: > Hi, > > I was wondering if a document describing how the differents parts of Qemu > worked together existed and if someone could pinpoint it to me. (or explain > it to me ;) ) > This project is so big that I don't know where to begin. Yes

Re: [Qemu-devel] Global overview

2014-04-15 Thread Laszlo Ersek
On 04/15/14 11:58, Dr. David Alan Gilbert wrote: > * Kevin Grandemange (grandemange.ke...@gmail.com) wrote: >> Hi, >> >> I was wondering if a document describing how the differents parts of Qemu >> worked together existed and if someone could pinpoint it to me. (or explain >> it to me ;) ) >> This

Re: [Qemu-devel] '.' IDs and certain names breaks -global and -set

2014-04-15 Thread Peter Crosthwaite
On Tue, Apr 15, 2014 at 1:38 AM, Andreas Färber wrote: > Am 14.04.2014 09:13, schrieb Markus Armbruster: >> Alistair Francis writes: >> >>> On Wed, Apr 9, 2014 at 9:58 PM, Peter Crosthwaite >>> wrote: On Wed, Apr 9, 2014 at 7:56 PM, Markus Armbruster wrote: > We have a number of

[Qemu-devel] guest crash due to missing clock interrupt

2014-04-15 Thread Damjan Marion
Originally I reported this issue to ubuntu folks, and it is documented on the launchpad[1] but would like to bring this into attention also on this list. Basically seems that on some occasions clock interrupts are not happening for longer periods of time causing that some VMs are crashing (i.e

Re: [Qemu-devel] [PATCH trivial 2/3] vl: remove redundant local variable 'res'

2014-04-15 Thread Chen Gang
On 04/15/2014 04:43 PM, Markus Armbruster wrote: > Peter Crosthwaite writes: > >> On Tue, Apr 8, 2014 at 10:02 PM, Chen Gang wrote: >>> In function, if no additional resources to free before quit, commonly, >>> need not use additional local variable 'res' to notice about it. So >>> remove it to

Re: [Qemu-devel] [PATCH trivial 1/3] vl: remove useless 'continue'

2014-04-15 Thread Chen Gang
On 04/15/2014 04:50 PM, Markus Armbruster wrote: > Chen Gang writes: > >> Normal "if (...) {...} else {...}" is enough in "while(...) {...}", not >> need additional useless 'continue'. >> >> Signed-off-by: Chen Gang >> --- >> vl.c |1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/

Re: [Qemu-devel] [PATCH trivial 3/3] vl: remove local variable 'args' in the middle of code block

2014-04-15 Thread Chen Gang
On 04/15/2014 04:56 PM, Markus Armbruster wrote: > Chen Gang writes: > >> For C code, it does not recommend to define a local variable in the >> middle of code block without "{...}". The original author may want to >> zero members not mentioned in structure assignment. >> >> So recommend to use

Re: [Qemu-devel] report a suspect bug about arm gic

2014-04-15 Thread Peter Maydell
On 15 April 2014 02:33, zhuxiaodong wrote: > I am a user of qemu. I found that in qemu2.0.0-rc0 the gic model was > updated. However, it seems loss ability to bind irqs to any specified core > when the board includes multiple cortex-a9 cores. The problematic codes > maybe locate at hw/intc

Re: [Qemu-devel] [PATCH v7 1/8] virtio: endian-ambivalent targets using legacy virtio

2014-04-15 Thread Alexander Graf
On 04/15/2014 10:40 AM, Greg Kurz wrote: On Mon, 14 Apr 2014 15:08:23 +0200 Alexander Graf wrote: On 14.04.14 14:55, Michael S. Tsirkin wrote: On Mon, Apr 14, 2014 at 02:40:04PM +0200, Alexander Graf wrote: On 14.04.14 14:37, Michael S. Tsirkin wrote: On Mon, Apr 14, 2014 at 02:29:20PM +020

Re: [Qemu-devel] Global overview

2014-04-15 Thread Kevin Grandemange
Thanks a lot, that's exactly what I was searching for. 2014-04-15 12:26 GMT+02:00 Laszlo Ersek : > On 04/15/14 11:58, Dr. David Alan Gilbert wrote: > > * Kevin Grandemange (grandemange.ke...@gmail.com) wrote: > >> Hi, > >> > >> I was wondering if a document describing how the differents parts of

[Qemu-devel] [Bug 1263747] Re: Arm64 fails to run a binary which runs OK on real hardware

2014-04-15 Thread Peter Maydell
The (re)implementation of this instruction for mainline never had this bug. ** Changed in: qemu Status: New => Fix Committed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1263747 Title: Ar

Re: [Qemu-devel] [PATCH trivial 3/3] vl: remove local variable 'args' in the middle of code block

2014-04-15 Thread Peter Crosthwaite
On Tue, Apr 8, 2014 at 10:05 PM, Chen Gang wrote: > For C code, it does not recommend to define a local variable in the > middle of code block without "{...}". The original author may want to > zero members not mentioned in structure assignment. > > So recommend to use structure initializing block

Re: [Qemu-devel] [PATCH trivial 3/3] vl: remove local variable 'args' in the middle of code block

2014-04-15 Thread Chen Gang
On 04/15/2014 08:29 PM, Peter Crosthwaite wrote: > On Tue, Apr 8, 2014 at 10:05 PM, Chen Gang wrote: >> For C code, it does not recommend to define a local variable in the >> middle of code block without "{...}". The original author may want to >> zero members not mentioned in structure assignment

[Qemu-devel] [Bug 1287195] Re: validate_guest_space incorrectly enabled on AArch64

2014-04-15 Thread Peter Maydell
Fix will be in 2.0. ** Changed in: qemu Status: In Progress => Fix Committed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1287195 Title: validate_guest_space incorrectly enabled on AArch6

Re: [Qemu-devel] KVM call agenda for 2014-04-15

2014-04-15 Thread Alexander Graf
On 04/14/2014 07:37 PM, Juan Quintela wrote: Hi Please, send any topic that you are interested in covering. As mentioned in the last email I still have the sysbus -device topic on the list. If Alexey shows up I would also like to discuss the IOMMU bus patch and why we need a bus solely for l

Re: [Qemu-devel] KVM call agenfda for 2014-04-01

2014-04-15 Thread Andreas Färber
Am 10.04.2014 17:52, schrieb Peter Maydell: > On 10 April 2014 16:49, Alexander Graf wrote: >> For the next call, I would propose to revive the "platform bus" >> (aka: how to create non-PCI devices with -device) discussions >> to make sure we're all on the same page. > > I rather suspect we are n

[Qemu-devel] Replacing display contents on paused VMs causes only a short flicker

2014-04-15 Thread Erik Rull
Hi all, I try to replace the display content of a paused VM (to show the operator that it is actually paused and not usable), but it causes only a short flicker on the display instead of staying permanent. By routing an HMP based command to console.c I want to replace the display contents. I star

Re: [Qemu-devel] [PATCH v7 1/8] virtio: endian-ambivalent targets using legacy virtio

2014-04-15 Thread Greg Kurz
On Tue, 15 Apr 2014 13:35:03 +0200 Alexander Graf wrote: > On 04/15/2014 10:40 AM, Greg Kurz wrote: > > On Mon, 14 Apr 2014 15:08:23 +0200 > > Alexander Graf wrote: > > > >> On 14.04.14 14:55, Michael S. Tsirkin wrote: > >>> On Mon, Apr 14, 2014 at 02:40:04PM +0200, Alexander Graf wrote: >

Re: [Qemu-devel] KVM call agenda for 2014-04-15

2014-04-15 Thread Juan Quintela
Juan Quintela wrote: > Hi > > Please, send any topic that you are interested in covering. > > Thanks, Juan. As there are no topics, no call.

[Qemu-devel] [PATCH] block: prefer protocol_name over format_name in bdrv_iterate_format

2014-04-15 Thread Jeff Cody
Some block drivers have multiple BlockDriver instances with identical format_name fields (e.g. gluster, nbd). In those cases, the protocol_name is usually the more unique identifier (e.g. gluster+tcp). Both qemu-img and qemu will use bdrv_iterate_format() to list the supported formats when a help

[Qemu-devel] [Bug 1257352] Re: kvm hangs occasionally when switching out of the qemu console

2014-04-15 Thread James Hunt
Sorry - ignore that. However, problem persists on a separate fully- updated amd64 trusty system running kvm version 2.0.0~rc1+dfsg-0ubuntu3. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1257352 Titl

[Qemu-devel] [Bug 1100843] Re: Live Migration Causes Performance Issues

2014-04-15 Thread Bryan Quigley
QEMU 1.7 was released, Quantal has 10ish days left of support, and Raring is EOL ** Changed in: qemu Status: Fix Committed => Fix Released ** Changed in: qemu-kvm (Ubuntu Quantal) Status: Triaged => Invalid ** Changed in: qemu-kvm (Ubuntu Raring) Status: Triaged => Invalid

Re: [Qemu-devel] [Xen-devel] [PATCH 1/2] xen: fix usage of ENODATA

2014-04-15 Thread Egger, Christoph
On 15.04.14 10:33, Roger Pau Monne wrote: > ENODATA doesn't exist on FreeBSD, so ENODATA errors returned by the > hypervisor are translated to ENOENT. > > Also, the error code is returned in errno if the call returns -1, so > compare the error code with the value in errno instead of the value > re

Re: [Qemu-devel] [QEMU v6 PATCH 00/17] SMBIOS: build full tables in QEMU

2014-04-15 Thread Gabriel L. Somlo
On Tue, Apr 15, 2014 at 10:29:26AM +0200, Gerd Hoffmann wrote: > So, how about this plan: > > Leave the old interface code basically as-is. type0 and type1 > individual fields are passed like they are passed today. We don't > change to to pass full tables, and we don't extend that to new table >

[Qemu-devel] QEMU 2.0 RC with Spice

2014-04-15 Thread Rick Vernam
I have been trying out the 2.0 RCs, and I've noticed that when I use spice qemu aborts when I reboot the VM. This occurs on Win XP guest, Win 7 (64-bit) guest and Win 8 (64-bit) guest. Is this something that anybody else experiences? I don't care to divert anybody's energy if this a spice thing

Re: [Qemu-devel] [PATCH trivial 0/3] vl: simplify code for main() and get_boot_device()

2014-04-15 Thread Chen Gang
On 04/15/2014 04:49 PM, Markus Armbruster wrote: > Chen Gang writes: > >> In "vl.c", at least, we can simplify the code below, so can let readers >> read professional C code (especially for new readers, which often start >> reading code at main function). >> >> - remove useless 'continue' in mai

Re: [Qemu-devel] [PATCH] block: prefer protocol_name over format_name in bdrv_iterate_format

2014-04-15 Thread Max Reitz
On 15.04.2014 15:28, Jeff Cody wrote: Some block drivers have multiple BlockDriver instances with identical format_name fields (e.g. gluster, nbd). In those cases, the protocol_name is usually the more unique identifier (e.g. gluster+tcp). Both qemu-img and qemu will use bdrv_iterate_format() t

Re: [Qemu-devel] [PATCH] block: prefer protocol_name over format_name in bdrv_iterate_format

2014-04-15 Thread Kevin Wolf
Am 15.04.2014 um 15:28 hat Jeff Cody geschrieben: > Some block drivers have multiple BlockDriver instances with identical > format_name fields (e.g. gluster, nbd). In those cases, the > protocol_name is usually the more unique identifier (e.g. gluster+tcp). > > Both qemu-img and qemu will use bdr

Re: [Qemu-devel] [PATCH v2] block: Handle error of bdrv_getlength in bdrv_create_dirty_bitmap

2014-04-15 Thread Max Reitz
On 15.04.2014 05:15, Fam Zheng wrote: bdrv_getlength could fail, check the return value before using it. Signed-off-by: Fam Zheng --- v2: Make use of error_setg_errno and -errno. (Kevin) Signed-off-by: Fam Zheng --- block-migration.c | 28 block.c

Re: [Qemu-devel] QEMU 2.0 RC with Spice

2014-04-15 Thread Dr. David Alan Gilbert
* Rick Vernam (rtver...@hobi.com) wrote: > I have been trying out the 2.0 RCs, and I've noticed that when I use spice > qemu aborts when I reboot the VM. This occurs on Win XP guest, Win 7 > (64-bit) > guest and Win 8 (64-bit) guest. > Is this something that anybody else experiences? > I don't

Re: [Qemu-devel] KVM call agenda for 2014-04-15

2014-04-15 Thread Markus Armbruster
Juan Quintela writes: > Juan Quintela wrote: >> Hi >> >> Please, send any topic that you are interested in covering. >> >> Thanks, Juan. > > As there are no topics, no call. Did we have a call anyway? IRC log looks like we did...

Re: [Qemu-devel] [PATCH v7 1/8] virtio: endian-ambivalent targets using legacy virtio

2014-04-15 Thread Peter Maydell
On 15 April 2014 14:22, Greg Kurz wrote: > On Tue, 15 Apr 2014 13:35:03 +0200 > Alexander Graf wrote: >> That would defeat the purpose - the reason to have the helper inlined is >> to remove the conditional branch for x86. >> > > Sure but on the other hand, Peter does not like the idea of moving

Re: [Qemu-devel] [PATCH trivial 0/3] vl: simplify code for main() and get_boot_device()

2014-04-15 Thread Markus Armbruster
Chen Gang writes: > On 04/15/2014 04:49 PM, Markus Armbruster wrote: >> In future submissions, please send the patches in-reply-to the cover >> letter, not chained together in-reply-to the previous part. Check out >> git-send-email --no-chain-reply-to. >> > > OK, thanks. But excuse me, I use th

Re: [Qemu-devel] segfault while booting from saved snapshot

2014-04-15 Thread Shehbaz Jaffer
Thankyou for your reply. I do not face the error using qemu.1.7.1 version. 1 quick question: I want to create a qcow2 image file after my VM has completed booting; i.e., when I boot from this new snapshot, I should directly get to my VMs login prompt. How can i create such a qcow2 snapshot? I tr

Re: [Qemu-devel] KVM call agenda for 2014-04-15

2014-04-15 Thread Alexander Graf
On 04/15/2014 04:00 PM, Markus Armbruster wrote: Juan Quintela writes: Juan Quintela wrote: Hi Please, send any topic that you are interested in covering. Thanks, Juan. As there are no topics, no call. Did we have a call anyway? IRC log looks like we did... Yes, we did. Whoever attend

Re: [Qemu-devel] [PATCH v1 1/1] vl.c: Allow sysbus devices to be attached via commandline

2014-04-15 Thread Andreas Färber
Hi, Am 26.03.2014 01:47, schrieb Alistair Francis: > This patch introduces a new command line argument that allows > sysbus devices to be attached via the command line. > > This allows devices to be added after the machine init but > before anything is booted > > The new argument is -sysbusdev >

[Qemu-devel] [PATCH] do_interrupt64 shouldn't be called when GIF is not set.

2014-04-15 Thread Himanshu Chauhan
I am not very sure about the place(s) we would need this. But I think this is required. Please advise. When having SVM enabled, the do_interrupt64 shouldn't be called unless GIF is set otherwise this causes random behaviour especially bad TR in case guest is running. Signed-off-by: Himanshu Chauh

Re: [Qemu-devel] [PATCH target-arm v1 1/1] arm: translate.c: Fix smlald Instruction

2014-04-15 Thread Peter Maydell
On 4 April 2014 03:19, Peter Crosthwaite wrote: > The smlald (and probably smlsld) instruction was doing incorrect sign > extensions of the operands amongst 64bit result calculation. The > instruction psuedo-code is: > > operand2 = if m_swap then ROR(R[m],16) else R[m]; > product1 = SInt(R[n]<15

Re: [Qemu-devel] KVM call agenda for 2014-04-15

2014-04-15 Thread Eric Auger
On 04/15/2014 04:55 PM, Alexander Graf wrote: > On 04/15/2014 04:00 PM, Markus Armbruster wrote: >> Juan Quintela writes: >> >>> Juan Quintela wrote: Hi Please, send any topic that you are interested in covering. Thanks, Juan. >>> As there are no topics, no call. >> Did w

Re: [Qemu-devel] [PATCH] vexpress: Add support for the -bios flag to provide firmware

2014-04-15 Thread Peter Maydell
On 28 March 2014 17:57, Grant Likely wrote: > Right now to run firmware inside the QEMU VExpress model requires > padding out the firmware image to the size of the virtual flash and > passing it in via the -pflash argument. If the firmware image is passed > without padding, then QEMU will fail. Al

Re: [Qemu-devel] [PATCH] vexpress: Add support for the -bios flag to provide firmware

2014-04-15 Thread Peter Maydell
On 28 March 2014 17:57, Grant Likely wrote: > Right now to run firmware inside the QEMU VExpress model requires > padding out the firmware image to the size of the virtual flash and > passing it in via the -pflash argument. If the firmware image is passed > without padding, then QEMU will fail. A

Re: [Qemu-devel] [PATCH 33/35] pc: ACPI BIOS: reserve SRAT entry for hotplug mem hole

2014-04-15 Thread Igor Mammedov
On Tue, 15 Apr 2014 14:37:01 +0800 Hu Tao wrote: > On Mon, Apr 14, 2014 at 06:44:42PM +0200, Igor Mammedov wrote: > > On Mon, 14 Apr 2014 15:25:01 +0800 > > Hu Tao wrote: > > > > > On Fri, Apr 04, 2014 at 03:36:58PM +0200, Igor Mammedov wrote: > > > > Needed for Windows to use hotplugged memory

Re: [Qemu-devel] [Qemu-trivial] [PATCH] arch_init.c: remove duplicate function

2014-04-15 Thread 陈梁
> On 04/15/14 01:55, Michael R. Hines wrote: >> On 04/14/2014 05:19 PM, Laszlo Ersek wrote: >>> On 04/14/14 04:27, Amos Kong wrote: We already have a function buffer_is_zero() in util/cutils.c Signed-off-by: Amos Kong --- arch_init.c | 9 ++--- 1 file changed,

Re: [Qemu-devel] KVM call agenda for 2014-04-15

2014-04-15 Thread 陈梁
Hi Do we have any plan to support migration by multi net card? Thanks ChenLiang > > 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, conta

Re: [Qemu-devel] [PATCH] Add remove_boot_device_path() function for hot-unplug device

2014-04-15 Thread lijun
On 04/15/2014 02:04 AM, Andreas Färber wrote: Am 13.04.2014 15:24, schrieb Jun Li: Add remove_boot_device_path() function to remove bootindex when hot-unplug a device. This patch fixed virtio-blk/virtio-net/scsi-disk/scsi-generic device. So it has fixed bug1086603, ref: https://bugzilla.redhat.

Re: [Qemu-devel] [PATCH] Add remove_boot_device_path() function for hot-unplug device

2014-04-15 Thread lijun
On 04/15/2014 12:18 AM, Markus Armbruster wrote: lijun writes: Hi Markus, Really appreciate your review first. I almost a new participant. And I read other's patches very little. So maybe this patch is duplicate to one of Marcel's patch. But I really do not know. And I really don't copying

Re: [Qemu-devel] [PATCH v1 1/3] qdev: Expose the qdev id string as a prop

2014-04-15 Thread Andreas Färber
Am 15.04.2014 04:21, schrieb Peter Crosthwaite: > So clients can set the top level id string. > > Signed-off-by: Peter Crosthwaite Anthony had nack'ed Paolo's attempt to generalize the qdev id to QOM, so I'm not sure if we should really do this even if just on device level. The id= is used to co

Re: [Qemu-devel] [PATCH 1/2] xen: fix usage of ENODATA

2014-04-15 Thread Anthony PERARD
On Tue, Apr 15, 2014 at 10:33:08AM +0200, Roger Pau Monne wrote: > ENODATA doesn't exist on FreeBSD, so ENODATA errors returned by the > hypervisor are translated to ENOENT. > > Also, the error code is returned in errno if the call returns -1, so > compare the error code with the value in errno in

Re: [Qemu-devel] KVM call agenda for 2014-04-15

2014-04-15 Thread Peter Maydell
Some fuller comments below on the parts of the call which I had opinions on. On 15 April 2014 15:55, Alexander Graf wrote: > > 2) -device for non-PCI devices > > There are 2 reasons we want to create "platform" devices using -device on > the command line. One is that Xilinx is trying to create a

Re: [Qemu-devel] [PATCH 1/2] xen: fix usage of ENODATA

2014-04-15 Thread Roger Pau Monné
On 15/04/14 18:15, Anthony PERARD wrote: > On Tue, Apr 15, 2014 at 10:33:08AM +0200, Roger Pau Monne wrote: >> ENODATA doesn't exist on FreeBSD, so ENODATA errors returned by the >> hypervisor are translated to ENOENT. >> >> Also, the error code is returned in errno if the call returns -1, so >> co

Re: [Qemu-devel] KVM call agenda for 2014-04-15

2014-04-15 Thread Markus Armbruster
Alexander Graf writes: > On 04/15/2014 04:00 PM, Markus Armbruster wrote: >> Juan Quintela writes: >> >>> Juan Quintela wrote: Hi Please, send any topic that you are interested in covering. Thanks, Juan. >>> As there are no topics, no call. >> Did we have a call anyway?

Re: [Qemu-devel] [PATCH v4 1/4] util/fifo: Generalise naming scheme

2014-04-15 Thread Beniamino Galvani
On Mon, Apr 14, 2014 at 08:18:20PM -0700, Peter Crosthwaite wrote: > Generalise the names of the FIFO API to not include the "8". > > The exception is the push/pop functions for which we: > > s/fifo8_pop/fifo_pop8 > s/fifo8_push/fifo_push8 > > This prepares support for generalising FIFO support

Re: [Qemu-devel] Unresponsive linux guest once migrated

2014-04-15 Thread Marcus
I can answer some of the questions. It's been 3 months or so since I looked into it. I ended up disabling kvmclock from the qemu command line and moving on. I saw it with CentOS 6.5 and Ubuntu 12.04 guests. Sending the guest to the BIOS CLI or PXE would not reproduce the issue. I didn't attempt an

[Qemu-devel] [PATCH] target-ppc: Eliminate Magic Number MSR Masks

2014-04-15 Thread Tom Musta
Use MSR mnemonics from cpu.h instead of magic numbers for the CPUPPCState.msr_mask initialization. There is one bit in the 401x2 (and subsequent) model that I could not find any documentation for. It is open coded at little endian bit position 20: pcc->msr_mask = (1ull << 20) |

Re: [Qemu-devel] [PATCH v4 2/4] util/fifo: Generalise for common integer widths

2014-04-15 Thread Beniamino Galvani
On Mon, Apr 14, 2014 at 08:18:56PM -0700, Peter Crosthwaite wrote: > Add support for 16, 32 and 64 bit width FIFOs. The push and pop > functions are replicated to accept all four different integer types. > The element width of the FIFO is set at creation time. > > The backing storage for all eleme

Re: [Qemu-devel] [PATCH v4 3/4] ssi: pl022: Send debug info to stderr

2014-04-15 Thread Beniamino Galvani
On Mon, Apr 14, 2014 at 08:19:31PM -0700, Peter Crosthwaite wrote: > To disentangle it from the monitor. > > Signed-off-by: Peter Crosthwaite > --- > > hw/ssi/pl022.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/ssi/pl022.c b/hw/ssi/pl022.c > index fd479ef..84bf8

Re: [Qemu-devel] [PATCH v1 0/3] Zynq SLCR Rewrite and Update

2014-04-15 Thread Peter Maydell
On 3 April 2014 06:37, Peter Crosthwaite wrote: > Hi Peter, > > Iv'e finally gotten around to looking at SLCR and its coverity issues. > Ended up doing a rewrite of the whole thing to action (P1). > > Some styling updates to go with it as well (P2-3). Reviewed-by: Peter Maydell thanks -- PMM

Re: [Qemu-devel] [PATCH v4 4/4] ssi: pl022: Convert to use FIFO

2014-04-15 Thread Beniamino Galvani
On Mon, Apr 14, 2014 at 08:20:06PM -0700, Peter Crosthwaite wrote: > Use the FIFO API to factor out the ring buffer implementation code. > Cleans up the somewhat verbose VMS description as well (version bump > required). > > Signed-off-by: Peter Crosthwaite > --- > > hw/ssi/pl022.c | 101 > +++

Re: [Qemu-devel] [PATCH target-arm v1 1/1] net: cadence_gem: Make phy respond to broadcast

2014-04-15 Thread Peter Maydell
On 4 April 2014 07:55, Peter Crosthwaite wrote: > Phys must respond to address 0 by specification. Implement. > > Signed-off-by: Nathan Rossi > Signed-off-by: Peter Crosthwaite Reviewed-by: Peter Maydell thanks -- PMM

Re: [Qemu-devel] Unresponsive linux guest once migrated

2014-04-15 Thread Dr. David Alan Gilbert
* Marcus (shadow...@gmail.com) wrote: > I can answer some of the questions. It's been 3 months or so since I > looked into it. I ended up disabling kvmclock from the qemu command > line and moving on. I saw it with CentOS 6.5 and Ubuntu 12.04 guests. > Sending the guest to the BIOS CLI or PXE would

[Qemu-devel] [PATCH] target-arm: implement WFE/YIELD as a yield for AArch64

2014-04-15 Thread Rob Herring
From: Rob Herring Like was done for AArch32 for WFE, implement both WFE and YIELD as a yield operation. This speeds up multi-core system emulation. Signed-off-by: Rob Herring --- target-arm/translate-a64.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/target-arm/translate-a64.c b/t

Re: [Qemu-devel] [PATCH] target-arm: implement WFE/YIELD as a yield for AArch64

2014-04-15 Thread Peter Maydell
On 15 April 2014 20:00, Rob Herring wrote: > From: Rob Herring > > Like was done for AArch32 for WFE, implement both WFE and YIELD as a > yield operation. This speeds up multi-core system emulation. > > Signed-off-by: Rob Herring Reviewed-by: Peter Maydell thanks -- PMM

[Qemu-devel] [Bug 1270397] Re: Systemd segfaults after live migration

2014-04-15 Thread Dr. David Alan Gilbert
Hi Mateusz, How are you migrating? If you are using the compress/xbzrle option, try turning it off and see if that helps, there are some corruption fixes for xbzrle in 2.0.0. Dave -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

Re: [Qemu-devel] Unresponsive linux guest once migrated

2014-04-15 Thread Cole Robinson
On 04/15/2014 02:53 PM, Dr. David Alan Gilbert wrote: > * Marcus (shadow...@gmail.com) wrote: > >> As mentioned by others, the mode of reproduction is to launch the vm, >> wait about an hour, and then try to migrate it >> >> Here's are example qemu command lines (as generated from libvirt with

Re: [Qemu-devel] Unresponsive linux guest once migrated

2014-04-15 Thread Marcin Gibuła
W dniu 2014-04-15 20:53, Dr. David Alan Gilbert pisze: * Marcus (shadow...@gmail.com) wrote: I can answer some of the questions. It's been 3 months or so since I looked into it. I ended up disabling kvmclock from the qemu command line and moving on. I saw it with CentOS 6.5 and Ubuntu 12.04 gues

Re: [Qemu-devel] [PATCH v6] spapr_hcall: add address-translation-mode-on-interrupt resource in H_SET_MODE

2014-04-15 Thread Mike Day
On Tue, Apr 8, 2014 at 2:28 AM, Alexander Graf wrote: > On 22.03.14 14:03, Alexey Kardashevskiy wrote: >> >> This adds handling of the RESOURCE_ADDR_TRANS_MODE resource from >> the H_SET_MODE, for POWER8 (PowerISA 2.07) only. >> >> Signed-off-by: Alexey Kardashevskiy >> Reviewed-by: Mike Day > >

[Qemu-devel] [PATCH] qmp: object-add: Validate class before creating object

2014-04-15 Thread Eduardo Habkost
Currently it is very easy to crash QEMU by issuing an object-add command using an abstract class or a class that doesn't support TYPE_USER_CREATABLE as parameter. Example: with the following QMP command: (QEMU) object-add qom-type=cpu id=foo QEMU aborts at: ERROR:qom/object.c:335:object

[Qemu-devel] [Bug 1307473] Re: guest hang due to missing clock interrupt

2014-04-15 Thread Serge Hallyn
Reproduced just as easily with qemu.org git HEAD. Again, this appears to only be a case when using -smp 4 while locking to 1 cpu with cpuset. ** Also affects: qemu Importance: Undecided Status: New -- You received this bug notification because you are a member of qemu- devel-ml, which

Re: [Qemu-devel] [PATCH] qmp: object-add: Validate class before creating object

2014-04-15 Thread Matthew Rosato
On 04/15/2014 04:01 PM, Eduardo Habkost wrote: > Currently it is very easy to crash QEMU by issuing an object-add command > using an abstract class or a class that doesn't support > TYPE_USER_CREATABLE as parameter. > > Example: with the following QMP command: > > (QEMU) object-add qom-type=c

Re: [Qemu-devel] KVM call agenda for 2014-04-15

2014-04-15 Thread Alexander Graf
> Am 15.04.2014 um 18:56 schrieb Markus Armbruster : > > Alexander Graf writes: > >>> On 04/15/2014 04:00 PM, Markus Armbruster wrote: >>> Juan Quintela writes: >>> Juan Quintela wrote: > Hi > > Please, send any topic that you are interested in covering. > > Thank

Re: [Qemu-devel] [PATCH v1 1/3] qdev: Expose the qdev id string as a prop

2014-04-15 Thread Peter Crosthwaite
On Wed, Apr 16, 2014 at 2:16 AM, Andreas Färber wrote: > Am 15.04.2014 04:21, schrieb Peter Crosthwaite: >> So clients can set the top level id string. >> >> Signed-off-by: Peter Crosthwaite > > Anthony had nack'ed Paolo's attempt to generalize the qdev id to QOM, so > I'm not sure if we should r

[Qemu-devel] Are Qemu builds supported for arm host?

2014-04-15 Thread New B
Hi, I keep getting this error: "must include QEMU headers” emitted from tcg/tcg-op.h. I have no idea how come this is happening. BTW, the same setup I have (build env + 1.7.1-stable qemu sources) to successfully build a functional qemu-system-arm for x86-64 host. Any info/pointers are

Re: [Qemu-devel] [PATCH trivial 0/3] vl: simplify code for main() and get_boot_device()

2014-04-15 Thread Chen Gang
On 04/15/2014 10:51 PM, Markus Armbruster wrote: > Chen Gang writes: > >> On 04/15/2014 04:49 PM, Markus Armbruster wrote: >>> In future submissions, please send the patches in-reply-to the cover >>> letter, not chained together in-reply-to the previous part. Check out >>> git-send-email --no-ch

Re: [Qemu-devel] [PATCH v2 1/2] usb-hid: Add high speed mouse configuration

2014-04-15 Thread Ján Veselý
Hi, I found some more time to work on this. However, I'm not sure I understand the purpose of compat property: I noticed that the only place it's set is in PC_COMPAT_1_3 (is this what you meant by older devices?), I guess same setting for FS mouse and kbd should be added there as well. The switch

[Qemu-devel] [PATCH trivial v2 0/2] vl: simplify code for main()

2014-04-15 Thread Chen Gang
In "vl.c", at least, we can simplify the code below, so can let readers read professional C code (especially for new readers, which often start reading code at main function). - Remove useless 'continue' in main(). - Eliminate a superfluous local variable in main(). Signed-off-by: Chen Gang -

  1   2   >