Signed-off-by: Wanlong Gao
---
Makefile.target | 2 +-
cpus.c | 14
include/sysemu/cpus.h | 1 -
include/sysemu/sysemu.h | 3 +
numa.c | 182
vl.c| 139 +---
As you know, QEMU can't direct it's memory allocation now, this may cause
guest cross node access performance regression.
And, the worse thing is that if PCI-passthrough is used,
direct-attached-device uses DMA transfer between device and qemu process.
All pages of the guest will be pinned by get_u
If the total number of the assigned numa nodes memory is not
equal to the assigned ram size, it will write the wrong data
to ACPI talb, then the guest will ignore the wrong ACPI table
and recognize all memory to one node. It's buggy, we should
check it to ensure that we write the right data to ACPI
Add qmp command query-numa to show guest NUMA information.
Reviewed-by: Luiz Capitulino
Signed-off-by: Wanlong Gao
---
numa.c | 66
qapi-schema.json | 36 +++
qmp-commands.hx | 49 +++
Set the guest numa nodes memory policies using the mbind(2)
system call node by node.
After this patch, we are able to set guest nodes memory policies
through the QEMU options, this arms to solve the guest cross
nodes memory access performance issue.
And as you all know, if PCI-passthrough is used,
Add "-numa mem," option like following as Paolo suggested:
-numa mem,nodeid=0,size=1G
This new option will make later coming memory hotplug better.
We will use the new options to specify nodes memory info,
and just remain "-numa node,mem=xx" as legacy.
Reviewed-by: Laszlo Ersek
Signed-off-
libnuma choosed 128 for MAX_NODES, so we follow libnuma here.
Signed-off-by: Wanlong Gao
---
include/sysemu/sysemu.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/sysemu/sysemu.h b/include/sysemu/sysemu.h
index 291aa6a..807619e 100644
--- a/include/sysemu/sysemu.h
+
Signed-off-by: Wanlong Gao
---
include/sysemu/sysemu.h | 3 +-
numa.c | 148 +++-
qapi-schema.json| 30 ++
vl.c| 11 +++-
4 files changed, 114 insertions(+), 78 deletions(-)
diff --git a/include/
Add the numa_info structure to contain the numa nodes memory,
VCPUs information and the future added numa nodes host memory
policies.
Reviewed-by: Eduardo Habkost
Signed-off-by: Andre Przywara
Signed-off-by: Wanlong Gao
---
hw/i386/pc.c| 12
include/sysemu/sysemu.h |
The memory policy setting format is like:
policy={default|membind|interleave|preferred}[,relative=true],host-nodes=N-N
And we are adding this setting as a suboption of "-numa mem,",
the memory policy then can be set like following:
-numa node,nodeid=0,cpus=0 \
-numa node,nodeid=1,cpus=1
Signed-off-by: Wanlong Gao
---
qapi-schema.json | 19 ++-
1 file changed, 18 insertions(+), 1 deletion(-)
diff --git a/qapi-schema.json b/qapi-schema.json
index db539b6..1043e57 100644
--- a/qapi-schema.json
+++ b/qapi-schema.json
@@ -4223,7 +4223,8 @@
##
{ 'union': 'NumaOption
Reviewed-by: Luiz Capitulino
Signed-off-by: Wanlong Gao
---
hmp.c | 57 +
hmp.h | 1 +
monitor.c | 21 +
3 files changed, 59 insertions(+), 20 deletions(-)
diff --git a/hmp.c b/hmp.c
index 32ee285..d6dedd2 1006
Signed-off-by: liguang
---
include/migration/vmstate.h |4
savevm.c| 31 +++
2 files changed, 35 insertions(+), 0 deletions(-)
diff --git a/include/migration/vmstate.h b/include/migration/vmstate.h
index 9d09e60..977cf52 100644
--- a/inc
Signed-off-by: liguang
---
default-configs/arm-softmmu.mak |2 +
hw/timer/Makefile.objs |2 +
hw/timer/allwinner-a10-pit.c | 253 ++
include/hw/timer/allwinner-a10-pit.h | 57
4 files changed, 314 insertions(+), 0 dele
Signed-off-by: liguang
---
hw/arm/Makefile.objs |2 +-
hw/arm/allwinner-a10.c | 77
include/hw/arm/allwinner-a10.h | 36 ++
3 files changed, 114 insertions(+), 1 deletions(-)
create mode 100644 hw/arm/allwinner-a1
Signed-off-by: liguang
---
hw/arm/Makefile.objs |2 +-
hw/arm/cubieboard.c | 52 ++
2 files changed, 53 insertions(+), 1 deletions(-)
create mode 100644 hw/arm/cubieboard.c
diff --git a/hw/arm/Makefile.objs b/hw/arm/Makefile.objs
index b9e5
Signed-off-by: liguang
---
hw/intc/Makefile.objs |1 +
hw/intc/allwinner-a10-pic.c | 218 +++
include/hw/intc/allwinner-a10-pic.h | 40 +++
3 files changed, 259 insertions(+), 0 deletions(-)
create mode 100644 hw/intc/allwinner-a10
lay a foundation for allwinner A10 SoC with a cortex-a8
processor, and will add more devices later.
v2: split timer and interrupt controller emulation into
their corresponding files.
v3:
1. change loader_start address
2. add 64-bit counter
3. fixup fail to clear interrup status
On Mi, 2013-12-04 at 17:02 +1000, Dave Airlie wrote:
> So I've hit a bit of a init ordering issue that I'm not sure how best to
> solve,
>
> Just some background:
> In order for the virt GPU and the UI layer (SDL or GTK etc) to
> interact properly over OpenGL use, I have created and OpenGL provid
Your cover letter lacks a proper subject, and your patches lack proper
threading.
git-send-email threads properly (unless you mess up its configuration).
On Tue, Dec 03, 2013 at 08:59:52PM +, Peter Maydell wrote:
> On 3 December 2013 20:41, Michael S. Tsirkin wrote:
> > On Tue, Dec 03, 2013 at 06:30:46PM +, Peter Maydell wrote:
> >> On 3 December 2013 16:28, Michael S. Tsirkin wrote:
> >> > const VMStateDescription vmstate_pcie_aer_log =
Il 04/12/2013 08:56, Liu, Jinsong ha scritto:
> From 256484fd75d4eb4d248e5e0f493f16182da59dc2 Mon Sep 17 00:00:00 2001
> From: Liu Jinsong
> Date: Wed, 4 Dec 2013 16:56:49 +0800
> Subject: [PATCH v2 2/2] target-i386: Intel MPX
>
> Add some MPX related definiation, and hardcode sizes and offsets
>
Il 04/12/2013 05:55, Alexey Kardashevskiy ha scritto:
> Normally the user is expected to eject DVD if it is not locked by
> the guest. eject_device() makes few checks and calls bdrv_close()
> if DVD is not in use.
>
> However it is still possible to eject DVD even if it is in use.
> For that, QEMU
Il 04/12/2013 05:55, Alexey Kardashevskiy ha scritto:
> This prevents the guest from preventing DVD medium removal when
> there is no medium.
>
> Without this, if the user has ejected a DVD, it is possible to
> have a block device with an open tray, no media but locked.
I need to check this again
Signed-off-by: Wenchao Xia
---
tests/qemu-iotests/058 | 19 ++-
tests/qemu-iotests/058.out | 12
2 files changed, 30 insertions(+), 1 deletions(-)
diff --git a/tests/qemu-iotests/058 b/tests/qemu-iotests/058
index cf50857..14584cd 100755
--- a/tests/qemu-iote
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 introduce function
bdrv_snapshot_load_tmp_by_id_or_name() that call bdrv_snapshot_load_tmp()
twice to keep old search logic. bdrv_snapshot_load_tmp_by_id_or_name() return
int to le
Now qemu-img convert have similar options as qemu-nbd for internal
snapshot.
Signed-off-by: Wenchao Xia
---
qemu-img-cmds.hx |4 ++--
qemu-img.c | 44 +++-
qemu-img.texi| 12
3 files changed, 45 insertions(+), 15 deletions(-)
Eric Blake writes:
> On 12/03/2013 01:33 PM, Igor Mammedov wrote:
>
Also, is it worth adding asserts and/or compiler annotations to require
that the Error **err argument of functions be non-NULL, to ensure that
callers are always passing either a valid destination or one of the
>>>
On Mon, Dec 2, 2013 at 4:54 AM, Chunyan Liu wrote:
> 2013/11/25 Kevin Wolf
>>
>> Am 21.11.2013 um 09:51 hat Stefan Hajnoczi geschrieben:
>> > On Thu, Nov 21, 2013 at 11:33:56AM +0800, Chunyan Liu wrote:
>> > > 2013/11/20 Stefan Hajnoczi
>> > >
>> > > > On Wed, Nov 20, 2013 at 04:50:29PM +0800, C
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 '--snapshot'
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 tip what util is missing, instead
of printing
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
---
block/snapshot.c | 18 ++
include/block/snapshot.h |8
qemu-nbd.c | 46 ++
Paolo Bonzini writes:
> Il 04/12/2013 05:55, Alexey Kardashevskiy ha scritto:
>> Normally the user is expected to eject DVD if it is not locked by
>> the guest. eject_device() makes few checks and calls bdrv_close()
>> if DVD is not in use.
>>
>> However it is still possible to eject DVD even if
Paolo Bonzini writes:
> Il 04/12/2013 05:55, Alexey Kardashevskiy ha scritto:
>> This prevents the guest from preventing DVD medium removal when
>> there is no medium.
>>
>> Without this, if the user has ejected a DVD, it is possible to
>> have a block device with an open tray, no media but lock
Signed-off-by: Wenchao Xia
---
qemu-nbd.c|1 +
qemu-nbd.texi |2 ++
2 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/qemu-nbd.c b/qemu-nbd.c
index fe6053c..136e8c9 100644
--- a/qemu-nbd.c
+++ b/qemu-nbd.c
@@ -79,6 +79,7 @@ static void usage(const char *name)
#endif
"\n
On Fri, Nov 22, 2013 at 11:00:24AM -0500, Paul Moore wrote:
> > Developers will only be happy with seccomp if it's easy and rewarding to
> > support/debug.
>
> Agreed.
>
> As a developer, how do you feel about the audit/syslog based approach I
> mentioned earlier?
I used the commands you posted
On 12/03/2013 04:15 PM, Peter Crosthwaite wrote:
On Tue, Dec 3, 2013 at 6:48 PM, Sergey Fedorov wrote:
TTBCR has additional fields PD0 and PD1 when using Short-descriptor
translation table format on a CPU with TrustZone feature support.
Signed-off-by: Sergey Fedorov
---
target-arm/helper.c
On 12/03/2013 04:17 PM, Peter Crosthwaite wrote:
On Tue, Dec 3, 2013 at 6:48 PM, Sergey Fedorov wrote:
From: Svetlana Fedoseeva
Signed-off-by: Svetlana Fedoseeva
Signed-off-by: Sergey Fedorov
---
target-arm/helper.c |4
1 file changed, 4 insertions(+)
diff --git a/target-arm/h
On Tue, Nov 26, 2013 at 01:59:41AM -, benjamin_zb wrote:
> I met some filesysterm errors in a sles guest on KVM. My system environment
> is:
> HOST:
> suse 10, the kernel version is 2.6.32.43
> Qemu-KVM 1.2
> Libvirt 1.0
> guest OS:
> suse 10, the kernel version is 2.6.32.43
> VMs
On 12/03/2013 04:51 PM, Peter Maydell wrote:
On 3 December 2013 12:20, Peter Crosthwaite
wrote:
On Tue, Dec 3, 2013 at 6:48 PM, Sergey Fedorov wrote:
From: Svetlana Fedoseeva
Define CPU monitor mode. Adjust core registers banking. Adjust CPU VM
state info. Provide CPU mode name for monitor
Strictly speaking, this is only required for has_zero_init() == false,
but it's easy enough to just do a cluster-aligned write that is padded
with zeros after the header.
This fixes that after 'qemu-img create' header extensions are attempted
to be parsed that are really just random leftover data.
On 12/03/2013 12:48 PM, Sergey Fedorov wrote:
This patch set implements a basic support of CPU core TrustZone feature. The
following major functionalities are implemented:
* CPU monitor mode
* Separate code translation for each secure state
* CPACR & NSACR co-processor access control
On Tue, Dec 03, 2013 at 02:57:52PM +0100, Max Reitz wrote:
> Leaving the backing file open although it is not needed anymore can
> cause problems if it is opened through a block driver which allows
> exclusive access only and if the create function of the block driver
> used for the top image (the
On Tue, Dec 03, 2013 at 03:22:28PM -, Florian Miksch wrote:
Hi,
This does not look like a bug. Please see the explanation below...
Inlining Florian's reproducer script so we can discuss it via email:
> #!/bin/bash
> # This script does create a backing image and overlay to
> # reproduce the r
On Wed, Dec 04, 2013 at 11:06:36AM +0100, Kevin Wolf wrote:
> Strictly speaking, this is only required for has_zero_init() == false,
> but it's easy enough to just do a cluster-aligned write that is padded
> with zeros after the header.
>
> This fixes that after 'qemu-img create' header extensions
On Wed, Dec 4, 2013 at 7:50 PM, Fedorov Sergey wrote:
>
> On 12/03/2013 04:15 PM, Peter Crosthwaite wrote:
>>
>> On Tue, Dec 3, 2013 at 6:48 PM, Sergey Fedorov
>> wrote:
>>>
>>> TTBCR has additional fields PD0 and PD1 when using Short-descriptor
>>> translation table format on a CPU with TrustZon
On Tue, Dec 03, 2013 at 06:24:24PM +, Peter Maydell wrote:
> On 3 December 2013 16:28, Michael S. Tsirkin wrote:
> > For example:
> >
> > https://bugzilla.redhat.com/show_bug.cgi?id=588133#c8
> > https://bugzilla.redhat.com/show_bug.cgi?id=588133#c9
>
> I get "not authorized" errors on
On Wed, Dec 4, 2013 at 8:01 PM, Fedorov Sergey wrote:
>
> On 12/03/2013 04:51 PM, Peter Maydell wrote:
>>
>> On 3 December 2013 12:20, Peter Crosthwaite
>> wrote:
>>>
>>> On Tue, Dec 3, 2013 at 6:48 PM, Sergey Fedorov
>>> wrote:
From: Svetlana Fedoseeva
Define CPU monitor mo
On 4 December 2013 00:48, Richard Henderson wrote:
> On 12/04/2013 01:32 PM, Peter Maydell wrote:
>> You're right that we can just make this function return the TCGv
>> temp rather than making the caller pass one in. Are you suggesting
>> the 64-bit case should return cpu_X[reg] rather than a copy
Hi Sergey,
On Wed, Dec 4, 2013 at 8:08 PM, Fedorov Sergey wrote:
> On 12/03/2013 12:48 PM, Sergey Fedorov wrote:
>>
>> This patch set implements a basic support of CPU core TrustZone feature.
>> The
>> following major functionalities are implemented:
>>* CPU monitor mode
>>* Separate code
On 4 December 2013 10:08, Fedorov Sergey wrote:
> On 12/03/2013 12:48 PM, Sergey Fedorov wrote:
>>
>> This patch set implements a basic support of CPU core TrustZone feature.
> We'd like this patch series finally to be merged into mainstream. What
> should be done to achieve this goal?
I'd like
On 4 December 2013 10:58, Peter Crosthwaite
wrote:
> So what im proposing is just a slightly more general patch. Is it
> really any more complicated than just applying your change pattern for
> the hyp mode?
I think it would be, because of the wrinkle that hyp mode doesn't
have a banked LR, so th
>
> Almost there. Migration (vmstate) is still missing.
>
Like this:
==
>From faead85c0dbe62da896e0ed9e165d98e10216968 Mon Sep 17 00:00:00 2001
From: Liu Jinsong
Date: Wed, 4 Dec 2013 16:56:49 +0800
Subject: [PATCH 2/2] target-i386: Intel MPX
Add some MPX related definiation,
On Wed, Dec 4, 2013 at 6:09 PM, liguang wrote:
> Signed-off-by: liguang
> ---
> hw/arm/Makefile.objs |2 +-
> hw/arm/allwinner-a10.c | 77
>
> include/hw/arm/allwinner-a10.h | 36 ++
> 3 files changed, 114 insert
Floating point is an extension to the instruction set rather than
a coprocessor, so call it directly from the ARM and Thumb decode
functions.
Signed-off-by: Will Newton
Reviewed-by: Peter Maydell
---
target-arm/translate.c | 32 +++-
1 file changed, 27 insertions(+),
This adds support for the VSEL floating point selection instruction
which was added in ARMv8.
Signed-off-by: Will Newton
Reviewed-by: Peter Maydell
---
target-arm/translate.c | 135 -
1 file changed, 134 insertions(+), 1 deletion(-)
diff --git a/
This adds support for the ARMv8 Advanced SIMD VMAXNM and VMINNM
instructions.
Signed-off-by: Will Newton
---
target-arm/translate.c | 31 ++-
1 file changed, 22 insertions(+), 9 deletions(-)
Changes in v8:
- Use VFP helper instead of adding a NEON specific one
- Dr
This series adds support for three new instructions added in ARMv8 -
VSEL, VMINNM and VMAXNM.
Will Newton (6):
target-arm: Move call to disas_vfp_insn out of disas_coproc_insn.
target-arm: Implement ARMv8 VSEL instruction.
softfloat: Remove unused argument from MINMAX macro.
softfloat: Add
This adds support for the ARMv8 floating point VMAXNM and VMINNM
instructions.
Signed-off-by: Will Newton
Reviewed-by: Peter Maydell
---
target-arm/helper.c| 25 +
target-arm/helper.h| 5 +
target-arm/translate.c | 50
The nan_exp argument is not used, so remove it.
Signed-off-by: Will Newton
Reviewed-by: Peter Maydell
---
fpu/softfloat.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/fpu/softfloat.c b/fpu/softfloat.c
index 7ba51b6..97bf627 100644
--- a/fpu/softfloat.c
+++ b/fpu/sof
Add floatnn_minnum() and floatnn_maxnum() functions which are equivalent
to the minNum() and maxNum() functions from IEEE 754-2008. They are
similar to min() and max() but differ in the handling of QNaN arguments.
Signed-off-by: Will Newton
---
fpu/softfloat.c | 32 ++
On 12/04/2013 08:33 PM, Markus Armbruster wrote:
> Paolo Bonzini writes:
>
>> Il 04/12/2013 05:55, Alexey Kardashevskiy ha scritto:
>>> Normally the user is expected to eject DVD if it is not locked by
>>> the guest. eject_device() makes few checks and calls bdrv_close()
>>> if DVD is not in use.
On 12/04/2013 08:35 PM, Markus Armbruster wrote:
> Paolo Bonzini writes:
>
>> Il 04/12/2013 05:55, Alexey Kardashevskiy ha scritto:
>>> This prevents the guest from preventing DVD medium removal when
>>> there is no medium.
>>>
>>> Without this, if the user has ejected a DVD, it is possible to
>>
On 12/04/2013 03:18 PM, Peter Maydell wrote:
On 4 December 2013 10:58, Peter Crosthwaite
wrote:
So what im proposing is just a slightly more general patch. Is it
really any more complicated than just applying your change pattern for
the hyp mode?
I think it would be, because of the wrinkle th
On 4 December 2013 12:33, Fedorov Sergey wrote:
>
> On 12/04/2013 03:18 PM, Peter Maydell wrote:
>>
>> On 4 December 2013 10:58, Peter Crosthwaite
>> wrote:
>>>
>>> So what im proposing is just a slightly more general patch. Is it
>>> really any more complicated than just applying your change pat
A few things missing from the previous patch:
* add 074 to the group file
* label new output "074" instead of "048"
* remove "qemu-io> " prompts from output (qemu.git/master merge
conflict)
Signed-off-by: Stefan Hajnoczi
---
Fam: can I squash these missing things into your patch?
tests/qe
On 12/04/2013 03:13 PM, Peter Maydell wrote:
On 4 December 2013 10:08, Fedorov Sergey wrote:
On 12/03/2013 12:48 PM, Sergey Fedorov wrote:
This patch set implements a basic support of CPU core TrustZone feature.
We'd like this patch series finally to be merged into mainstream. What
should be
On Wed, Dec 04, 2013 at 04:50:59PM +1100, Alexey Kardashevskiy wrote:
> On 12/04/2013 01:47 AM, Eduardo Habkost wrote:
> > On Tue, Dec 03, 2013 at 02:30:48PM +0100, Andreas Färber wrote:
> >> Am 03.12.2013 00:03, schrieb Alexey Kardashevskiy:
> >>> On 12/03/2013 09:09 AM, Andreas Färber wrote:
> >>
Alexey Kardashevskiy writes:
> On 12/04/2013 08:35 PM, Markus Armbruster wrote:
>> Paolo Bonzini writes:
>>
>>> Il 04/12/2013 05:55, Alexey Kardashevskiy ha scritto:
This prevents the guest from preventing DVD medium removal when
there is no medium.
Without this, if the user
On 2013年12月04日 20:45, Stefan Hajnoczi wrote:
A few things missing from the previous patch:
* add 074 to the group file
* label new output "074" instead of "048"
* remove "qemu-io> " prompts from output (qemu.git/master merge
conflict)
Signed-off-by: Stefan Hajnoczi
---
Fam: can I squa
Alexey Kardashevskiy writes:
> On 12/04/2013 08:33 PM, Markus Armbruster wrote:
>> Paolo Bonzini writes:
>>
>>> Il 04/12/2013 05:55, Alexey Kardashevskiy ha scritto:
Normally the user is expected to eject DVD if it is not locked by
the guest. eject_device() makes few checks and calls
The existing approach clearly doesn't support the full range of options
that users specify on the command-line.
Bugs. It will get fixed in time with more testing/debugging. Eduardo is
working on improving the testing and RH's QA folks are working hard to shake
out the bugs too. I just poste
On 12/04/2013 07:39 AM, Stefan Hajnoczi wrote:
On Fri, Nov 22, 2013 at 11:00:24AM -0500, Paul Moore wrote:
Developers will only be happy with seccomp if it's easy and rewarding to
support/debug.
Agreed.
As a developer, how do you feel about the audit/syslog based approach I
mentioned earlie
On Wed, Dec 04, 2013 at 09:06:57AM +0800, Fam Zheng wrote:
> This series adds cache mode option in the iotests framework. Test cases are
> updated to make use of cache mode and mask supported modes.
>
> v6: [05] Recover disappeared two lines. (Benoît)
> Added Wenchao's reviewed-by lines to oth
On Fri, Nov 29, 2013 at 08:52:24PM +0100, Antonios Motakis wrote:
> @@ -91,15 +91,27 @@ static int vhost_net_get_fd(NetClientState *backend)
> }
> }
>
> -struct vhost_net *vhost_net_init(NetClientState *backend, int devfd,
> - bool force)
> +struct vhost_net
On Fri, Nov 29, 2013 at 08:52:23PM +0100, Antonios Motakis wrote:
> diff --git a/hw/virtio/vhost-backend.c b/hw/virtio/vhost-backend.c
> index 05de174..80defe4 100644
> --- a/hw/virtio/vhost-backend.c
> +++ b/hw/virtio/vhost-backend.c
> @@ -25,9 +25,18 @@ static int vhost_kernel_call(struct vhost_d
On Fri, Nov 29, 2013 at 08:52:21PM +0100, Antonios Motakis wrote:
> In this patch series we would like to introduce our approach for putting a
> virtio-net backend in an external userspace process. Our eventual target is to
> run the network backend in the Snabbswitch ethernet switch, while receivi
On Thu, Oct 10, 2013 at 05:07:18PM +0200, Paolo Bonzini wrote:
> @@ -298,30 +278,31 @@ static void handle_notify(EventNotifier *e)
> vring_disable_notification(s->vdev, &s->vring);
>
> for (;;) {
> -head = vring_pop(s->vdev, &s->vring, iov, end, &out_num,
> &in_num)
On Thu, Oct 10, 2013 at 05:07:15PM +0200, Paolo Bonzini wrote:
> Now that the memory API is thread-safe, we can use it in
> virtio-blk-dataplane and replace hostmem.[ch]. This series does this,
> and also changes the vring API to use VirtQueueElement (with an eye
> towards migration). With this c
On 11/29/2013 12:52 PM, Antonios Motakis wrote:
> Adding a new tap network backend option - vhostsock. It points
> to a named unix domain socket, that will be used to communicate
> with vhost-user backend. This is a temporary work around; in future
> versions of this series vhost-user will be made
From: Markus Armbruster
Commit 4f193e3 added the test, but screwed up in-tree builds
(SRCDIR=.): the tests's output overwrites the expected output, and is
thus compared to itself.
Cc: qemu-sta...@nongnu.org
Reported-by: Laszlo Ersek
Reviewed-by: Andreas Färber
Reviewed-by: Laszlo Ersek
Signed
From: Markus Armbruster
Forgotten in commit 6046c62 and 3464700.
Cc: qemu-sta...@nongnu.org
Reviewed-by: Andreas Färber
Reviewed-by: Laszlo Ersek
Signed-off-by: Markus Armbruster
Signed-off-by: Michael Tokarev
(cherry picked from commit 9dbb52e862458935c250bac9e71d5a87da4e33e9)
Conflicts:
Hi everyone,
The following new patches are queued for QEMU stable v1.6.2:
https://github.com/mdroth/qemu/commits/stable-1.6-staging
The release is planned for 2013-12-09:
http://wiki.qemu.org/Planning/1.6
Please respond here or CC qemu-sta...@nongnu.org on any patches you
think should be inclu
From: Stefan Weil
These functions were copies from include/linux/bitopts.h.
Signed-off-by: Stefan Weil
Reviewed-by: Richard Henderson
(cherry picked from commit 6aa25b4a7bb10c48c3054f268d5be98e42ea42c0)
Signed-off-by: Michael Roth
---
include/qemu/bitops.h | 80 +++
From: Amit Shah
All the backends implement an io watcher tag for callbacks. Move it to
CharDriverState from each backend's struct to make accessing the tag from
backend-neutral functions easier.
This will be used later to cancel a callback on chardev detach from a
frontend.
CC:
Reviewed-by: G
From: Stefan Weil
It is used by qemu-ppc64 when running Debian's busybox-static.
Cc: qemu-stable
Signed-off-by: Stefan Weil
Reviewed-by: Richard Henderson
(cherry picked from commit d285bf784b6234e994ce73c05c82c9fb6429df00)
Signed-off-by: Michael Roth
---
tcg/tci/tcg-target.c |1 -
tci
From: Amit Shah
This deduplicates code used a lot of times.
CC:
Reviewed-by: Gerd Hoffmann
Signed-off-by: Amit Shah
(cherry picked from commit 26da70c72524eb22c946ab19ec98a217b8252f7e)
Signed-off-by: Michael Roth
---
qemu-char.c | 62 ++
From: Stefan Weil
Signed-off-by: Stefan Weil
(cherry picked from commit 3df2b8fde949be86d8a78923c992fdd698d4ea4c)
Signed-off-by: Michael Roth
---
target-arm/iwmmxt_helper.c |2 +-
tcg/optimize.c | 12
tci.c |8
3 files changed,
From: Amos Kong
The buffer content might be read out more than once, currently
we just repeatedly read the first data block, buffer offset is
missing.
Cc: qemu-sta...@nongnu.org
Signed-off-by: Amos Kong
Message-id: 1385023371-8198-3-git-send-email-ak...@redhat.com
Signed-off-by: Anthony Liguori
From: Hans de Goede
Now that we no longer have MIN_REARM_TIMER_NS a bug in the audio subsys has
clearly shown it self by trying to make a timer fire every nano second.
Note we have a similar problem in 1.6, 1.5 and older but there
MIN_REARM_TIMER_NS limits the wakeups caused by audio being activ
From: Matthew Daley
Commit 4472beae modified the semantics of ioreq_{un,}map so that they are
idempotent if called when they're not needed (ie., twice in a row). However,
it neglected to handle the case where batch mapping is not being used (the
default), and one of the grants fails to map. In th
From: Stefan Hajnoczi
qdev_device_add() leaks the created device upon failure. I suspect this
problem crept in because qdev_free() unparents the device but does not
drop a reference - confusing name.
Cc: qemu-sta...@nongnu.org
Signed-off-by: Stefan Hajnoczi
Reviewed-by: Eric Blake
Signed-off-
From: Bandan Das
PCIBus registers a vmstate during init. Unregister it upon
removal/unplug.
Signed-off-by: Bandan Das
Cc: qemu-sta...@nongnu.org
Reviewed-by: Andreas Färber
Signed-off-by: Michael S. Tsirkin
(cherry picked from commit 5c397242d5d53c1adecce31817bb439383cf8228)
Signed-off-by: M
From: Alex Williamson
When an assigned device is initialized it copies the device config
space into the emulated config space. Unfortunately multifunction is
setup prior to the device initfn and gets clobbered. We need to
restore it just like pci-assign does.
Cc: qemu-sta...@nongnu.org
Signed-
From: Igor Mammedov
User is able to crash running QEMU when following monitor
command is called:
device_add intel-hda-generic
Crash is caused by assertion in object_initialize_with_type()
when type is abstract.
Checking if type is abstract before instance is created in
qdev_device_add() allow
From: Vlad Yasevich
Save the result of the call to object_get_canonical_path()
so we can free it.
Cc: qemu-sta...@nongnu.org
Signed-off-by: Vlad Yasevich
Reviewed-by: Amos Kong
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Andreas Färber
(cherry picked from commit 2d3aa28cc2cf382aa04cd577e0be5
On 12/04/2013 02:11 AM, Markus Armbruster wrote:
> No objection to asserting that the caller passed an error object when
> the error object is the only way to signal failure. You can't force
> your callers to check for failure, but the assertion could help prevent
> accidental misuse.
>
> Assert
On 12/04/2013 08:21 AM, Eduardo Otubo wrote:
On 12/04/2013 07:39 AM, Stefan Hajnoczi wrote:
On Fri, Nov 22, 2013 at 11:00:24AM -0500, Paul Moore wrote:
Developers will only be happy with seccomp if it's easy and
rewarding to
support/debug.
Agreed.
As a developer, how do you feel about th
From: Amit Shah
If a frontend device releases the chardev (via unplug), the chr handlers
are set to NULL via qdev's exit callbacks invoking
qemu_chr_add_handlers(). If the chardev had a pending operation, a
callback will be invoked, which will try to access data in the
just-released frontend, ca
On Wed, Dec 04, 2013 at 05:10:53PM +0800, Wenchao Xia wrote:
> 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
1 - 100 of 231 matches
Mail list logo
