Re: [Qemu-devel] RFH: boot from virtio cdrom?

2013-06-06 Thread Gerd Hoffmann
On 06/06/13 17:10, Philipp Hahn wrote: > Hello, > > I'm using libvirt to manage my VMs and configured one VM to boot from > a CDROM connected via virtio. This does neither work with QEMU-1.1.2 > nor with QEMU-1.5; neither with SeaBIOS is 1.7.0 nor 1.7.2. Doesn't work with virtio-blk, virtio-scsi

[Qemu-devel] RFH: boot from virtio cdrom?

2013-06-06 Thread Philipp Hahn
Hello, I'm using libvirt to manage my VMs and configured one VM to boot from a CDROM connected via virtio. This does neither work with QEMU-1.1.2 nor with QEMU-1.5; neither with SeaBIOS is 1.7.0 nor 1.7.2. /root/qemu-1.5.0+dfsg/qemu-build/x86_64-softmmu/qemu-system-x86_64 \ -M pc-1.1 \ -enabl

Re: [Qemu-devel] [PATCH v6 00/11] rdma: migration support

2013-06-06 Thread Michael R. Hines
OK, this looks excellent. I think we're ready for a PULL request now - I will submit to Juan - I've already got a signed-off from Paolo & Eric. I think we've got sufficient testing now. I'm expecting to get access to a big machine in the next week or two (256G machine) - and I should be able

Re: [Qemu-devel] [PATCH] arm/zynq: Add software reset

2013-06-06 Thread Peter Crosthwaite
Hi Sebastian, I'd like to get this merged with the fix I suggested. Are you able to remake or ok if I ammend and resend your patch? Regards, Peter On Tue, May 14, 2013 at 9:49 AM, Peter Crosthwaite wrote: > Hi Sebastian, > > Thanks for your contribution, > > On Wed, May 8, 2013 at 10:55 PM, Seb

[Qemu-devel] [PATCH arm-devs v1 1/1] char/cadence_uart: Fix reset for unattached instances

2013-06-06 Thread peter . crosthwaite
From: Peter Crosthwaite commit 1db8b5efe0c2b5000e50691eea61264a615f43de introduced an issue where QEMU would segfault if you have an unattached Cadence UART. Fix by guarding the flush-on-reset logic on there being a qemu_chr attachment. Reported-by: Soren Brinkmann Signed-off-by: Peter Crosthw

Re: [Qemu-devel] [PATCH] block/m25p80: Update Micron entries

2013-06-06 Thread Peter Crosthwaite
Hi Ed, On Wed, Jun 5, 2013 at 1:59 AM, Ed Maste wrote: > - Split 32Mb and 256Mb parts into a11 and a13 variants. > - Add the 4K sector flag to the 128Mb parts. (These entries were taken from > the Linux kernel list, which is missing the flag.) > - Fill out the table of sizes with entries for 6

Re: [Qemu-devel] [QEMU question] Disk hot plugging without working PCI hotplug - possible?

2013-06-06 Thread Alexey Kardashevskiy
On 06/07/2013 12:29 AM, Frederic Konrad wrote: > On 06/06/2013 15:59, Alexey Kardashevskiy wrote: >> On 06.06.2013 22:44, Frederic Konrad wrote: >>> On 06/06/2013 12:13, Alexey Kardashevskiy wrote: Hi! For the pseries platform (server PPC64) we do not support PCI hotplug yet. >>

[Qemu-devel] [PATCH v1 1/1] xilinx_axidma: Do not set DMA .notify to NULL after notify

2013-06-06 Thread peter . crosthwaite
From: Wendy Liang If a stream notify function is not ready, it may re-populate the notify call- back to indicate it should be re-polled later. This break in this usage, as immediately following the notify() call, .notify is set to NULL. reverse the ordering of the notify call and NULL assignment

Re: [Qemu-devel] [PATCH] virtio-net: put virtio net header inline with data

2013-06-06 Thread Rusty Russell
"Michael S. Tsirkin" writes: > For small packets we can simplify xmit processing by linearizing buffers > with the header: most packets seem to have enough head room we can use > for this purpose. > > Since some older hypervisors (e.g. qemu before version 1.5) > required that header is the first s

Re: [Qemu-devel] [PATCH RFC] virtio-pci: support config layout in BAR1

2013-06-06 Thread Rusty Russell
"Michael S. Tsirkin" writes: > Some setups don't support enabling BAR0 (IO BAR). Reasons range from CPU > limitations (e.g. on some powerpc setups) to architecture limmitations > (e.g. a setup with >15 PCI bridges, with one virtio device behind each, > on x86). > > PCI Express spec made IO optiona

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-06-06 Thread Peter Crosthwaite
Ping! We are stuck in limbo here and I would like to get either V1 or V2 (this patch) of this through. Paolo, If you are not happy with this patch, does V1 address your concerns (which was its original intention). Regards, Peter On Sun, Jun 2, 2013 at 9:13 AM, Peter Crosthwaite wrote: > HI Da

Re: [Qemu-devel] [PATCH] virtio-net: put virtio net header inline with data

2013-06-06 Thread Jason Wang
On 06/06/2013 05:55 PM, Michael S. Tsirkin wrote: > For small packets we can simplify xmit processing by linearizing buffers > with the header: most packets seem to have enough head room we can use > for this purpose. > > Since some older hypervisors (e.g. qemu before version 1.5) > required that h

[Qemu-devel] [PATCH v1 5/5] intc/xilinx_intc: Dont lower IRQ when HIE cleared

2013-06-06 Thread peter . crosthwaite
From: Peter Crosthwaite This is a little strange. It is lowering the parent IRQ pin on input when HIE is cleared. There is no such behaviour in the real hardware. ISR changes based on interrupt pin state are already guarded on HIE being set. So we can just delete this if in its entirety. Signed

