Re: [Qemu-devel] [PATCH 1/7] qmp: introduce device-list-properties command

2012-07-27 Thread Luiz Capitulino
On Fri, 27 Jul 2012 08:37:13 -0500 Anthony Liguori aligu...@us.ibm.com wrote: This can be used in conjunction with qom-list-types to determine the supported set of devices and their parameters. Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- qapi-schema.json | 28

Re: [Qemu-devel] [PATCH 36/47] host-utils: add ffsl and flsl

2012-07-27 Thread Eric Blake
On 07/24/2012 05:04 AM, Paolo Bonzini wrote: We can provide fast versions based on the other functions defined by host-utils.h. Some care is required on glibc, which provides ffsl already. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- host-utils.h | 45

Re: [Qemu-devel] [PATCH 2/7] qapi: mark QOM commands stable

2012-07-27 Thread Luiz Capitulino
On Fri, 27 Jul 2012 08:37:14 -0500 Anthony Liguori aligu...@us.ibm.com wrote: We've had a cycle to tweak. It is time to commit to supporting them. qmp_qom_get() and qpm_qom_set() still use the legacy monitor interface, can't we convert it to the qapi? Signed-off-by: Anthony Liguori

[Qemu-devel] [PATCH 03/32] lsi: use qdev_reset_all

2012-07-27 Thread Paolo Bonzini
By first resetting the devices, lsi_soft_reset will find the queue already cleared so there is no need to do that forcibly (which may also leak SCSIRequests, and/or worse due to dangling references to the lsi_request in the hba_private field). Signed-off-by: Paolo Bonzini pbonz...@redhat.com ---

Re: [Qemu-devel] [PATCH 3/7] qapi: add query-machines command

2012-07-27 Thread Luiz Capitulino
On Fri, 27 Jul 2012 08:37:15 -0500 Anthony Liguori aligu...@us.ibm.com wrote: This provides the same output as -M ? but in a structured way. Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- qapi-schema.json | 28 qmp-commands.hx |6 ++ vl.c

[Qemu-devel] [PATCH 05/32] lsi: avoid redundant tests of s-current != NULL

2012-07-27 Thread Paolo Bonzini
Simplify the code by checking against req-hba_private directly, and asserting that it is non-NULL before a command is completed or canceled. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/lsi53c895a.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git

[Qemu-devel] [PATCH 30/32] virtio-scsi: report parameter change events

2012-07-27 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/virtio-scsi.c | 16 1 file changed, 16 insertions(+) diff --git a/hw/virtio-scsi.c b/hw/virtio-scsi.c index bf18861..c4a5b22 100644 --- a/hw/virtio-scsi.c +++ b/hw/virtio-scsi.c @@ -27,6 +27,7 @@ /* Feature Bits */

Re: [Qemu-devel] [PATCH 5/7] qapi: add query-cpudefs command

2012-07-27 Thread Luiz Capitulino
On Fri, 27 Jul 2012 08:37:17 -0500 Anthony Liguori aligu...@us.ibm.com wrote: This command attempts to map to the behavior of -cpu ?. Unfortunately, the output of this command differs wildly across targets. To accomodate this, we use a weak symbol to implement a default version of the

[Qemu-devel] [PATCH 22/32] scsi: add tracepoint for scsi_req_cancel

2012-07-27 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/scsi-bus.c |1 + trace-events |1 + 2 files changed, 2 insertions(+) diff --git a/hw/scsi-bus.c b/hw/scsi-bus.c index dd0cdd0..543d9d2 100644 --- a/hw/scsi-bus.c +++ b/hw/scsi-bus.c @@ -1496,6 +1496,7 @@ void

Re: [Qemu-devel] [PATCH 0/7] qapi: add commands to remove the need to parse -help output

2012-07-27 Thread Luiz Capitulino
On Fri, 27 Jul 2012 08:37:12 -0500 Anthony Liguori aligu...@us.ibm.com wrote: This series implements the necessary commands to implements danpb's idea to remove -help parsing in libvirt. We would introduce all of these commands in 1.2 and then change the -help output starting in 1.3. I've

[Qemu-devel] [PATCH 23/32] scsi: introduce hotplug() and hot_unplug() interfaces for SCSI bus

2012-07-27 Thread Paolo Bonzini
From: Cong Meng m...@linux.vnet.ibm.com Add two interfaces hotplug() and hot_unplug() to scsi bus info. The scsi bus can implement these two interfaces to signal the HBA driver of guest kernel to add/remove the scsi device in question. Signed-off-by: Sen Wang senw...@linux.vnet.ibm.com

[Qemu-devel] [PATCH 09/32] scsi-disk: make discard asynchronous

2012-07-27 Thread Paolo Bonzini
By making discard asynchronous, we can reuse all the error handling code that is used for other commands. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/scsi-disk.c | 22 ++ 1 file changed, 10 insertions(+), 12 deletions(-) diff --git a/hw/scsi-disk.c

[Qemu-devel] [PATCH 20/32] scsi-disk: Fail medium writes with proper sense for readonly LUNs

