[Qemu-devel] [PATCH v2] translate-all: Fix formatting of dump output

2013-09-22 Thread Stefan Weil
The page dump writes a table with 3 abi_ulong values in each row. These values take 8 or 16 characters (depending on sizeof abi_ulong). Fix the table headings to be aligned with the table columns. old: startend size prot 00012000-00012021e000 0021e000 rwx

[Qemu-devel] [PATCH v2] block: Fix compiler warning (-Werror=uninitialized)

2013-09-22 Thread Stefan Weil
The patch fixes a warning from gcc (Debian 4.6.3-14+rpi1) 4.6.3: block/stream.c:141:22: error: ‘copy’ may be used uninitialized in this function [-Werror=uninitialized] This is not a real bug - a better compiler would not complain. Now 'copy' has always a defined value, so the check for ret = 0

Re: [Qemu-devel] in_asm substitute for accel=kvm:tcg

2013-09-22 Thread Gleb Natapov
On Thu, Sep 19, 2013 at 08:49:51PM +0300, Andriy Gapon wrote: on 19/09/2013 19:53 Paolo Bonzini said the following: Il 19/09/2013 16:36, Andriy Gapon ha scritto: Not sure how the code ends up at 0x9315 after that. Events are dropped, probably corresponding to more emulation. I've got

Re: [Qemu-devel] [PATCH] target-i386: Enable x2apic by default on more recent CPU models

2013-09-22 Thread Gleb Natapov
On Fri, Sep 20, 2013 at 04:15:17PM -0300, Eduardo Habkost wrote: This enables x2apic on the following CPU models: Conroe, Penryn, Nehalem, Westmere, Opteron_G[12345]. Normally we try to keep the CPU model definitions as close as the real CPUs as possible, but x2apic can be emulated by KVM

Re: [Qemu-devel] [PATCH V4 0/3] qemu-iotests: add test for fd passing via SCM rights

2013-09-22 Thread Wenchao Xia
On 09/20/2013 11:11 PM, Stefan Hajnoczi wrote: On Mon, Sep 09, 2013 at 01:57:20PM +0200, Stefan Hajnoczi wrote: On Fri, Sep 06, 2013 at 11:24:31AM +0800, Wenchao Xia wrote: tests/qemu-iotests/socket_scm_helper.c | 135 After a clean QEMU build ./check -raw

[Qemu-devel] [PATCH] Extend qemu-ga's 'guest-info' command to expose flag 'success-response'

2013-09-22 Thread Mark Wu
Now we have several qemu-ga commands not returning response on success. It has been documented in qga/qapi-schema.json already. This patch exposes the 'success-response' flag by extending 'guest-info' command. With this change, the clients can handle the command response more flexibly. Changes:

[Qemu-devel] [PATCH 0/4] qom: add helpers for integer properties

2013-09-22 Thread Michael S. Tsirkin
Add helper functions for adding read-only properties, that work in the case where the value is in memory. Michael S. Tsirkin (4): qemu: add Error to typedefs qom: pull in qemu/typedefs qom: cleanup struct Error references qom: add pointer to int property helpers include/qemu/typedefs.h

[Qemu-devel] [PATCH 2/4] qom: pull in qemu/typedefs

2013-09-22 Thread Michael S. Tsirkin
As usual so we can use typedefs without header dependencies. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/qom/object.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/qom/object.h b/include/qom/object.h index 1a7b71a..5b3b743 100644 --- a/include/qom/object.h +++

[Qemu-devel] [PATCH 1/4] qemu: add Error to typedefs

2013-09-22 Thread Michael S. Tsirkin
This is so qom headers can use it without pulling in extra headers. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/qemu/typedefs.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/qemu/typedefs.h b/include/qemu/typedefs.h index c5c6e36..bb5acf9 100644 ---

[Qemu-devel] [PATCH 4/4] qom: add pointer to int property helpers

2013-09-22 Thread Michael S. Tsirkin
Make it easy to add read-only helpers for simple integer properties in memory. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/qom/object.h | 21 qom/object.c | 56 2 files changed, 77 insertions(+)

