From: Wenchao Xia xiaw...@linux.vnet.ibm.com
This case can't run when IMGPROTO=nbd, since it needs to create some
internal snapshot which would fail for EOF write request, even when
TEST_IMG is exported with -f raw in common.rc, so set _supported_proto
to file.
_require_command() is changed to
From: Fam Zheng f...@redhat.com
Format raw doesn't always work on certain file systems (e.g. tmpfs).
Use qcow2 to make the allocation status explicit and split into a new
case.
[Resolved merge conflict due to qemu-io prompt filter, added 074 to
group file, and fixed up s/048/074/ copy-paste
06.12.2013 16:52, Alexander Graf wrote:
When we're running in non-64bit mode with qemu-system-x86_64 we can
still end up with virtual addresses that are above the 32bit boundary
if a segment offset is set up.
GNU Hurd does exactly that. It sets the segment offset to 0x8000 and
puts its
It will allow to reuse field in different BUSes, reducing code duplication.
Field is intended fot replacing 'hotplug_qdev' field in PCIBus and also
will allow not to add equivalent field to DimmBus with possiblitity
to refactor other BUSes to use it instead of custom field.
Signed-off-by: Igor
This patch changes bdrv_co_do_pwritev() to actually be what its name
promises. If requests aren't properly aligned, it performs a RMW.
Requests touching the same block are serialised against the RMW request.
Further optimisation of this is possible by differentiating types of
requests (concurrent
SMTP does not preserve newlines. This is normally not a problem if the
email body uses DOS or UNIX newlines consistently. In 051.out we mix
UNIX newlines with DOS newlines (since QEMU monitor output uses \r\n).
This patch filters the QEMU monitor output so the golden master file
uses UNIX
From: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Reviewed-by: Peter Lieven p...@kamp.de
Signed-off-by: Stefan Hajnoczi stefa...@redhat.com
---
block/qcow2.c | 2 ++
block/qed.c | 2 ++
block/vdi.c | 1 +
block/vhdx.c | 3 +++
block/vpc.c | 1 +
5
On Fri, Dec 06, 2013 at 09:23:22AM -0800, H. Peter Anvin wrote:
On 12/06/2013 05:46 AM, Borislav Petkov wrote:
I'm guessing this and the struct lwp_struct above is being added so that
you can have the LWP XSAVE area size? If so, you don't need it: LWP
XSAVE area is 128 bytes at offset 832
Am 06.12.2013 19:48, schrieb Michael Tokarev:
06.12.2013 16:52, Alexander Graf wrote:
When we're running in non-64bit mode with qemu-system-x86_64 we can
still end up with virtual addresses that are above the 32bit boundary
if a segment offset is set up.
GNU Hurd does exactly that. It sets
From: Paolo Bonzini pbonz...@redhat.com
Writing zeroes to a file can be done by punching a hole if
MAY_UNMAP is set.
Note that in this case ENOTSUP is not ignored, but makes
the block layer fall back to the generic implementation.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by:
04.12.2013 10:27, Antony Pavlov wrote:
scripts/checkpatch.pl reports about some style problems,
this commit fixes some of them:
Thanks, applied to trivial-patches queue.
I'm a bit afraid of this sort of patches - there are LOTS of
coding style issues in current qemu code base, and applying
a
From: Fam Zheng f...@redhat.com
This replaces _unsupported_qemu_io_options and check for support of
current cache mode, and allow to provide a default if user didn't
specify.
Signed-off-by: Fam Zheng f...@redhat.com
Reviewed-by: Wenchao Xia xiaw...@linux.vnet.ibm.com
Signed-off-by: Stefan
06.12.2013 19:08, Paolo Bonzini wrote:
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -2089,7 +2089,7 @@ ETEXI
DEF(iscsi, HAS_ARG, QEMU_OPTION_iscsi,
-iscsi [user=user][,password=password]\n
[,header-digest=CRC32C|CR32C-NONE|NONE-CRC32C|NONE\n
-
Series is replacement for a more simple attempt to generalize hotplug API
http://lists.gnu.org/archive/html/qemu-devel/2013-11/msg02529.html
promted by Paolo in this patch review.
--
Refactor PCI specific hotplug API to a more generic/reusable one.
Model it after SCSI-BUS like hotplug API
On 12/06/2013 10:22 AM, Kevin Wolf wrote:
For an O_DIRECT request to succeed, it's not only necessary that all
base addresses in the qiov are aligned, but also that each lengh in it
s/lengh/length/
is aligned.
Signed-off-by: Kevin Wolf kw...@redhat.com
---
block.c | 3 +++
1 file
06.12.2013 22:43, Stefan Weil wrote:
The memory region can be included by value instead of by reference in the
device state (like it is done in other SoCs).
Signed-off-by: Stefan Weil s...@weilnetz.de
---
hw/arm/highbank.c |7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
From: Stefan Weil s...@weilnetz.de
This reduces the dependencies on trace.h.
Only one source file which needs hcd-ehci.h also needs trace.h.
Signed-off-by: Stefan Weil s...@weilnetz.de
Signed-off-by: Michael Tokarev m...@tls.msk.ru
---
hw/usb/hcd-ehci.c |1 +
hw/usb/hcd-ehci.h |1 -
2
From: Stefan Weil s...@weilnetz.de
Signed-off-by: Stefan Weil s...@weilnetz.de
Reviewed-by: Gerd Hoffmann kra...@redhat.com
Signed-off-by: Michael Tokarev m...@tls.msk.ru
---
trace-events |2 ++
ui/console.c | 11 +++
2 files changed, 5 insertions(+), 8 deletions(-)
diff --git
Here's the first trivial-patches pull request after 1.7.
It contains the few patches which were requested for 1.7
but hasn't been applied, plus a few more patches which
weren't supposed to be included in 1.7.
Please pull.
Thanks,
/mjt
The following changes since commit
From: Stefan Weil s...@weilnetz.de
This reduces the dependencies on trace.h.
Only two source files which need console.h also need trace.h.
Signed-off-by: Stefan Weil s...@weilnetz.de
Signed-off-by: Michael Tokarev m...@tls.msk.ru
---
hw/display/vmware_vga.c |1 +
include/ui/console.h|
From: Fam Zheng f...@redhat.com
Signed-off-by: Fam Zheng f...@redhat.com
Reviewed-by: Stefan Weil s...@weilnetz.de
Signed-off-by: Michael Tokarev m...@tls.msk.ru
---
.gitignore |1 +
1 file changed, 1 insertion(+)
diff --git a/.gitignore b/.gitignore
index 5584b5f..1c9d63d 100644
---
From: Stefan Weil s...@weilnetz.de
Signed-off-by: Stefan Weil s...@weilnetz.de
Signed-off-by: Michael Tokarev m...@tls.msk.ru
---
trace-events |5 +
ui/gtk.c | 19 +--
2 files changed, 10 insertions(+), 14 deletions(-)
diff --git a/trace-events b/trace-events
index
Provide generic hotplug interface for devices.
Intended for replacing hotplug mechanism used by
PCI/PCIE/SHPC code.
Signed-off-by: Igor Mammedov imamm...@redhat.com
---
it's scsi-bus like interface, but abstracted from bus altogether
since all current users care about in hotplug handlers, it's
From: Stefan Weil s...@weilnetz.de
Most code already used QEMUTimer without the redundant 'struct' keyword.
Signed-off-by: Stefan Weil s...@weilnetz.de
Reviewed-by: Andreas Färber afaer...@suse.de
Signed-off-by: Michael Tokarev m...@tls.msk.ru
---
hw/char/cadence_uart.c |4 ++--
From: Antony Pavlov antonynpav...@gmail.com
scripts/checkpatch.pl reports about some style problems,
this commit fixes some of them:
ERROR: space prohibited before open square bracket '['
+.fields = (VMStateField []) {
ERROR: space prohibited after that '!' (ctx:BxW)
+if (!
From: Stefan Weil s...@weilnetz.de
Warning from ccc-analyzer:
libcacard/cac.c:192:13: warning: Value stored to 'ret' is never read
ret = VCARD_DONE;
^ ~~
Here 'ret' is assigned a value inside of a switch statement and also after
that switch statement.
From: Don Koch dk...@verizon.com
Prevent a call to put_kbd if null.
On shutdown of some OSes, the keyboard handler goes away before the
system is down. If a key is typed during this window, qemu crashes.
Signed-off-by: Don Koch dk...@verizon.com
Signed-off-by: Michael Tokarev m...@tls.msk.ru
From: Stefan Weil s...@weilnetz.de
Warning from ccc-analyzer:
libcacard/vcard_emul_nss.c:937:9: warning:
Value stored to 'cert_count' is never read
cert_count = options-vreader[i].cert_count;
^~~
Signed-off-by: Stefan Weil
On 6 December 2013 19:12, Michael Tokarev m...@tls.msk.ru wrote:
06.12.2013 22:43, Stefan Weil wrote:
The memory region can be included by value instead of by reference in the
device state (like it is done in other SoCs).
Signed-off-by: Stefan Weil s...@weilnetz.de
---
hw/arm/highbank.c |
From: Stefan Weil s...@weilnetz.de
gcc 4.8.2 reports this warning when extra warnings are enabled (-Wextra):
CCm68k-softmmu/hw/m68k/mcf5206.o
hw/i386/acpi-build.c: In function ‘build_append_nameseg’:
hw/i386/acpi-build.c:294:5: error:
function might be possible candidate for ‘gnu_printf’
From: Zhi Yong Wu wu...@linux.vnet.ibm.com
Signed-off-by: Zhi Yong Wu wu...@linux.vnet.ibm.com
Reviewed-by: Fam Zheng f...@redhat.com
Signed-off-by: Michael Tokarev m...@tls.msk.ru
---
hw/net/virtio-net.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/net/virtio-net.c
From: Stefan Weil s...@weilnetz.de
gcc 4.8.2 reports this warning when extra warnings are enabled (-Wextra):
CCqobject/qerror.o
qobject/qerror.c: In function ‘qerror_from_info’:
qobject/qerror.c:53:5: error:
function might be possible candidate for ‘gnu_printf’ format attribute
This definition has been moved to prep.c.
Signed-off-by: Michael Tokarev m...@tls.msk.ru
---
hw/ppc/mac.h |1 -
1 file changed, 1 deletion(-)
diff --git a/hw/ppc/mac.h b/hw/ppc/mac.h
index 1e578dd..c1faf9c 100644
--- a/hw/ppc/mac.h
+++ b/hw/ppc/mac.h
@@ -34,7 +34,6 @@
#define MAX_CPUS 1
On Fri, Dec 06, 2013 at 10:06:37AM +0100, Paolo Bonzini wrote:
Il 04/12/2013 08:58, Wanlong Gao ha scritto:
According to this patch set, we are able to set guest nodes memory policy
like following:
-numa node,nodeid=0,cpus=0, \
-numa mem,size=1024M,policy=membind,host-nodes=0-1 \
Paolo Bonzini wrote:
Il 07/12/2013 01:20, Qiaowei Ren ha scritto:
This patch defines xstate feature and extends struct xsave_hdr_struct
to support Intel MPX.
Signed-off-by: Qiaowei Ren qiaowei@intel.com
Signed-off-by: Xudong Hao xudong@intel.com
Signed-off-by: Liu Jinsong
Il 06/12/2013 20:04, Michael Tokarev ha scritto:
06.12.2013 19:08, Paolo Bonzini wrote:
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -2089,7 +2089,7 @@ ETEXI
DEF(iscsi, HAS_ARG, QEMU_OPTION_iscsi,
-iscsi [user=user][,password=password]\n
Anthony,
The following changes since commit 7dc65c02fe3fb8f3146ce0b9ff5fec5945329f0e:
Open 2.0 development tree (2013-11-27 14:02:45 -0800)
are available in the git repository at:
git://github.com/awilliam/qemu-vfio.git tags/vfio-pci-for-qemu-20131206.0
for you to fetch changes up to
We were relying on msix_unset_vector_notifiers() to release all the
vectors when we disable MSI-X, but this only happens when MSI-X is
still enabled on the device. Perform further cleanup by releasing
any remaining vectors listed as in-use after this call. This caused
a leak of IRQ routes on
When MSI is enabled on Nvidia GeForce cards the driver seems to
acknowledge the interrupt by writing a 0xff byte to the MSI capability
ID register using the PCI config space mirror at offset 0x88000 from
BAR0. Without this, the device will only fire a single interrupt.
VFIO handles the PCI
EDIT: I should have mentioned that this is using QEMU 1.6.1, but the
problem also occurs with 1.3.1. Also, in case it makes a difference, I
installed QEMU using Homebrew.
** Description changed:
Whenever I run ``qemu-system-i386 -curses ...'' (with or without a ``-k
en-gb'') on OS X 10.9,
Public bug reported:
Whenever I run ``qemu-system-i386 -curses ...'' (with or without a ``-k
en-gb'') on OS X 10.9, the keyboard does not work properly. For example,
when attempting to switch to the QEMU console with Alt+2, I get:
``Warning: no scancode found for keysym 226
Warning: no scancode
Add and remove groups from the KVM virtual VFIO device as we make
use of them. This allows KVM to optimize for performance and
correctness based on properties of the group.
Signed-off-by: Alex Williamson alex.william...@redhat.com
---
hw/misc/vfio.c | 67
It's sometimes useful to be able to verify interrupts are passing
through correctly.
Signed-off-by: Alex Williamson alex.william...@redhat.com
---
hw/misc/vfio.c | 24
1 file changed, 20 insertions(+), 4 deletions(-)
diff --git a/hw/misc/vfio.c b/hw/misc/vfio.c
index
Update to tag v3.13-rc3 (374b105797c3d4f29c685f3be535c35f5689b30e)
Signed-off-by: Alex Williamson alex.william...@redhat.com
---
linux-headers/asm-arm/kvm.h |3 +
linux-headers/asm-powerpc/epapr_hcalls.h |4 +
linux-headers/asm-powerpc/kvm.h | 86
On 12/06/2013 12:05 PM, Liu, Jinsong wrote:
Since Peter already said the same, please undo these changes.
Also, how is XSTATE_EAGER used? Should MPX be disabled when xsaveopt
is disabled on the kernel command line? (Liu, how would this affect
the KVM patches, too?)
Paolo
Currently
Don't duplicate the array length computation in the memset
when plain sizeof can produce the correct results.
Signed-off-by: Richard Henderson r...@twiddle.net
---
cputlb.c| 2 +-
translate-all.c | 5 ++---
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/cputlb.c
The size of tlb_table is 4k on a 64-bit host. For overwriting
memory at this size, cacheline tricks can help.
Signed-off-by: Richard Henderson r...@twiddle.net
---
cputlb.c | 19 ++-
1 file changed, 2 insertions(+), 17 deletions(-)
diff --git a/cputlb.c b/cputlb.c
index
Il 06/12/2013 21:48, Alex Williamson ha scritto:
/* Extra debugging, trap acceleration paths for more logging */
#define VFIO_ALLOW_MMAP 1
#define VFIO_ALLOW_KVM_INTX 1
+#define VFIO_ALLOW_KVM_MSI 1
+#define VFIO_ALLOW_KVM_MSIX 1
Why not make these device properties instead?
Paolo
H. Peter Anvin wrote:
On 12/06/2013 12:05 PM, Liu, Jinsong wrote:
Since Peter already said the same, please undo these changes.
Also, how is XSTATE_EAGER used? Should MPX be disabled when
xsaveopt is disabled on the kernel command line? (Liu, how would
this affect the KVM patches, too?)
Paolo Bonzini pbonz...@redhat.com writes:
Resetting should be done in post-order, not pre-order. However,
qdev_walk_children and qbus_walk_children do not allow this. Fix
it by adding two extra arguments to the functions.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
---
This patch defines xstate feature and extends struct xsave_hdr_struct
to support Intel MPX.
Signed-off-by: Qiaowei Ren qiaowei@intel.com
Signed-off-by: Xudong Hao xudong@intel.com
Signed-off-by: Liu Jinsong jinsong@intel.com
---
arch/x86/include/asm/processor.h | 12
-Original Message-
From: Liu, Jinsong
Sent: Saturday, December 07, 2013 6:13 AM
To: H. Peter Anvin; Paolo Bonzini; Ren, Qiaowei
Cc: k...@vger.kernel.org; x...@kernel.org; Xudong Hao;
qemu-devel@nongnu.org; linux-ker...@vger.kernel.org; Ingo Molnar; Thomas
Gleixner
Subject: RE:
Retain the existing gen_aa32_* inlines, to aid compilation for A64.
Cc: Peter Maydell peter.mayd...@linaro.org
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-arm/translate.c | 57 ++
1 file changed, 25 insertions(+), 32 deletions(-)
Signed-off-by: Antony Pavlov antonynpav...@gmail.com
---
target-arm/cpu-qom.h | 1 +
target-arm/cpu.c | 13 +
2 files changed, 14 insertions(+)
diff --git a/target-arm/cpu-qom.h b/target-arm/cpu-qom.h
index b55306a..116320c 100644
--- a/target-arm/cpu-qom.h
+++
[RFC 1/2] ARM: cpu: add hivecs property (high vectors on reset)
[RFC 2/2] ARM: arm_cpu_reset: make possible to use high vectors for
On Sat, Dec 7, 2013 at 10:55 AM, Antony Pavlov antonynpav...@gmail.com wrote:
Signed-off-by: Antony Pavlov antonynpav...@gmail.com
---
target-arm/cpu-qom.h | 1 +
target-arm/cpu.c | 13 +
2 files changed, 14 insertions(+)
diff --git a/target-arm/cpu-qom.h
ping
http://patchwork.ozlabs.org/patch/295442/
From: Andreas Färber [afaer...@suse.de]
Sent: Friday, November 29, 2013 6:04 PM
To: Petar Jovanovic; qemu-devel@nongnu.org
Cc: Petar Jovanovic; aurel...@aurel32.net; qemu-stable
Subject: Re: [Qemu-devel]
On Fri, 06 Dec 2013 22:59:29 +0400
Michael Tokarev m...@tls.msk.ru wrote:
04.12.2013 10:27, Antony Pavlov wrote:
scripts/checkpatch.pl reports about some style problems,
this commit fixes some of them:
Thanks, applied to trivial-patches queue.
I'm a bit afraid of this sort of patches -
On Sat, Dec 7, 2013 at 10:55 AM, Antony Pavlov antonynpav...@gmail.com wrote:
Signed-off-by: Antony Pavlov antonynpav...@gmail.com
---
target-arm/cpu.c | 4
1 file changed, 4 insertions(+)
diff --git a/target-arm/cpu.c b/target-arm/cpu.c
index f838499..6f548c7 100644
---
-Original Message-
From: H. Peter Anvin [mailto:h...@zytor.com]
Sent: Saturday, December 07, 2013 9:07 AM
To: Ren, Qiaowei; Liu, Jinsong; Paolo Bonzini
Cc: k...@vger.kernel.org; x...@kernel.org; Xudong Hao;
qemu-devel@nongnu.org; linux-ker...@vger.kernel.org; Ingo Molnar; Thomas
On 12/06/2013 05:16 PM, Ren, Qiaowei wrote:
Jinsong think that both kvm and host depend on these feature definition
header file, so we firstly submit these files depended on.
Yes, but we can't turn on the feature without proper protection. Either
way, they are now in tip:x86/cpufeature.
Am 29.11.2013 11:17, schrieb Paolo Bonzini:
This series fixes hot-unplug of virtio devices, which can crash due to
dangling pointer accesses.
The current implementation supports guest-initiated hot-unplug via the
virtio_bus_destroy_device function, but not hot-unplugging the virtio
device
Cc: Michael Walle mich...@walle.cc
Signed-off-by: Richard Henderson r...@twiddle.net
---
target-lm32/translate.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/target-lm32/translate.c b/target-lm32/translate.c
index 6ea0ecd..3d2f2e8 100644
---
On Sat, Dec 7, 2013 at 5:24 AM, Peter Maydell peter.mayd...@linaro.org wrote:
On 6 December 2013 19:12, Michael Tokarev m...@tls.msk.ru wrote:
06.12.2013 22:43, Stefan Weil wrote:
The memory region can be included by value instead of by reference in the
device state (like it is done in other
201 - 264 of 264 matches
Mail list logo