2012-07-27 Thread Paolo Bonzini
From: Ronnie Sahlberg ronniesahlb...@gmail.com Add sense code for DATA_PROTECT/WRITE_PROTECTED and return this error for any WRITE*/WRITE_VERIFY* calls if the device is readonly=on, i.e. write-protected Signed-off-by: Ronnie Sahlberg ronniesahlb...@gmail.com Signed-off-by: Paolo Bonzini

[Qemu-devel] [PATCH 12/32] scsi-disk: separate read_data/write_data implementation for emulate_reqops

2012-07-27 Thread Paolo Bonzini
The previous patch only separated the send_command callback. Use different implementations also for read_data and write_data. The latter is still unreachable, so it aborts for now. read_data passes the data buffer that was prepared and completes the command. Signed-off-by: Paolo Bonzini

[Qemu-devel] [PATCH 24/32] scsi: establish precedence levels for unit attention

2012-07-27 Thread Paolo Bonzini
When a device is resized, we will report a unit attention condition for CAPACITY DATA HAS CHANGED. However, we should ensure that this condition does not override a more important unit attention condition. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- hw/scsi-bus.c | 52

Re: [Qemu-devel] [PATCH V2 1/5] QMP, Introduce xen-set-global-dirty-log command.

2012-07-27 Thread Luiz Capitulino
On Fri, 20 Jul 2012 15:34:38 +0100 Anthony PERARD anthony.per...@citrix.com wrote: This command is used during a migration of a guest under Xen. It calls memory_global_dirty_log_start or memory_global_dirty_log_stop according to the argument pass to the command. Signed-off-by: Anthony

[Qemu-devel] [PATCH 01/32] megasas: Replace trace_megasas_dcmd_dump_frame()

2012-07-27 Thread Paolo Bonzini
From: Hannes Reinecke h...@suse.de trace_megasas_dcmd_dump_frame() takes 9 arguments, which is rather much. Plus the trace infrastructure doesn't support it. As we can get the information via other means it's pointless to have it in the driver, so rather use some proper trace point here and

Re: [Qemu-devel] [PATCH] vl: add -libvirt-caps option for libvirt to stop parsing help output

2012-07-27 Thread Daniel P. Berrange
On Fri, Jul 27, 2012 at 01:23:03PM +0200, Markus Armbruster wrote: Daniel P. Berrange berra...@redhat.com writes: The above would take care of probably 50% of the current libvirt capabilities probing, including a portion of the -help stuff. Then there is all the rest of the crap we detect

Re: [Qemu-devel] [PATCH 0/7] qapi: add commands to remove the need to parse -help output

2012-07-27 Thread Daniel P. Berrange
On Fri, Jul 27, 2012 at 01:21:01PM -0300, Luiz Capitulino wrote: On Fri, 27 Jul 2012 08:37:12 -0500 Anthony Liguori aligu...@us.ibm.com wrote: This series implements the necessary commands to implements danpb's idea to remove -help parsing in libvirt. We would introduce all of these

Re: [Qemu-devel] qemu_rearm_alarm_timer: do not call rearm if the next deadline is INT64_MAX

2012-07-27 Thread Andreas Färber
Am 12.06.2012 10:24, schrieb Andreas Färber: Am 29.05.2012 15:35, schrieb Stefano Stabellini: qemu_rearm_alarm_timer partially duplicates the code in qemu_next_alarm_deadline to figure out if it needs to rearm the timer. If it calls qemu_next_alarm_deadline, it always rearms the timer even if

Re: [Qemu-devel] [PATCH] Fixing sh4 serial abort

2012-07-27 Thread Rob Landley
On 07/27/2012 09:32 AM, Peter Maydell wrote: On 27 July 2012 14:45, Rob Landley r...@landley.net wrote: I.E. sci_getreg(port, SCFCR) move to before checking whether or not we'll ever possibly use the result. SCFCR is 0x18 and QEMU calls abort() on an attempt to read from an unimplemented

Re: [Qemu-devel] [PATCH] vl: add -libvirt-caps option for libvirt to stop parsing help output

2012-07-27 Thread Markus Armbruster
Daniel P. Berrange berra...@redhat.com writes: On Fri, Jul 27, 2012 at 01:23:03PM +0200, Markus Armbruster wrote: Daniel P. Berrange berra...@redhat.com writes: The above would take care of probably 50% of the current libvirt capabilities probing, including a portion of the -help stuff.

Re: [Qemu-devel] [PATCH 3/7] qapi: add query-machines command

2012-07-27 Thread Eric Blake
On 07/27/2012 07:37 AM, Anthony Liguori wrote: This provides the same output as -M ? but in a structured way. Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- qapi-schema.json | 28 qmp-commands.hx |6 ++ vl.c | 31

Re: [Qemu-devel] [PATCH] Fixing sh4 serial abort

2012-07-27 Thread Peter Maydell
On 27 July 2012 18:16, Rob Landley r...@landley.net wrote: On 07/27/2012 09:32 AM, Peter Maydell wrote: On 27 July 2012 14:45, Rob Landley r...@landley.net wrote: I.E. sci_getreg(port, SCFCR) move to before checking whether or not we'll ever possibly use the result. SCFCR is 0x18 and QEMU