[Qemu-devel] [PATCH v1 4/5] intc/xilinx_intc: Inhibit write to ISR when HIE

2013-06-06 Thread peter . crosthwaite
From: Peter Crosthwaite When the Hardware Interrupt Enable (HIE) bit is set, software cannot change ISR. Add write guard accordingly. Signed-off-by: Peter Crosthwaite --- hw/intc/xilinx_intc.c | 5 + 1 file changed, 5 insertions(+) diff --git a/hw/intc/xilinx_intc.c b/hw/intc/xilinx_intc

[Qemu-devel] [PATCH v1 3/5] intc/xilinx_intc: Handle level interrupt retriggering

2013-06-06 Thread peter . crosthwaite
From: Peter Crosthwaite Acking a level sensitive interrupt should have no effect if the interrupt pin is still asserted. The current implementation requires and edge condition to occur for setting a level sensitive IRQ, which means an ACK can clear a level sensitive interrupt, until the original

Re: [Qemu-devel] [PATCH 10/21] memory: make section size a 128-bit integer

2013-06-06 Thread Paolo Bonzini
Il 06/06/2013 21:23, Alexey Kardashevskiy ha scritto: > On 06/07/2013 11:09 AM, Paolo Bonzini wrote: >> Il 06/06/2013 04:36, Alexey Kardashevskiy ha scritto: > diff --git a/hw/misc/vfio.c b/hw/misc/vfio.c > index 693a9ff..c89676b 100644 > --- a/hw/misc/vfio.c > +++ b/hw/misc/vfio.c

Re: [Qemu-devel] [PATCH v1 1/5] intc/xilinx_intc: Use qemu_set_irq

2013-06-06 Thread Peter Crosthwaite
email accidentally sent with cc supression, so cc qemu-trivial as intended. On Fri, Jun 7, 2013 at 12:38 PM, wrote: > From: Peter Crosthwaite > > Use qemu_set_irq rather than if-elsing qemu_irq_(lower|raise). No > functional change, just reduces verbosity. > > Cc: qemu-triv...@nongnu.org > > Si

[Qemu-devel] [PATCH v1 2/5] intc/xilinx_intc: Don't clear level sens. IRQs without ACK

2013-06-06 Thread peter . crosthwaite
From: Peter Crosthwaite For level sensitive interrupts, ISR bits are cleared when the input pin is lowered. This is incorrect. Only software can clear ISR bits (via IAR or direct write to ISR with !MER(2)). Signed-off-by: Peter Crosthwaite --- hw/intc/xilinx_intc.c | 8 +--- 1 file change

[Qemu-devel] [PATCH v1 0/5] Xilinx Intc Fixes

2013-06-06 Thread peter . crosthwaite
From: Peter Crosthwaite Various fixups to the Xilinx Interrupt controller following a review against TRM and RTL descriptions. Tested as working for microblazeel Linux. Peter Crosthwaite (5): intc/xilinx_intc: Use qemu_set_irq intc/xilinx_intc: Don't clear level sens. IRQs without ACK i

[Qemu-devel] [PATCH v1 1/5] intc/xilinx_intc: Use qemu_set_irq

2013-06-06 Thread peter . crosthwaite
From: Peter Crosthwaite Use qemu_set_irq rather than if-elsing qemu_irq_(lower|raise). No functional change, just reduces verbosity. Cc: qemu-triv...@nongnu.org Signed-off-by: Peter Crosthwaite --- hw/intc/xilinx_intc.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a

Re: [Qemu-devel] [0/10] Clean up PCI code to allow for multiple root buses (v2)

2013-06-06 Thread David Gibson
On Thu, Jun 06, 2013 at 10:04:58AM -0500, Anthony Liguori wrote: > "Michael S. Tsirkin" writes: > > > On Thu, Jun 06, 2013 at 06:48:44PM +1000, David Gibson wrote: > >> The current PCI subsystem has kind of half-hearted support for > >> multiple independent root buses - aka PCI domains - in the f

Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read

