[Qemu-devel] [Bug 1018530] Re: No write access in a 9p/virtfs shared folder

2012-07-10 Thread M. Mohan Kumar
Georg, pass-through security model needs root privilege, if you want to run qemu as non-root user either you have to use mapped security model or proxy fs driver. But libvirt does not have support for proxy FS driver. I posted a patch few months ago to libvirt for enabling the same. I will do the

Re: [Qemu-devel] [PATCH] iov: Fix do_send_recv() for MinGW (also fixes a build breakage)

2012-07-10 Thread Michael Tokarev
On 11.07.2012 09:09, Stefan Weil wrote: > Commit 25e5e4c7 broke compilation for non POSIX hosts (e.g. MinGW) > because it partially replaced "ret" by "count". > > It also changed the handling of EINTR in a wrong way. > > The patch restores the old code for these two changes. > > Signed-off-by: S

[Qemu-devel] [PATCH] iov: Fix do_send_recv() for MinGW (also fixes a build breakage)

2012-07-10 Thread Stefan Weil
Commit 25e5e4c7 broke compilation for non POSIX hosts (e.g. MinGW) because it partially replaced "ret" by "count". It also changed the handling of EINTR in a wrong way. The patch restores the old code for these two changes. Signed-off-by: Stefan Weil --- iov.c | 11 ++- 1 file change

Re: [Qemu-devel] [PATCH 2/2] vfio-powerpc: added VFIO support

2012-07-10 Thread Benjamin Herrenschmidt
On Wed, 2012-07-11 at 12:54 +1000, Alexey Kardashevskiy wrote: > > Why do you need this, aren't the extension checks sufficient for this to > > be a nop for you? > > > It uses ioapic_remove_gsi_eoi_notifier() so it needs some #ifdef anyway. And > as we do not support > kvm_irqchip_in_kernel(), t

Re: [Qemu-devel] [PATCH 2/2] vfio-powerpc: added VFIO support

2012-07-10 Thread Alexey Kardashevskiy
On 11/07/12 02:55, Alex Williamson wrote: > On Tue, 2012-07-10 at 15:51 +1000, Alexey Kardashevskiy wrote: >> The patch enables VFIO on POWER. >> >> It literally does the following: >> >> 1. POWERPC IOMMU support (the kernel counterpart is required) >> >> 2. Added #ifdef TARGET_PPC64 for EOI handle

Re: [Qemu-devel] [PATCH 0/2] RFC: powerpc-vfio: adding support

2012-07-10 Thread Alexey Kardashevskiy
On 11/07/12 02:57, Alex Williamson wrote: > On Tue, 2012-07-10 at 15:51 +1000, Alexey Kardashevskiy wrote: >> The two patches in this set are supposed to add VFIO support for POWER. >> >> The first one adds one more step in the initalizaion sequence which I am not >> sure is correct. >> >> The seco

Re: [Qemu-devel] [libvirt] Problem setting CPU topology

2012-07-10 Thread Doug Goldstein
On Tue, Jul 10, 2012 at 2:02 PM, Eduardo Habkost wrote: > On Tue, Jul 10, 2012 at 11:54:05AM +0200, Christophe Fergeau wrote: >> On Sat, Jul 07, 2012 at 07:10:53PM +0300, Zeeshan Ali (Khattak) wrote: >> > Hi, >> >I'm trying to set exact CPU topology to qemu-kvm domains to match >> > host's top

Re: [Qemu-devel] [PATCH 4/5] qom-cpu: during cpu reset, it will reset its child

2012-07-10 Thread liu ping fan
On Tue, Jul 10, 2012 at 4:41 PM, Paolo Bonzini wrote: > Il 10/07/2012 08:16, Liu Ping Fan ha scritto: >> This will give the embeded logic module, such as apic has the >> opportunity to reset. >> >> Signed-off-by: Liu Ping Fan >> --- >> qom/cpu.c | 16 >> 1 files changed, 16 in

Re: [Qemu-devel] [PATCH 4/5] qom-cpu: during cpu reset, it will reset its child

2012-07-10 Thread liu ping fan
On Tue, Jul 10, 2012 at 6:12 PM, Andreas Färber wrote: > Am 10.07.2012 10:41, schrieb Paolo Bonzini: >> Il 10/07/2012 08:16, Liu Ping Fan ha scritto: >>> This will give the embeded logic module, such as apic has the >>> opportunity to reset. >>> >>> Signed-off-by: Liu Ping Fan >>> --- >>> qom/cp

Re: [Qemu-devel] [PATCH 2/5] qom: introduce object_is_type_str(), so we can judge its type.

2012-07-10 Thread liu ping fan
On Tue, Jul 10, 2012 at 4:39 PM, Paolo Bonzini wrote: > Il 10/07/2012 08:16, Liu Ping Fan ha scritto: >> Signed-off-by: Liu Ping Fan >> --- >> include/qemu/object.h |2 ++ >> qom/object.c |6 ++ >> 2 files changed, 8 insertions(+), 0 deletions(-) >> >> diff --git a/include/q

Re: [Qemu-devel] [PATCH 1/5] qdev: introduce qdev_create_kid(Object *parent, const char *type)

2012-07-10 Thread liu ping fan
On Tue, Jul 10, 2012 at 4:45 PM, Paolo Bonzini wrote: > Il 10/07/2012 08:16, Liu Ping Fan ha scritto: >> DeviceState can be created as kid of DeviceState/CPUState, not neccesary >> attached to bus. This will be helpful to simulate the real hardware >> submodule which sits inside package. >> >> Sig