Re: [Qemu-devel] [PATCH RFC v4 18/18] MAINTAINERS: Document virtex_ml507 machine

2012-07-27 Thread Edgar E. Iglesias
Acked-by: Edgar E. Iglesias edgar.igles...@gmail.com Im currently traveling but can apply when Im back. Thanks Edgar --- Sent from my phone On Jul 26, 2012 2:29 PM, Andreas Färber afaer...@suse.de wrote: It's a ppc440, so place it close to Bamboo. Signed-off-by: Andreas Färber

Re: [Qemu-devel] [PATCH 3/7] qapi: add query-machines command

2012-07-27 Thread Anthony Liguori
Eric Blake ebl...@redhat.com writes: On 07/27/2012 07:37 AM, Anthony Liguori wrote: This provides the same output as -M ? but in a structured way. Signed-off-by: Anthony Liguori aligu...@us.ibm.com --- qapi-schema.json | 28 qmp-commands.hx |6 ++

Re: [Qemu-devel] [PATCH 3/7] qapi: add query-machines command

2012-07-27 Thread Eric Blake
On 07/27/2012 12:12 PM, Anthony Liguori wrote: Eric Blake ebl...@redhat.com writes: +# @name: the name of the machine +# +# @alias: #optional an alias for the machine name Can a machine have more than one alias? No, it can't. Really, alias is only used to map 'pc' - 'pc-1.1'. It's

Re: [Qemu-devel] [PATCH 5/7] qapi: add query-cpudefs command

2012-07-27 Thread Eric Blake
On 07/27/2012 07:37 AM, Anthony Liguori wrote: This command attempts to map to the behavior of -cpu ?. Unfortunately, the output of this command differs wildly across targets. To accomodate this, we use a weak symbol to implement a default version of the s/accomodate/accommodate/ command

Re: [Qemu-devel] [PATCH v3 2/2] block: Support GlusterFS as a QEMU block backend

2012-07-27 Thread Blue Swirl
On Wed, Jul 25, 2012 at 6:00 AM, Bharata B Rao bhar...@linux.vnet.ibm.com wrote: block: Support GlusterFS as a QEMU block backend. From: Bharata B Rao bhar...@linux.vnet.ibm.com This patch adds gluster as the new block backend in QEMU. This gives QEMU the ability to boot VM images from

Re: [Qemu-devel] [RFC PATCH 0/6] virtio-trace: Support virtio-trace

2012-07-27 Thread Blue Swirl
On Wed, Jul 25, 2012 at 8:15 AM, Masami Hiramatsu masami.hiramatsu...@hitachi.com wrote: (2012/07/25 5:26), Blue Swirl wrote: The following patch set provides a low-overhead system for collecting kernel tracing data of guests by a host in a virtualization environment. A guest OS generally

Re: [Qemu-devel] Funny -m arguments can crash

2012-07-27 Thread Blue Swirl
On Wed, Jul 25, 2012 at 11:49 AM, Markus Armbruster arm...@redhat.com wrote: Asking for funny memory sizes sometimes crashes for me. For instance -m 0.8 or the equivalent -m 838860b. -m 0.7 appears to work. Should we round memory sizes? How? Where? We had a discussion about this recently:

Re: [Qemu-devel] [PATCH] block: Add -drive detect_zero=on|off option to detect all zero writes.

2012-07-27 Thread Paolo Bonzini
Il 27/07/2012 15:58, Anthony Liguori ha scritto: Note that if you want to test this feature, you must use a qcow2 version 3 file. To create that, do: qemu-img create -f qcow2 -o compat=1.1 name size Ordinary qcow2 (v2) and raw do NOT know how to treat zero blocks specially, so you won't

Re: [Qemu-devel] [PATCH v5 5/6] ps2: output warning when event queue full

2012-07-27 Thread Blue Swirl
On Thu, Jul 26, 2012 at 4:49 AM, Amos Kong ak...@redhat.com wrote: Event would be ignored if ps2 queue is full, this patch added a warning in ignore path. Signed-off-by: Amos Kong ak...@redhat.com --- hw/ps2.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git

[Qemu-devel] [Bug 897750] Re: libvirt/kvm problem with disk attach/detach/reattach on running virt

2012-07-27 Thread Scott Moser
For the cloud-images specifically, this problem was solved once before in bug 450463. our cloud-images have the following in their /etc/modules: # /etc/modules: kernel modules to load at boot time. # # This file contains the names of kernel modules that should be loaded # at boot time, one per

Re: [Qemu-devel] [PATCH 4/7] compiler: add macro for GCC weak symbols

2012-07-27 Thread Blue Swirl
On Fri, Jul 27, 2012 at 3:31 PM, Anthony Liguori aligu...@us.ibm.com wrote: Peter Maydell peter.mayd...@linaro.org writes: On 27 July 2012 15:27, Anthony Liguori aligu...@us.ibm.com wrote: Peter Maydell peter.mayd...@linaro.org writes: The GCC manual says Weak symbols are supported for ELF

[Qemu-devel] [PATCH V4 09/12] hw/sd.c: convert SD state to QOM object