2013-06-06 Thread Fam Zheng
On Thu, 06/06 12:01, Richard W.M. Jones wrote: > On Thu, Jun 06, 2013 at 02:25:46PM +0800, Fam Zheng wrote: > > v7: > > 13: Added: > > curl: change timeout to 30 seconds > > I tested this against: > > (1) HTTP to Apache server over slow but local wifi. > > (2) HTTP to a remote ISO (on an

Re: [Qemu-devel] [PATCH 10/21] memory: make section size a 128-bit integer

2013-06-06 Thread Alexey Kardashevskiy
On 06/07/2013 11:09 AM, Paolo Bonzini wrote: > Il 06/06/2013 04:36, Alexey Kardashevskiy ha scritto: diff --git a/hw/misc/vfio.c b/hw/misc/vfio.c index 693a9ff..c89676b 100644 --- a/hw/misc/vfio.c +++ b/hw/misc/vfio.c @@ -1953,7 +1953,7 @@ static void vfio_listener_region_a

Re: [Qemu-devel] [PATCH 10/21] memory: make section size a 128-bit integer

2013-06-06 Thread Paolo Bonzini
Il 06/06/2013 04:36, Alexey Kardashevskiy ha scritto: >> > diff --git a/hw/misc/vfio.c b/hw/misc/vfio.c >> > index 693a9ff..c89676b 100644 >> > --- a/hw/misc/vfio.c >> > +++ b/hw/misc/vfio.c >> > @@ -1953,7 +1953,7 @@ static void vfio_listener_region_add(MemoryListener >> > *listener, >> > }

Re: [Qemu-devel] [PATCH 00/39] Delay destruction of memory regions to instance_finalize

2013-06-06 Thread Paolo Bonzini
Il 05/06/2013 05:50, Andreas Färber ha scritto: > Am 04.06.2013 20:51, schrieb Paolo Bonzini: >> This series changes all PCI devices (the sole to support hotplug >> _and_ use MemoryRegions) to do memory_region_del_subregion at >> unrealize time, and memory_region_destroy at instance_finalize >> tim

Re: [Qemu-devel] [PATCH 11/39] msix: split msix_free from msix_uninit

2013-06-06 Thread Paolo Bonzini
Il 05/06/2013 06:32, Michael S. Tsirkin ha scritto: > On Wed, Jun 05, 2013 at 09:48:19AM +0200, Paolo Bonzini wrote: >> Il 05/06/2013 06:53, Michael S. Tsirkin ha scritto: >>> On Wed, Jun 05, 2013 at 12:40:00AM +0200, Paolo Bonzini wrote: Il 05/06/2013 00:03, Michael S. Tsirkin ha scritto: >>>

Re: [Qemu-devel] [QEMU question] Disk hot plugging without working PCI hotplug - possible?

2013-06-06 Thread Alexey Kardashevskiy
On 07.06.2013 0:29, Frederic Konrad wrote: > On 06/06/2013 15:59, Alexey Kardashevskiy wrote: >> On 06.06.2013 22:44, Frederic Konrad wrote: >>> On 06/06/2013 12:13, Alexey Kardashevskiy wrote: Hi! For the pseries platform (server PPC64) we do not support PCI hotplug yet. H

Re: [Qemu-devel] [PATCH v6 00/11] rdma: migration support

2013-06-06 Thread Chegu Vinod
On 6/1/2013 9:09 PM, Michael R. Hines wrote: All, I have successfully performed over 1000+ back-to-back RDMA migrations automatically looped *in a row* using a heavy-weight memory-stress benchmark here at IBM. Migration success is done by capturing the actual serial console output of the virt

[Qemu-devel] vhost-scsi and pscsi

2013-06-06 Thread Badari
Hi Nicholas, I am trying to test vhost-scsi with pscsi lio. I am able to create a target and pass it to QEMU. I don't see any error from tcm_vhost or qemu. But the guest VM doesn't see any SCSI device. What am I missing ? Here is how I am using pscsi target. wwpn="naa.6001405bd4e8476e" tpgt=

Re: [Qemu-devel] [PATCH] fix Coverity scan SIGN_EXTENSION error

2013-06-06 Thread Andreas Färber
Am 06.06.2013 17:12, schrieb Gerd Hoffmann: > Cc: Markus Armbruster > Signed-off-by: Gerd Hoffmann Please indicate in the subject that this is about qxl when queuing. :) Cheers, Andreas > --- > hw/display/qxl-render.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [Qemu-devel] [PATCH] virtio-net: put virtio net header inline with data

2013-06-06 Thread Dave Jones
On Thu, Jun 06, 2013 at 02:59:44PM -0500, Jesse Larrew wrote: > >pr_debug("%s: xmit %p %pM\n", vi->dev->name, skb, dest); > > + if (vi->mergeable_rx_bufs) > > + hdr_len = sizeof hdr->mhdr; > > + else > > + hdr_len = sizeof hdr->hdr; > > All conditionals need braces

[Qemu-devel] [Bug 1188018] [NEW] qemu monitor does not suppot rbd "savevm" command

2013-06-06 Thread liu.changpeng
Public bug reported: 1. I used ceph rbd as my block device, /usr/local/bin/qemu-system-x86_64 -drive format=rbd,file=rbd:rbd/sles.img:rbd_cache=true,cache=writeback -boot c -m 1024 -enable-kvm -vnc 0.0.0.0:0 -monitor stdio 2. when in monitor command line "savevm", it reports "Error -95 while wri

[Qemu-devel] [Bug 1185395] [NEW] qemu-1.5.0 savevm error -95 while writing vm with ceph-rbd as storage-backend

2013-06-06 Thread Launchpad Bug Tracker
You have been subscribed to a public bug: With a running VM I encounter this strange behaviour, former qemu-versions don't show up such an error. Perhaps this comes from the rbd-backend in qemu-1.5.0 in combination with ceph-0.56.6? ( -95 might be a general "Operation not supported" error? ) U

Re: [Qemu-devel] [PATCH] virtio-net: put virtio net header inline with data

2013-06-06 Thread Jesse Larrew
On 06/06/2013 03:09 PM, Dave Jones wrote: > On Thu, Jun 06, 2013 at 02:59:44PM -0500, Jesse Larrew wrote: > > > > pr_debug("%s: xmit %p %pM\n", vi->dev->name, skb, dest); > > > +if (vi->mergeable_rx_bufs) > > > +hdr_len = sizeof hdr->mhdr; > > > +else >

Re: [Qemu-devel] [PATCH 7/7] smbios: Check R in -smbios type=0, release=R parses okay

2013-06-06 Thread Markus Armbruster
Laszlo Ersek writes: > On 06/06/13 18:27, Markus Armbruster wrote: >> Signed-off-by: Markus Armbruster >> --- >> hw/i386/smbios.c | 5 - >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/hw/i386/smbios.c b/hw/i386/smbios.c >> index 68bd6d0..88a1360 100644 >> --- a/hw/i38

Re: [Qemu-devel] [PATCH] virtio-net: put virtio net header inline with data

