[Qemu-devel] [PATCH v4 1/4] firmware: introduce sysfs driver for QEMU's fw_cfg device

2015-11-13 Thread Gabriel L. Somlo
From: Gabriel Somlo Make fw_cfg entries of type "file" available via sysfs. Entries are listed under /sys/firmware/qemu_fw_cfg/by_key, in folders named after each entry's selector key. Filename, selector value, and size read-only attributes are included for each entry. Also, a "raw" attribute all

[Qemu-devel] [PATCH v4 3/4] firmware: create directory hierarchy for sysfs fw_cfg entries

2015-11-13 Thread Gabriel L. Somlo
From: Gabriel Somlo Each fw_cfg entry of type "file" has an associated 56-char, nul-terminated ASCII string which represents its name. While the fw_cfg device doesn't itself impose any specific naming convention, QEMU developers have traditionally used path name semantics (i.e. "etc/acpi/rsdp") t

[Qemu-devel] [PATCH v4 0/4] SysFS driver for QEMU fw_cfg device

2015-11-13 Thread Gabriel L. Somlo
From: "Gabriel Somlo" Allow access to QEMU firmware blobs, passed into the guest VM via the fw_cfg device, through SysFS entries. Blob meta-data (e.g. name, size, and fw_cfg key), as well as the raw binary blob data may be accessed. The SysFS access location is /sys/firmware/qemu_fw_cfg/... and

[Qemu-devel] [PATCH v4 2/4] kobject: export kset_find_obj() for module use