2012-07-27 Thread Igor Mitsyanko
A straightforward conversion of SD card implementation to a proper QEMU object. Wrapper functions were introduced for SDClass methods in order to avoid SD card users modification. Because of this, name change for several functions in hw/sd.c was required. Signed-off-by: Igor Mitsyanko

[Qemu-devel] [Bug 1030104] [NEW] Parallel build doesn't work after make clean

2012-07-27 Thread Igor Mitsyanko
Public bug reported: After running make clean qemu won't build with -j option. When I run ./configure make clean and then make -j5, following errors occur: GEN config-host.h GEN trace.h GEN qemu-options.def GEN qmp-commands.h GEN qapi-types.h GEN qapi-visit.h GEN

[Qemu-devel] [PATCH V4 06/12] hw/sd.c: make sd_dataready() return bool

2012-07-27 Thread Igor Mitsyanko
For the sake of code clarity Signed-off-by: Igor Mitsyanko i.mitsya...@samsung.com Reviewed-by: Peter Maydell peter.mayd...@linaro.org --- hw/sd.c |2 +- hw/sd.h |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/sd.c b/hw/sd.c index d4a8927..a51d18d 100644 ---

[Qemu-devel] [PATCH V4 11/12] SD card: introduce spi property for SD card objects

2012-07-27 Thread Igor Mitsyanko
And drop passing is_spi argument to SDCardClass::init function. spi property could be set only while SD card object is not attached to any BlockDriverState. It defaults to false. Signed-off-by: Igor Mitsyanko i.mitsya...@samsung.com Cc: Paul Brook p...@codesourcery.com --- hw/sd.c | 33

[Qemu-devel] [PATCH V4 08/12] hw/sd.c: add SD card save/load support

2012-07-27 Thread Igor Mitsyanko
This patch updates SD card model to support save/load of card's state. Signed-off-by: Igor Mitsyanko i.mitsya...@samsung.com --- hw/sd.c | 88 +- 1 files changed, 64 insertions(+), 24 deletions(-) diff --git a/hw/sd.c b/hw/sd.c index

[Qemu-devel] [PATCH V4 03/12] hw/sd.c: introduce wrapper for conversion address to wp group

2012-07-27 Thread Igor Mitsyanko
Add wrapper function sd_addr_to_wpnum() to replace long address--wg_group conversion line. Signed-off-by: Igor Mitsyanko i.mitsya...@samsung.com --- hw/sd.c | 27 +-- 1 files changed, 13 insertions(+), 14 deletions(-) diff --git a/hw/sd.c b/hw/sd.c index

[Qemu-devel] [PATCH V4 10/12] SD card users: optimize access to SDClass methods

2012-07-27 Thread Igor Mitsyanko
Rather that repeatedly call SD_GET_CLASS() in a loop, call it once before a loop starts. Signed-off-by: Igor Mitsyanko i.mitsya...@samsung.com --- hw/omap_mmc.c|9 + hw/pl181.c |7 --- hw/pxa2xx_mmci.c |6 -- 3 files changed, 13 insertions(+), 9 deletions(-)

[Qemu-devel] [PATCH V4 01/12] hw/sd.c: convert wp_groups in SDState to bitfield

2012-07-27 Thread Igor Mitsyanko
Representing each group write protection flag with only one bit instead of int variable significantly reduces memory consumption. Signed-off-by: Igor Mitsyanko i.mitsya...@samsung.com --- hw/sd.c | 33 +++-- 1 files changed, 19 insertions(+), 14 deletions(-) diff

[Qemu-devel] [PATCH V4 07/12] hw/sd.c: make sd_wp_addr() return bool

2012-07-27 Thread Igor Mitsyanko
For the sake of code clarity Signed-off-by: Igor Mitsyanko i.mitsya...@samsung.com --- hw/sd.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/sd.c b/hw/sd.c index a51d18d..20ebd8e 100644 --- a/hw/sd.c +++ b/hw/sd.c @@ -544,7 +544,7 @@ static void

[Qemu-devel] [PATCH V4 00/12] SD save/load support, SD qomification and bug fixes

2012-07-27 Thread Igor Mitsyanko
v3-v4 PATCH11: spi property now can be set until SD card is not attached to BlockDriverState, not when card in idle state; PATCH12: device-id property renamed to drive, few code movements; PATCH13 droped, its functionality is available in BlockDriverState; v2-v3 Patchset modified in such a way

[Qemu-devel] [PATCH V4 12/12] hw/sd.c: introduce SD card drive property

2012-07-27 Thread Igor Mitsyanko
Setting drive SD card property ties SD card with BlockDriverState of the same name. This property can be set dynamically, allowing for SD card hot-insert. With drive property we no longer need SDClass::init method, all work is done in property setter. Signed-off-by: Igor Mitsyanko

[Qemu-devel] [PATCH V4 04/12] hw/sd.c: favour SD card type (SDSC or SDHC) when performing erase

2012-07-27 Thread Igor Mitsyanko
Standard capacity cards SDSC use byte unit address while SDHC and SDXC Cards use block unit address (512 bytes) when setting ERASE_START and ERASE_END with CMD32 and CMD33, we have to account for this. Signed-off-by: Igor Mitsyanko i.mitsya...@samsung.com --- hw/sd.c |6 -- 1 files