2013-06-06 Thread Jesse Larrew
Hi Michael! On 06/06/2013 04:55 AM, Michael S. Tsirkin wrote: > diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c > index c9e0038..d35a097 100644 > --- a/drivers/net/virtio_net.c > +++ b/drivers/net/virtio_net.c > @@ -106,6 +106,9 @@ struct virtnet_info { > /* Has control virt

Re: [Qemu-devel] [PATCH 6/7] smbios: Fix -smbios type=0, release=... for big endian hosts

2013-06-06 Thread Markus Armbruster
Laszlo Ersek writes: > On 06/06/13 18:27, Markus Armbruster wrote: >> Classic endianness bug due to careless dirty coding: assuming reading >> a byte from an int variable gets the least significant byte. >> >> Signed-off-by: Markus Armbruster >> --- >> hw/i386/smbios.c | 4 ++-- >> 1 file chan

Re: [Qemu-devel] [PATCH 5/7] smbios: Clean up smbios_add_field() parameters

2013-06-06 Thread Markus Armbruster
Laszlo Ersek writes: > On 06/06/13 18:27, Markus Armbruster wrote: >> Having size preceed the associated pointer is odd. Swap them, and fix >> up the types. > > Can you proceed to fix the spelling of "precede"? :) Sure.

Re: [Qemu-devel] [PATCH 4/7] Use sizeof(qemu_uuid) instead of literal 16

2013-06-06 Thread Markus Armbruster
Laszlo Ersek writes: > On 06/06/13 18:27, Markus Armbruster wrote: >> Signed-off-by: Markus Armbruster >> --- >> arch_init.c | 3 ++- >> hw/nvram/fw_cfg.c | 2 +- >> include/sysemu/sysemu.h | 2 +- >> 3 files changed, 4 insertions(+), 3 deletions(-) >> >> diff --git a/arch_in

Re: [Qemu-devel] [PATCH] console: nicer initial screen

2013-06-06 Thread Anthony Liguori
Gerd Hoffmann writes: > Now that we have a function to create a fancy DisplaySurface with a > message for the user, to handle non-existing graphics hardware, we > can make it more generic and use it for other things too. > > This patch adds a text line to the in initial DisplaySurface, notifying

Re: [Qemu-devel] [PATCH] fix Coverity scan SIGN_EXTENSION error

2013-06-06 Thread Laszlo Ersek
On 06/06/13 17:12, Gerd Hoffmann wrote: > Cc: Markus Armbruster > Signed-off-by: Gerd Hoffmann > --- > hw/display/qxl-render.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/display/qxl-render.c b/hw/display/qxl-render.c > index f511a62..269b1a7 100644 > --- a/hw

Re: [Qemu-devel] [PATCH 7/7] smbios: Check R in -smbios type=0, release=R parses okay

2013-06-06 Thread Laszlo Ersek
On 06/06/13 18:27, Markus Armbruster wrote: > Signed-off-by: Markus Armbruster > --- > hw/i386/smbios.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/hw/i386/smbios.c b/hw/i386/smbios.c > index 68bd6d0..88a1360 100644 > --- a/hw/i386/smbios.c > +++ b/hw/i386/smbios

Re: [Qemu-devel] [PATCH 6/7] smbios: Fix -smbios type=0, release=... for big endian hosts

2013-06-06 Thread Laszlo Ersek
On 06/06/13 18:27, Markus Armbruster wrote: > Classic endianness bug due to careless dirty coding: assuming reading > a byte from an int variable gets the least significant byte. > > Signed-off-by: Markus Armbruster > --- > hw/i386/smbios.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletion

Re: [Qemu-devel] [PATCH 5/7] smbios: Clean up smbios_add_field() parameters

2013-06-06 Thread Laszlo Ersek
On 06/06/13 18:27, Markus Armbruster wrote: > Having size preceed the associated pointer is odd. Swap them, and fix > up the types. Can you proceed to fix the spelling of "precede"? :) > Signed-off-by: Markus Armbruster > --- > arch_init.c | 2 +- > hw/i386/smbios.c | 26

Re: [Qemu-devel] [PATCH 1/7] error-report.h: Supply missing include

2013-06-06 Thread Laszlo Ersek
On 06/06/13 18:27, Markus Armbruster wrote: > Missed in commit e5924d8. > > Signed-off-by: Markus Armbruster > --- > include/qemu/error-report.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/qemu/error-report.h b/include/qemu/error-report.h > index c902cc1..14c1719 100644 > -

Re: [Qemu-devel] [PATCH 4/7] Use sizeof(qemu_uuid) instead of literal 16

2013-06-06 Thread Laszlo Ersek
On 06/06/13 18:27, Markus Armbruster wrote: > Signed-off-by: Markus Armbruster > --- > arch_init.c | 3 ++- > hw/nvram/fw_cfg.c | 2 +- > include/sysemu/sysemu.h | 2 +- > 3 files changed, 4 insertions(+), 3 deletions(-) > > diff --git a/arch_init.c b/arch_init.c > index 5d7187

Re: [Qemu-devel] [PATCH 3/7] smbios: Convert to error_report()

2013-06-06 Thread Laszlo Ersek
On 06/06/13 18:27, Markus Armbruster wrote: > Improves diagnistics from ad hoc messages like > > Invalid SMBIOS UUID string > > to > > qemu-system-x86_64: -smbios type=1,uuid=gaga: Invalid UUID > > Signed-off-by: Markus Armbruster > --- > arch_init.c | 1 - > hw/i386/smbios.c |

Re: [Qemu-devel] [PATCH 2/7] log.h: Supply missing includes

2013-06-06 Thread Laszlo Ersek
On 06/06/13 18:27, Markus Armbruster wrote: > has always been missing. Rest missed in commit eeacee4. > > Signed-off-by: Markus Armbruster > --- > include/qemu/log.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/include/qemu/log.h b/include/qemu/log.h > index 6b0db02..fd76f91 10

Re: [Qemu-devel] SEV and WFE instructions on ARM

2013-06-06 Thread Peter Maydell
On 6 June 2013 15:34, Sebastian Huber wrote: > I want to use Qemu to test some SMP code. For this I set up Qemu to fire up > two Cortex-A9 MPCore CPUs. I have the following ticket lock implementation: QEMU is not a very good choice for this sort of testing, because we do not implement any of:

[Qemu-devel] [PATCH 5/5] tcg-arm: Use AT_PLATFORM to detect the host ISA

2013-06-06 Thread Richard Henderson
With this we can generate armv7 insns even when the OS compiles for a lower common denominator. The macros are arranged so that when we do compile for a given ISA, all of the runtime checks for that ISA are optimized away. Signed-off-by: Richard Henderson --- tcg/arm/tcg-target.c | 22 +

[Qemu-devel] [PATCH 3/5] tcg-arm: Make use of conditional availability of opcodes for divide

2013-06-06 Thread Richard Henderson
We can now detect and use divide instructions at runtime, rather than having to restrict their availability to compile-time. Signed-off-by: Richard Henderson --- tcg/arm/tcg-target.c | 16 ++-- tcg/arm/tcg-target.h | 14 -- 2 files changed, 22 insertions(+), 8 deletions(-

[Qemu-devel] [PATCH 4/5] tcg-arm: Simplify logic in detecting the ARM ISA in use

2013-06-06 Thread Richard Henderson
GCC 4.8 defines a handy __ARM_ARCH symbol that we can use, which will make us nicely forward compatible with ARMv8 AArch32. Signed-off-by: Richard Henderson --- tcg/arm/tcg-target.c | 62 +--- 1 file changed, 20 insertions(+), 42 deletions(-) diff

[Qemu-devel] [PATCH 2/5] tcg: Simplify logic using TCG_OPF_NOT_PRESENT

2013-06-06 Thread Richard Henderson
Expand the definition of "not present" to include "should not be present". This means we can simplify the logic surrounding the generic tcg opcodes for which the host backend ought not be providing definitions. Signed-off-by: Richard Henderson --- tcg/tcg-opc.h | 26 +++--- t

[Qemu-devel] [PATCH 0/5] tcg-arm: Runtime detection of architecture

2013-06-06 Thread Richard Henderson
Two prepatory generic tcg patches, to allow non-constant values for the various TCG_TARGET_HAS_foo macros. When in patch 3 this gets used, the code inlined in the translators will be able to check the relevant variable and emit either division opcode or the call to the division subroutine. Perhap

[Qemu-devel] [PATCH 1/5] tcg: Allow non-constant control macros

2013-06-06 Thread Richard Henderson
This allows TCG_TARGET_HAS_* to be a variable rather than a constant, which allows easier support for differing ISA levels for the host. Signed-off-by: Richard Henderson --- tcg/tcg-opc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tcg/tcg-opc.h b/tcg/tcg-opc.h index 4246

Re: [Qemu-devel] [PATCH qom-cpu 00/15 v8] target-i386: convert CPU features into properties, part 1

2013-06-06 Thread Eduardo Habkost
On Wed, Jun 05, 2013 at 07:57:11PM +0100, Peter Maydell wrote: > On 5 June 2013 19:31, Eduardo Habkost wrote: > > On Wed, Jun 05, 2013 at 07:29:46PM +0200, Andreas Färber wrote: > >> Am 05.06.2013 19:17, schrieb Eduardo Habkost: > >> > We have had this discussion before, and I remember Anthony say

[Qemu-devel] [Bug 1185395] Re: qemu-1.5.0 savevm error -95 while writing vm with ceph-rbd as storage-backend

2013-06-06 Thread Oliver Francke
** Package changed: ubuntu => qemu -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1185395 Title: qemu-1.5.0 savevm error -95 while writing vm with ceph-rbd as storage- backend Status in QEMU: N

[Qemu-devel] [PATCH 5/7] smbios: Clean up smbios_add_field() parameters

2013-06-06 Thread Markus Armbruster
Having size preceed the associated pointer is odd. Swap them, and fix up the types. Signed-off-by: Markus Armbruster --- arch_init.c | 2 +- hw/i386/smbios.c | 26 ++ include/hw/i386/smbios.h | 2 +- 3 files changed, 16 insertions(+), 14 deletions(

Re: [Qemu-devel] [PATCH v2] e600 core for MPC86xx processors

2013-06-06 Thread Scott Wood
On 06/06/2013 06:44:31 AM, Andreas Färber wrote: Am 26.05.2013 19:41, schrieb Julio Guerra: > MPC86xx processors are based on the e600 core, which is not the case > in qemu where it is based on the 7400 processor. > > This patch creates the e600 core and instantiates the MPC86xx > processors base

[Qemu-devel] [PATCH 7/7] smbios: Check R in -smbios type=0, release=R parses okay

2013-06-06 Thread Markus Armbruster
Signed-off-by: Markus Armbruster --- hw/i386/smbios.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/hw/i386/smbios.c b/hw/i386/smbios.c index 68bd6d0..88a1360 100644 --- a/hw/i386/smbios.c +++ b/hw/i386/smbios.c @@ -140,7 +140,10 @@ static void smbios_build_type_0_field

[Qemu-devel] [PATCH 6/7] smbios: Fix -smbios type=0, release=... for big endian hosts

2013-06-06 Thread Markus Armbruster
Classic endianness bug due to careless dirty coding: assuming reading a byte from an int variable gets the least significant byte. Signed-off-by: Markus Armbruster --- hw/i386/smbios.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/i386/smbios.c b/hw/i386/smbios.c ind

[Qemu-devel] [PATCH 3/7] smbios: Convert to error_report()

2013-06-06 Thread Markus Armbruster
Improves diagnistics from ad hoc messages like Invalid SMBIOS UUID string to qemu-system-x86_64: -smbios type=1,uuid=gaga: Invalid UUID Signed-off-by: Markus Armbruster --- arch_init.c | 1 - hw/i386/smbios.c | 24 2 files changed, 12 insertions(+), 13 d

[Qemu-devel] [PATCH 1/7] error-report.h: Supply missing include

2013-06-06 Thread Markus Armbruster
Missed in commit e5924d8. Signed-off-by: Markus Armbruster --- include/qemu/error-report.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/qemu/error-report.h b/include/qemu/error-report.h index c902cc1..14c1719 100644 --- a/include/qemu/error-report.h +++ b/include/qemu/error-report

[Qemu-devel] [PATCH 4/7] Use sizeof(qemu_uuid) instead of literal 16

2013-06-06 Thread Markus Armbruster
Signed-off-by: Markus Armbruster --- arch_init.c | 3 ++- hw/nvram/fw_cfg.c | 2 +- include/sysemu/sysemu.h | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/arch_init.c b/arch_init.c index 5d71870..aa24660 100644 --- a/arch_init.c +++ b/arch_init.c @@ -1029

[Qemu-devel] [PATCH 0/7] Some -smbios work

2013-06-06 Thread Markus Armbruster
Better error messages, a bit of code cleanup, and a big endian fix. Not addressed: qemu_uuid_parse() sets an SMBIOS field by side effect. Gross! Markus Armbruster (7): error-report.h: Supply missing include log.h: Supply missing includes smbios: Convert to error_report() Use sizeof(qemu_u

[Qemu-devel] [PATCH 2/7] log.h: Supply missing includes

2013-06-06 Thread Markus Armbruster
has always been missing. Rest missed in commit eeacee4. Signed-off-by: Markus Armbruster --- include/qemu/log.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/include/qemu/log.h b/include/qemu/log.h index 6b0db02..fd76f91 100644 --- a/include/qemu/log.h +++ b/include/qemu/log.h @@ -2,6

[Qemu-devel] [Bug 1185395] Re: qemu-1.5.0 savevm error -95 while writing vm with ceph-rbd as storage-backend

2013-06-06 Thread Oliver Francke
** Project changed: qemu => ubuntu -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1185395 Title: qemu-1.5.0 savevm error -95 while writing vm with ceph-rbd as storage- backend Status in Ubuntu:

Re: [Qemu-devel] [PATCH 2/2] Add monitor command mem-nodes

2013-06-06 Thread Eduardo Habkost
On Thu, Jun 06, 2013 at 05:30:16PM +0800, Wanlong Gao wrote: > On 06/05/2013 11:54 PM, Eduardo Habkost wrote: > > On Wed, Jun 05, 2013 at 07:57:42AM -0500, Anthony Liguori wrote: > >> Wanlong Gao writes: > >> > >>> Add monitor command mem-nodes to show the huge mapped > >>> memory nodes locations.

[Qemu-devel] [Bug 1185395] Re: qemu-1.5.0 savevm error -95 while writing vm with ceph-rbd as storage-backend

2013-06-06 Thread Oliver Francke
Sorry for not providing enough informations, here we go: typical longish start for qemu is: /usr/local/bin/qemu-system-x86_64 -usbdevice tablet -enable-kvm -daemonize -pidfile /var/run/qemu-server/760.pid -monitor unix:/var/run /qemu-server/760.mon,server,nowait -vnc unix:/var/run/qemu- server/76

Re: [Qemu-devel] [PATCH 2/2] KVM: PPC: Add dummy kvm_arch_init_irq_routing()

2013-06-06 Thread Scott Wood
On 06/05/2013 10:59:06 PM, Alexey Kardashevskiy wrote: From: Scott Wood The common KVM code insists on calling kvm_arch_init_irq_routing() as soon as it sees kernel header support for it (regardless of whether QEMU supports it). Provide a dummy function to satisfy this. Unlike x86, PPC does n

[Qemu-devel] [PATCH 1/2] fbdev: add linux framebuffer display driver.

2013-06-06 Thread Gerd Hoffmann
Display works, requires truecolor framebuffer with 16 or 32 bpp on the host. 32bpp is recommended. The framebuffer is used as-is, qemu doesn't try to switch modes. With LCD displays mode switching is pretty pointless IMHO, also it wouldn't work anyway with the most common fbdev drivers (vesafb,

[Qemu-devel] [PATCH 2/2] fbdev: add monitor command to enable/disable

2013-06-06 Thread Gerd Hoffmann
This patch adds a fbdev monitor command to enable/disable the fbdev display at runtime to both qmp and hmp. qmp: framebuffer-display enable=on|off scale=on|off device=/dev/fb hmp: framebuffer-display on|off Signed-off-by: Gerd Hoffmann --- hmp-commands.hx | 15 +++ hmp.c

[Qemu-devel] [PATCH] console: nicer initial screen

2013-06-06 Thread Gerd Hoffmann
Now that we have a function to create a fancy DisplaySurface with a message for the user, to handle non-existing graphics hardware, we can make it more generic and use it for other things too. This patch adds a text line to the in initial DisplaySurface, notifying the user that the display isn't i

[Qemu-devel] [PATCH] spice: Add -spice disable-agent-file-transfer cmdline option (rhbz#961850)

2013-06-06 Thread Hans de Goede
Signed-off-by: Hans de Goede --- qemu-options.hx | 7 +-- ui/spice-core.c | 12 2 files changed, 17 insertions(+), 2 deletions(-) diff --git a/qemu-options.hx b/qemu-options.hx index bf94862..b62e542 100644 --- a/qemu-options.hx +++ b/qemu-options.hx @@ -917,8 +917,8 @@ DEF("sp

Re: [Qemu-devel] [0/10] Clean up PCI code to allow for multiple root buses (v2)

2013-06-06 Thread Anthony Liguori
"Michael S. Tsirkin" writes: > On Thu, Jun 06, 2013 at 06:48:44PM +1000, David Gibson wrote: >> The current PCI subsystem has kind of half-hearted support for >> multiple independent root buses - aka PCI domains - in the form of the >> PCIHostBus structure and its domain field. However, it doesn

[Qemu-devel] [PATCH v4 04/10] Add a script to extract VSS SDK headers on POSIX system

2013-06-06 Thread Tomoki Sekiyama
VSS SDK(*) setup.exe is only runnable on Windows. This adds a script to extract VSS SDK headers on POSIX-systems using msitools. * http://www.microsoft.com/en-us/download/details.aspx?id=23490 From: Paolo Bonzini Signed-off-by: Paolo Bonzini Signed-off-by: Tomoki Sekiyama --- scripts/extrac

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

2013-06-06 Thread Tomoki Sekiyama
Support guest-fsfreeze-freeze and guest-fsfreeze-thaw commands for Windows guests. When fsfreeze command is issued, it calls the VSS requester to freeze filesystems and applications. On thaw command, it again tells the VSS requester to thaw them. This also adds calling of initialize functions for

[Qemu-devel] [PATCH] fix Coverity scan SIGN_EXTENSION error

2013-06-06 Thread Gerd Hoffmann
Cc: Markus Armbruster Signed-off-by: Gerd Hoffmann --- hw/display/qxl-render.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/display/qxl-render.c b/hw/display/qxl-render.c index f511a62..269b1a7 100644 --- a/hw/display/qxl-render.c +++ b/hw/display/qxl-render.c @@ -19

[Qemu-devel] [PATCH v4 01/10] configure: Support configuring c++ compiler

2013-06-06 Thread Tomoki Sekiyama
Add configuration for c++ compiler (${cross_prefix}g++ as default) in configure and Makefiles. Currently, usage of c++ language is only for access to Windows VSS using COM+ services in qemu-guest-agent for Windows. Signed-off-by: Tomoki Sekiyama --- configure | 13 + rules.mak |

[Qemu-devel] [PATCH 0/1] spice: Add -spice disable-agent-file-transfer cmdline option (v2)

2013-06-06 Thread Hans de Goede
Hi All, Now with documentation for the new option... Regards, Hans

[Qemu-devel] [PATCH v4 07/10] qemu-ga: Add Windows VSS requester to quiesce applications and filesystems

2013-06-06 Thread Tomoki Sekiyama
Add VSS requester functions for to qemu-ga. This provides facility to request VSS service in Windows guest to quiesce applications and filesystems. This function is only supported in Windows 2003 or later. In older guests, this function does nothing. In several versions of Windows which don't supp

[Qemu-devel] [PATCH v4 05/10] qemu-ga: Add configure options to specify path to Windows/VSS SDK

2013-06-06 Thread Tomoki Sekiyama
To enable VSS support in qemu-ga for Windows, header files included in VSS SDK are required. The VSS support is enabled by the configure option like below: ./configure --with-vss-sdk="/path/to/VSS SDK" If the path is omitted, it tries to search the headers from default paths and VSS support is e

[Qemu-devel] [PATCH v4 10/10] QMP/qemu-ga-client: make timeout longer for guest-fsfreeze-freeze command

2013-06-06 Thread Tomoki Sekiyama
guest-fsfreeze-freeze command can take longer than 3 seconds when heavy disk I/O is running. To avoid unexpected timeout, this changes the timeout to 60 seconds (timeout of pre-commit phase of VSS). Signed-off-by: Tomoki Sekiyama --- QMP/qemu-ga-client |4 +++- 1 file changed, 3 insertions(+

Re: [Qemu-devel] kFreeBSD and USB support

2013-06-06 Thread Hans de Goede
Hi, On 06/06/2013 02:47 PM, Gerd Hoffmann wrote: Hi, This effectively disables host usb support for kFreeBSD. And since I guess the kernel headers are the same on regular FreeBSD, I think the same issue happens on regular FreeBSD too, that is, host usb does not work there as well. What is

[Qemu-devel] [PATCH v4 06/10] qemu-ga: Add Windows VSS provider to quiesce applications on fsfreeze

2013-06-06 Thread Tomoki Sekiyama
Implements a basic stub of software VSS provider. Currently, this module only provides a relay function of events between qemu-guest-agent and Windows VSS when VSS finished filesystem freeze and when qemu snapshot is done. In the future, this module could be extended to support the other VSS funct

[Qemu-devel] [PATCH v4 03/10] checkpatch.pl: check .cpp files

2013-06-06 Thread Tomoki Sekiyama
Enable checkpatch.pl to apply the same checks as C source files for C++ files with .cpp extensions. It also adds some exceptions for C++ sources to suppress errors for: - <> used in C++ template arguments (e.g. template ) - :: used to represent namespaces (e.g. SomeClass::method()) - : used

[Qemu-devel] [PATCH v4 09/10] qemu-ga: install Windows VSS provider on `qemu-ga -s install'

2013-06-06 Thread Tomoki Sekiyama
Register QGA VSS provider library into Windows when qemu-ga is installed as Windows service ('-s install' option). It is deregistered when the service is uninstalled ('-s uninstall' option). Signed-off-by: Tomoki Sekiyama --- qga/main.c |8 1 file changed, 8 insertions(+) diff --gi

[Qemu-devel] [PATCH v4 02/10] Add c++ keywords to QAPI helper script

2013-06-06 Thread Tomoki Sekiyama
Add c++ keywords to avoid errors in compiling with c++ compiler. This also renames class member of PciDeviceInfo to q_class. Signed-off-by: Tomoki Sekiyama --- hmp.c |2 +- hw/pci/pci.c|2 +- scripts/qapi.py | 12 +++- 3 files changed, 13 insertions(+), 3 deletion

[Qemu-devel] [PATCH v4 00/10] qemu-ga: fsfreeze on Windows using VSS

2013-06-06 Thread Tomoki Sekiyama
Hi, This patch series adds fsfreeze support for Windows qemu-guest-agent. changes from v3: -[01/10] Use c++ instead of g++ in configureing C++ compiler if neither $cross_prefix nor $CXX is specified. -[02/10] Add "alternative representations" as a reserved keywords in qapi.py. -[03/1

[Qemu-devel] [PATCH] spice: Add -spice disable-agent-file-transfer cmdline option (rhbz#961850)

2013-06-06 Thread Hans de Goede
Signed-off-by: Hans de Goede --- ui/spice-core.c | 12 1 file changed, 12 insertions(+) diff --git a/ui/spice-core.c b/ui/spice-core.c index bcc4199..2faac93 100644 --- a/ui/spice-core.c +++ b/ui/spice-core.c @@ -446,6 +446,9 @@ static QemuOptsList qemu_spice_opts = { .

Re: [Qemu-devel] [PATCH 1/2] device_tree: Fix build with latest libfdt

2013-06-06 Thread Gerd Hoffmann
On 06/06/13 16:39, Hans de Goede wrote: > We override libfdt_env.h with our own copy, and the latest libfdt expects > libfdt_env.h to define fdt##_t types. Looks sane to me, but I'll leave that to the fdt experts ... cheers, Gerd

Re: [Qemu-devel] [PATCH 2/2] usb/host-libusb: Fix building with libusb git master code

2013-06-06 Thread Gerd Hoffmann
On 06/06/13 16:39, Hans de Goede wrote: > The next libusb release will deprecate libusb_get_port_path, and since > we compile with -Werror, this breaks the build. Patch added to usb patch queue. thanks, Gerd

[Qemu-devel] [PATCH 0/2] 2 compile / build fixes

2013-06-06 Thread Hans de Goede
Hi All, The joys of working with a cutting edge distro on my workstation ... Regards, Hans

[Qemu-devel] [PATCH 2/2] usb/host-libusb: Fix building with libusb git master code

2013-06-06 Thread Hans de Goede
The next libusb release will deprecate libusb_get_port_path, and since we compile with -Werror, this breaks the build. Signed-off-by: Hans de Goede --- hw/usb/host-libusb.c | 4 1 file changed, 4 insertions(+) diff --git a/hw/usb/host-libusb.c b/hw/usb/host-libusb.c index 3a582c5..0c12b0f

[Qemu-devel] [PATCH 1/2] device_tree: Fix build with latest libfdt

2013-06-06 Thread Hans de Goede
We override libfdt_env.h with our own copy, and the latest libfdt expects libfdt_env.h to define fdt##_t types. Signed-off-by: Hans de Goede --- include/libfdt_env.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/include/libfdt_env.h b/include/libfdt_env.h index 3667d4c..1cd8766 10064

[Qemu-devel] SEV and WFE instructions on ARM

2013-06-06 Thread Sebastian Huber
Hello, I want to use Qemu to test some SMP code. For this I set up Qemu to fire up two Cortex-A9 MPCore CPUs. I have the following ticket lock implementation: static inline void _ARM_Data_memory_barrier( void ) { __asm__ volatile ( "dmb" : : : "memory" ); } static inline void _ARM_Data_sy

Re: [Qemu-devel] [QEMU question] Disk hot plugging without working PCI hotplug - possible?

2013-06-06 Thread Frederic Konrad
On 06/06/2013 15:59, Alexey Kardashevskiy wrote: On 06.06.2013 22:44, Frederic Konrad wrote: On 06/06/2013 12:13, Alexey Kardashevskiy wrote: Hi! For the pseries platform (server PPC64) we do not support PCI hotplug yet. However we still want to hot plug disks. As a workaround, we could add m

Re: [Qemu-devel] [PATCH 1/2] Revert "migration: do not sent zero pages in bulk stage"

2013-06-06 Thread Peter Lieven
On 06.06.2013 14:16, Eric Blake wrote: - Original Message - Not sending zero pages breaks migration if a page is zero at the source but not at the destination. This can e.g. happen if different BIOS versions are used at source and destination. It has also been reported that migration on

Re: [Qemu-devel] [QEMU question] Disk hot plugging without working PCI hotplug - possible?

2013-06-06 Thread Alexey Kardashevskiy
On 06.06.2013 22:44, Frederic Konrad wrote: > On 06/06/2013 12:13, Alexey Kardashevskiy wrote: >> Hi! >> >> For the pseries platform (server PPC64) we do not support PCI hotplug >> yet. >> However we still want to hot plug disks. >> >> As a workaround, we could add multiple SCSI host devices >> (vi

Re: [Qemu-devel] [PATCH v2 1/2] audio/intel-hda: Fix Inheritance hierachy

2013-06-06 Thread Gerd Hoffmann
On 06/06/13 07:34, peter.crosthwa...@xilinx.com wrote: > From: Peter Crosthwaite > > The ich6 and ich9 variants either need to inherit one from the other, > or both from a common base class, otherwise its not possible to create > a QOM cast macro for use by the shared implementation functions. >

  1   2   >