[Qemu-devel] [PATCH 3/4] qom: cleanup struct Error references

2013-09-22 Thread Michael S. Tsirkin
now that a typedef for struct Error is available, use it in qom/object.h to match coding style rules. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/qom/object.h | 51 +-- 1 file changed, 25 insertions(+), 26 deletions(-) diff --git

Re: [Qemu-devel] [PATCH v3 0/4] timers thread-safe stuff

2013-09-22 Thread liu ping fan
On Wed, Sep 18, 2013 at 9:54 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Tue, Aug 27, 2013 at 11:20:59AM +0800, Liu Ping Fan wrote: Saw the Alex's patches has been merged, rebase mine onto his. v3: 1. rename seqlock_read_check as seqlock_read_retry 2. Document timerlist were

Re: [Qemu-devel] in_asm substitute for accel=kvm:tcg

2013-09-22 Thread Andriy Gapon
on 22/09/2013 09:31 Gleb Natapov said the following: Which kernel version is this? What BSD version? $ uname -a Linux kvm 3.8.0-27-generic #40-Ubuntu SMP Tue Jul 9 00:17:05 UTC 2013 x86_64 x86_64 x86_64 GNU/Linux FreeBSD is 9.x. -- Andriy Gapon

Re: [Qemu-devel] [PATCH 00/11] virtio: cleanup and fix hot-unplug

2013-09-22 Thread Paolo Bonzini
Il 21/09/2013 21:17, Michael S. Tsirkin ha scritto: On Fri, Sep 20, 2013 at 04:57:49PM +0200, Paolo Bonzini wrote: This series fixes hot-unplug of virtio devices, which can crash due to dangling pointer accesses. Could you please describe the sequence of steps that makes qemu crash? See

[Qemu-devel] [PATCH v4 0/4] timers thread-safe stuff

2013-09-22 Thread Liu Ping Fan
v4: fix commit log for protect timers_state's clock with seqlock (Thanks for Alex) v3: 1. rename seqlock_read_check as seqlock_read_retry 2. Document timerlist were protected by BQL, and discard private lock around qemu_event_wait(tl-ev). v2: 1. fix comment in commit and code 2. fix

[Qemu-devel] [PATCH v4 1/4] seqlock: introduce read-write seqlock

2013-09-22 Thread Liu Ping Fan
This lets the read-side access run outside the BQL. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- include/qemu/seqlock.h | 72 ++ 1 file changed, 72 insertions(+) create mode 100644 include/qemu/seqlock.h diff --git

[Qemu-devel] [PATCH v4 2/4] timer: protect timers_state's clock with seqlock

2013-09-22 Thread Liu Ping Fan
QEMU_CLOCK_VIRTUAL may be read outside BQL. This will make its foundation, i.e. timers_state exposed to race condition. Using private lock to protect it. After this patch, reading QEMU_CLOCK_VIRTUAL is thread safe unless use_icount is true, in which case the existing callers still rely on the BQL

[Qemu-devel] [PATCH v4 1/4] seqlock: introduce read-write seqlock

2013-09-22 Thread Liu Ping Fan
This lets the read-side access run outside the BQL. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- include/qemu/seqlock.h | 72 ++ 1 file changed, 72 insertions(+) create mode 100644 include/qemu/seqlock.h diff --git

[Qemu-devel] [PATCH v4 4/4] timer: make qemu_clock_enable sync between disable and timer's cb

2013-09-22 Thread Liu Ping Fan
After disabling the QemuClock, we should make sure that no QemuTimers are still in flight. To implement that with light overhead, we resort to QemuEvent. The caller of disabling will wait on QemuEvent of each timerlist. Note, qemu_clock_enable(foo,false) can _not_ be called from timer's cb. And

[Qemu-devel] [PATCH v4 3/4] qemu-thread: add QemuEvent