[Qemu-devel] [PATCH V4 05/12] hw/sd.c: convert binary variables to bool

2012-07-27 Thread Igor Mitsyanko
Several members of SDState have type int when they actually are binary variables. Change type of these variables to bool to improve code readability. Change SD API to be in consistency with new variables type. Signed-off-by: Igor Mitsyanko i.mitsya...@samsung.com Reviewed-by: Peter Maydell

[Qemu-devel] [Bug 988291] Re: virsh -c qemu+ssh://root@source/system migrate --domain client --desturi qemu+ssh://root@destination/system fails

2012-07-27 Thread FL
*** This bug is a duplicate of bug 989452 *** https://bugs.launchpad.net/bugs/989452 Hi, How do you turning off gssapi-auth ? -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/988291 Title: virsh

[Qemu-devel] [PATCH 0/3] Attempting to run Microport UNIX (ca 1987)

2012-07-27 Thread Matthew Ogilvie
I've recently been trying to get an ancient version of UNIX I used to use working in qemu: Micoport UNIX System V/386, v 2.1 (ca 1987). I used this from about 1987 until about 1996, when I first got Linux. With a couple of small patches, I can get qemu to boot and run UNIX from the first

[Qemu-devel] [PATCH 1/3] fix some debug printf 64bit format strings

2012-07-27 Thread Matthew Ogilvie
Signed-off-by: Matthew Ogilvie mmogilvi_q...@miniinfo.net --- hw/cirrus_vga.c | 4 ++-- hw/ide/cmd646.c | 5 +++-- hw/ide/via.c| 5 +++-- 3 files changed, 8 insertions(+), 6 deletions(-) diff --git a/hw/cirrus_vga.c b/hw/cirrus_vga.c index 623dd68..24ddea6 100644 --- a/hw/cirrus_vga.c +++

[Qemu-devel] [PATCH 3/3] HACKS for vga text mode with Microport UNIX (ca 1987)

2012-07-27 Thread Matthew Ogilvie
Background: I don't really think it is appropriate to include this patch in it's current form, but I'm posting it to illustrate some of the obscure things that Microport UNIX System V/386 v 2.1 (ca 1987) is trying to do to the video card. On real hardware, this version of UNIX

[Qemu-devel] [Bug 897750] Re: libvirt/kvm problem with disk attach/detach/reattach on running virt

2012-07-27 Thread Serge Hallyn
pci_hotplog actually is available, at least on precise. But it is not loaded (acpiphp is). After I modprobe pci_hotplug, the problem goes away. Thanks wangpan. ** Changed in: qemu-kvm (Ubuntu) Assignee: (unassigned) = Stefan Bader (stefan-bader-canonical) -- You received this bug

[Qemu-devel] [Bug 897750] Re: libvirt/kvm problem with disk attach/detach/reattach on running virt

2012-07-27 Thread Serge Hallyn
Hi Stefan, I'm assigning this bug to you to get your input about which package I need to target it to. It's invalid (I believe) for qemu-kvm. We need server cloud guests to have pci_hotplog auto-loaded at boot (to avoid this bug). Should I assign to package linux for that? -- You received

[Qemu-devel] [PATCH 2/3] target-i386/translate.c: mov to/from crN/drN: ignore mod bits

2012-07-27 Thread Matthew Ogilvie
Microport UNIX System V/386 v 2.1 (ca 1987) uses mod R/M bytes for the control register mov instructions where the mod bits are 0, even though the 80386 spec claims they are always 1's. The fact that it ran at all clearly indicates the real chips (at least 386 and 486) just ignores the bits and

[Qemu-devel] [PATCH V4 02/12] hw/sd.c: make sd_wp_addr() accept 64 bit address argument

2012-07-27 Thread Igor Mitsyanko
Currently sd_wp_addr() accepts 32 bit address arguments therefore implicitly restricting SD card address range. Change address argument type to uint64_t. Signed-off-by: Igor Mitsyanko i.mitsya...@samsung.com --- hw/sd.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

Re: [Qemu-devel] [RFC PATCH] vfio: VFIO PCI driver for Qemu

