Re: [Qemu-devel] [Qemu-ppc] [PATCH v2] spapr: populate ibm, loc-code

2015-03-30 Thread Alexey Kardashevskiy
On 03/30/2015 04:34 PM, Nikunj A Dadhania wrote: Nikunj A Dadhania writes: David Gibson writes: On Mon, Mar 30, 2015 at 01:18:01PM +1100, Alexey Kardashevskiy wrote: On 03/27/2015 08:49 PM, Nikunj A Dadhania wrote: Each hardware instance has a platform unique location code. The OF device

Re: [Qemu-devel] [PATCH 1/6] Split serial-isa into its own config option

2015-03-30 Thread Markus Armbruster
David Gibson writes: > At present, the core device model code for 8250-like serial ports > (serial.c) and the code for serial ports attached to ISA-style legacy IO > (serial-isa.c) are both controlled by the CONFIG_ISA variable. > > There are lots and lots of embedded platforms that have 8250-lik

Re: [Qemu-devel] remaining patches for rc2?

2015-03-30 Thread Cornelia Huck
On Fri, 27 Mar 2015 16:42:18 + Peter Maydell wrote: > This is the set of things I have on the list for "should ideally > get into rc2": > * AArch64 migration fixes > * VNC patches to fix CVEs > * recent exec.c changes broke ppc OSX/OpenBIOS boot > > Anything else? > > thanks > -- PMM >

Re: [Qemu-devel] [PATCH 2/6] Remove monitor.c dependency on CONFIG_I8259

2015-03-30 Thread Markus Armbruster
David Gibson writes: > The hmp commands "irq" and "pic" are a bit of a mess. They're implemented > on a number of targets, but not all. On sparc32 and LM32 they do target > specific things, but on the remainder (i386, ppc and mips) they call into > the i8259 PIC code. Where "info pic" does abs

Re: [Qemu-devel] [PATCH v5 0/7] QEMU memory hot unplug support

2015-03-30 Thread Paolo Bonzini
On 30/03/2015 04:53, Zhu Guihua wrote: > Question: > > When migrating from old qemu to one with this hot remove feature, I found > this feature would not work any more in new qemu. > > The reason is that DSDT table will not re-generate in new qemu when > migration. > So the hot remove will not

Re: [Qemu-devel] [PATCH for-2.3?] qom: Fix object_property_add_alias() with [*]

2015-03-30 Thread Paolo Bonzini
On 29/03/2015 19:19, Andreas Färber wrote: > Commit 8074264 (qom: Add description field in ObjectProperty struct) > introduced property descriptions and copied them for alias properties. > > Instead of using the caller-supplied property name, use the returned > property name for setting the desc

Re: [Qemu-devel] [Migration Bug? ] Occasionally, the content of VM's memory is inconsistent between Source and Destination of migration

2015-03-30 Thread Dr. David Alan Gilbert
* zhanghailiang (zhang.zhanghaili...@huawei.com) wrote: > On 2015/3/27 18:18, Dr. David Alan Gilbert wrote: > >* zhanghailiang (zhang.zhanghaili...@huawei.com) wrote: > >>On 2015/3/26 11:52, Li Zhijian wrote: > >>>On 03/26/2015 11:12 AM, Wen Congyang wrote: > On 03/25/2015 05:50 PM, Juan Quinte

[Qemu-devel] [PULL for-2.3 0/4] s390x bugfixes for 2.3

2015-03-30 Thread Cornelia Huck
The following changes since commit 627f91b1f80fecc73d00727181a9ddb6162cc30e: Merge remote-tracking branch 'remotes/jnsnow/tags/ide-pull-request' into staging (2015-03-28 10:10:04 +) are available in the git repository at: git://github.com/cohuck/qemu tags/s390x-20150330

[Qemu-devel] [PULL for-2.3 4/4] s390x/ipl: avoid sign extension

2015-03-30 Thread Cornelia Huck
Make s390_update_iplstate() return uint32_t to avoid sign extensions for cssids > 127. While this doesn't matter in practice yet (as nobody supports MCSS-E and thus won't see the real cssid), play safe. Reported-by: Paolo Bonzini Reviewed-by: Jason J. Herne Signed-off-by: Cornelia Huck --- hw/

[Qemu-devel] [PULL for-2.3 3/4] s390x: do not include ram_addr.h

2015-03-30 Thread Cornelia Huck
From: Paolo Bonzini ram_addr.h is an internal interface and it is not needed anyway by hw/s390x/ipl.c. Cc: Christian Borntraeger Reviewed-by: Thomas Huth Signed-off-by: Paolo Bonzini Message-Id: <1427295389-5054-1-git-send-email-pbonz...@redhat.com> Signed-off-by: Christian Borntraeger Signe

[Qemu-devel] [PULL for-2.3 1/4] virtio-ccw: fix range check for SET_VQ

2015-03-30 Thread Cornelia Huck
VIRTIO_PCI_QUEUE_MAX is already too big; a malicious guest would be able to trigger a write beyond the VirtQueue structure. Cc: qemu-sta...@nongnu.org Reviewed-by: David Hildenbrand Acked-by: Christian Borntraeger Signed-off-by: Cornelia Huck --- hw/s390x/virtio-ccw.c | 2 +- 1 file changed, 1

[Qemu-devel] [PULL for-2.3 2/4] virtio-ccw: range check in READ_VQ_CONF

2015-03-30 Thread Cornelia Huck
Processing for READ_VQ_CONF needs to check whether the requested queue value is actually in the supported range and post a channel program check if not. Cc: qemu-sta...@nongnu.org Reviewed-by: David Hildenbrand Acked-by: Christian Borntraeger Signed-off-by: Cornelia Huck --- hw/s390x/virtio-cc

Re: [Qemu-devel] [PATCH v5 20/45] Modify savevm handlers for postcopy