2013-09-22 Thread Liu Ping Fan
This emulates Win32 manual-reset events using futexes or conditional variables. Typical ways to use them are with multi-producer, single-consumer data structures, to test for a complex condition whose elements come from different threads: for (;;) { qemu_event_reset(ev); ...

Re: [Qemu-devel] in_asm substitute for accel=kvm:tcg

2013-09-22 Thread Gleb Natapov
On Sun, Sep 22, 2013 at 11:05:37AM +0300, Andriy Gapon wrote: on 22/09/2013 09:31 Gleb Natapov said the following: Which kernel version is this? What BSD version? $ uname -a Linux kvm 3.8.0-27-generic #40-Ubuntu SMP Tue Jul 9 00:17:05 UTC 2013 x86_64 x86_64 x86_64 GNU/Linux That's pretty

[Qemu-devel] [PATCH 1/2] tests: build the helper program in main build process

2013-09-22 Thread Wenchao Xia
This is a quick way to update helper program when qemu main code is changed or built, instead of adding new Makefile under test/qemu-iotest. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- tests/Makefile |8 +++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git

[Qemu-devel] [PATCH 0/2] build: trivial patches for test

2013-09-22 Thread Wenchao Xia
Wenchao Xia (2): tests: build the helper program in main build process build: add command check-clean Makefile |1 - tests/Makefile | 16 ++-- 2 files changed, 14 insertions(+), 3 deletions(-)

[Qemu-devel] [PATCH 2/2] build: add command check-clean

2013-09-22 Thread Wenchao Xia
This command will package the clean operations in tests. Now root Makefile simply calls the command and do not care the details of it any more. Original the built binaries for test will not be removed, now they will be deleted in clean operation. Signed-off-by: Wenchao Xia

[Qemu-devel] [RFC] sync NIC's MAC maintained in NICConf as soon as emualted NIC's MAC changed in guest

2013-09-22 Thread Zhanghaoyu (A)
Hi, all Do live migration if emulated NIC's MAC has been changed, RARP with wrong MAC address will broadcast via qemu_announce_self in destination, so, long time network disconnection probably happen. I want to do below works to resolve this problem, 1. change NICConf's MAC as soon as emulated

Re: [Qemu-devel] [RFC] sync NIC's MAC maintained in NICConf as soon as emualted NIC's MAC changed in guest

2013-09-22 Thread Michael S. Tsirkin
On Sun, Sep 22, 2013 at 08:35:29AM +, Zhanghaoyu (A) wrote: Hi, all Do live migration if emulated NIC's MAC has been changed, RARP with wrong MAC address will broadcast via qemu_announce_self in destination, so, long time network disconnection probably happen. Good catch. I want to

[Qemu-devel] [PATCH V2 0/4] export internal snapshot by qemu-nbd

2013-09-22 Thread Wenchao Xia
This series allow user to read internal snapshot's contents without qemu-img convert. V2: Address Stefan's comments: 02: add 'fall through' comments in the case statement. 03: add doc about the difference of internal snapshot and backing chain snapshot, which is used in previous

[Qemu-devel] [PATCH V2 4/4] qemu-iotests: add 058 internal snapshot export with qemu-nbd case

2013-09-22 Thread Wenchao Xia
Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- tests/qemu-iotests/058 | 87 tests/qemu-iotests/058.out | 26 + tests/qemu-iotests/group |1 + 3 files changed, 114 insertions(+), 0 deletions(-) create mode 100755

[Qemu-devel] [PATCH V2 1/4] snapshot: distinguish id and name in load_tmp

2013-09-22 Thread Wenchao Xia
Since later this function will be used so improve it. The only caller of it now is qemu-img, and it is not impacted by call the function twice to keep old search logic. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- block/qcow2-snapshot.c| 16 ++-- block/qcow2.h

[Qemu-devel] [PATCH V2 3/4] qemu-nbd: add doc for internal snapshot export

2013-09-22 Thread Wenchao Xia
Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- qemu-nbd.c|8 +++- qemu-nbd.texi |8 +++- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/qemu-nbd.c b/qemu-nbd.c index e450d04..8cb4bf1 100644 --- a/qemu-nbd.c +++ b/qemu-nbd.c @@ -80,7 +80,13 @@ static

[Qemu-devel] [PATCH V2 2/4] qemu-nbd: support internal snapshot export

2013-09-22 Thread Wenchao Xia
Now it is possible to directly export an internal snapshot, which can be used to probe the snapshot's contents without qemu-img convert. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- qemu-nbd.c | 54 +- 1 files changed, 53

Re: [Qemu-devel] in_asm substitute for accel=kvm:tcg

2013-09-22 Thread Andriy Gapon
on 22/09/2013 11:17 Gleb Natapov said the following: On Sun, Sep 22, 2013 at 11:05:37AM +0300, Andriy Gapon wrote: on 22/09/2013 09:31 Gleb Natapov said the following: Which kernel version is this? What BSD version? $ uname -a Linux kvm 3.8.0-27-generic #40-Ubuntu SMP Tue Jul 9 00:17:05 UTC

Re: [Qemu-devel] [SeaBIOS] [PATCH v2 1/5] linker: utility to patch in-memory ROM files

2013-09-22 Thread Michael S. Tsirkin
On Thu, Jul 25, 2013 at 08:06:27PM -0400, Kevin O'Connor wrote: On Thu, Jul 25, 2013 at 03:55:56PM +0300, Michael S. Tsirkin wrote: On Mon, Jul 15, 2013 at 11:01:02AM +0300, Michael S. Tsirkin wrote: On Sun, Jul 14, 2013 at 02:24:52PM -0400, Kevin O'Connor wrote: I'd prefer to see this

Re: [Qemu-devel] [SeaBIOS] [PATCH v2 1/5] linker: utility to patch in-memory ROM files

2013-09-22 Thread Michael S. Tsirkin
On Sun, Sep 22, 2013 at 01:49:58PM +0300, Michael S. Tsirkin wrote: On Thu, Jul 25, 2013 at 08:06:27PM -0400, Kevin O'Connor wrote: On Thu, Jul 25, 2013 at 03:55:56PM +0300, Michael S. Tsirkin wrote: On Mon, Jul 15, 2013 at 11:01:02AM +0300, Michael S. Tsirkin wrote: On Sun, Jul 14, 2013

[Qemu-devel] [PATCH] sPAPR: implement route_intx_to_irq to get gsi of pci device.

2013-09-22 Thread Liu Ping Fan
This is useful when pci assignment happens on sPAPR. Signed-off-by: Liu Ping Fan pingf...@linux.vnet.ibm.com --- This patch will apply on patches which enable xics in kernel. --- hw/intc/xics.c| 5 + hw/ppc/spapr_pci.c| 14 ++ include/hw/ppc/xics.h | 1 + 3 files

[Qemu-devel] [PATCH 1/2] block: fix backing file overriding

2013-09-22 Thread Fam Zheng
Providing backing.file.filename doesn't override backing file as expected: $ x86_64-softmmu/qemu-system-x86_64 -drive \ file=/tmp/child.qcow2,backing.file.filename=/tmp/fake.qcow2 qemu-system-x86_64: -drive \ file=/tmp/child.qcow2,backing.file.filename=/tmp/fake.qcow2:

[Qemu-devel] [PATCH 2/2] qemu-iotests: add test for backing file overriding

2013-09-22 Thread Fam Zheng
Test that backing.file.filename option can be parsed and override the backing file from image (backing file reflected with info block). Signed-off-by: Fam Zheng f...@redhat.com --- tests/qemu-iotests/051 | 17 - tests/qemu-iotests/051.out | 11 +++ 2 files changed, 27

[Qemu-devel] [PATCH 0/2] block: fix backing file overriding

2013-09-22 Thread Fam Zheng
The backing.file.filename option is not working as expected: if there's also a backing file name from the format driver, adding this option fails bdrv_open; if there's no backing file name info in the image, info block doesn't show the overrided file name. A test case is updated to catch these

[Qemu-devel] [PATCH] .gitignore: ignore tests/qemu-iotests/socket_scm_helper

2013-09-22 Thread Fam Zheng
Signed-off-by: Fam Zheng f...@redhat.com --- tests/qemu-iotests/.gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/qemu-iotests/.gitignore b/tests/qemu-iotests/.gitignore index 62b4002..0541f80 100644 --- a/tests/qemu-iotests/.gitignore +++ b/tests/qemu-iotests/.gitignore @@

[Qemu-devel] [PATCH v4 05/23] fw_cfg: interface to trigger callback on read

2013-09-22 Thread Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/nvram/fw_cfg.h | 4 hw/nvram/fw_cfg.c | 33 - 2 files changed, 32 insertions(+), 5 deletions(-) diff --git a/include/hw/nvram/fw_cfg.h b/include/hw/nvram/fw_cfg.h index f60dd67..2ab0fc2

[Qemu-devel] [PATCH v4 00/23] qemu: generate acpi tables for the guest

2013-09-22 Thread Michael S. Tsirkin
This code can also be found here: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git acpi While this patch still uses info not available in QOM, I think it's reasonable to merge it and then refactor as QOM properties cover more ground. In particular, merging this patchset blocks other projects

[Qemu-devel] [PATCH v4 01/23] qemu: add Error to typedefs

2013-09-22 Thread Michael S. Tsirkin
This is so qom headers can use it without pulling in extra headers. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/qemu/typedefs.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/qemu/typedefs.h b/include/qemu/typedefs.h index a4c1b84..46c3599 100644 ---

[Qemu-devel] [PATCH v4 12/23] acpi: add rules to compile ASL source

2013-09-22 Thread Michael S. Tsirkin
Detect presence of IASL compiler and use it to process ASL source. If not there, use pre-compiled files in-tree. Add script to update the in-tree files. Note: distros are known to silently update iasl so detect correct iasl flags for the installed version on each run as opposed to at configure

[Qemu-devel] [PATCH v4 06/23] loader: support for unmapped ROM blobs

2013-09-22 Thread Michael S. Tsirkin
Support ROM blobs not mapped into guest memory: same as ROM files really but use caller's buffer. Support incoking callback on access and return memory pointer making it easier for caller to update memory if necessary. Signed-off-by: Michael S. Tsirkin m...@redhat.com Reviewed-by: Laszlo Ersek

[Qemu-devel] [PATCH v4 08/23] pcie_host: expose address format

2013-09-22 Thread Michael S. Tsirkin
Callers pass in the address so it's helpful for them to be able to decode it. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/pci/pcie_host.h | 21 + hw/pci/pcie_host.c | 21 - 2 files changed, 21 insertions(+), 21 deletions(-)

[Qemu-devel] [PATCH v4 15/23] i386: add bios linker/loader

2013-09-22 Thread Michael S. Tsirkin
This adds a dynamic bios linker/loader. This will be used by acpi table generation code to: - load each table in the appropriate memory segment - link tables to each other - fix up checksums after said linking Signed-off-by: Michael S. Tsirkin m...@redhat.com ---

[Qemu-devel] [PATCH v4 22/23] hpet: add API to find it

2013-09-22 Thread Michael S. Tsirkin
Add API to find HPET using QOM. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/timer/hpet.h | 2 ++ hw/timer/hpet.c | 5 + 2 files changed, 7 insertions(+) diff --git a/include/hw/timer/hpet.h b/include/hw/timer/hpet.h index 757f79f..ab44bd3 100644 ---

[Qemu-devel] [PATCH v4 09/23] q35: use macro for MCFG property name

2013-09-22 Thread Michael S. Tsirkin
Useful to make it accessible through QOM. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/pci/pcie_host.h | 2 ++ hw/pci-host/q35.c | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/include/hw/pci/pcie_host.h b/include/hw/pci/pcie_host.h index

[Qemu-devel] [PATCH v4 07/23] pcie_host: expose UNMAPPED macro

2013-09-22 Thread Michael S. Tsirkin
Make it possible to test unmapped status through QMP. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/pci/pcie_host.h | 3 +++ hw/pci/pcie_host.c | 3 --- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/include/hw/pci/pcie_host.h

[Qemu-devel] [PATCH v4 16/23] loader: allow adding ROMs in done callbacks

2013-09-22 Thread Michael S. Tsirkin
Don't abort if machine done callbacks add ROMs. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/loader.h | 1 + hw/core/loader.c| 6 +- vl.c| 3 +++ 3 files changed, 9 insertions(+), 1 deletion(-) diff --git a/include/hw/loader.h b/include/hw/loader.h

[Qemu-devel] [PATCH v4 10/23] q35: expose mmcfg size as a property

2013-09-22 Thread Michael S. Tsirkin
Address is already exposed, expose size for symmetry. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/pci/pcie_host.h | 1 + hw/pci-host/q35.c | 14 ++ 2 files changed, 15 insertions(+) diff --git a/include/hw/pci/pcie_host.h b/include/hw/pci/pcie_host.h

[Qemu-devel] [PATCH v4 17/23] i386: define pc guest info

2013-09-22 Thread Michael S. Tsirkin
This defines a structure that will be used to fill in acpi tables where relevant properties are not yet available using QOM. Signed-off-by: Michael S. Tsirkin m...@redhat.com Reviewed-by: Laszlo Ersek ler...@redhat.com --- include/hw/i386/pc.h | 10 ++ hw/i386/pc.c | 35

[Qemu-devel] [PATCH v4 21/23] pvpanic: add API to access io port

2013-09-22 Thread Michael S. Tsirkin
Add API to find pvpanic device and get its io port. Will be used to fill in guest info structure. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/i386/pc.h | 1 + hw/misc/pvpanic.c| 13 - 2 files changed, 13 insertions(+), 1 deletion(-) diff --git

[Qemu-devel] [PATCH v4 14/23] loader: use file path size from fw_cfg.h

2013-09-22 Thread Michael S. Tsirkin
Avoid a bit of code duplication, make max file path constant reusable. Suggested-by: Laszlo Ersek ler...@redhat.com Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/nvram/fw_cfg.h | 4 +++- hw/core/loader.c | 2 +- 2 files changed, 4 insertions(+), 2 deletions(-) diff

[Qemu-devel] [PATCH v4 18/23] acpi/piix: add macros for acpi property names

2013-09-22 Thread Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/i386/pc.h | 10 ++ hw/acpi/piix4.c | 6 +++--- 2 files changed, 13 insertions(+), 3 deletions(-) diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h index 7a343f8..f966cef 100644 --- a/include/hw/i386/pc.h +++

Re: [Qemu-devel] [SeaBIOS] [PATCH v2 1/5] linker: utility to patch in-memory ROM files

2013-09-22 Thread Michael S. Tsirkin
On Sun, Sep 22, 2013 at 02:18:45PM +0300, Michael S. Tsirkin wrote: On Sun, Sep 22, 2013 at 01:49:58PM +0300, Michael S. Tsirkin wrote: On Thu, Jul 25, 2013 at 08:06:27PM -0400, Kevin O'Connor wrote: On Thu, Jul 25, 2013 at 03:55:56PM +0300, Michael S. Tsirkin wrote: On Mon, Jul 15, 2013

[Qemu-devel] [PATCH v4 19/23] piix: APIs for pc guest info

2013-09-22 Thread Michael S. Tsirkin
This adds APIs that will be used to fill in guest acpi tables. Some required information is still lacking in QOM, so we fall back on lookups by type and returning explicit types. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/acpi/piix4.h | 8 include/hw/i386/pc.h

[Qemu-devel] [PATCH v4 20/23] ich9: APIs for pc guest info

2013-09-22 Thread Michael S. Tsirkin
This adds APIs that will be used to fill in acpi tables, implemented using QOM, to various ich9 components. Some information is still missing in QOM, so we fall back on lookups by type instead. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/acpi/ich9.h| 2 ++

[Qemu-devel] [PATCH v4 03/23] qom: cleanup struct Error references

2013-09-22 Thread Michael S. Tsirkin
now that a typedef for struct Error is available, use it in qom/object.h to match coding style rules. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/qom/object.h | 51 +-- 1 file changed, 25 insertions(+), 26 deletions(-) diff --git

[Qemu-devel] [PATCH v4 02/23] qom: pull in qemu/typedefs

2013-09-22 Thread Michael S. Tsirkin
As usual so we can use typedefs without header dependencies. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/qom/object.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/qom/object.h b/include/qom/object.h index 1a7b71a..5b3b743 100644 --- a/include/qom/object.h +++

[Qemu-devel] [PATCH v4 04/23] qom: add pointer to int property helpers

2013-09-22 Thread Michael S. Tsirkin
Make it easy to add read-only helpers for simple integer properties in memory. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/qom/object.h | 21 qom/object.c | 56 2 files changed, 77 insertions(+)

[Qemu-devel] [PATCH] virtio-net: broken RX filtering logic fixed

2013-09-22 Thread Dmitry Fleytman
From: Dmitry Fleytman dfley...@redhat.com Upon processing of VIRTIO_NET_CTRL_MAC_TABLE_SET command multicast list overwrites unicast list in mac_table. This leads to broken logic for both unicast and multicast RX filtering. Signed-off-by: Dmitry Fleytman dfley...@redhat.com ---

[Qemu-devel] the way i read the registers may wrong

2013-09-22 Thread Peter Cheung
Hi all I am using the following code to read registers, i can read the value of EIP, but when i dump the value pointed by EIP, all are zero, byte code should not be zero, so i think my code is wrong. Please point me out thanks? static void gkd_read_registers(CPUState *cpu, char *buffer) {

Re: [Qemu-devel] [Xen-devel] RESEND [Xen-unstable][Qemu-xen] HVM Guest reading of Expansion ROM from passthroughed PCI device returns data from emulated VGA rom

2013-09-22 Thread Pasi Kärkkäinen
On Sun, Sep 22, 2013 at 05:00:58PM +0200, Sander Eikelenboom wrote: I'm trying to get secondary vga-passthrough on a HVM guest to work with a AMD HD6570 and the native kernel radeon driver and kernel modesetting. So the guest still gets the emulated stdvga or cirrus device(used in my

Re: [Qemu-devel] [RFC 00/16] TCG indirect registers

2013-09-22 Thread Max Filippov
On Fri, Sep 20, 2013 at 1:24 AM, Richard Henderson r...@twiddle.net wrote: This is an attempt to improve performance of target-sparc by exposing the windowed registers as TCG globals, and all the optimization that we can do there. This is done via allowing tcg_global_mem_new to be used with

Re: [Qemu-devel] [PATCH v4 22/23] hpet: add API to find it

2013-09-22 Thread Paolo Bonzini
Il 22/09/2013 15:38, Michael S. Tsirkin ha scritto: Add API to find HPET using QOM. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/timer/hpet.h | 2 ++ hw/timer/hpet.c | 5 + 2 files changed, 7 insertions(+) diff --git a/include/hw/timer/hpet.h

Re: [Qemu-devel] [PATCH v4 22/23] hpet: add API to find it

2013-09-22 Thread Michael S. Tsirkin
On Sun, Sep 22, 2013 at 09:22:09PM +0200, Paolo Bonzini wrote: Il 22/09/2013 15:38, Michael S. Tsirkin ha scritto: Add API to find HPET using QOM. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/timer/hpet.h | 2 ++ hw/timer/hpet.c | 5 + 2 files

Re: [Qemu-devel] [PATCH] sPAPR: implement route_intx_to_irq to get gsi of pci device.

2013-09-22 Thread Alexander Graf
Am 22.09.2013 um 13:47 schrieb Liu Ping Fan qemul...@gmail.com: This is useful when pci assignment happens on sPAPR. This patch doesn't sound useful on its own to me, thus probably belongs in a greater patch set. And without even a clear commit message that explains why exactly this is going

Re: [Qemu-devel] [PATCH 1/2] tests: build the helper program in main build process

2013-09-22 Thread Michael Tokarev
22.09.2013 12:30, Wenchao Xia пишет: This is a quick way to update helper program when qemu main code is changed or built, instead of adding new Makefile under test/qemu-iotest. Why? [] +# Build the help program automatically + +all: $(QEMU_IOTESTS_HELPERS-y) This appears to be somewhat

Re: [Qemu-devel] [PATCH] .gitignore: ignore tests/qemu-iotests/socket_scm_helper

2013-09-22 Thread Wenchao Xia
Reviewed-by: Wenchao Xiaxiaw...@linux.vnet.ibm.com Signed-off-by: Fam Zhengf...@redhat.com --- tests/qemu-iotests/.gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/qemu-iotests/.gitignore b/tests/qemu-iotests/.gitignore index 62b4002..0541f80 100644 ---

Re: [Qemu-devel] [PATCH 1/2] tests: build the helper program in main build process

2013-09-22 Thread Wenchao Xia
On 09/23/2013 04:36 AM, Michael Tokarev wrote: 22.09.2013 12:30, Wenchao Xia пишет: This is a quick way to update helper program when qemu main code is changed or built, instead of adding new Makefile under test/qemu-iotest. Why? [] I think it is a bit overkill to have a new Makefile for

Re: [Qemu-devel] [PATCH] sPAPR: implement route_intx_to_irq to get gsi of pci device.

2013-09-22 Thread liu ping fan
On Mon, Sep 23, 2013 at 4:02 AM, Alexander Graf ag...@suse.de wrote: Am 22.09.2013 um 13:47 schrieb Liu Ping Fan qemul...@gmail.com: This is useful when pci assignment happens on sPAPR. This patch doesn't sound useful on its own to me, thus probably belongs in a greater patch set. Yes, I

Re: [Qemu-devel] [PATCH] sPAPR: implement route_intx_to_irq to get gsi of pci device.

2013-09-22 Thread Alexey Kardashevskiy
On 09/22/2013 09:47 PM, Liu Ping Fan wrote: This is useful when pci assignment happens on sPAPR. I have almost the same patch in my queue already, it will enable irqfd for both INTX and MSI, I am just waiting till in-kernel XICS patchset gets in upstream and then I'll post it. Signed-off-by:

Re: [Qemu-devel] [PATCH v4 00/12] xics: reworks and in-kernel support

2013-09-22 Thread Alexey Kardashevskiy
On 09/16/2013 02:10 PM, Alexey Kardashevskiy wrote: On 09/10/2013 02:26 PM, Alexey Kardashevskiy wrote: On 09/04/2013 12:56 PM, Alexey Kardashevskiy wrote: On 08/30/2013 03:28 PM, Alexey Kardashevskiy wrote: Yet another try with XICS and XICS-KVM. v3-v4: Addressed multiple comments from

Re: [Qemu-devel] [PATCH 5/5] hw: arm_gic_kvm: Add KVM VGIC save/restore logic

2013-09-22 Thread Christoffer Dall
On Sat, Sep 21, 2013 at 06:38:19PM +0900, Peter Maydell wrote: On 21 September 2013 06:46, Christoffer Dall christoffer.d...@linaro.org wrote: On Sat, Sep 21, 2013 at 06:22:23AM +0900, Peter Maydell wrote: *) for getting TCG-KVM and KVM-with-non-host-CPU cases right we need to do

Re: [Qemu-devel] [PATCH] sPAPR: implement route_intx_to_irq to get gsi of pci device.

2013-09-22 Thread liu ping fan
On Mon, Sep 23, 2013 at 9:59 AM, Alexey Kardashevskiy a...@ozlabs.ru wrote: On 09/22/2013 09:47 PM, Liu Ping Fan wrote: This is useful when pci assignment happens on sPAPR. I have almost the same patch in my queue already, it will enable irqfd for both INTX and MSI, I am just waiting till