2015-11-13 Thread Gabriel L. Somlo
From: Gabriel Somlo Signed-off-by: Gabriel Somlo --- lib/kobject.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/kobject.c b/lib/kobject.c index 7cbccd2..90d1be6 100644 --- a/lib/kobject.c +++ b/lib/kobject.c @@ -861,6 +861,7 @@ struct kobject *kset_find_obj(struct kset *kset, const c

[Qemu-devel] [PATCH v4 4/4] devicetree: update documentation for fw_cfg ARM bindings

2015-11-13 Thread Gabriel L. Somlo
From: Gabriel Somlo Remove redundant details from Documentation/devicetree/bindings/arm/fw-cfg.txt, and replace them with a pointer to the more comprehensive fw_cfg documentation privided by Documentation/ABI/testing/sysfs-firmware-qemu_fw_cfg, leaving the specific ARM DTB node description in pla

[Qemu-devel] [PATCH v5 2/5] pc: fw_cfg: move ioport base constant to pc.h

2015-11-13 Thread Gabriel L. Somlo
Move BIOS_CFG_IOPORT define from pc.c to pc.h, and rename it to FW_CFG_IO_BASE. Cc: Marc Marí Signed-off-by: Gabriel Somlo Reviewed-by: Laszlo Ersek --- hw/i386/pc.c | 5 ++--- include/hw/i386/pc.h | 2 ++ 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/hw/i386/pc.c b/hw

[Qemu-devel] [PATCH v5 4/5] acpi: arm: add fw_cfg device node to dsdt

2015-11-13 Thread Gabriel L. Somlo
Add a fw_cfg device node to the ACPI DSDT. This is mostly informational, as the authoritative fw_cfg MMIO region(s) are listed in the Device Tree. However, since we are building ACPI tables, we might as well be thorough while at it... Signed-off-by: Gabriel Somlo Reviewed-by: Laszlo Ersek Tested

[Qemu-devel] [PATCH v5 0/5] add ACPI node for fw_cfg on pc and arm

2015-11-13 Thread Gabriel L. Somlo
New since v4: - rebased on top of Marc's DMA series - drop machine compat dependency for insertion into x86/ssdt (patch 3/5), following agreement between Igor and Eduardo - [mm]io register range now covers DMA register as well, if available. - s/

[Qemu-devel] [PATCH v5 1/5] fw_cfg: expose control register size in fw_cfg.h

2015-11-13 Thread Gabriel L. Somlo
Expose the size of the control register (FW_CFG_CTL_SIZE) in fw_cfg.h. Add comment to fw_cfg_io_realize() pointing out that since the 8-bit data register is always subsumed by the 16-bit control register in the port I/O case, we use the control register width as the *total* width of the (classic, n

[Qemu-devel] [PATCH v5 3/5] acpi: pc: add fw_cfg device node to ssdt

2015-11-13 Thread Gabriel L. Somlo
Add a fw_cfg device node to the ACPI SSDT. While the guest-side firmware can't utilize this information (since it has to access the hard-coded fw_cfg device to extract ACPI tables to begin with), having fw_cfg listed in ACPI will help the guest kernel keep a more accurate inventory of in-use IO por

[Qemu-devel] [PATCH v5 5/5] fw_cfg: document ACPI device node information

2015-11-13 Thread Gabriel L. Somlo
Signed-off-by: Gabriel Somlo Reviewed-by: Laszlo Ersek --- docs/specs/fw_cfg.txt | 9 + 1 file changed, 9 insertions(+) diff --git a/docs/specs/fw_cfg.txt b/docs/specs/fw_cfg.txt index 2099ad9..5414140 100644 --- a/docs/specs/fw_cfg.txt +++ b/docs/specs/fw_cfg.txt @@ -84,6 +84,15 @@ Sel

[Qemu-devel] [PATCH v3 1/2] target-ppc: Define kvmppc_read_int_dt()

2015-11-13 Thread Sukadev Bhattiprolu
Extract code from the function kvmppc_read_int_cpu_dt() that actually reads the file into a separate function, so it can be called from other places. Signed-off-by: Sukadev Bhattiprolu --- target-ppc/kvm.c | 36 ++-- 1 file changed, 22 insertions(+), 14 deletions(

[Qemu-devel] [PATCH v3 2/2] target-ppc: Implement rtas_get_sysparm(PROCESSOR_MODULE_INFO)

2015-11-13 Thread Sukadev Bhattiprolu
Implement RTAS_SYSPARM_PROCESSOR_MODULE_INFO parameter to rtas_get_sysparm() call in qemu. This call returns the processor module (socket), chip and core information as specified in section 7.3.16.17 of LoPAPR v2.7. We walk the /proc/device-tree to determine the number of modules(aka sockets), chi

Re: [Qemu-devel] [PATCH 1/2] add support for KVM_CAP_SPLIT_IRQCHIP

2015-11-13 Thread Eric Blake
On 11/13/2015 04:25 PM, Matt Gingell wrote: [meta-comment:] This patch was sent without an In-Reply-To header tying it back to the 0/2 cover letter, which means mailers render it as its own thread. Git 'send-email' can do proper threading with the right settings (although I'm not sure exactly wh

Re: [Qemu-devel] [PATCH v11 22/28] qapi: Simplify visiting of alternate types

2015-11-13 Thread Eric Blake
On 11/10/2015 11:51 PM, Eric Blake wrote: > Previously, working with alternates required two lookup arrays > and some indirection: for type Foo, we created Foo_qtypes[] > which maps each qtype to a value of the generated FooKind enum, > then look up that value in FooKind_lookup[] like we do for oth

[Qemu-devel] [PATCH 1/2] add support for KVM_CAP_SPLIT_IRQCHIP

2015-11-13 Thread Matt Gingell
This patch adds the initial plumbing for split IRQ chip mode via KVM_CAP_SPLIT_IRQCHIP. In addition to option processing, a number of kvm_*_in_kernel macros are defined to help clarify which component is where. Signed-off-by: Matt Gingell --- hw/core/machine.c| 48 +++

[Qemu-devel] [PATCH 2/2] add support for KVM_CAP_SPLIT_IRQCHIP

2015-11-13 Thread Matt Gingell
This patch adds support for split IRQ chip mode. When KVM_CAP_SPLIT_IRQCHIP is enabled: 1.) The PIC, PIT, and IOAPIC are implemented in userspace while the LAPIC is implemented by KVM. 2.) The software IOAPIC delivers interrupts to the KVM LAPIC via kvm_set_irq. Interrupt delivery

[Qemu-devel] [PATCH 0/2] add support for KVM_CAP_SPLIT_IRQCHIP

2015-11-13 Thread Matt Gingell
Hi, The following patches adds support for the new KVM split irqchip interface discussed on the KVM mailing list. [kvm] KVM: x86: Split the APIC from the rest of IRQCHIP. http://thread.gmane.org/gmane.comp.emulators.kvm.devel/135918 Our testing found some issues with the implementation of split

Re: [Qemu-devel] [PATCH V4 1/6] ide/atapi: make PIO read requests async

2015-11-13 Thread Peter Lieven
> Am 13.11.2015 um 23:42 schrieb John Snow : > > > >> On 11/12/2015 11:30 AM, Peter Lieven wrote: >> PIO read requests on the ATAPI interface used to be sync blk requests. >> This has two significant drawbacks. First the main loop hangs util an >> I/O request is completed and secondly if the I

Re: [Qemu-devel] [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close()

2015-11-13 Thread John Snow
On 11/12/2015 01:23 AM, Fam Zheng wrote: > On Mon, 11/09 23:39, Max Reitz wrote: >> bdrv_delete() is not very happy about deleting BlockDriverStates with >> dirty bitmaps still attached to them. In the past, we got around that >> very easily by relying on bdrv_close_all() bypassing bdrv_delete(),

Re: [Qemu-devel] [PATCH for 2.5 0/2] minor fixes for guest-exec

2015-11-13 Thread Michael Roth
Quoting Denis V. Lunev (2015-11-12 07:36:19) > Signed-off-by: Yuri Pudgorodskiy > Signed-off-by: Denis V. Lunev > CC: Michael Roth > > Yuri Pudgorodskiy (2): > [qga] Bugfix of default env processing for guest-exec. > [qga] Enable lookup for the PATH from the passed envp for guest-exec. Thanks,

Re: [Qemu-devel] [PATCH V4 0/6] ide: avoid main-loop hang on CDROM/NFS failure

2015-11-13 Thread John Snow
On 11/12/2015 11:30 AM, Peter Lieven wrote: > This series aims at avoiding a hanging main-loop if a vserver has a > CDROM image mounted from a NFS share and that NFS share goes down. > Typical situation is that users mount an CDROM ISO to install something > and then forget to eject that CDROM af

Re: [Qemu-devel] [PATCH V4 1/6] ide/atapi: make PIO read requests async

2015-11-13 Thread John Snow
On 11/12/2015 11:30 AM, Peter Lieven wrote: > PIO read requests on the ATAPI interface used to be sync blk requests. > This has two significant drawbacks. First the main loop hangs util an > I/O request is completed and secondly if the I/O request does not > complete (e.g. due to an unresponsive

[Qemu-devel] [PULL for-2.5 2/2] qga: allow to lookup in PATH from the passed envp for guest-exec

2015-11-13 Thread Michael Roth
From: Yuri Pudgorodskiy This was original behaviour before GLIB gspawn() rework and we rely on this behaviour. Signed-off-by: Yuri Pudgorodskiy Signed-off-by: Denis V. Lunev CC: Michael Roth Signed-off-by: Michael Roth --- qga/commands.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(

[Qemu-devel] [PULL for-2.5 0/2] qemu-ga patch queue for 2.5

2015-11-13 Thread Michael Roth
The following changes since commit 8337c6cbc37c6b2184f41bab3eaff47d5e68012a: Update version for v2.5.0-rc0 release (2015-11-13 17:10:36 +) are available in the git repository at: git://github.com/mdroth/qemu.git tags/qga-pull-2015-11-13-tag for you to fetch changes up to 8144e432d8c8e80

[Qemu-devel] [PULL for-2.5 1/2] qga: fix for default env processing for guest-exec

2015-11-13 Thread Michael Roth
From: Yuri Pudgorodskiy envp == NULL must be passed inside gspawn() if it was not passed with the command line. Original code inherits environment from the QGA, which is wrong. Signed-off-by: Yuri Pudgorodskiy Signed-off-by: Denis V. Lunev CC: Michael Roth Signed-off-by: Michael Roth --- qg

[Qemu-devel] [ANNOUNCE] QEMU 2.5.0-rc0 is now available

2015-11-13 Thread Michael Roth
Hello, On behalf of the QEMU Team, I'd like to announce the availability of the first release candidate for the QEMU 2.5 release. This release is meant for testing purposes and should not be used in a production environment. http://wiki.qemu.org/download/qemu-2.5.0-rc0.tar.bz2 You can help impr

Re: [Qemu-devel] [PATCH] crypto: avoid two coverity false positive error reports

2015-11-13 Thread Eric Blake
On 11/13/2015 10:45 AM, Daniel P. Berrange wrote: > In qcrypto_tls_creds_get_path() coverity complains that > we are checking '*creds' for NULL, despite having > dereferenced it previously. This is harmless bug due > to fact that the trace call was too early. Moving it > after the cleanup gets the

Re: [Qemu-devel] [PATCH v11 19/28] qapi: Change munging of CamelCase enum values

2015-11-13 Thread Eric Blake
On 11/13/2015 11:13 AM, Markus Armbruster wrote: > We need c_name() to protect ticklish identifiers only when its result is > used as identifier. Not when it's *part* of an identifier, > e.g. prefixed with qapi_, or camel_to_upper(type_name) + '_'. > > We can protect even when we don't need to,

Re: [Qemu-devel] [PATCH v4 6/7] qom: replace object property list with GHashTable

2015-11-13 Thread Andreas Färber
Am 13.11.2015 um 22:00 schrieb Christian Borntraeger: > On 11/13/2015 07:14 PM, Andreas Färber wrote: >> Am 13.10.2015 um 14:37 schrieb Daniel P. Berrange: >>> From: Pavel Fedin >>> >>> ARM GICv3 systems with large number of CPUs create lots of IRQ pins. Since >>> every pin is represented as a pro

Re: [Qemu-devel] [PATCH v2] target-arm: Clean up DISAS_UPDATE usage in AArch32 translation code

2015-11-13 Thread Sergey Fedorov
On 10.11.2015 15:15, Peter Maydell wrote: > So the way the 32-bit code works for singlestep is complicated > because of the need to handle the conditional instructions, > which means you get a lot more cases like "this is a conditional > SWI" that need to be handled. A quick summary of some of the

Re: [Qemu-devel] [PATCH v13 09/13] add check reset mechanism when hotplug vfio device

2015-11-13 Thread Alex Williamson
On Fri, 2015-11-13 at 11:28 +0800, Cao jin wrote: > > On 11/12/2015 07:51 PM, Michael S. Tsirkin wrote: > > On Wed, Nov 11, 2015 at 06:34:27PM +0800, Cao jin wrote: > >> From: Chen Fan > >> > >> Since we support multi-function hotplug. the function 0 indicate > >> the closure of the slot, so we h

Re: [Qemu-devel] [PATCH v4 6/7] qom: replace object property list with GHashTable

2015-11-13 Thread Christian Borntraeger
On 11/13/2015 07:14 PM, Andreas Färber wrote: > Am 13.10.2015 um 14:37 schrieb Daniel P. Berrange: >> From: Pavel Fedin >> >> ARM GICv3 systems with large number of CPUs create lots of IRQ pins. Since >> every pin is represented as a property, number of these properties becomes >> very large. Ever

Re: [Qemu-devel] [PATCH v2 1/1] target-ppc: Implement rtas_get_sysparm(PROCESSOR_MODULE_INFO)

2015-11-13 Thread Sukadev Bhattiprolu
Thomas Huth [th...@redhat.com] wrote: | On 10/11/15 05:22, Sukadev Bhattiprolu wrote: | [...] | > | > +static int file_read_buf(char *file_name, char *buf, int len) | > | > +{ | > | > +int rc; | > | > +FILE *fp; | > | > + | > | > +fp = fopen(file_name, "r"); | > | > +if (!fp) { | >

Re: [Qemu-devel] [PATCH v2 1/1] target-ppc: Implement rtas_get_sysparm(PROCESSOR_MODULE_INFO)

2015-11-13 Thread Sukadev Bhattiprolu
David Gibson [da...@gibson.dropbear.id.au] wrote: | On Mon, Nov 09, 2015 at 08:22:32PM -0800, Sukadev Bhattiprolu wrote: | > David Gibson [da...@gibson.dropbear.id.au] wrote: | > | On Wed, Nov 04, 2015 at 03:06:05PM -0800, Sukadev Bhattiprolu wrote: | > | > Implement RTAS_SYSPARM_PROCESSOR_MODULE_I

[Qemu-devel] [PULL 3/6] ahci/qtest: don't use tcp sockets for migration tests

2015-11-13 Thread John Snow
Signed-off-by: John Snow Message-id: 1447108074-20609-1-git-send-email-js...@redhat.com --- tests/ahci-test.c | 25 - 1 file changed, 20 insertions(+), 5 deletions(-) diff --git a/tests/ahci-test.c b/tests/ahci-test.c index 59d387c..1bb7410 100644 --- a/tests/ahci-test.c

[Qemu-devel] [PULL 4/6] qtest/ahci: always specify image format

2015-11-13 Thread John Snow
Signed-off-by: John Snow Message-id: 1447439479-16775-2-git-send-email-js...@redhat.com --- tests/ahci-test.c | 51 +-- 1 file changed, 29 insertions(+), 22 deletions(-) diff --git a/tests/ahci-test.c b/tests/ahci-test.c index 1bb7410..9bf5f2c 1006

[Qemu-devel] [PULL 5/6] libqos: add qemu-img presence check

2015-11-13 Thread John Snow
To allow tests to optionally exercise additional tests that require the qemu-img tool that may not be present in all builds. Signed-off-by: John Snow Message-id: 1447439479-16775-3-git-send-email-js...@redhat.com --- tests/libqos/libqos.c | 28 +++- tests/libqos/libqos.h

[Qemu-devel] [PULL 1/6] atapi: add byte_count_limit helper

2015-11-13 Thread John Snow
Signed-off-by: John Snow Tested-by: Mark Cave-Ayland Message-id: 1447095959-10046-2-git-send-email-js...@redhat.com --- hw/ide/atapi.c | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/hw/ide/atapi.c b/hw/ide/atapi.c index cf0b78e..42f78fb 100644 --- a/hw/id

[Qemu-devel] [PULL 2/6] atapi: Prioritize unknown cmd error over BCL error

2015-11-13 Thread John Snow
If we don't know about the command at all, we need to prioritize that failure above the zero byte-count-limit failure. This fixes a failure in the sparc64 NetBSD 7.0 installer bootup. Reported-by: Mark Cave-Ayland Signed-off-by: John Snow Tested-by: Mark Cave-Ayland Message-id: 1447095959-1004

[Qemu-devel] [PULL 0/6] Ide patches

2015-11-13 Thread John Snow
The following changes since commit 8337c6cbc37c6b2184f41bab3eaff47d5e68012a: Update version for v2.5.0-rc0 release (2015-11-13 17:10:36 +) are available in the git repository at: https://github.com/jnsnow/qemu.git tags/ide-pull-request for you to fetch changes up to 917158dc3b22924922dc

[Qemu-devel] [PULL 6/6] qtest/ahci: use raw format when qemu-img is absent

2015-11-13 Thread John Snow
If we don't have the qemu-img tool, use the raw format for tests and skip the high-sector LBA48 tests. Signed-off-by: John Snow Message-id: 1447439479-16775-4-git-send-email-js...@redhat.com --- tests/ahci-test.c | 41 - 1 file changed, 32 insertions(+), 9

Re: [Qemu-devel] [PATCH] include/hw: field 'offset' in struct Property should be ptrdiff_t as int causes overflow

2015-11-13 Thread John Snow
On 11/13/2015 01:36 PM, Andreas Färber wrote: > Am 13.11.2015 um 19:32 schrieb John Snow: >> On 11/12/2015 12:41 PM, Andreas Färber wrote: >>> [...] Testing >>> got stuck in ahci though, investigating. >>> >>> Thanks, >>> Andreas >>> >> >> Did you ever reproduce this, or does it seem to just be a

[Qemu-devel] [QEMU PATCH] target-i386: Disable rdtscp on Opteron_G* CPU models

2015-11-13 Thread Eduardo Habkost
KVM can't virtualize rdtscp on AMD CPUs yet, so there's no point in enabling it by default on AMD CPU models, as all we are getting are confused users because of the "host doesn't support requested feature" warnings. Disable rdtscp on Opteron_G* models, but keep compatibility on pc-*-2.4 and older

Re: [Qemu-devel] [PATCH for-2.5 v2] tests: Ignore recent test binaries

2015-11-13 Thread John Snow
On 11/12/2015 10:47 PM, Eric Blake wrote: > Commits 6c6f312d and bd797fc1 added new tests (test-blockjob-txn > and test-timed-average, respectively), but did not mark them for > exclusion in .gitignore. > > Signed-off-by: Eric Blake > --- > > v2: retitle; also ignore test-timed-average > > t

Re: [Qemu-devel] [PATCH COLO-Frame v10 12/38] COLO: Save PVM state to secondary side when do checkpoint

2015-11-13 Thread Dr. David Alan Gilbert
* zhanghailiang (zhang.zhanghaili...@huawei.com) wrote: > On 2015/11/7 2:59, Dr. David Alan Gilbert wrote: > >* zhanghailiang (zhang.zhanghaili...@huawei.com) wrote: > >>The main process of checkpoint is to synchronize SVM with PVM. > >>VM's state includes ram and device state. So we will migrate P

Re: [Qemu-devel] [PATCH 09/16] target-arm: Support multiple address spaces in page table walks

2015-11-13 Thread Peter Maydell
On 9 November 2015 at 10:58, Peter Maydell wrote: > On 9 November 2015 at 10:51, Paolo Bonzini wrote: >> >> >> On 05/11/2015 19:15, Peter Maydell wrote: >>> If we have a secure address space, use it in page table walks: >>> * when doing the physical accesses to read descriptors, >>>make them

Re: [Qemu-devel] [PATCH] include/hw: field 'offset' in struct Property should be ptrdiff_t as int causes overflow

2015-11-13 Thread Andreas Färber
Am 13.11.2015 um 19:32 schrieb John Snow: > On 11/12/2015 12:41 PM, Andreas Färber wrote: >> [...] Testing >> got stuck in ahci though, investigating. >> >> Thanks, >> Andreas >> > > Did you ever reproduce this, or does it seem to just be a race? Once I updated to a later git commit I was no long

Re: [Qemu-devel] [PATCH COLO-Frame v10 17/38] COLO: synchronize PVM's state to SVM periodically

2015-11-13 Thread Dr. David Alan Gilbert
* zhanghailiang (zhang.zhanghaili...@huawei.com) wrote: > Do checkpoint periodically, the default interval is 200ms. > > Signed-off-by: zhanghailiang > Signed-off-by: Li Zhijian > --- > migration/colo.c | 14 ++ > 1 file changed, 14 insertions(+) > > diff --git a/migration/colo.c b

Re: [Qemu-devel] [PATCH] include/hw: field 'offset' in struct Property should be ptrdiff_t as int causes overflow

2015-11-13 Thread John Snow
On 11/12/2015 12:41 PM, Andreas Färber wrote: > Am 11.11.2015 um 09:54 schrieb Markus Armbruster: >> Peter Maydell writes: >>> On 25 August 2015 at 15:17, Markus Armbruster wrote: Stumbled over this while throwing away old mail. Andreas, what do you think? >>> >>> Seems right to me -

[Qemu-devel] [PATCH v2 2/3] libqos: add qemu-img presence check

2015-11-13 Thread John Snow
To allow tests to optionally exercise additional tests that require the qemu-img tool that may not be present in all builds. Signed-off-by: John Snow --- tests/libqos/libqos.c | 28 +++- tests/libqos/libqos.h | 1 + 2 files changed, 24 insertions(+), 5 deletions(-) diff

[Qemu-devel] [PATCH v2 3/3] qtest/ahci: use raw format when qemu-img is absent

2015-11-13 Thread John Snow
If we don't have the qemu-img tool, use the raw format for tests and skip the high-sector LBA48 tests. Signed-off-by: John Snow --- tests/ahci-test.c | 41 - 1 file changed, 32 insertions(+), 9 deletions(-) diff --git a/tests/ahci-test.c b/tests/ahci-test

[Qemu-devel] [PATCH v2 1/3] qtest/ahci: always specify image format

2015-11-13 Thread John Snow
Signed-off-by: John Snow --- tests/ahci-test.c | 51 +-- 1 file changed, 29 insertions(+), 22 deletions(-) diff --git a/tests/ahci-test.c b/tests/ahci-test.c index 59d387c..6d9ac84 100644 --- a/tests/ahci-test.c +++ b/tests/ahci-test.c @@ -48,6 +48

[Qemu-devel] [PATCH v2 0/3] qtest/ahci: skip qcow2 tests

2015-11-13 Thread John Snow
Skip tests that need qcow2 to function when qemu-img isn't set or otherwise present. When running under raw mode, we no longer properly test LBA48 reads/writes beyond 137Gb. ahci-test will skip past any //io//lba48//high test. While we're at it, try to make failures due to missing qemu-img binari

Re: [Qemu-devel] [PATCH v2 0/7] vl: graphics stubs + #ifdef cleanup

2015-11-13 Thread Eduardo Habkost
On Thu, Nov 12, 2015 at 07:49:51PM -0200, Eduardo Habkost wrote: > On Thu, Nov 12, 2015 at 02:17:53PM -0700, Eric Blake wrote: > > On 11/12/2015 12:02 PM, Eduardo Habkost wrote: > > > Clean up the graphics initialization code to reduce the number of > > > > > > > Lame of git for eating lines that

Re: [Qemu-devel] [Qemu-block] [PATCH V5] block/nfs: add support for setting debug level

2015-11-13 Thread Jeff Cody
On Fri, Nov 13, 2015 at 05:45:09PM +0800, Stefan Hajnoczi wrote: > On Mon, Nov 09, 2015 at 08:09:33AM +0100, Peter Lieven wrote: > > recent libnfs versions support logging debug messages. Add > > support for it in qemu through an URL parameter. > > > > Example: > > qemu -cdrom nfs://127.0.0.1/iso

Re: [Qemu-devel] [PATCH 3/3] qtest/ahci: use raw format when qemu-img is absent

2015-11-13 Thread John Snow
On 11/12/2015 02:51 PM, John Snow wrote: > If we don't have the qemu-img tool, use the raw format > for tests and skip the high-sector LBA48 tests. > > Signed-off-by: John Snow > --- > tests/ahci-test.c | 41 - > 1 file changed, 32 insertions(+), 9 delet

Re: [Qemu-devel] [PATCH v4 6/7] qom: replace object property list with GHashTable

2015-11-13 Thread Andreas Färber
Am 13.10.2015 um 14:37 schrieb Daniel P. Berrange: > From: Pavel Fedin > > ARM GICv3 systems with large number of CPUs create lots of IRQ pins. Since > every pin is represented as a property, number of these properties becomes > very large. Every property add first makes sure there's no duplicate

Re: [Qemu-devel] [PATCH v11 19/28] qapi: Change munging of CamelCase enum values

2015-11-13 Thread Markus Armbruster
Eric Blake writes: > On 11/10/2015 11:51 PM, Eric Blake wrote: >> When munging enum values, the fact that we were passing the entire >> prefix + value through camel_to_upper() meant that enum values >> spelled with CamelCase could be turned into CAMEL_CASE. However, >> this provides a potential

Re: [Qemu-devel] [PATCH] MAINTAINERS: Update TCG CPU cores section

2015-11-13 Thread Peter Crosthwaite
On Fri, Nov 13, 2015 at 9:53 AM, Paolo Bonzini wrote: > These are the people that I think have been touching it lately > or reviewing patches. > > Signed-off-by: Paolo Bonzini > --- > MAINTAINERS | 17 + > 1 file changed, 13 insertions(+), 4 deletions(-) > > diff --git a/MAINTA

[Qemu-devel] could someone else handle the "misc" patches for a few months?

2015-11-13 Thread Paolo Bonzini
Hi, I would like someone else to handle miscellaneous patches for the rest of the 2.5 release as well as for 2.6. The main thing to cover here is the build stuff, especially configure, and the documentation. I would still handle cpu-exec.c/exec.c/translate-all.c and friends, for which I've sent

[Qemu-devel] [PATCH 2/3] sun4u: split NPT and INT_DIS accesses between timer and compare registers

2015-11-13 Thread Mark Cave-Ayland
Accesses to the timer register high bit should only set NPT, whilst accesses to the timer compare register high bit should only set INT_DIS. This fixes issues with the timer being unexpectedly disabled whilst trying to boot FreeBSD SPARC64. Signed-off-by: Mark Cave-Ayland --- hw/sparc64/sun4u.c

[Qemu-devel] [PATCH 3/3] target-sparc: implement NPT timer bit

2015-11-13 Thread Mark Cave-Ayland
If the NPT bit is set in the timer register, all non-supervisor read accesses to the register should fail with a privilege exception. Signed-off-by: Mark Cave-Ayland --- target-sparc/helper.c| 10 -- target-sparc/helper.h|2 +- target-sparc/translate.c | 18 ++

[Qemu-devel] [PATCH 1/3] sun4u: split out NPT and INT_DIS into separate CPUTimer fields

2015-11-13 Thread Mark Cave-Ayland
Currently there is confusion between use of these bits for the timer and timer compare registers (while they both have the same value, the behaviour is different). Split into two separate CPUTimer fields so we can always reference the correct value. Signed-off-by: Mark Cave-Ayland --- hw/sparc64

[Qemu-devel] [PATCH 0/3] sun4u: timer fixups

2015-11-13 Thread Mark Cave-Ayland
Whilst trying to boot FreeBSD SPARC64, it became apparent that the existing timer code doesn't behave correctly with respect to the NPT and INT_DIS bits. This patchset corrects this behaviour and allows FreeBSD SPARC64 boot to progress further. Many thanks to Marius Strobl for pointing me in the

Re: [Qemu-devel] [PULL 2/2] hw/misc: Add support for ADC controller in Xilinx Zynq 7000

2015-11-13 Thread Peter Crosthwaite
On Fri, Nov 13, 2015 at 9:08 AM, Paolo Bonzini wrote: > > > On 12/11/2015 16:23, Peter Maydell wrote: >> From: Guenter Roeck >> >> Add support for the Xilinx XADC core used in Zynq 7000. >> >> References: >> - Zynq-7000 All Programmable SoC Technical Reference Manual >> - 7 Series FPGAs and Zynq-

[Qemu-devel] [PATCH] MAINTAINERS: Update TCG CPU cores section

2015-11-13 Thread Paolo Bonzini
These are the people that I think have been touching it lately or reviewing patches. Signed-off-by: Paolo Bonzini --- MAINTAINERS | 17 + 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 9e1fa72..f67a026 100644 --- a/MAINTAINERS +++ b

Re: [Qemu-devel] [PATCH v1 0/2] hw/microblaze: Set MicroBlaze CPU version

2015-11-13 Thread Peter Crosthwaite
On Fri, Nov 13, 2015 at 9:00 AM, Edgar E. Iglesias wrote: > From: "Edgar E. Iglesias" > > Hi, > > This series avoids runtime warnings when using the > petalogix s3adsp1800 and ml605 microblaze boards by > explicitly selecting the MicroBlaze CPU versions. > > The versions match the DTBs we ship. >

Re: [Qemu-devel] [PATCH v11 19/28] qapi: Change munging of CamelCase enum values

2015-11-13 Thread Eric Blake
On 11/10/2015 11:51 PM, Eric Blake wrote: > When munging enum values, the fact that we were passing the entire > prefix + value through camel_to_upper() meant that enum values > spelled with CamelCase could be turned into CAMEL_CASE. However, > this provides a potential collision (both OneTwo and

[Qemu-devel] [PATCH] crypto: avoid two coverity false positive error reports

2015-11-13 Thread Daniel P. Berrange
In qcrypto_tls_creds_get_path() coverity complains that we are checking '*creds' for NULL, despite having dereferenced it previously. This is harmless bug due to fact that the trace call was too early. Moving it after the cleanup gets the desired semantics. In qcrypto_tls_creds_check_cert_key_purp

Re: [Qemu-devel] [PATCH QEMU v3 2/2] xen: fix usage of xc_domain_create in domain builder

2015-11-13 Thread Stefano Stabellini
On Fri, 13 Nov 2015, Roger Pau Monne wrote: > Due to the addition of HVMlite and the requirement to always provide a valid > xc_domain_configuration_t, xc_domain_create now always takes an arch domain > config, which can be NULL in order to mimic previous behaviour. > > Add a small stub called xen

Re: [Qemu-devel] [PATCH] vfio: Fix handling VFIO_IOMMU_GET_INFO results

2015-11-13 Thread Alex Williamson
On Fri, 2015-11-13 at 12:33 +0300, Pavel Fedin wrote: > Hello! > > > > If we fix qemu, it will automatically start working with all > > > available kernels which are there in the wild. If we fix kernel, older > > > versions will still not work, however they can. > > > That's why i think that we

Re: [Qemu-devel] [PATCH V4 0/6] ide: avoid main-loop hang on CDROM/NFS failure

2015-11-13 Thread John Snow
On 11/13/2015 06:13 AM, Mark Cave-Ayland wrote: > On 12/11/15 16:30, Peter Lieven wrote: > >> This series aims at avoiding a hanging main-loop if a vserver has a >> CDROM image mounted from a NFS share and that NFS share goes down. >> Typical situation is that users mount an CDROM ISO to install

Re: [Qemu-devel] [PATCH v2 3/4] usb-mtp: Add support for inotify based file monitoring

2015-11-13 Thread Bandan Das
Gerd Hoffmann writes: > Hi, > >> +switch (mask) { >> +case IN_CREATE: >> +if (event->mask & IN_ISDIR) { >> +/* >> + * Add the watchpoint first so we >> + * don't miss events in this subdir >> +

Re: [Qemu-devel] [PULL v2 5/7] aio: Introduce aio-epoll.c

2015-11-13 Thread Paolo Bonzini
On 09/11/2015 11:08, Stefan Hajnoczi wrote: > From: Fam Zheng > > To minimize code duplication, epoll is hooked into aio-posix's > aio_poll() instead of rolling its own. This approach also has both > compile-time and run-time switchability. > > 1) When QEMU starts with a small number of fds in

Re: [Qemu-devel] [PULL 2/2] hw/misc: Add support for ADC controller in Xilinx Zynq 7000

2015-11-13 Thread Paolo Bonzini
On 12/11/2015 16:23, Peter Maydell wrote: > From: Guenter Roeck > > Add support for the Xilinx XADC core used in Zynq 7000. > > References: > - Zynq-7000 All Programmable SoC Technical Reference Manual > - 7 Series FPGAs and Zynq-7000 All Programmable SoC XADC > Dual 12-Bit 1 MSPS Analog-to-

Re: [Qemu-devel] [PULL v3 00/15] Tracing patches

2015-11-13 Thread Edgar E. Iglesias
On Fri, Nov 13, 2015 at 11:28:41AM +, Peter Maydell wrote: > On 13 November 2015 at 10:30, Peter Maydell wrote: > > I also now get a handful of extra warnings in the 'make check' output: > > > > /aarch64/qom/xilinx-zynq-a9: OK > > /aarch64/qom/xlnx-e

[Qemu-devel] [PATCH v1 2/2] petalogix-ml605: Set the MicroBlaze CPU version to 8.10.a

2015-11-13 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Set the MicroBlaze CPU version to 8.10.a avoiding a runtime warning due to an unset CPU version. Signed-off-by: Edgar E. Iglesias --- hw/microblaze/petalogix_ml605_mmu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/microblaze/petalogix_ml605_mmu.c b/hw/micr

[Qemu-devel] [PATCH v1 1/2] s3adsp1800: Set the MicroBlaze CPU version to 7.10.d

2015-11-13 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Set the MicroBlaze CPU version to 7.10.d avoiding a runtime warning due to an unset CPU version. Signed-off-by: Edgar E. Iglesias --- hw/microblaze/petalogix_s3adsp1800_mmu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/microblaze/petalogix_s3adsp1800_mmu.c

[Qemu-devel] [PATCH v1 0/2] hw/microblaze: Set MicroBlaze CPU version

2015-11-13 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" Hi, This series avoids runtime warnings when using the petalogix s3adsp1800 and ml605 microblaze boards by explicitly selecting the MicroBlaze CPU versions. The versions match the DTBs we ship. Cheers, Edgar Edgar E. Iglesias (2): s3adsp1800: Set the MicroBlaze CPU

Re: [Qemu-devel] [PATCH COLO-Frame v10 18/38] COLO failover: Introduce a new command to trigger a failover

2015-11-13 Thread Eric Blake
On 11/03/2015 04:56 AM, zhanghailiang wrote: > We leave users to choose whatever heartbeat solution they want, if the > heartbeat > is lost, or other errors they detect, they can use experimental command > 'x_colo_lost_heartbeat' to tell COLO to do failover, COLO will do operations > accordingly.

Re: [Qemu-devel] [PATCH v7 23/35] nvdimm: implement NVDIMM device abstract

2015-11-13 Thread Vladimir Sementsov-Ogievskiy
On 02.11.2015 12:13, Xiao Guangrong wrote: Introduce "nvdimm" device which is based on dimm device type 128K memory region which is the minimum namespace label size required by NVDIMM Namespace Spec locates at the end of backend memory device is reserved for label data We can use "-m 1G,maxmem=

Re: [Qemu-devel] [PATCH COLO-Frame v10 10/38] COLO: Add a new RunState RUN_STATE_COLO

2015-11-13 Thread Eric Blake
On 11/03/2015 04:56 AM, zhanghailiang wrote: > Guest will enter this state when paused to save/restore VM state > under colo checkpoint. > > Cc: Eric Blake > Cc: Markus Armbruster > Signed-off-by: zhanghailiang > Signed-off-by: Li Zhijian > Signed-off-by: Gonglei > Reviewed-by: Dr. David Alan

[Qemu-devel] [PATCH QEMU v3 2/2] xen: fix usage of xc_domain_create in domain builder

2015-11-13 Thread Roger Pau Monne
Due to the addition of HVMlite and the requirement to always provide a valid xc_domain_configuration_t, xc_domain_create now always takes an arch domain config, which can be NULL in order to mimic previous behaviour. Add a small stub called xen_domain_create that encapsulates the correct call to x

Re: [Qemu-devel] [PATCH COLO-Frame v10 09/38] COLO: Implement colo checkpoint protocol

2015-11-13 Thread Eric Blake
On 11/03/2015 04:56 AM, zhanghailiang wrote: > We need communications protocol of user-defined to control the checkpoint > process. > > The new checkpoint request is started by Primary VM, and the interactive > process > like below: > Checkpoint synchronizing points, > >P

Re: [Qemu-devel] [PATCH] net: convert qemu_log to error_report

2015-11-13 Thread Thomas Huth
On 13/11/15 13:58, Paolo Bonzini wrote: > Ensure that the error is printed with the proper timestamp. > > Signed-off-by: Paolo Bonzini > --- > net/dump.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/net/dump.c b/net/dump.c > index dd0555f..ab71ffc 100644 > --- a/net/

Re: [Qemu-devel] [PATCH COLO-Frame v10 05/38] migration: Integrate COLO checkpoint process into migration

2015-11-13 Thread Eric Blake
On 11/03/2015 04:56 AM, zhanghailiang wrote: > Add a migrate state: MIGRATION_STATUS_COLO, enter this migration state > after the first live migration successfully finished. > > We reuse migration thread, so if colo is enabled by user, migration thread > will > go into the process of colo. > > S

Re: [Qemu-devel] [PATCH COLO-Frame v10 16/38] COLO: Flush PVM's cached RAM into SVM's memory

2015-11-13 Thread Dr. David Alan Gilbert
* zhanghailiang (zhang.zhanghaili...@huawei.com) wrote: > During the time of VM's running, PVM may dirty some pages, we will transfer > PVM's dirty pages to SVM and store them into SVM's RAM cache at next > checkpoint > time. So, the content of SVM's RAM cache will always be some with PVM's memory

Re: [Qemu-devel] [PATCH COLO-Frame v10 15/38] ram/COLO: Record pages received from PVM by re-using migration dirty bitmap

2015-11-13 Thread Dr. David Alan Gilbert
* zhanghailiang (zhang.zhanghaili...@huawei.com) wrote: > We need to record the address of the dirty pages that received from PVM, > It will help flushing pages that cached into SVM. > > Signed-off-by: zhanghailiang > --- > v10: > - New patch split from v9's patch 13 > - Rebase to master to use '

Re: [Qemu-devel] [PATCH for-2.5] mac_dbdma: always initialize channel field in DBDMA_channel

2015-11-13 Thread Mark Cave-Ayland
On 12/11/15 21:24, Hervé Poussineau wrote: > dbdma_from_ch() uses channel field to return the right DBDMA object. > Previous code was working if guest OS was only using registered DMA channels. > However, it lead to QEMU crashes if guest OS was using unregistered DMA > channels. > > Signed-off-b

Re: [Qemu-devel] [PATCH COLO-Frame v10 14/38] COLO: Load VMState into qsb before restore it

2015-11-13 Thread Dr. David Alan Gilbert
* zhanghailiang (zhang.zhanghaili...@huawei.com) wrote: > We should not destroy the state of SVM (Secondary VM) until we receive the > whole > state from the PVM (Primary VM), in case the primary fails in the middle of > sending > the state, so, here we cache the device state in Secondary before

Re: [Qemu-devel] [PATCH COLO-Frame v10 02/38] migration: Introduce capability 'x-colo' to migration

2015-11-13 Thread Eric Blake
On 11/03/2015 04:56 AM, zhanghailiang wrote: > We add helper function colo_supported() to indicate whether > colo is supported or not, with which we use to control whether or not > showing 'x-colo' string to users, they can use qmp command > 'query-migrate-capabilities' or hmp command 'info migrate

Re: [Qemu-devel] [PATCH v2 1/7] vl: Add DT_COCOA DisplayType value

2015-11-13 Thread Andreas Färber
Am 12.11.2015 um 20:02 schrieb Eduardo Habkost: > Instead of reusing DT_SDL for Cocoa, use DT_COCOA to indicate > that a Cocoa display was requested. > > configure already ensures CONFIG_COCOA and CONFIG_SDL are never > set at the same time. The only case where DT_SDL is used outside > a #ifdef CO

Re: [Qemu-devel] [PATCH v4 0/5] handle vhost reset/start/stop correctly

2015-11-13 Thread Yuanhan Liu
On Fri, Nov 13, 2015 at 12:24:52PM +0200, Michael S. Tsirkin wrote: > On Fri, Nov 13, 2015 at 10:03:29AM +0800, Yuanhan Liu wrote: > > On Thu, Nov 12, 2015 at 04:44:19PM +0200, Michael S. Tsirkin wrote: > > > On Thu, Nov 12, 2015 at 10:08:15PM +0800, Yuanhan Liu wrote: > > > > On Thu, Nov 12, 2015

Re: [Qemu-devel] [PATCH COLO-Frame v10 13/38] COLO: Load PVM's dirty pages into SVM's RAM cache temporarily

2015-11-13 Thread Dr. David Alan Gilbert
* zhanghailiang (zhang.zhanghaili...@huawei.com) wrote: > We should not load PVM's state directly into SVM, because there maybe some > errors happen when SVM is receving data, which will break SVM. > > We need to ensure receving all data before load the state into SVM. We use > an extra memory to

Re: [Qemu-devel] [PATCH 1/3] monitor: avoid clang shifting negative signed warning

2015-11-13 Thread Eduardo Habkost
On Tue, Nov 10, 2015 at 03:57:33PM +, Stefan Hajnoczi wrote: > clang 3.7.0 on x86_64 warns about the following: > > target-i386/monitor.c:38:22: warning: shifting a negative signed value is > undefined [-Wshift-negative-value] > addr |= -1LL << 48; > ^ > > Sign

[Qemu-devel] block/vpc: Fix vhd extra sectors issue

2015-11-13 Thread Lucian Petrut
At the moment, qemu-img extends new image virtual sizes based on the CHS algorithm provided by the VHD specs in order to ensure that the disk geometry (and payload as seen by some guests which use the CHS value) can fit in the requested disk. This patch drops this behavior, as it breaks compatibi

Re: [Qemu-devel] [PATCH 3/3] virtio: optimize virtio_access_is_big_endian() for little-endian targets

2015-11-13 Thread Greg Kurz
On Fri, 13 Nov 2015 15:42:53 +0100 Cornelia Huck wrote: > On Fri, 13 Nov 2015 10:28:31 +0100 > Greg Kurz wrote: > > > On Thu, 12 Nov 2015 19:08:59 +0100 > > Cornelia Huck wrote: > > > > > On Mon, 09 Nov 2015 18:58:34 +0100 > > > Greg Kurz wrote: > > > > > > > When adding cross-endian suppor

Re: [Qemu-devel] [PATCH v3 1/3] target-i386: add a subsection for migrating vcpu's TSC rate

2015-11-13 Thread Eduardo Habkost
On Fri, Nov 13, 2015 at 10:23:54AM +0800, Haozhong Zhang wrote: > On 11/11/15 22:27, Haozhong Zhang wrote: > > On 11/11/15 12:16, Eduardo Habkost wrote: > [...] > > > > diff --git a/hw/i386/pc_q35.c b/hw/i386/pc_q35.c > > > > index 2f8f396..858ed69 100644 > > > > --- a/hw/i386/pc_q35.c > > > > +++

Re: [Qemu-devel] [PATCH] nand: fix flash erase when oob is in memory

2015-11-13 Thread Ricard Wanderlof
On Fri, 13 Nov 2015, Ricard Wanderlof wrote: > For the "main area on file, oob in memory" case, fix the shifts so that > we erase the correct number of pages. > ... Sorry about the duplicate post. Initially used the wrong email address when subscribing so when I subscribed with the correct on

  1   2   >