2015-03-30 Thread Paolo Bonzini
On 24/03/2015 23:32, David Gibson wrote: >>> OK, that one I prefer. Are you OK with: >>> qemu_savevm_state_complete_precopy calls -> >>> save_live_complete_precopy >>> >>> qemu_savevm_state_complete_postcopy calls -> >>> save_live_complete_postcopy >>> >>> ? > Sounds ok to me. Fwiw, I was th

Re: [Qemu-devel] [PATCH v5 22/45] postcopy: OS support test

2015-03-30 Thread Paolo Bonzini
On 13/03/2015 11:41, Dr. David Alan Gilbert wrote: >>> > > +#ifdef HOST_X86_64 >>> > > +#ifndef __NR_userfaultfd >>> > > +#define __NR_userfaultfd 323 >> > >> > Sholdn't this come from the kernel headers imported in the previous >> > patch? Rather than having an arch-specific hack. > The header

Re: [Qemu-devel] [PATCH v5 23/45] migrate_start_postcopy: Command to trigger transition to postcopy

2015-03-30 Thread Paolo Bonzini
On 19/03/2015 10:33, Dr. David Alan Gilbert wrote: >> > But still pointless. Atomicity isn't magic pixie dust; it only makes >> > sense if you're making atomic specific operations that need to be. >> > Simple integer loads and stores are already atomic. Unless at least >> > some of the atomic o

Re: [Qemu-devel] [PATCH v5 23/45] migrate_start_postcopy: Command to trigger transition to postcopy