2012-07-27 Thread Alex Williamson
On Fri, 2012-07-27 at 19:22 +, Blue Swirl wrote: On Wed, Jul 25, 2012 at 5:03 PM, Alex Williamson diff --git a/hw/vfio_pci.c b/hw/vfio_pci.c new file mode 100644 index 000..e9ae421 --- /dev/null +++ b/hw/vfio_pci.c @@ -0,0 +1,2030 @@ +/* + * vfio based device assignment

[Qemu-devel] Status update for tcm_vhost driver merge

2012-07-27 Thread Nicholas A. Bellinger
Hello folks, The RFC-v5 patch for tcm_vhost kernel code was sent out for review a bit less than 24 hours ago, and thus far there has not been any additional comments.. Thanks to everyone who has been participating in the various threads over the past week and giving their feedback! Also, just a

Re: [Qemu-devel] [PULL] UniCore32 PUV3 machine support

2012-07-27 Thread Blue Swirl
On Mon, Jul 23, 2012 at 2:29 AM, g...@mprc.pku.edu wrote: are available in the git repository at: git://github.com/gxt/QEMU.git unicore32 I'm getting these errors, please fix: CClibhw64/hw/puv3_gpio.o /src/qemu/hw/puv3_gpio.c: In function 'puv3_gpio_read':

Re: [Qemu-devel] [PATCH 4/7] compiler: add macro for GCC weak symbols

2012-07-27 Thread Anthony Liguori
Blue Swirl blauwir...@gmail.com writes: On Fri, Jul 27, 2012 at 3:31 PM, Anthony Liguori aligu...@us.ibm.com wrote: Peter Maydell peter.mayd...@linaro.org writes: On 27 July 2012 15:27, Anthony Liguori aligu...@us.ibm.com wrote: Peter Maydell peter.mayd...@linaro.org writes: The GCC manual

Re: [Qemu-devel] [PATCH 2/3] target-i386/translate.c: mov to/from crN/drN: ignore mod bits

2012-07-27 Thread malc
On Fri, 27 Jul 2012, Matthew Ogilvie wrote: Microport UNIX System V/386 v 2.1 (ca 1987) uses mod R/M bytes for the control register mov instructions where the mod bits are 0, even though the 80386 spec claims they are always 1's. The fact that it ran at all clearly indicates the real chips

Re: [Qemu-devel] [PATCH 4/7] compiler: add macro for GCC weak symbols

2012-07-27 Thread Blue Swirl
On Fri, Jul 27, 2012 at 8:51 PM, Anthony Liguori aligu...@us.ibm.com wrote: Blue Swirl blauwir...@gmail.com writes: On Fri, Jul 27, 2012 at 3:31 PM, Anthony Liguori aligu...@us.ibm.com wrote: Peter Maydell peter.mayd...@linaro.org writes: On 27 July 2012 15:27, Anthony Liguori

[Qemu-devel] [RFC 00/27]: add new error format

2012-07-27 Thread Luiz Capitulino
[Please, read below why this is an RFC] This series implements the 'Plan for error handling in QMP' as described by Anthony in this email: http://lists.gnu.org/archive/html/qemu-devel/2012-07/msg03764.html Basically, this replaces almost all error classes by GenericError (the exception are

[Qemu-devel] [PATCH 03/27] qerror: QERR_DEVICE_ENCRYPTED: add filename info to human msg

2012-07-27 Thread Luiz Capitulino
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- qerror.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qerror.c b/qerror.c index 082de98..b2ed0e3 100644 --- a/qerror.c +++ b/qerror.c @@ -81,7 +81,7 @@ static const QErrorStringTable qerror_table[] = { }, {

[Qemu-devel] [PATCH 17/27] qapi: qapi-types.h: don't include qapi/qapi-types-core.h

2012-07-27 Thread Luiz Capitulino
qapi-types.h needs only qemu-common.h. Including qapi-types-core.h causes problems when qerror.h or error.h includes qapi-types.h. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- scripts/qapi-types.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[Qemu-devel] [PATCH 20/27] qapi-schema: add ErrorClass enum

2012-07-27 Thread Luiz Capitulino
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- qapi-schema.json | 26 ++ 1 file changed, 26 insertions(+) diff --git a/qapi-schema.json b/qapi-schema.json index 7500754..b1325c4 100644 --- a/qapi-schema.json +++ b/qapi-schema.json @@ -3,6 +3,32 @@ # QAPI

[Qemu-devel] [PATCH 21/27] qerror: qerror_table: don't use C99 struct initializers

2012-07-27 Thread Luiz Capitulino
This allows for changing QERR_ macros to initialize two struct members at the same time. See next commit for more details. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- qerror.c | 280 +++ qerror.h | 2 +- 2 files changed,

[Qemu-devel] [PATCH 10/27] qmp: query-block: add 'valid_encryption_key' field

2012-07-27 Thread Luiz Capitulino
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- block.c | 1 + qapi-schema.json | 7 +-- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/block.c b/block.c index ce7eb8f..b230f16 100644 --- a/block.c +++ b/block.c @@ -2445,6 +2445,7 @@ BlockInfoList

[Qemu-devel] [PATCH 26/27] error, qerror: pass desc string to error calls

2012-07-27 Thread Luiz Capitulino
FIXME: broke error_is_type() and qemu-ga. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- error.c | 5 ++- qerror.c | 44 ++-- qerror.h | 143 +++ 3 files changed, 77 insertions(+), 115 deletions(-) diff

[Qemu-devel] [PATCH 14/27] error: drop functions used to get error data

2012-07-27 Thread Luiz Capitulino
They are not used anymore and all current errors' data are going to be dropped. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- error.c | 23 --- error.h | 10 -- error_int.h | 1 - 3 files changed, 34 deletions(-) diff --git a/error.c b/error.c

[Qemu-devel] [PATCH 19/27] qapi: don't convert enum strings to lowercase

2012-07-27 Thread Luiz Capitulino
Next commit will introduce enum strings in camel case. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- scripts/qapi-types.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/qapi-types.py b/scripts/qapi-types.py index 9b7da96..cf601ae 100644 ---

[Qemu-devel] [PATCH 04/27] qerror: reduce public exposure

2012-07-27 Thread Luiz Capitulino
Make functions only used in qerror.c static. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- qerror.c | 10 +- qerror.h | 5 - 2 files changed, 5 insertions(+), 10 deletions(-) diff --git a/qerror.c b/qerror.c index b2ed0e3..ce0499b 100644 --- a/qerror.c +++ b/qerror.c @@

[Qemu-devel] [PATCH 18/27] qapi: generate correct enum names for camel case enums

2012-07-27 Thread Luiz Capitulino
That's, generate ERROR_CLASS_GENERIC_ERROR instead of ERROR_CLASS_GENERICERROR. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- scripts/qapi-types.py | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/scripts/qapi-types.py b/scripts/qapi-types.py index

[Qemu-devel] [PATCH 08/27] qerror: don't delay error message construction

2012-07-27 Thread Luiz Capitulino
Today, the error message is only constructed when it's used. This commit changes that to construct the error message when the error object is built (ie. when the error is reported). This eliminates the need of storing a pointer to qerror_table[], which will be dropped soon, and also simplifies

[Qemu-devel] [PATCH 11/27] hmp: hmp_cont(): don't rely on QERR_DEVICE_ENCRYPTED

2012-07-27 Thread Luiz Capitulino
Today, hmp_cont() checks for QERR_DEVICE_ENCRYPTED in order to know if qmp_cont() failed due to an encrypted device. If it did, hmp_cont() accesses QERR_DEVICE_ENCRYPTED's data member 'device' to precisely know for which device an encryption key must be set. However, all errors data members are

[Qemu-devel] [PATCH 07/27] qerror: qerror_format(): return an allocated string

2012-07-27 Thread Luiz Capitulino
Simplifies current and future users. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- error.c | 5 + qerror.c | 10 -- qerror.h | 2 +- 3 files changed, 10 insertions(+), 7 deletions(-) diff --git a/error.c b/error.c index a52b771..b630b05 100644 --- a/error.c +++

[Qemu-devel] [PATCH 09/27] error: don't delay error message construction

2012-07-27 Thread Luiz Capitulino
Today, the error message is only constructed when it's used. This commit changes that to construct the error message when the error object is built (ie. when the error is reported). This simplifies the Error object. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- error.c | 8 +---

[Qemu-devel] [PATCH 27/27] qerror: drop qerror_table and qerror_format()

2012-07-27 Thread Luiz Capitulino
They not used anymore. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- qerror.c | 404 --- qerror.h | 1 - 2 files changed, 405 deletions(-) diff --git a/qerror.c b/qerror.c index f443261..e1606d3 100644 --- a/qerror.c +++

[Qemu-devel] [PATCH 06/27] qerror: QError: drop file, linenr, func

2012-07-27 Thread Luiz Capitulino
They have never been fully used and conflict with future error related improvements. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- qerror.c | 20 +++- qerror.h | 8 +--- 2 files changed, 4 insertions(+), 24 deletions(-) diff --git a/qerror.c b/qerror.c index

[Qemu-devel] [PATCH 13/27] error: error_is_type(): don't use error_get_field()

2012-07-27 Thread Luiz Capitulino
It's going to be dropped. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- error.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/error.c b/error.c index acb10a2..cdeeafe 100644 --- a/error.c +++ b/error.c @@ -113,7 +113,7 @@ bool error_is_type(Error *err, const

[Qemu-devel] [PATCH 23/27] qerror: use ErrorClass for QERR_ macro

2012-07-27 Thread Luiz Capitulino
This commit replaces the place holder value for the ErrorClass argument being passed to all current error calls to the real ErrorClass value. That is, all current errors are mapped to GenericError, except the following errors: CommandNotFound, DeviceNotActive, DeviceNotFound, KVMMissingCap,

[Qemu-devel] [PATCH 02/27] qerror: QERR_AMBIGUOUS_PATH: drop %(object) from human msg

2012-07-27 Thread Luiz Capitulino
Actually, renames it to 'object'. This must be what the original author meant to write, as there's no 'object' in the error's data member. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- qerror.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qerror.c b/qerror.c

[Qemu-devel] [PATCH 12/27] hmp: hmp_change(): don't use error_get_field()

2012-07-27 Thread Luiz Capitulino
Use the 'device' passed by the user and call qmp_query_block() to get the 'filename' info. error_get_field() is going to be dropped by a future commit. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- hmp.c | 37 - 1 file changed, 28 insertions(+), 9

[Qemu-devel] [PATCH 22/27] error, qerror: add ErrorClass argument to error functions

2012-07-27 Thread Luiz Capitulino
The new argument is added to functions qerror_report(), error_set() and error_is_type(). qerror_report_err() is also updated to take care of it. However, the new argument is not used yet. It's only stored in Error and QError. The QERR_ macros are also changed to contain an ErrorClass value.

[Qemu-devel] [PATCH 16/27] hmp: hmp.h: include qdict.h

2012-07-27 Thread Luiz Capitulino
hmp.h is relying on qdict.h being provided by qapi-types.h. However, qapi-types.h won't provide it anymore. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- hmp.h | 1 + 1 file changed, 1 insertion(+) diff --git a/hmp.h b/hmp.h index 8d2b0d7..3275522 100644 --- a/hmp.h +++ b/hmp.h @@

[Qemu-devel] [PATCH 25/27] qapi: qapi.py: allow the ' character be escaped

2012-07-27 Thread Luiz Capitulino
A future commit will add a new qapi script which escapes that character. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- scripts/qapi.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/scripts/qapi.py b/scripts/qapi.py index e062336..9aa518f 100644 ---

Re: [Qemu-devel] [PATCH 4/7] compiler: add macro for GCC weak symbols

2012-07-27 Thread Anthony Liguori
Blue Swirl blauwir...@gmail.com writes: On Fri, Jul 27, 2012 at 8:51 PM, Anthony Liguori aligu...@us.ibm.com wrote: If someone comes along and actively maintains support for another compiler, we can revisit. But otherwise, there's no practical reason to avoid extensions. Because it's more

[Qemu-devel] [PATCH 24/27] qmp: switch to the new error format on the wire

2012-07-27 Thread Luiz Capitulino
IMPORTANT: this commit BREAKS QMP's compatibility for the error response. This commit changes QMP's wire protocol to make use of the simpler error format introduced by previous commits. There are two important (and mostly incompatible) changes: 1. Almost all error classes have been replaced by

[Qemu-devel] [PATCH 01/27] monitor: drop unused monitor debug code

2012-07-27 Thread Luiz Capitulino
In the old QMP days, this code was used to find out QMP commands that might be calling monitor_printf() down its call chain. This is almost impossible to happen today, because the qapi converted commands don't even have a monitor object. Besides, it's been more than a year since I used this last

[Qemu-devel] [PATCH 15/27] block: block_int: include qerror.h

2012-07-27 Thread Luiz Capitulino
Several block/ files are relying on qerror.h being provided by qapi-types.h. However, qapi-types.h won't provide it anymore. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- block_int.h | 1 + 1 file changed, 1 insertion(+) diff --git a/block_int.h b/block_int.h index d72317f..00892e8

[Qemu-devel] [PATCH 05/27] qerror: drop qerror_abort()

2012-07-27 Thread Luiz Capitulino
qerror_abort() depends on the 'file', 'func' and 'linenr' members of QError. However, these members are going to be dropped by the next commit, so let's drop qerror_abort() in favor of printing an error message to stderr plus an abort() call. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com

[Qemu-devel] [PATCH] iSCSI: generate default initiator-name from vm-name

2012-07-27 Thread Ronnie Sahlberg
List, Please find a patch that updates the iSCSI block driver to automatically generate a 'unique' initiator-name based on the name given to the virtual machine. Normally users will always have to set the initiator-name properly since LUN masking on the iscsi targets would almost always

[Qemu-devel] [PATCH] ISCSI: Pick default initiator-name based on the name of the VM

2012-07-27 Thread Ronnie Sahlberg
This patch updates the iscsi layer to automatically pick a 'unique' initiator-name based on the name of the vm in case the user has not set an explicit iqn-name to use. Save the -name of the vm so that we can use it in the default iscsi name. Add a new variable that holds the name for the vm as

[Qemu-devel] [PATCH] SCSI: STARTSTOPUNIT fix for PWRCND!=0

2012-07-27 Thread Ronnie Sahlberg
Paolo, List Please find a trivial fix for the STARTSTOPUNIT emulation. This command will only eject/load media if powercondition is 0. With this patch, QEMU SCSI emulation now pass all my SSU tests : $ ./bin/iscsi-test --dataloss --test=*startstop* iscsi://127.0.0.1:32601/iqn.qemu.scsi-test/3

[Qemu-devel] [PATCH] SCSI: STARTSTOPUNIT only eject/load media if powercondition is 0

2012-07-27 Thread Ronnie Sahlberg
The START STOP UNIT command will only eject/load media if power condition is zero. If power condition is !0 then LOEJ and START will be ignored. From MMC (sbc contains similar wordings too) The Power Conditions field requests the block device to be placed in the power condition defined in

Re: [Qemu-devel] [RFC PATCH] vfio: VFIO PCI driver for Qemu

2012-07-27 Thread Alexey Kardashevskiy
On 28/07/12 05:22, Blue Swirl wrote: On Wed, Jul 25, 2012 at 5:03 PM, Alex Williamson + +static void vfio_enable_intx_kvm(VFIODevice *vdev) +{ +#ifdef CONFIG_KVM These shouldn't be needed. The device will not be useful without KVM, so the file shouldn't be compiled for non-KVM case at all.

Re: [Qemu-devel] [PATCH v3 2/2] block: Support GlusterFS as a QEMU block backend

2012-07-27 Thread Bharata B Rao
On Fri, Jul 27, 2012 at 06:44:04PM +, Blue Swirl wrote: +struct GlusterOpts { static Sure. +bool optional; +char defval[10]; const char *defval? Sure I could. +char *value; +} GlusterOpts[] = { +{false, , NULL }, +{false, , NULL }, +{true,

<    1   2