[Qemu-devel] buildbot failure in qemu on block_mingw32

2012-07-10 Thread qemu
The Buildbot has detected a new failure on builder block_mingw32 while building qemu. Full details are available at: http://buildbot.b1-systems.de/qemu/builders/block_mingw32/builds/262 Buildbot URL: http://buildbot.b1-systems.de/qemu/ Buildslave for this Build: kraxel_rhel61 Build Reason: The

Re: [Qemu-devel] [PATCH 2/2] vfio-powerpc: added VFIO support

2012-07-10 Thread Benjamin Herrenschmidt
On Wed, 2012-07-11 at 10:17 +1000, Alexey Kardashevskiy wrote: > So the current one would be SPAPR_TCE_32? No, the iommu type is SPAPR_TCE, but the *window* info you get here is the 32-bit window. My thinking is add some versionning and a bunch of reserved fields to that info struct so we can stic

[Qemu-devel] buildbot failure in qemu on default_ppc

2012-07-10 Thread qemu
The Buildbot has detected a new failure on builder default_ppc while building qemu. Full details are available at: http://buildbot.b1-systems.de/qemu/builders/default_ppc/builds/446 Buildbot URL: http://buildbot.b1-systems.de/qemu/ Buildslave for this Build: qemu-ppc.opensuse.org Build Reason:

Re: [Qemu-devel] [PATCH 2/2] vfio-powerpc: added VFIO support

2012-07-10 Thread Alexey Kardashevskiy
On 11/07/12 10:04, Benjamin Herrenschmidt wrote: > On Wed, 2012-07-11 at 09:55 +1000, Alexey Kardashevskiy wrote: >> On 11/07/12 08:26, Scott Wood wrote: >>> On 07/10/2012 12:51 AM, Alexey Kardashevskiy wrote: The patch enables VFIO on POWER. It literally does the following: >>>

Re: [Qemu-devel] [PATCH 2/2] vfio-powerpc: added VFIO support