2015-03-30 Thread Paolo Bonzini
On 23/03/2015 03:20, David Gibson wrote: >>> 1) There's no barrier after the write, so there's no guarantee >>> the other thread will eventually see it (in practice we've got >>> other pthread ops we take so we will get a barrier somewhere, >>> and most CPUs eventually do propagate the store). >

Re: [Qemu-devel] [Qemu-ppc] [PATCH v2] spapr: populate ibm, loc-code

2015-03-30 Thread Nikunj A Dadhania
Alexey Kardashevskiy writes: > On 03/30/2015 04:34 PM, Nikunj A Dadhania wrote: >> Nikunj A Dadhania writes: >> >>> David Gibson writes: >>> On Mon, Mar 30, 2015 at 01:18:01PM +1100, Alexey Kardashevskiy wrote: > On 03/27/2015 08:49 PM, Nikunj A Dadhania wrote: >> Each hardware ins

[Qemu-devel] Help for backport virtio to older kernel 2.6.18

2015-03-30 Thread Halsey Pian
Hi All, It seems that virtio disk driver is included in kernel version started from 2.6.32, but for 2.6.18 (using redhat5 as VM), we have to backport the virtio driver. And I found that there is a link in link below, but cannot be opened. Could you provide help on it? Thanks! http://ww

Re: [Qemu-devel] [PATCH v5 30/45] Postcopy: Postcopy startup in migration thread

2015-03-30 Thread Paolo Bonzini
On 26/03/2015 12:05, Dr. David Alan Gilbert wrote: >>> > > +qemu_system_wakeup_request(QEMU_WAKEUP_REASON_OTHER); >>> > > +*old_vm_running = runstate_is_running(); >> > >> > I think that needs some explanation. Why are you doing a wakeup on >> > the source host? > This matches the exist

[Qemu-devel] [PATCH v6 1/2] qom: Add can_be_deleted callback to UserCreatableClass

2015-03-30 Thread Lin Ma
If backends implement the can_be_deleted and it returns false, Then the qmp_object_del won't delete the given backends. Signed-off-by: Lin Ma --- include/qom/object_interfaces.h | 13 + qmp.c | 5 + qom/object_interfaces.c | 12 3 f

[Qemu-devel] [PATCH v6 0/2] Add generic can_be_deleted to UserCreatableClass.

2015-03-30 Thread Lin Ma
The patchset adds a generic can_be_deleted callback to UserCreatableClass. It prevents removing a usercreatable object if the callback returns false. Backends could implement the callback if it shoudn't be removed while it's in use. Thank Peter Crosthwaite, Paolo Bonzini, Andreas Färber and Igor

Re: [Qemu-devel] [PATCH 2/6] Remove monitor.c dependency on CONFIG_I8259

2015-03-30 Thread Markus Armbruster
Markus Armbruster writes: > David Gibson writes: > >> The hmp commands "irq" and "pic" are a bit of a mess. They're implemented >> on a number of targets, but not all. On sparc32 and LM32 they do target >> specific things, but on the remainder (i386, ppc and mips) they call into >> the i8259 P

[Qemu-devel] [PATCH v6 2/2] hostmem: Prevent removing an in-use memory backend

2015-03-30 Thread Lin Ma
showing a memory device whose memdev is removed leads an assert: (qemu) object_add memory-backend-ram,id=ram0,size=128M (qemu) device_add pc-dimm,id=d0,memdev=ram0 (qemu) object_del ram0 (qemu) info memory-devices ** ERROR:qom/object.c:1274:object_get_canonical_path_component:\

Re: [Qemu-devel] [PATCH 0/6] Clean up ISA dependencies so we make ISA optional to build

2015-03-30 Thread Markus Armbruster
David Gibson writes: > On Tue, Mar 10, 2015 at 10:56:22AM -0400, Luiz Capitulino wrote: >> On Tue, 10 Mar 2015 15:20:29 +0100 >> "Michael S. Tsirkin" wrote: >> >> > On Fri, Mar 06, 2015 at 03:18:20PM +1100, David Gibson wrote: >> > > At present, ISA bus support is always included in the build f

Re: [Qemu-devel] [PATCH v6 0/2] Add generic can_be_deleted to UserCreatableClass.

2015-03-30 Thread Paolo Bonzini
On 30/03/2015 10:36, Lin Ma wrote: > The patchset adds a generic can_be_deleted callback to UserCreatableClass. > It prevents removing a usercreatable object if the callback returns false. > > Backends could implement the callback if it shoudn't be removed while it's > in use. > > Thank Peter C

Re: [Qemu-devel] [PATCH] vhost: logs sharing

2015-03-30 Thread Michael S. Tsirkin
On Fri, Mar 20, 2015 at 04:53:24PM +0800, Jason Wang wrote: > Currently we allocate one vhost log per vhost device. This is sub > optimal when: > > - Guest has several device with vhost as backend > - Guest has multiqueue devices > > In the above cases, we can avoid the memory allocation by shari

Re: [Qemu-devel] [PATCH v5 6/7] acpi: add hardware implementation for memory hot unplug

2015-03-30 Thread Igor Mammedov
On Fri, 27 Mar 2015 17:20:39 +0800 Zhu Guihua wrote: > - implements QEMU hardware part of memory hot unplug protocol > described at "docs/spec/acpi_mem_hotplug.txt" > - handles memory remove notification event > - handles device eject notification > > Signed-off-by: Zhu Guihua > --- > docs/s

Re: [Qemu-devel] [RFC PATCH v2 22/23] spapr: Support ibm, dynamic-reconfiguration-memory

2015-03-30 Thread Bharata B Rao
On Thu, Mar 26, 2015 at 02:44:17PM +1100, David Gibson wrote: > > +/* > > + * TODO: Take care of sparsemem configuration ? > > + */ > > +static uint64_t numa_node_end(uint32_t nodeid) > > +{ > > +uint32_t i = 0; > > +uint64_t addr = 0; > > + > > +do { > > +addr += numa_info[i].n

Re: [Qemu-devel] [v3][PATCH 2/2] libxl: introduce gfx_passthru_kind

2015-03-30 Thread Ian Campbell
On Mon, 2015-03-30 at 09:28 +0800, Chen, Tiejun wrote: > Sounds it should be a legacy fix to qemu-xen-tranditional :) So lets do > it now, > > @@ -326,6 +326,10 @@ static char ** > libxl__build_device_model_args_old(libxl__gc *gc, > } > if (libxl_defbool_val(b_info->u.hvm.gfx

[Qemu-devel] [PATCH 3/3] MAINTAINERS: Add Fam Zheng as Null block driver maintainer

2015-03-30 Thread Fam Zheng
Signed-off-by: Fam Zheng --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index d7e9ba2..30073a3 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1172,6 +1172,12 @@ S: Supported F: block/gluster.c T: git git://github.com/codyprime/qemu-kvm-jtc.gi

[Qemu-devel] [PATCH 2/3] block/null: Support reopen

2015-03-30 Thread Fam Zheng
Reopen is used in block-commit. With this always-succeed operation, it is now possible to test committing to a null drive, by specifying "null-aio://" or "null-co://" as the backing image when creating the qcow2 image. Signed-off-by: Fam Zheng --- block/null.c | 8 1 file changed, 8 ins

[Qemu-devel] [PATCH 0/3] null driver patches

2015-03-30 Thread Fam Zheng
The first patch was the same one from 03/18 which is reviewed by Eric. The second patch allows testing commit to a null image. The third adds a stanza to MAINTAINERS. Fam Zheng (3): block/null: Latency simulation by adding new option "latency-ns" block/null: Support reopen MAINTAINERS: Add

[Qemu-devel] [PATCH 1/3] block/null: Latency simulation by adding new option "latency-ns"

2015-03-30 Thread Fam Zheng
Aio context switch should just work because the requests will be drained, so the scheduled timer(s) on the old context will be freed. Signed-off-by: Fam Zheng Reviewed-by: Eric Blake --- block/null.c | 59 ++-- qapi/block-core.json | 5 ++

Re: [Qemu-devel] [PATCH v3 1/2] VFIO: Clear stale MSIx table during EEH reset

2015-03-30 Thread Gavin Shan
On Fri, Mar 27, 2015 at 05:00:25PM +1100, David Gibson wrote: >On Thu, Mar 26, 2015 at 04:35:01PM +1100, Gavin Shan wrote: >> The PCI device MSIx table is cleaned out in hardware after EEH PE >> reset. However, we still hold the stale MSIx entries in QEMU, which >> should be cleared accordingly. Ot

Re: [Qemu-devel] [PATCH v3 1/2] VFIO: Clear stale MSIx table during EEH reset

2015-03-30 Thread Gavin Shan
On Mon, Mar 30, 2015 at 01:39:16PM +1100, David Gibson wrote: >On Thu, Mar 26, 2015 at 04:35:01PM +1100, Gavin Shan wrote: >> The PCI device MSIx table is cleaned out in hardware after EEH PE >> reset. However, we still hold the stale MSIx entries in QEMU, which >> should be cleared accordingly. Ot

Re: [Qemu-devel] [PATCH 2/2] sPAPR: Reenable EEH functionality on reboot

2015-03-30 Thread Gavin Shan
On Mon, Mar 30, 2015 at 01:40:04PM +1100, David Gibson wrote: >On Thu, Mar 26, 2015 at 04:35:02PM +1100, Gavin Shan wrote: >> When rebooting the guest, some PEs might be in frozen state. The >> contained PCI devices won't work properly if their frozen states >> aren't cleared in time. One case runn

Re: [Qemu-devel] MemoryRegionOps' {min,max}_access_size fields

2015-03-30 Thread Paolo Bonzini
On 27/03/2015 17:21, Jan Beulich wrote: > Hello, > > I'm trying to understand what the idea behind these is. For one, > without .valid.accepts set I can't see the two respective .valid > fields take effect at all. Yet all examples I looked at don't set > .valid.accepts. What's the deal here? Th

Re: [Qemu-devel] ppc vga output breakage since commit c3c1bb99

2015-03-30 Thread Paolo Bonzini
On 28/03/2015 20:19, Mark Cave-Ayland wrote: > On 28/03/15 19:04, BALATON Zoltan wrote: > >> Hello, >> >> Commit c3c1bb99d1c11978d9ce94d1bdcf0705378c1459 (exec: Respect >> as_tranlsate_internal length clamp) seems to break vga output with >> qemu-system-ppc on x86_64 host. Since this commit, the

Re: [Qemu-devel] [PATCH] tcg: optimise memory layout of TCGTemp

2015-03-30 Thread Laurent Desnogues
Hello, On Fri, Mar 27, 2015 at 10:09 PM, Emilio G. Cota wrote: > On Fri, Mar 27, 2015 at 09:55:03 +, Alex Bennée wrote: >> Have you been able to measure any performance improvement with these new >> structures? In theory, if aligned with cache lines, performance should >> improve but real num

Re: [Qemu-devel] [PATCH v2 1/2] cpu/apic: drop icc bus/bridge/

2015-03-30 Thread Chen Fan
On 03/23/2015 05:43 PM, Igor Mammedov wrote: On Mon, 23 Mar 2015 17:07:29 +0800 Chen Fan wrote: On 03/23/2015 04:23 PM, Igor Mammedov wrote: On Mon, 23 Mar 2015 13:54:23 +0800 Chen Fan wrote: ICC bus was invented only to provide hotplug capability to CPU and APIC because at the time being

Re: [Qemu-devel] ppc vga output breakage since commit c3c1bb99

2015-03-30 Thread Mark Cave-Ayland
On 30/03/15 10:51, Paolo Bonzini wrote: > On 28/03/2015 20:19, Mark Cave-Ayland wrote: >> On 28/03/15 19:04, BALATON Zoltan wrote: >> >>> Hello, >>> >>> Commit c3c1bb99d1c11978d9ce94d1bdcf0705378c1459 (exec: Respect >>> as_tranlsate_internal length clamp) seems to break vga output with >>> qemu-sy

Re: [Qemu-devel] ppc vga output breakage since commit c3c1bb99

2015-03-30 Thread Paolo Bonzini
On 30/03/2015 12:20, Mark Cave-Ayland wrote: >> > >> > Of course, there's a QEMU regression too and I'm thinking of how to fix it. > Hmmm that's interesting because the documentation refers to both > registers being 16-bit: http://wiki.osdev.org/Bochs_VBE_Extensions. And > indeed the pseudo-code

[Qemu-devel] [PATCH v3] spapr: populate ibm,loc-code

2015-03-30 Thread Nikunj A Dadhania
Each hardware instance has a platform unique location code. The OF device tree that describes a part of a hardware entity must include the “ibm,loc-code” property with a value that represents the location code for that hardware entity. Introduce an rtas call to populate ibm,loc-code. 1) PCI passt

[Qemu-devel] [Bug 1438144] [NEW] Page sizes are not interpreted correctly for E500/E500MC

2015-03-30 Thread WGH
Public bug reported: http://cache.freescale.com/files/32bit/doc/ref_manual/E500CORERM.pdf - see 2.12.5.2 MAS Register 1 (MAS1), p. 2-41 http://cache.freescale.com/files/32bit/doc/ref_manual/E500MCRM.pdf - see 2.16.6.2 MAS Register 1 (MAS1), p. 2-54 According to these documents, variable page si

[Qemu-devel] [RFC PATCH v2] acpi: add reset register to fadt

2015-03-30 Thread Reza Jelveh
Some operating systems such as FreeBSD and Mac OSX need the reset_register section of the FADT filled to know which port to write to for a system reset. Signed-off-by: Reza Jelveh --- hw/i386/acpi-build.c | 9 + hw/i386/acpi-defs.h | 2 ++ 2 files changed, 11 insertions(+) diff --git a

Re: [Qemu-devel] arm: semihosting: Preliminary AArch64 support

2015-03-30 Thread Leon Alrae
On 27/03/2015 17:33, Liviu Ionescu wrote: > >> On 27 Mar 2015, at 19:15, Leon Alrae wrote: >> >> ... introducing separate "-semihosting-arg" option to pass input arguments > > if we'll ever go for this solution, I would call it "-semihosting-cmdline", > since it should include the entire comman

[Qemu-devel] [PATCH for-2.3 3/6] ide: there is only one data port

2015-03-30 Thread Paolo Bonzini
IDE PIO data must be written, for example, at 0x1f0. You cannot do word or dword writes to 0x1f1..0x1f3 to access the data register. Adjust the ide_portio_list accordingly. Cc: John Snow Signed-off-by: Paolo Bonzini --- hw/ide/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) d

[Qemu-devel] [PATCH for-2.3 2/6] gus: clean up MemoryRegionPortio

2015-03-30 Thread Paolo Bonzini
Remove 16-bit reads/writes, since ioport.c is able to synthesize them. Remove the two MIDI registers (0x300 and 0x301) from gus_portio_list1, and add the second MIDI register (0x301) to gus_portio_list2. Tested with 2nd Reality. Signed-off-by: Paolo Bonzini --- hw/audio/gus.c | 22 ++---

[Qemu-devel] [PATCH for-2.3 0/3] ioport fixes

2015-03-30 Thread Paolo Bonzini
These fix the problem with OpenBIOS's accesses to the VBE DISPI ports. The problem stems from weird memory regions created by ioport.c. Patches 1-3 simplify some of the MemoryRegionPortio arrays. This makes it easier to ascertain the effect of patch 6, since the patch only affects MemoryRegionPor

[Qemu-devel] [PULL] tricore patches for 2.3-rc2

2015-03-30 Thread Bastian Koppelmann
.git tags/pull-tricore-20150330 for you to fetch changes up to f1fdaf552974ee2ef6ec1ba3cf1e18c2951533e1: target-tricore: fix CACHEA/I_POSTINC/PREINC using data register.. (2015-03-30 13:39:38 +0200) TriCor

[Qemu-devel] [PATCH for-2.3 1/6] sb16: remove useless mixer_write_indexw

2015-03-30 Thread Paolo Bonzini
ioport.c is already able to split a 16-bit access into two 8-bit accesses to consecutive ports. Tested with Epic Pinball. Signed-off-by: Paolo Bonzini --- hw/audio/sb16.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/hw/audio/sb16.c b/hw/audio/sb16.c index 444eb9e..b052de5 100644 --

[Qemu-devel] [PATCH for-2.3 4/6] ioport: remove wrong comment

2015-03-30 Thread Paolo Bonzini
ioport.c has not been using an alias since commit b40acf9 (ioport: Switch dispatching to memory core layer, 2013-06-24). Remove the obsolete comment. Signed-off-by: Paolo Bonzini --- ioport.c | 4 1 file changed, 4 deletions(-) diff --git a/ioport.c b/ioport.c index 783a3ae..eb954e3 10064

[Qemu-devel] [PATCH for-2.3 5/6] ioport: loosen assertions on emulation of 16-bit ports

2015-03-30 Thread Paolo Bonzini
Right now, ioport.c assumes that the entire range specified with MemoryRegionPortio includes a region with size == 1. This however is not true for the VBE DISPI ports, which are 16-bit only. The next patch will make these regions' length equal to two, which can cause the assertions to trigger. R

[Qemu-devel] [PATCH for-2.3 6/6] ioport: reserve the whole range of an I/O port in the AddressSpace

2015-03-30 Thread Paolo Bonzini
When an I/O port is more than 1 byte long, ioport.c is currently creating "short" regions, for example 0x1ce-0x1ce for the 16-bit Bochs index port. When I/O ports are memory mapped, and thus accessed via a subpage_ops memory region, subpage_accepts gets confused because it finds a hole at 0x1cf an

[Qemu-devel] [PULL] target-tricore: fix CACHEA/I_POSTINC/PREINC using data register..

2015-03-30 Thread Bastian Koppelmann
..for address calculation instead address registers. Signed-off-by: Bastian Koppelmann --- target-tricore/translate.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/target-tricore/translate.c b/target-tricore/translate.c index bbcfee9..54a48cd 100644 --- a/target-tri

Re: [Qemu-devel] ppc vga output breakage since commit c3c1bb99

2015-03-30 Thread Peter Crosthwaite
On Mon, Mar 30, 2015 at 8:28 PM, Paolo Bonzini wrote: > > > On 30/03/2015 12:20, Mark Cave-Ayland wrote: >>> > >>> > Of course, there's a QEMU regression too and I'm thinking of how to fix >>> > it. Can the address_space_translate_address() length clamp be made conditional on non-MMIO access as

Re: [Qemu-devel] ppc vga output breakage since commit c3c1bb99

2015-03-30 Thread Paolo Bonzini
On 30/03/2015 13:45, Peter Crosthwaite wrote: > Can the address_space_translate_address() length clamp be made > conditional on non-MMIO access as the RC fix? I submitted > c3c1bb99d1c11978d9ce94d1bdcf0705378c1459 as I think its the right > thing to do regardless of memory type, but in reality it

Re: [Qemu-devel] [PATCH v5 1/2] qom: Add can_be_deleted callback to UserCreatableClass

2015-03-30 Thread Igor Mammedov
On Sat, 28 Mar 2015 01:53:28 +0800 Lin Ma wrote: > > 在 2015年03月28日 01:20, Igor Mammedov 写道: > > On Sat, 28 Mar 2015 00:55:29 +0800 > > Lin Ma wrote: > > > >> If backends implement the can_be_deleted and it returns false, > >> Then the qmp_object_del won't delete the given backends. > >> > >> Si

[Qemu-devel] KVM call agenda for 2015-03-31

2015-03-30 Thread Juan Quintela
Hi Please, send any topic that you are interested in covering. Call details: By popular demand, a google calendar public entry with it https://www.google.com/calendar/embed?src=dG9iMXRqcXAzN3Y4ZXZwNzRoMHE4a3BqcXNAZ3JvdXAuY2FsZW5kYXIuZ29vZ2xlLmNvbQ (Let me know if you have any problems wit

Re: [Qemu-devel] [PATCH v6 0/2] Add generic can_be_deleted to UserCreatableClass.

2015-03-30 Thread Igor Mammedov
On Mon, 30 Mar 2015 16:36:27 +0800 Lin Ma wrote: Paolo, Could you take it through your tree for 2.3, pls? > The patchset adds a generic can_be_deleted callback to UserCreatableClass. > It prevents removing a usercreatable object if the callback returns false. > > Backends could implement the ca

[Qemu-devel] [PATCH for-2.3] pc: acpi: fix pvpanic regression

2015-03-30 Thread Igor Mammedov
Commit cd61cb2 pc: acpi-build: generate pvpanic device description dynamically introduced regression changing pvpanic device HID from QEMU0001 to QEMU0002. Fix AML generated code so that pvpanic device would keep its original HID. i.e. QEMU0001 Signed-off-by: Igor Mammedov Reported-by: Gal Hamm

Re: [Qemu-devel] [PATCH for-2.3] pc: acpi: fix pvpanic regression

2015-03-30 Thread Paolo Bonzini
On 30/03/2015 14:18, Igor Mammedov wrote: > Commit cd61cb2 pc: acpi-build: generate pvpanic device description > dynamically > > introduced regression changing pvpanic device HID from > QEMU0001 to QEMU0002. > Fix AML generated code so that pvpanic device > would keep its original HID. i.e. QE

Re: [Qemu-devel] arm: semihosting: Preliminary AArch64 support

2015-03-30 Thread Liviu Ionescu
> On 30 Mar 2015, at 14:44, Leon Alrae wrote: > > ... QEMU parser doesn't seem to support the same sub-argument used multiple > times (always the last value is used): > -semihosting-config arg="argument 1",arg="argument 2",arg="argument 3" yes, unfortunately it is not supported. please note th

Re: [Qemu-devel] [RFC] acpi: add reset register to fadt

2015-03-30 Thread Igor Mammedov
On Sat, 28 Mar 2015 15:46:53 +0100 Reza Jelveh wrote: > Some operating systems such as FreeBSD and Mac OSX need the reset_register > section of the FADT filled to know which port to write to for a system reset. > > What is the right way to set the reset_val and the reset addr in this case? > ---

Re: [Qemu-devel] [PATCH for-2.3] pc: acpi: fix pvpanic regression

2015-03-30 Thread Igor Mammedov
On Mon, 30 Mar 2015 14:27:05 +0200 Paolo Bonzini wrote: > > > On 30/03/2015 14:18, Igor Mammedov wrote: > > Commit cd61cb2 pc: acpi-build: generate pvpanic device description > > dynamically > > > > introduced regression changing pvpanic device HID from > > QEMU0001 to QEMU0002. > > Fix AML

Re: [Qemu-devel] [PATCH v2 1/2] cpu/apic: drop icc bus/bridge/

2015-03-30 Thread Igor Mammedov
On Mon, 30 Mar 2015 18:12:06 +0800 Chen Fan wrote: > > On 03/23/2015 05:43 PM, Igor Mammedov wrote: > > On Mon, 23 Mar 2015 17:07:29 +0800 > > Chen Fan wrote: > > > >> On 03/23/2015 04:23 PM, Igor Mammedov wrote: > >>> On Mon, 23 Mar 2015 13:54:23 +0800 > >>> Chen Fan wrote: > >>> > ICC b

Re: [Qemu-devel] [PATCH 3/3] MAINTAINERS: Add Fam Zheng as Null block driver maintainer

2015-03-30 Thread Eric Blake
On 03/30/2015 03:23 AM, Fam Zheng wrote: > Signed-off-by: Fam Zheng > --- > MAINTAINERS | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index d7e9ba2..30073a3 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -1172,6 +1172,12 @@ S: Supported > F: blo

Re: [Qemu-devel] [PATCH 2/3] block/null: Support reopen

2015-03-30 Thread Eric Blake
On 03/30/2015 03:23 AM, Fam Zheng wrote: > Reopen is used in block-commit. With this always-succeed operation, it > is now possible to test committing to a null drive, by specifying > "null-aio://" or "null-co://" as the backing image when creating the > qcow2 image. > > Signed-off-by: Fam Zheng

Re: [Qemu-devel] [PATCH] replaced get_ticks_per_sec() with constant

2015-03-30 Thread Stefan Hajnoczi
On Sat, Mar 28, 2015 at 06:22:11PM +0200, Emil Condrea wrote: > diff --git a/target-arm/helper.c b/target-arm/helper.c > index 10886c5..504530a 100644 > --- a/target-arm/helper.c > +++ b/target-arm/helper.c > @@ -649,7 +649,7 @@ void pmccntr_sync(CPUARMState *env) > uint64_t temp_ticks; > >

Re: [Qemu-devel] [PATCH] replaced get_ticks_per_sec() with constant

2015-03-30 Thread Stefan Hajnoczi
On Sat, Mar 28, 2015 at 06:22:11PM +0200, Emil Condrea wrote: > This replaces all calls to get_ticks_per_sec() with > NSEC_PER_SEC defined in /include/qemu/timer.h > > Signed-off-by: Emil Condrea > --- > audio/audio.c | 2 +- > audio/noaudio.c| 4 ++-- > audio/spiceau

Re: [Qemu-devel] [RFC] acpi: add reset register to fadt

2015-03-30 Thread Reza Jelveh
On Mon, Mar 30, 2015 at 2:36 PM, Igor Mammedov wrote: > you are extending structure beyond of what specified by ACPI 1.0b spec, > that might break guests. > We probably can't change revision since Windows ACPI implementation > is mostly 1.0b based so we are stuck with it. > Patch needs to be test

Re: [Qemu-devel] [PATCH v2] fsdev/virtfs-proxy-helper: Fix possible overflow

2015-03-30 Thread Stefan Hajnoczi
On Sat, Mar 14, 2015 at 10:00:16AM +0800, Shannon Zhao wrote: > It's detected by coverity. As max of sockaddr_un.sun_path is > sizeof(helper.sun_path), should check the length of source > and use strncpy instead of strcpy. > > Signed-off-by: Shannon Zhao > Signed-off-by: Shannon Zhao > --- >

[Qemu-devel] [Bug 1392504] Re: libvirt not relabeling devices on USB Passthrough

2015-03-30 Thread nickmaelao
I have done the suggested in comment 23 , but the problem persist, after i lunch the guest, i receive the following erros, and lsusb in the guest doesnt show the attached usb device: Host: ll /dev/bus/usb/001/ crw-rw-r-- 1 libvirt-qemu kvm 189, 9 Mar 29 21:21 010 lsusb: Bus 001 Device 010:

[Qemu-devel] [Bug 1437970] [NEW] qemu-system-x86_64 - two mouse pointers & fast scrolling problem

2015-03-30 Thread LPNow
Public bug reported: Hello, System Specs Host: Slackware 14.1 x86_64 Openbox 3.5.2 tint2 panel (svn version) Nvidia GTX660M nvidia-driver-346.35 Screen: 17" @1920x1080@60Hz Guest - Slackware 14.1 x86_64 XFce 4.10 Screen 17" @1920x1080 xf86-video-vmware 13.0.1 QEMU 2.2.1 I sta

[Qemu-devel] [Bug 1392504] Re: libvirt not relabeling devices on USB Passthrough

2015-03-30 Thread nickmaelao
Sorry, i posted the wrong output, i had modified the ownership of usb the device /dev/bus/usb/001/010 to libvirt-qemu:kvm, but the problem persists. I configured libvirt to run as root, but the problem is the same, in syslog im still receiving Mar 29 22:54:42 ivan-desktop kernel: [174324.769452]

[Qemu-devel] [Bug 1437970] Re: qemu-system-x86_64 - two mouse pointers & fast scrolling problem

2015-03-30 Thread LPNow
** Description changed: Hello, System Specs Host: Slackware 14.1 x86_64 Openbox 3.5.2 tint2 panel (svn version) Nvidia GTX660M nvidia-driver-346.35 Screen: 17" @1920x1080@60Hz Logitech USB mouse Guest - Slackware 14.1 x86_64 XFce 4.10 Scree

[Qemu-devel] [Bug 1437970] Re: qemu-system-x86_64 - two mouse pointers & fast scrolling problem

2015-03-30 Thread LPNow
** Attachment added: "qemu_2.jpg" https://bugs.launchpad.net/qemu/+bug/1437970/+attachment/4360353/+files/qemu_2.jpg ** Description changed: Hello, System Specs Host: Slackware 14.1 x86_64 Openbox 3.5.2 tint2 panel (svn version) Nvidia GTX660M nvidia-driver-34

[Qemu-devel] [Bug 1437970] Re: qemu-system-x86_64 - two mouse pointers & fast scrolling problem

2015-03-30 Thread LPNow
** Description changed: Hello, System Specs Host: Slackware 14.1 x86_64 Openbox 3.5.2 tint2 panel (svn version) Nvidia GTX660M nvidia-driver-346.35 Screen: 17" @1920x1080@60Hz + Logitech USB mouse Guest - Slackware 14.1 x86_64 XFce 4.10 Scree

[Qemu-devel] [Bug 1437970] Re: qemu-system-x86_64 - two mouse pointers & fast scrolling problem

2015-03-30 Thread LPNow
** Description changed: Hello, System Specs Host: Slackware 14.1 x86_64 Openbox 3.5.2 tint2 panel (svn version) Nvidia GTX660M nvidia-driver-346.35 Screen: 17" @1920x1080@60Hz Guest - Slackware 14.1 x86_64 XFce 4.10 Screen 17" @1920x1080 xf

Re: [Qemu-devel] [RFC] acpi: add reset register to fadt

2015-03-30 Thread Igor Mammedov
On Mon, 30 Mar 2015 15:27:26 +0200 Reza Jelveh wrote: > On Mon, Mar 30, 2015 at 2:36 PM, Igor Mammedov wrote: > > > you are extending structure beyond of what specified by ACPI 1.0b spec, > > that might break guests. > > We probably can't change revision since Windows ACPI implementation > > is

Re: [Qemu-devel] [RFC] acpi: add reset register to fadt

2015-03-30 Thread Paolo Bonzini
On 29/03/2015 19:38, Marcel Apfelbaum wrote: > On 03/28/2015 05:46 PM, Reza Jelveh wrote: >> Some operating systems such as FreeBSD and Mac OSX need the >> reset_register >> section of the FADT filled to know which port to write to for a system >> reset. >> >> What is the right way to set the res

[Qemu-devel] [PATCH 0/2] virtfs-proxy-helper: small fixes

2015-03-30 Thread Stefan Hajnoczi
These fixes are not critical but it wouldn't hurt to get them into QEMU 2.3. See patches for descriptions. Stefan Hajnoczi (2): virtfs-proxy-helper: add missing long option terminator virtfs-proxy-helper: fail gracefully if socket path is too long fsdev/virtfs-proxy-helper.c | 8 +++- 1

[Qemu-devel] [PATCH 1/2] virtfs-proxy-helper: add missing long option terminator

2015-03-30 Thread Stefan Hajnoczi
The getopt_long(3) long options array must have a zeroed terminator. This patch solves a segmentation fault when an unknown command-line option is encountered: $ fsdev/virtfs-proxy-helper --help Segmentation fault (core dumped) Signed-off-by: Stefan Hajnoczi --- fsdev/virtfs-proxy-helper.c

[Qemu-devel] [PATCH 2/2] virtfs-proxy-helper: fail gracefully if socket path is too long

2015-03-30 Thread Stefan Hajnoczi
Replace the assertion check with graceful failure when the socket path is too long. Programs should not crash on invalid input. Print an error message and exit properly. Cc: Shannon Zhao Signed-off-by: Stefan Hajnoczi --- fsdev/virtfs-proxy-helper.c | 7 ++- 1 file changed, 6 insertions(+

Re: [Qemu-devel] [RFC] acpi: add reset register to fadt

2015-03-30 Thread Paolo Bonzini
On 30/03/2015 15:27, Reza Jelveh wrote: > you are extending structure beyond of what specified by ACPI 1.0b spec, > that might break guests. > We probably can't change revision since Windows ACPI implementation > is mostly 1.0b based so we are stuck with it. > Patch needs to b

Re: [Qemu-devel] [PATCH v2] fsdev/virtfs-proxy-helper: Fix possible overflow

2015-03-30 Thread Aneesh Kumar K.V
Stefan Hajnoczi writes: > On Sat, Mar 14, 2015 at 10:00:16AM +0800, Shannon Zhao wrote: >> It's detected by coverity. As max of sockaddr_un.sun_path is >> sizeof(helper.sun_path), should check the length of source >> and use strncpy instead of strcpy. >> >> Signed-off-by: Shannon Zhao >> Signed

Re: [Qemu-devel] [PATCH v5 22/45] postcopy: OS support test

2015-03-30 Thread Dr. David Alan Gilbert
* Paolo Bonzini (pbonz...@redhat.com) wrote: > > > On 13/03/2015 11:41, Dr. David Alan Gilbert wrote: > >>> > > +#ifdef HOST_X86_64 > >>> > > +#ifndef __NR_userfaultfd > >>> > > +#define __NR_userfaultfd 323 > >> > > >> > Sholdn't this come from the kernel headers imported in the previous > >> >

Re: [Qemu-devel] [PATCH v5 22/45] postcopy: OS support test

2015-03-30 Thread Paolo Bonzini
On 30/03/2015 16:07, Dr. David Alan Gilbert wrote: >>> > > >> > >> > You would just require new _installed_ kernel headers. Then you can use >> > linux/userfaultfd.h and syscall.h (the latter from glibc, includes >> > asm/unistd.h to get syscall numbers). >> > >> > linux-headers/ is useful fo

Re: [Qemu-devel] [PATCH RFC for-2.3? 2/8] pc87312: Create isa-parallel in-place and add alias par0-chardev property

2015-03-30 Thread Markus Armbruster
Andreas Färber writes: > Move the parallel_hds[] code to the PReP machine. Could use a brief explanation *why* we move it. > Signed-off-by: Andreas Färber > --- > hw/isa/pc87312.c | 25 +++-- > hw/ppc/prep.c| 9 + > include/hw/isa/pc87312.h |

[Qemu-devel] [PATCH v5 0/7] Block Throttle Group Support

2015-03-30 Thread Alberto Garcia
One more try, the changes in this series are smaller compared to the previous one, the most important fixes are related to the tests. V5: - Fixed and expanded the tests. There's a new one that checks that configuration changes affect the whole throttling group. - Refactored the code a bit: new

[Qemu-devel] [PATCH 7/7] throttle: Update throttle infrastructure copyright

2015-03-30 Thread Alberto Garcia
Signed-off-by: Alberto Garcia --- include/qemu/throttle.h | 8 +--- util/throttle.c | 8 +--- 2 files changed, 10 insertions(+), 6 deletions(-) diff --git a/include/qemu/throttle.h b/include/qemu/throttle.h index 2c560db..5af76f0 100644 --- a/include/qemu/throttle.h +++ b/include

[Qemu-devel] [PATCH 5/7] throttle: acquire the ThrottleGroup lock in bdrv_swap()

2015-03-30 Thread Alberto Garcia
bdrv_swap() touches the fields of a BlockDriverState that are protected by the ThrottleGroup lock. Although those fields end up in their original place, they are temporarily swapped in the process, so there's a chance that an operation on a member of the same group happening on a different thread c

[Qemu-devel] [PATCH 2/7] throttle: Add throttle group infrastructure

2015-03-30 Thread Alberto Garcia
Signed-off-by: Alberto Garcia --- block/Makefile.objs | 1 + block/throttle-groups.c | 261 include/block/block_int.h | 1 + include/block/throttle-groups.h | 39 ++ 4 files changed, 302 insertions(+) create mode 100644

[Qemu-devel] [PATCH 6/7] throttle: add the name of the ThrottleGroup to BlockDeviceInfo

2015-03-30 Thread Alberto Garcia
Signed-off-by: Alberto Garcia --- block/qapi.c | 3 +++ hmp.c| 6 -- qapi/block-core.json | 4 +++- 3 files changed, 10 insertions(+), 3 deletions(-) diff --git a/block/qapi.c b/block/qapi.c index 65af057..ee823d1 100644 --- a/block/qapi.c +++ b/block/qapi.c @@ -91,6

[Qemu-devel] [PATCH 3/7] throttle: Add throttle group infrastructure tests

2015-03-30 Thread Alberto Garcia
Signed-off-by: Alberto Garcia --- tests/test-throttle.c | 70 +-- 1 file changed, 68 insertions(+), 2 deletions(-) diff --git a/tests/test-throttle.c b/tests/test-throttle.c index 458f577..e87e6ae 100644 --- a/tests/test-throttle.c +++ b/tests/test

[Qemu-devel] [PATCH 1/7] throttle: Extract timers from ThrottleState into a separate structure

2015-03-30 Thread Alberto Garcia
From: Benoît Canet Group throttling will share ThrottleState between multiple bs. As a consequence the ThrottleState will be accessed by multiple aio context. Timers are tied to their aio context so they must go out of the ThrottleState structure. This commit paves the way for each bs of a comm

Re: [Qemu-devel] [PATCH RFC for-2.3? 0/8] prep: Fix pc87312 for -device usage

2015-03-30 Thread Markus Armbruster
Andreas Färber writes: > Hello Markus et al., > > This series attempts to fix the -device pc87312 issues you reported. > > I can't add alias properties for devices that don't get created before > realize. > Therefore this involves moving code for various ISA devices, to enable us > to initialize

[Qemu-devel] [PATCH v4 04/15] target-s390x: Introduce cpu models

2015-03-30 Thread Michael Mueller
This patch implements the static part of the s390 cpu class definitions. It defines s390 cpu models by means of virtual cpu ids (enum) which contain information on the cpu generation, the machine class, the GA number and the machine type. The cpu id is used to instantiate a cpu class per cpu model.

[Qemu-devel] [PATCH v4 00/15] s390x cpu model implementation

2015-03-30 Thread Michael Mueller
This patch set in combination with its kernel kvm patch set proposes an implementation of S390 cpu models. The origin of this item is to provide a means for management interfaces like libvirt to draw decisions if life guest migration to a target hypervisor is reasonable. A migration constraint is

[Qemu-devel] [PATCH v4 09/15] target-s390x: Add cpu class initialization routines

2015-03-30 Thread Michael Mueller
This patch provides routines to dynamically update the previously defined S390 cpu classes in the current host context. The main function performing this process is s390_setup_cpu_classes(). It takes the current host context and a facility list mask as parameter to setup the classes accordingly. It

  1   2   3   >