2012-07-10 Thread Benjamin Herrenschmidt
On Wed, 2012-07-11 at 09:55 +1000, Alexey Kardashevskiy wrote: > On 11/07/12 08:26, Scott Wood wrote: > > On 07/10/2012 12:51 AM, Alexey Kardashevskiy wrote: > >> The patch enables VFIO on POWER. > >> > >> It literally does the following: > >> > >> 1. POWERPC IOMMU support (the kernel counterpart i

Re: [Qemu-devel] [PATCH 2/2] vfio-powerpc: added VFIO support

2012-07-10 Thread Alexey Kardashevskiy
On 11/07/12 08:26, Scott Wood wrote: > On 07/10/2012 12:51 AM, Alexey Kardashevskiy wrote: >> The patch enables VFIO on POWER. >> >> It literally does the following: >> >> 1. POWERPC IOMMU support (the kernel counterpart is required) > [snip] >> +/* API for POWERPC IOMMU */ >> + >

Re: [Qemu-devel] [PATCH] hw/imx_avic.c: Avoid format error when target_phys_addr_t is 64 bits

2012-07-10 Thread Peter Chubb
> "Peter" == Peter Maydell writes: Peter> Add a missing cast to avoid gcc complaining about format string Peter> errors when printing an expression based on a Peter> target_phys_addr_t. Peter> Signed-off-by: Peter Maydell Reviewed-by: Peter Chubb -- Dr Peter Chubb

Re: [Qemu-devel] [PATCH 2/2] vfio-powerpc: added VFIO support

2012-07-10 Thread Scott Wood
On 07/10/2012 12:51 AM, Alexey Kardashevskiy wrote: > The patch enables VFIO on POWER. > > It literally does the following: > > 1. POWERPC IOMMU support (the kernel counterpart is required) [snip] > +/* API for POWERPC IOMMU */ > + > +#define POWERPC_IOMMU 2 > + > +str

Re: [Qemu-devel] [PATCH 2/2] vfio-powerpc: added VFIO support

2012-07-10 Thread Benjamin Herrenschmidt
On Tue, 2012-07-10 at 15:48 -0600, Alex Williamson wrote: > > specific hypervisor calls, not sure what a MemoryListener would do > > here ... > > Hmm, the guest directed iommu updates via hypercalls may not really fit > the MemoryListener model. I'm just trying to think of ways to avoid > having

Re: [Qemu-devel] [PATCH 2/2] vfio-powerpc: added VFIO support

2012-07-10 Thread Alex Williamson
On Wed, 2012-07-11 at 07:32 +1000, Benjamin Herrenschmidt wrote: > On Tue, 2012-07-10 at 10:55 -0600, Alex Williamson wrote: > > > > I wish you could do this through a MemoryListener like we do on x86. > > > > Can you elaborate ? TCE (iommu) manipulation on PAPR is done via > specific hypervisor

Re: [Qemu-devel] [PATCH v2 1/3] bitops: fix types

2012-07-10 Thread Peter Maydell
On 10 July 2012 21:01, Blue Swirl wrote: > On Tue, Jul 10, 2012 at 7:37 PM, Peter Maydell > wrote: >> On 10 July 2012 20:18, Blue Swirl wrote: >>> On Mon, Jul 9, 2012 at 7:49 AM, Markus Armbruster wrote: There is no consensus. I recognize the power of maintainers to force a change e

Re: [Qemu-devel] [PATCH 2/2] vfio-powerpc: added VFIO support

2012-07-10 Thread Benjamin Herrenschmidt
On Tue, 2012-07-10 at 10:55 -0600, Alex Williamson wrote: > > I wish you could do this through a MemoryListener like we do on x86. > Can you elaborate ? TCE (iommu) manipulation on PAPR is done via specific hypervisor calls, not sure what a MemoryListener would do here ... Cheers, Ben.

Re: [Qemu-devel] [PATCH 4/5] Avoid unportable %m format

2012-07-10 Thread Anthony Liguori
On 07/10/2012 04:12 PM, Stefan Weil wrote: Am 10.07.2012 22:53, schrieb Anthony Liguori: On 07/08/2012 06:51 AM, blauwir...@gmail.com wrote: From: Blue Swirl Replace %m format with explicit call to standard strerror(). Signed-off-by: Blue Swirl I would expect '%m' to be thread safe whereas

Re: [Qemu-devel] [PATCH 4/5] Avoid unportable %m format

2012-07-10 Thread Stefan Weil
Am 10.07.2012 22:53, schrieb Anthony Liguori: On 07/08/2012 06:51 AM, blauwir...@gmail.com wrote: From: Blue Swirl Replace %m format with explicit call to standard strerror(). Signed-off-by: Blue Swirl I would expect '%m' to be thread safe whereas strerror() isn't. I don't think this chang

Re: [Qemu-devel] [PATCH 4/5] Avoid unportable %m format

2012-07-10 Thread Anthony Liguori
On 07/08/2012 06:51 AM, blauwir...@gmail.com wrote: From: Blue Swirl Replace %m format with explicit call to standard strerror(). Signed-off-by: Blue Swirl I would expect '%m' to be thread safe whereas strerror() isn't. I don't think this change is actually good. You'd need to do somethin

[Qemu-devel] [QEMU RFC PATCH 4/7] i386: create apic_id_for_cpu() function

2012-07-10 Thread Eduardo Habkost
Currently we need a way to calculate the Initial APIC ID using only the CPU index, as the CPU hotplug interface doesn't have a method to send the correct APIC ID to the ACPI hotplug code at hotplug-time. Signed-off-by: Eduardo Habkost --- target-i386/cpu.c |2 +- target-i386/cpu.h | 17 +++

[Qemu-devel] [QEMU RFC PATCH 7/7] generate APIC IDs according to CPU topology

2012-07-10 Thread Eduardo Habkost
NOTE: this is unfinished, and will break live-migration between older and newer QEMU versions if applied as-is. We need to make the older machine-types keep the existing behavior when generating APIC IDs. Signed-off-by: Eduardo Habkost --- target-i386/cpu.h |7 +++ 1 file changed, 3 inse

[Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions

2012-07-10 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- target-i386/topology.h | 138 tests/.gitignore |1 + tests/Makefile |7 ++- tests/test-x86-cpuid.c | 108 + 4 files changed, 253 insertions(+), 1 deletio

[Qemu-devel] [QEMU RFC PATCH 5/7] pc: write lapic info (apic IDs) to fw_cfg so seabios can use it

2012-07-10 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- hw/pc.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/hw/pc.c b/hw/pc.c index 3b8e469..dc95fb8 100644 --- a/hw/pc.c +++ b/hw/pc.c @@ -71,6 +71,7 @@ #define FW_CFG_IRQ0_OVERRIDE (FW_CFG_ARCH_LOCAL + 2) #define FW_CFG_E820_TABLE

[Qemu-devel] [Seabios RFC PATCH 1/1] get lapic IDs from fw_cfg

2012-07-10 Thread Eduardo Habkost
This changes: - MADT table generation - SRAT table generation Still missing: - The _MAT method on SSDT Processor entries still return the wrong APIC ID (breaking CPU hotplug) - The MPTable generation code doesn't take the new APIC IDs into account Signed-off-by: Eduardo Habkost --- sr

[Qemu-devel] [RFC 0/7+1] QEMU APIC ID + topology bug + CPU hotplug

2012-07-10 Thread Eduardo Habkost
Hi, This is more a call for discussion than a request for comments in the actual code. Our problem today is: - Lots of QEMU and Seabios code have the assumption that Initial APIC ID == CPU "index" (in other words, that APIC IDs are always contiguous and start at 0); - However, the Initia

[Qemu-devel] [QEMU PATCH 3/7] kvm: set vcpu_id to APIC ID instead of CPU index

2012-07-10 Thread Eduardo Habkost
The CPU ID in KVM is supposed to be the APIC ID, so change the KVM_CREATE_VCPU call to match it. It didn't break anything yet because today the APIC ID is assumed to be == the CPU index, but this won't be true in the future. Signed-off-by: Eduardo Habkost --- kvm-all.c |2 +- 1 file changed,

[Qemu-devel] [QEMU PATCH 2/7] hw/apic.c: rename bit functions to not conflict with bitops.h

2012-07-10 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- hw/apic.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/hw/apic.c b/hw/apic.c index 60552df..d322fe3 100644 --- a/hw/apic.c +++ b/hw/apic.c @@ -50,7 +50,7 @@ static int ffs_bit(uint32_t value) ret

[Qemu-devel] [PATCH 1/7] cpus.h: include cpu-common.h

2012-07-10 Thread Eduardo Habkost
Needed for the definition of fprint_function. Signed-off-by: Eduardo Habkost --- cpus.h |2 ++ 1 file changed, 2 insertions(+) diff --git a/cpus.h b/cpus.h index 81bd817..061ff7f 100644 --- a/cpus.h +++ b/cpus.h @@ -1,6 +1,8 @@ #ifndef QEMU_CPUS_H #define QEMU_CPUS_H +#include "qemu-com

[Qemu-devel] [PATCH 0/5]: rename machine options to use dashes

2012-07-10 Thread Luiz Capitulino
Today, machine options use underscores to separate words (eg. kernel_irqchip), however upcoming QOM conversion wants to use dashes instead. This series converts all machine type options to use dashes. Command-line backwards compatibility is maintained by adding an alias for each changed option.

[Qemu-devel] [PATCH 4/5] machine: rename phandle_start to phandle-start

2012-07-10 Thread Luiz Capitulino
QOM conversion wants option names with dashes (instead of underscores), this commit does the change for the phandle_start machine option. The underscore is still supported through an option alias for backwards compatibility. Signed-off-by: Luiz Capitulino --- device_tree.c | 2 +- qemu-config.c

[Qemu-devel] [PATCH 3/5] machine: rename kvm_shadow_mem to kvm-shadow-mem

2012-07-10 Thread Luiz Capitulino
QOM conversion wants option names with dashes (instead of underscores), this commit does the change for the kvm_shadow_mem machine option. The underscore is still supported through an option alias for backwards compatibility. Signed-off-by: Luiz Capitulino --- qemu-config.c | 3 ++- qemu-op

Re: [Qemu-devel] [PATCH v2 1/3] bitops: fix types

2012-07-10 Thread Blue Swirl
On Tue, Jul 10, 2012 at 7:37 PM, Peter Maydell wrote: > On 10 July 2012 20:18, Blue Swirl wrote: >> On Mon, Jul 9, 2012 at 7:49 AM, Markus Armbruster wrote: >>> There is no consensus. I recognize the power of maintainers to force a >>> change even without consensus. Use it wisely. >> >> I thou

[Qemu-devel] [PATCH v3] console: Implementing blinking of cursor

2012-07-10 Thread Jan Kiszka
From: Jan Kiszka Let the text console cursor blink at 2 HZ. Signed-off-by: Jan Kiszka --- Changes in v3: - renamed cursor_blink_state to cursor_visible_phase and made it a bool console.c | 26 +- 1 files changed, 25 insertions(+), 1 deletions(-) diff --git a/conso

Re: [Qemu-devel] [PATCH 5/9] IPMI: Add a PC ISA type structure

2012-07-10 Thread Corey Minyard
On 07/10/2012 04:33 AM, Paolo Bonzini wrote: Here, you should add a "normal" chardev property so that people can use -chardev and -device to create the IPMI interface. The device can be created like this: -chardev ...,id=charipmi -device ipmi,interface=kcs,chardev=charipmi If the chardev i

[Qemu-devel] [PATCH 5/5] machine: rename dt_compatible to dt-compatible

2012-07-10 Thread Luiz Capitulino
QOM conversion wants option names with dashes (instead of underscores), this commit does the change for the dt_compatible machine option. The underscore is still supported through an option alias for backwards compatibility. Signed-off-by: Luiz Capitulino --- hw/ppce500_mpc8544ds.c | 2 +- qemu

[Qemu-devel] [PATCH 2/5] machine: rename kernel_irqchip to kernel-irqchip

2012-07-10 Thread Luiz Capitulino
QOM conversion wants option names with dashes (instead of underscores), this commit does the change for the kernel_irqchip machine option. The underscore is still supported through an option alias for backwards compatibility. Signed-off-by: Luiz Capitulino --- kvm-all.c | 2 +- qemu-confi

[Qemu-devel] [PATCH 1/5] qemu-option: add alias support

2012-07-10 Thread Luiz Capitulino
It allows for specifying an alias for each option name, see next commits examples. Signed-off-by: Luiz Capitulino --- qemu-option.c | 9 - qemu-option.h | 1 + 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/qemu-option.c b/qemu-option.c index bb3886c..59a1f6e 100644 --- a/

Re: [Qemu-devel] [PATCH v2] console: Implementing blinking of cursor

2012-07-10 Thread Jan Kiszka
On 2012-07-10 21:34, Blue Swirl wrote: > On Mon, Jul 9, 2012 at 2:53 PM, Jan Kiszka wrote: >> Let the text console cursor blink at 2 HZ. >> >> Signed-off-by: Jan Kiszka >> --- >> >> Changes in v2: >> - fixed semantic of CONSOLE_CURSOR_PERIOD and reduced frequency >> >> I know there was a concern

Re: [Qemu-devel] Disk geometry and migration

2012-07-10 Thread Blue Swirl
On Tue, Jul 10, 2012 at 6:34 PM, Markus Armbruster wrote: > Scenario: > > 1. Start a guest with a blank disk (need not be the only disk) and >default disk geometry. > > 2. Examine the disk's physical geometry > >Details depend on the device model. scsi-hd exposes it in mode pages >4 a

Re: [Qemu-devel] [PATCH v2 1/3] bitops: fix types

2012-07-10 Thread Peter Maydell
On 10 July 2012 20:18, Blue Swirl wrote: > On Mon, Jul 9, 2012 at 7:49 AM, Markus Armbruster wrote: >> There is no consensus. I recognize the power of maintainers to force a >> change even without consensus. Use it wisely. > > I thought I refuted all concrete arguments except performance. No,

Re: [Qemu-devel] [PATCH v2] console: Implementing blinking of cursor

2012-07-10 Thread Blue Swirl
On Mon, Jul 9, 2012 at 2:53 PM, Jan Kiszka wrote: > Let the text console cursor blink at 2 HZ. > > Signed-off-by: Jan Kiszka > --- > > Changes in v2: > - fixed semantic of CONSOLE_CURSOR_PERIOD and reduced frequency > > I know there was a concern regarding the approach in general, but I > still

Re: [Qemu-devel] [PATCH v2 3/3] bitops: use bool

2012-07-10 Thread Blue Swirl
On Mon, Jul 9, 2012 at 7:43 AM, Markus Armbruster wrote: > blauwir...@gmail.com writes: > >> From: Blue Swirl >> >> Use 'bool' type for return value of bit test functions. > > Matter of taste. 'bool' makes sense if you think of these functions as > predicates (ugly ones, with side effects). 'in

Re: [Qemu-devel] [PATCH v2 1/3] bitops: fix types

2012-07-10 Thread Blue Swirl
On Mon, Jul 9, 2012 at 7:49 AM, Markus Armbruster wrote: > blauwir...@gmail.com writes: > >> From: Blue Swirl >> >> Use 'unsigned int' for bit numbers instead of 'unsigned long' or >> 'int'. Adjust asserts. > > I'd like to lodge a formal objection to this part. > > There is no consensus. I recog

[Qemu-devel] [PATCH v17 3/9] Add cache handling functions

2012-07-10 Thread Orit Wasserman
Add LRU page cache mechanism. The page are accessed by their address. Signed-off-by: Benoit Hudzia Signed-off-by: Petter Svard Signed-off-by: Aidan Shribman Signed-off-by: Orit Wasserman --- Makefile.objs |1 + cutils.c |9 ++ include/qemu/page_cache.h |

Re: [Qemu-devel] [PATCH 1/5] Avoid GCC extension ?:

2012-07-10 Thread Blue Swirl
On Mon, Jul 9, 2012 at 7:35 AM, Markus Armbruster wrote: > blauwir...@gmail.com writes: > >> From: Blue Swirl >> >> Replace expr1 ?: expr2 with expr1 ? expr1 : expr2 as K&R intended. >> >> If expr1 has side effects, introduce a temporary variable. >> >> Signed-off-by: Blue Swirl >> --- >> block

Re: [Qemu-devel] [libvirt] Problem setting CPU topology

2012-07-10 Thread Eduardo Habkost
On Tue, Jul 10, 2012 at 11:54:05AM +0200, Christophe Fergeau wrote: > On Sat, Jul 07, 2012 at 07:10:53PM +0300, Zeeshan Ali (Khattak) wrote: > > Hi, > >I'm trying to set exact CPU topology to qemu-kvm domains to match > > host's topology. In my case, host topology is: 1 socket, 2 cores and 2 >

Re: [Qemu-devel] [PATCH] usb: selective endpoint initialization

2012-07-10 Thread Erik Rull
Erik Rull wrote: Gerd Hoffmann wrote: Add support for (re-)initializing endpoints which belong to a specific interface only. Use this in usb-host when changing altsetting for an interface, so other interfaces are not disturbed. Hi Gerd, I tested it on my AMD test system where the issue didn

[Qemu-devel] Disk geometry and migration

2012-07-10 Thread Markus Armbruster
Scenario: 1. Start a guest with a blank disk (need not be the only disk) and default disk geometry. 2. Examine the disk's physical geometry Details depend on the device model. scsi-hd exposes it in mode pages 4 and 5. ide-hd in command IDENTIFY, and in its translation from CHS to L

Re: [Qemu-devel] [PATCH v4] sheepdog: do not blindly memset all read buffers

2012-07-10 Thread MORITA Kazutaka
At Tue, 10 Jul 2012 16:12:27 +0200, Christoph Hellwig wrote: > > Only buffers that map to unallocated blocks need to be zeroed. > > Signed-off-by: Christoph Hellwig > > --- > block/sheepdog.c | 37 ++--- > 1 file changed, 18 insertions(+), 19 deletions(-) Ack

Re: [Qemu-devel] [PATCH] Add --disable-bridge-helper configure flag

2012-07-10 Thread Andreas Färber
Am 10.07.2012 12:43, schrieb Fabien Chouteau: > Bridge helper uses ioctl's not available on old Linux versions, we add > this flag to disable the build. > > Signed-off-by: Fabien Chouteau > --- > Makefile |2 +- > configure |8 > 2 files changed, 9 insertions(+), 1 deletion(-)

Re: [Qemu-devel] KVM call minutes July 10th

2012-07-10 Thread Andreas Färber
Hi, Am 10.07.2012 16:55, schrieb Juan Quintela: > > We discussed this - CPU VMState series by Juan Question: Could we assign CPU-specific VMStateDescription to DeviceState::vmsd? > - cpu_index: is that transmited? Yes (Juan). we need to be sure that > cpu_common and cpu use the same index/w

Re: [Qemu-devel] [PATCH v3] Fixes related to processing of qemu's -numa option

2012-07-10 Thread Eduardo Habkost
On Thu, Jul 05, 2012 at 09:49:40AM -0700, Chegu Vinod wrote: > Changes since v2: >- Using "unsigned long *" for the node_cpumask[]. >- Use bitmap_new() instead of g_malloc0() for allocation. >- Don't rely on "max_cpus" since it may not be initialized > before the numa related qemu

Re: [Qemu-devel] [PATCH 0/2] RFC: powerpc-vfio: adding support

2012-07-10 Thread Alex Williamson
On Tue, 2012-07-10 at 15:51 +1000, Alexey Kardashevskiy wrote: > The two patches in this set are supposed to add VFIO support for POWER. > > The first one adds one more step in the initalizaion sequence which I am not > sure is correct. > > The second patch adds actual VFIO support. It is not rea

Re: [Qemu-devel] [PATCH 2/2] vfio-powerpc: added VFIO support

2012-07-10 Thread Alex Williamson
On Tue, 2012-07-10 at 15:51 +1000, Alexey Kardashevskiy wrote: > The patch enables VFIO on POWER. > > It literally does the following: > > 1. POWERPC IOMMU support (the kernel counterpart is required) > > 2. Added #ifdef TARGET_PPC64 for EOI handlers initialisation. > > 3. Added vfio_get_contai

[Qemu-devel] [PATCH 1/1] s390x: fix s390 virtio aliases

2012-07-10 Thread Alexander Graf
Some of the virtio devices have the same frontend name, but actually implement different devices behind the scenes through aliases. The indicator which device type to use is the architecture. On s390, we want s390 virtio devices. On everything else, we want PCI devices. Reflect this in the alias

[Qemu-devel] [PULL 0/1] s390 patch queue 2012-07-10

2012-07-10 Thread Alexander Graf
Hi Mike, This is my current patch queue for s390 for 1.1. It contains an important fix to make virtio devices accessible via their aliases. Please pull. Alex The following changes since commit b7093f294c330c4db789c077dac9d8611e4f8ee0: Jason Wang (1): rtl8139: validate rx ring before

[Qemu-devel] [PATCH 1/1] s390: autodetect map private

2012-07-10 Thread Alexander Graf
From: Christian Borntraeger By default qemu will use MAP_PRIVATE for guest pages. This will write protect pages and thus break on s390 systems that dont support this feature. Therefore qemu has a hack to always use MAP_SHARED for s390. But MAP_SHARED has other problems (no dirty pages tracking, a

[Qemu-devel] [PULL 0/1] s390 patch queue 2012-07-10

2012-07-10 Thread Alexander Graf
Hi Blue / Aurelien, This is my current patch queue for s390. It only includes a single patch that enables for better memory allocation with kvm on newer kernels. Please pull. Alex The following changes since commit 92336855975805d88c7979f53bc05c2d47abab04: Anthony Liguori (1): megasa

Re: [Qemu-devel] [PATCH 4/9] Add a base IPMI interface

2012-07-10 Thread Corey Minyard
On 07/10/2012 04:17 AM, Daniel P. Berrange wrote: On Mon, Jul 09, 2012 at 02:17:04PM -0500, miny...@acm.org wrote: diff --git a/qemu-options.hx b/qemu-options.hx index 125a4da..823f6bc 100644 --- a/qemu-options.hx +++ b/qemu-options.hx @@ -2204,6 +2204,41 @@ Three button serial mouse. Configure

Re: [Qemu-devel] [PATCH 1/2] linux-user: Factor out guest space probing into a function

2012-07-10 Thread Peter Maydell
On 10 July 2012 16:57, Meador Inge wrote: > Signed-off-by: Meador Inge > --- > linux-user/elfload.c | 111 > +++--- > linux-user/qemu.h| 11 + > 2 files changed, 89 insertions(+), 33 deletions(-) > > diff --git a/linux-user/elfload.c b/linu

[Qemu-devel] [PATCH] ehci: raise irq in the frame timer

2012-07-10 Thread Gerd Hoffmann
With the async schedule being kicked from other places than the frame timer (commit 0f588df8b3688b00e77aabaa32e26ece5f19bd39) it may happen that we call ehci_commit_interrupt() more than once per frame. Move the call from the async schedule handler to the frame timer to restore old irq behavior, w

[Qemu-devel] [Bug 680350] Re: fail to compile qemu-kvm-0.13.0

2012-07-10 Thread Peter Maydell
This is a compilation error for a really old version of qemu-kvm; it has never been a bug in QEMU master (see commit 7c80eef). So I'm closing it as invalid. ** Changed in: qemu Status: Incomplete => Invalid -- You received this bug notification because you are a member of qemu- devel-ml,

[Qemu-devel] [PATCH 2/2] linux-user: Use init_guest_space when -R and -B are specified

2012-07-10 Thread Meador Inge
Modify the driver to initialize the guest address space when -R or -B is specified so that the reserved memory space can be probed. Calling 'mmap' just once as is currently done is not guaranteed to succeed since the host address space validation might fail. Signed-off-by: Meador Inge --- linux

[Qemu-devel] [PATCH 1/2] linux-user: Factor out guest space probing into a function

2012-07-10 Thread Meador Inge
Signed-off-by: Meador Inge --- linux-user/elfload.c | 111 +++--- linux-user/qemu.h| 11 + 2 files changed, 89 insertions(+), 33 deletions(-) diff --git a/linux-user/elfload.c b/linux-user/elfload.c index f3b1552..44b4bdb 100644 --- a/linux-

[Qemu-devel] [PATCH 0/2] Probe the guest memory space when using -R

2012-07-10 Thread Meador Inge
Hi, This patch series fixes an issue that was discussed here [1] where -R can fail when the mapped address space fails validation. I fixed this issue by (1) refactoring the guest space probing code into a single function for initialing the guest space and (2) by calling the guest space initializi

Re: [Qemu-devel] [PATCH 0/4 v2] target-i386: move tcg intialization inside CPU object

2012-07-10 Thread Igor Mammedov
ping On 06/25/2012 03:55 PM, Igor Mammedov wrote: v2: - drop usage of prev_debug_excp_handler consistently in all users - split from reset patches to avoid confusion of inter-dependency Compile & Run tested: target-i386: tcg and kvm mode i386-linux-user: running of /bin/ls Compile t

Re: [Qemu-devel] First shot at adding IPMI to qemu

2012-07-10 Thread Corey Minyard
On 07/10/2012 04:35 AM, Paolo Bonzini wrote: Il 09/07/2012 21:17, miny...@acm.org ha scritto: I had asked about getting an IPMI device into qemu and received some interest, and it's useful to me, so I've done some work to add it. The following patch set has a set of patches to add an IPMI KCS de

[Qemu-devel] [Bug 571432] Re: qemu-system-arm crashed with SIGSEGV in subpage_register()

2012-07-10 Thread Peter Maydell
This bug doesn't seem to have any reproduction instructions, and at least some of the command lines seem to be asking for 512MB of RAM on a versatilepb model, which isn't supported and will crash the guest unhelpfully. I'm marking it 'incomplete' for QEMU... ** Changed in: qemu Status: New

[Qemu-devel] [Bug 570574] Re: GDBStub Bug: Can't stop at start_kernel when debugging linux kernel

2012-07-10 Thread Peter Maydell
This works for me with QEMU 1.1 and GDB 7.4: gdb successfully stops at the start_kernel breakpoint. So I'm going to close it as 'fix released' on the assumption that we fixed the bug at some point between QEMU 0.12 and 1.1... ** Changed in: qemu Status: New => Fix Released -- You receive

[Qemu-devel] [Bug 944753] Re: ARM: CORTEX M, PRIMASK and FAULTMASK are misplaced, WINDOWS mingw 4.6.1 bug

2012-07-10 Thread Peter Maydell
This isn't a QEMU bug, and gcc 4.6.1 has fallen off the list of versions gcc upstream still maintains, so I'm afraid I'm closing it as invalid. I suggest you upgrade to a newer version of gcc... ** Changed in: qemu Status: New => Invalid -- You received this bug notification because you

[Qemu-devel] [Bug 1011142] Re: Arm emulation do a crash

2012-07-10 Thread Peter Maydell
This appears to be user confusion. ** Changed in: qemu Status: New => Invalid -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1011142 Title: Arm emulation do a crash Status in QEMU: Invali

[Qemu-devel] [Bug 960515] Re: CortexM, NVIC not reseted if qemu_system_reset_request() called

2012-07-10 Thread Peter Maydell
I added a reset function for the NVIC/systick as part of the GIC refactoring work I did recently. These changes should be in QEMU 1.1. ** Changed in: qemu Status: New => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to

[Qemu-devel] [Bug 929638] Re: qemu 1.0 unable to compile on the pandaboard ES

2012-07-10 Thread Peter Maydell
In QEMU 1.1 building on ARM hosts is supported again -- please can you retry with a clean 1.1 source tree? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.n

[Qemu-devel] [Bug 883136] Re: qemu on ARM hosts aborts on startup because makecontext() always fails

2012-07-10 Thread Peter Maydell
Fix released in upstream QEMU 1.1. ** Changed in: qemu Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/883136 Title: qemu on ARM hosts aborts on startup

[Qemu-devel] [PATCH 0/3] powerpc: adding MSI/MSIX support

2012-07-10 Thread Alexey Kardashevskiy
While some of my colleagues are overloaded or in vacations, I will give it another try here :) The patch I posted earlier is split to three to make it easier to review/comment. Each patch compiles but does not make sense without others. Alexey Kardashevskiy (3): pseries pci: preparing for MS

[Qemu-devel] [PATCH 3/8] scsi-disk: split scsi-disk reqops

2012-07-10 Thread Paolo Bonzini
Only checks for present medium were still done in scsi_send_command for emulated commands. So move those to scsi_disk_emulate_command and return different SCSIReqOps depending on the kind of command. Checks for present medium can be done unconditionally for the scsi_disk_dma_reqops case. Signed-

[Qemu-devel] [Bug 870990] Re: compile failure on ARMv7 hosts when compiled for thumb if --enable-debug

2012-07-10 Thread Peter Maydell
** Changed in: qemu Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/870990 Title: compile failure on ARMv7 hosts when compiled for thumb if --enable- de

[Qemu-devel] [Bug 556560] Re: -S doesn't freeze CPU as expected

2012-07-10 Thread Peter Maydell
-s -S works for me, so I'm going to assume this bug has been fixed at some point in the last 2 years. Reopen if it still persists in QEMU 1.1. ** Changed in: qemu Status: Incomplete => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is

[Qemu-devel] [Bug 883133] Re: qemu on ARM hosts asserts due to code buffer/libc heap conflict

2012-07-10 Thread Peter Maydell
** Changed in: qemu Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/883133 Title: qemu on ARM hosts asserts due to code buffer/libc heap conflict Status

[Qemu-devel] KVM call minutes July 10th

2012-07-10 Thread Juan Quintela
Hi We discussed this - cpu_index: is that transmited? Yes (Juan). we need to be sure that cpu_common and cpu use the same index/whatever it is changed - We used cpu_index for SEABIOS interface as apic_id (Eduardo) - cpu_hotplug makes that difficult. How to pass the apic_id to seabios wi

[Qemu-devel] [Bug 607794] Re: SEGFAULT caused by modifying the CPSR register

2012-07-10 Thread Peter Maydell
The patches that fix this bug are in QEMU 1.1. ** Changed in: qemu Status: New => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/607794 Title: SEGFAULT caused by modifying the

[Qemu-devel] [Bug 1015226] Re: arm realview pbx hung

2012-07-10 Thread Peter Maydell
I'm afraid you'll need to at least provide a copy of your kernel and initrd if you want anybody to try to debug this. ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.

[Qemu-devel] [PATCH 8/8] scsi-disk: support toggling the write cache

2012-07-10 Thread Paolo Bonzini
Finally, this uses the "plumbing" in the previous patch to add support for toggling the WCE bit of the caching mode page. Signed-off-by: Paolo Bonzini --- hw/scsi-disk.c |8 1 file changed, 8 insertions(+) diff --git a/hw/scsi-disk.c b/hw/scsi-disk.c index cd0c389..d4d3ae6 100644 -

[Qemu-devel] [PATCH 1/8] scsi-disk: make discard asynchronous

2012-07-10 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 --- Not really related to this series, but it is yet another patch I had around and it had a lot of conflicts with the ones that follow, s

[Qemu-devel] [Bug 982321] Re: User mode arm qemu segfault

2012-07-10 Thread Peter Maydell
** Changed in: qemu Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/982321 Title: User mode arm qemu segfault Status in QEMU: Fix Released Bug descrip

[Qemu-devel] [PATCH 6/8] scsi-disk: fix changeable values for MODE_PAGE_R_W_ERROR

2012-07-10 Thread Paolo Bonzini
The changeable values were not all-zeros for this mode page, fix it. Signed-off-by: Paolo Bonzini --- hw/scsi-disk.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/hw/scsi-disk.c b/hw/scsi-disk.c index e64d659..a7f1f14 100644 --- a/hw/scsi-disk.c +++ b/hw/scsi-disk.c @@ -1058,6 +1058,

[Qemu-devel] [Bug 597641] Re: SD card state change from ident to stby

2012-07-10 Thread Peter Maydell
** Changed in: qemu Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/597641 Title: SD card state change from ident to stby Status in QEMU: Fix Released

[Qemu-devel] tracing guest memory access

2012-07-10 Thread Ilia K.
Hi, I need to trace all data loads/stores of a guest OS running under qemu (for now, both guest and host are x86-64, and virtual addresses should be sufficient). Looking into qemu-devel archive I see that this subject is brought up not very frequently, but regularly. I know, there is a general trac

[Qemu-devel] [PATCH] pseries: added allocator for a block of IRQs

2012-07-10 Thread Alexey Kardashevskiy
The patch adds a simple helper which allocates a consecutive sequence of IRQs calling spapr_allocate_irq for each and checks that allocated IRQs go consequently. The patch is required for upcoming support of MSI/MSIX on POWER. Signed-off-by: Alexey Kardashevskiy --- hw/spapr.c | 20 ++

[Qemu-devel] [Bug 847638] Re: screendump broken for text mode

2012-07-10 Thread Cole Robinson
Should have been fixed by http://git.qemu.org/?p=qemu.git;a=commit;h=2313e998d7bf55d02499ee162fb4558af4fbb3f5 soon after this report ** Changed in: qemu Status: New => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEM

Re: [Qemu-devel] [Bug 1018530] Re: No write access in a 9p/virtfs shared folder

2012-07-10 Thread Serge Hallyn
Quoting M. Mohan Kumar (1018...@bugs.launchpad.net): > No, commit daf0b9aca9f67323266af1a92e8ea06f9d7bf408 added create > support proxy FS driver model. Local FS had support for creating files > much before. Yes, but that commit is not in v1.0 (according to qemu-kvm git history at least) -- You

[Qemu-devel] [PATCH 7/8] scsi-disk: parse MODE SELECT commands and parameters

2012-07-10 Thread Paolo Bonzini
This adds the bulk of the parsing code for MODE SELECT, including breaking out changes to different mode pages, and checking that only changeable values are modified. In order to report errors correctly two passes are made through the parameters; the first only looks for errors, the second actuall

Re: [Qemu-devel] [PATCH] qemu_find_file: check name as a straight path even if it has no '/'

2012-07-10 Thread Peter Maydell
Ping^3? This patch has been on the list over six weeks now... -- PMM On 28 June 2012 12:23, Peter Maydell wrote: > Ping^2? > > -- PMM > > On 19 June 2012 16:13, Peter Maydell wrote: >> Ping? (patch still applies cleanly to current master; patchwork URL >> http://patchwork.ozlabs.org/patch/16132

  1   2   3   >