Hi,
+del_boot_device_path(dev);
You can call this from device_finalize() instead of placing it into each
individual device.
Maybe put this call in device_finalize is not a good idea.
I have three reasons:
1. the device's some memory have been freed before call
Hi Zhang,
No I haven't seen such problem
Which kernel version are you running?
Host kernel: RHEL7-RC1(linux-3.10.0).
Does it include the latest lazy eli changes?
lazy eli or lazy eoi?
How to confirm whether lazy eli has been included?
Btw, hv_spinlocks=0xfff is a pretty huge value.
which
Best regards,
-Gonglei
-Original Message-
From: Eduardo Habkost [mailto:ehabk...@redhat.com]
Sent: Friday, August 01, 2014 10:46 PM
To: Gonglei (Arei)
Cc: qemu-devel@nongnu.org; chenliang (T); Huangweidong (C);
m...@redhat.com; a...@ozlabs.ru; hu...@cn.fujitsu.com;
Hi,
Subject: Re: [Qemu-devel] [PATCH v2 5/7] qmp: add set-bootindex command
On Fri, Jul 25, 2014 at 02:52:50PM +0800, arei.gong...@huawei.com wrote:
From: Gonglei arei.gong...@huawei.com
Adds set-bootindex id=xx,bootindex=xx,suffix=xx QMP command.
Example QMP command:
- { execute:
Hi,
On Thu, Jul 31, 2014 at 05:47:26PM +0800, arei.gong...@huawei.com wrote:
[...]
+void modify_boot_device_path(int32_t bootindex, DeviceState *dev,
+ const char *suffix)
+{
+FWBootEntry *i, *old_entry = NULL;
+
+assert(dev != NULL ||
On 2014/7/31 17:53, Michael S. Tsirkin wrote:
On Thu, Jul 31, 2014 at 05:26:41PM +0800, Chen, Tiejun wrote:
On 2014/7/31 17:10, Michael S. Tsirkin wrote:
On Thu, Jul 31, 2014 at 02:31:36PM +0800, Tiejun Chen wrote:
We'd like to split i440fx_init and then we can share something
with other
When optarg happens to contain \n like:
../configure --target-list='i386-softmmu
x86_64-softmmu'
make will fail with message:
config-host.mak:45: *** missing separator. Stop.
This patch fix this problem by replacing \n with space in optarg.
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
---
On Mon, Aug 04, 2014 at 12:21:17PM +0800, Chen Fan wrote:
Signed-off-by: Chen Fan chen.fan.f...@cn.fujitsu.com
Reviewed-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
Reviewed-by: Hu Tao hu...@cn.fujitsu.com
---
numa.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff
On Mon, Aug 04, 2014 at 12:21:18PM +0800, Chen Fan wrote:
string_output_get_string() uses g_string_free(str, false) to
transfer the 'str' pointer to callers and never free it.
Signed-off-by: Chen Fan chen.fan.f...@cn.fujitsu.com
Reviewed-by: Hu Tao hu...@cn.fujitsu.com
---
hmp.c|
On Mon, Aug 04, 2014 at 12:21:19PM +0800, Chen Fan wrote:
the memdev_list in hmp_info_memdev() is never freed.
so we use existent method qapi_free_MemdevList() to free it.
and also we can use qapi_free_MemdevList() to replace list loops
to clean up the memdev list in error path.
Gonglei (Arei) arei.gong...@huawei.com writes:
Hi,
On Thu, Jul 31, 2014 at 05:47:26PM +0800, arei.gong...@huawei.com wrote:
[...]
+void modify_boot_device_path(int32_t bootindex, DeviceState *dev,
+ const char *suffix)
+{
+FWBootEntry *i,
Please stop Cc'ing me emails sent to, at least, qemu-trivial@.
I'm about to filter personal emails which are also sent to
some mailinglists I receive. I'd not do that, because this is
a good practice to Cc someone like that for various really
important or urgent emails, and if I to apply such a
Hi,
On Thu, Jul 31, 2014 at 05:47:26PM +0800, arei.gong...@huawei.com
wrote:
[...]
+void modify_boot_device_path(int32_t bootindex, DeviceState *dev,
+ const char *suffix)
+{
+FWBootEntry *i, *old_entry = NULL;
+
+assert(dev !=
If user specifies a node number that exceeds the available numa nodes in
emulated system for pc-dimm device, the device will reports an invalid _PXM
to OSPM. Fix it by checking the node value.
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
---
hw/mem/pc-dimm.c | 5 +
1 file changed, 5
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
---
hw/i386/acpi-dsdt.dsl | 4 ++--
hw/i386/acpi-dsdt.hex.generated | 8
hw/i386/q35-acpi-dsdt.dsl | 4 ++--
hw/i386/ssdt-mem.dsl| 16
hw/i386/ssdt-misc.dsl | 2 +-
The error messages before and after patch are:
before:
qemu-system-x86_64: total memory for NUMA nodes (134217728) should equal RAM
size (2000)
after:
qemu-system-x86_64: total memory for NUMA nodes (0x800) should equal RAM
size (0x2000)
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
See each patch for the detail.
Hu Tao (3):
hw:i386: typo fix: MEMORY_HOPTLUG_DEVICE - MEMORY_HOTPLUG_DEVICE
pc-dimm: check if node property exceeds available numa nodes
numa: show hex number in error message for consistency and prefix them
with 0x
hw/i386/acpi-dsdt.dsl | 4
Gonglei (Arei) arei.gong...@huawei.com writes:
Hi,
Subject: Re: [Qemu-devel] [PATCH v2 5/7] qmp: add set-bootindex command
On Fri, Jul 25, 2014 at 02:52:50PM +0800, arei.gong...@huawei.com wrote:
From: Gonglei arei.gong...@huawei.com
Adds set-bootindex id=xx,bootindex=xx,suffix=xx QMP
Stefan Hajnoczi stefa...@gmail.com writes:
On Thu, Jul 31, 2014 at 05:14:12PM -0400, John Snow wrote:
On 07/31/2014 06:13 AM, Stefan Hajnoczi wrote:
On Wed, Jul 30, 2014 at 06:28:28PM -0400, John Snow wrote:
-static uint64_t pc_alloc(QGuestAllocator *allocator, size_t size)
+static inline
In function virtio_blk_handle_request, it may freed memory pointed by req,
So do not access member of req after calling this function.
Signed-off-by: zhanghailiang zhang.zhanghaili...@huawei.com
---
hw/block/virtio-blk.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git
The function monitor_fdset_dup_fd_find_remove() references member of 'mon_fdset'
which may be freed in function monitor_fdset_cleanup()
Signed-off-by: zhanghailiang zhang.zhanghaili...@huawei.com
---
monitor.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/monitor.c
Pointer 'ch' will be used in function 'l2cap_channel_open_req_msg' after
it was previously freed in 'l2cap_channel_open'.
Assigned it to NULL after it is freed.
Signed-off-by: zhanghailiang zhang.zhanghaili...@huawei.com
---
hw/bt/l2cap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
The function fstat() may fail, so check its return value.
Signed-off-by: zhanghailiang zhang.zhanghaili...@huawei.com
---
hw/misc/ivshmem.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/hw/misc/ivshmem.c b/hw/misc/ivshmem.c
index 768e528..2667e9f 100644
---
zhanghailiang (4):
l2cap: fix access freed memory
monitor: fix access freed memory
virtio-blk: fix reference a pointer which might be freed
ivshmem: check the value returned by fstat()
hw/block/virtio-blk.c | 5 +++--
hw/bt/l2cap.c | 2 +-
hw/misc/ivshmem.c | 5 -
Hi,
Subject: Re: [Qemu-devel] [PATCH v2 5/7] qmp: add set-bootindex command
On Fri, Jul 25, 2014 at 02:52:50PM +0800, arei.gong...@huawei.com wrote:
From: Gonglei arei.gong...@huawei.com
Adds set-bootindex id=xx,bootindex=xx,suffix=xx QMP command.
Example QMP command:
- {
Hi
I'm working on an extension to QEMU (target i386). This involves adding new
MSR's. I've got it working in non-KVM mode by adding these MSR's to the
state and adding extra cases to helper_wrmsr(), helper_rdmsr(). The guest
can now read/write these MSR's as expected. However, it fails when
On 4 August 2014 05:55, Alexey Kardashevskiy a...@ozlabs.ru wrote:
Our lawyers refused to provide any public advise on this :-/
Is that it, end of story? :)
Well, I 'd still like to get the license fixed to at least my
personal satisfaction, but if your lawyers won't confirm
whether what we
On Mon, Aug 04, 2014 at 09:05:39AM +1000, Peter Crosthwaite wrote:
On Sun, Aug 3, 2014 at 1:45 AM, Richard W.M. Jones rjo...@redhat.com wrote:
+max_bytes = UBOOT_MAX_GUNZIP_BYTES;
Why does u-boot's maximum size limit apply here?
We need some maximum to prevent people uploading a kernel
zhanghailiang writes:
Pointer 'ch' will be used in function 'l2cap_channel_open_req_msg' after
it was previously freed in 'l2cap_channel_open'.
Assigned it to NULL after it is freed.
Signed-off-by: zhanghailiang zhang.zhanghaili...@huawei.com
---
hw/bt/l2cap.c | 2 +-
1 file changed, 1
On 4 August 2014 08:12, Hu Tao hu...@cn.fujitsu.com wrote:
When optarg happens to contain \n like:
../configure --target-list='i386-softmmu
x86_64-softmmu'
make will fail with message:
config-host.mak:45: *** missing separator. Stop.
Why would you put newlines in the option string? This
On 4 August 2014 09:48, Richard W.M. Jones rjo...@redhat.com wrote:
On Mon, Aug 04, 2014 at 09:05:39AM +1000, Peter Crosthwaite wrote:
On Sun, Aug 3, 2014 at 1:45 AM, Richard W.M. Jones rjo...@redhat.com wrote:
+max_bytes = UBOOT_MAX_GUNZIP_BYTES;
Why does u-boot's maximum size limit
zhanghailiang writes:
The function monitor_fdset_dup_fd_find_remove() references member of
'mon_fdset'
which may be freed in function monitor_fdset_cleanup()
Signed-off-by: zhanghailiang zhang.zhanghaili...@huawei.com
---
monitor.c | 4 +++-
1 file changed, 3 insertions(+), 1
On Mon, Aug 4, 2014 at 6:48 PM, Richard W.M. Jones rjo...@redhat.com wrote:
On Mon, Aug 04, 2014 at 09:05:39AM +1000, Peter Crosthwaite wrote:
On Sun, Aug 3, 2014 at 1:45 AM, Richard W.M. Jones rjo...@redhat.com wrote:
+max_bytes = UBOOT_MAX_GUNZIP_BYTES;
Why does u-boot's maximum size
On Mon, Aug 4, 2014 at 2:21 PM, Chen Fan chen.fan.f...@cn.fujitsu.com wrote:
string_output_get_string() uses g_string_free(str, false) to
transfer the 'str' pointer to callers and never free it.
Signed-off-by: Chen Fan chen.fan.f...@cn.fujitsu.com
Reviewed-by: Peter Crosthwaite
The Saturday 02 Aug 2014 à 01:49:15 (+0200), Max Reitz wrote :
Depending on the changed options and the image format,
bdrv_amend_options() may take a significant amount of time. In these
cases, a way to be informed about the operation's status is desirable.
Since the operation is rather
From: Gonglei arei.gong...@huawei.com
imitate nearby code about using '!value' or 'value == NULL'
Signed-off-by: Gonglei arei.gong...@huawei.com
---
ui/spice-core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/ui/spice-core.c b/ui/spice-core.c
index 7bb91e6..1a2fb4b
From: Gonglei arei.gong...@huawei.com
imitate nearby code about using '!value' or 'value == NULL'
Signed-off-by: Gonglei arei.gong...@huawei.com
---
vl.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/vl.c b/vl.c
index fe451aa..04c5abd 100644
--- a/vl.c
+++ b/vl.c
@@
From: Gonglei arei.gong...@huawei.com
imitate nearby code about using '!value' or 'value == NULL'
Signed-off-by: Gonglei arei.gong...@huawei.com
---
qdev-monitor.c | 2 +-
qemu-char.c | 2 +-
util/qemu-sockets.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git
From: Gonglei arei.gong...@huawei.com
imitate nearby code about using '!value' or 'value == NULL'
Signed-off-by: Gonglei arei.gong...@huawei.com
---
hw/audio/gus.c | 2 +-
hw/audio/hda-codec.c | 3 ++-
hw/audio/sb16.c | 6 +++---
3 files changed, 6 insertions(+), 5 deletions(-)
diff
From: Gonglei arei.gong...@huawei.com
imitate nearby code about using '!value' or 'value == NULL'
Signed-off-by: Gonglei arei.gong...@huawei.com
---
hw/net/vmxnet3.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/hw/net/vmxnet3.c b/hw/net/vmxnet3.c
index
From: Gonglei arei.gong...@huawei.com
Yoda conditions lack readability, and QEMU has a
strict compiler configuration for checking a common
mistake like if (dev = NULL). Make it a written rule.
Signed-off-by: Gonglei arei.gong...@huawei.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
From: Gonglei arei.gong...@huawei.com
$WHATEVER: don't use 'Yoda conditions'
'Yoda conditions' are not part of idiomatic QEMU coding
style, so rewrite them in the more usual order.
v4:
- trivial typo for patch 1/8 suggested by Eric, thanks.
v3:
- rewrite CODINT_STYLE file suggested by Eric,
From: Gonglei arei.gong...@huawei.com
imitate nearby code about using '!value' or 'value == NULL'
Signed-off-by: Gonglei arei.gong...@huawei.com
---
hw/isa/isa-bus.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/isa/isa-bus.c b/hw/isa/isa-bus.c
index b28981b..cc85e53
From: Gonglei arei.gong...@huawei.com
imitate nearby code about using '!value' or 'value == NULL'
Signed-off-by: Gonglei arei.gong...@huawei.com
---
hw/usb/dev-audio.c | 2 +-
hw/usb/dev-mtp.c | 4 ++--
hw/usb/hcd-ehci.c | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git
The Saturday 02 Aug 2014 à 01:49:16 (+0200), Max Reitz wrote :
Now that bdrv_amend_options() supports a status callback, use it to
display a progress report.
Signed-off-by: Max Reitz mre...@redhat.com
---
qemu-img-cmds.hx | 4 ++--
qemu-img.c | 25 ++---
Hi,
This patchset fixes a memleak in virtio-rng init and moves from
error_set to error_setg.
Please pull.
The following changes since commit c79805802ba0463713c253307d99ebef56436b8c:
Open 2.2 development tree (2014-08-01 18:30:08 +0100)
are available in the git repository at:
On Fri, Aug 1, 2014 at 9:46 AM, Stefan Hajnoczi stefa...@gmail.com wrote:
On Mon, Jul 21, 2014 at 02:18:09PM +0400, Maria Kustova wrote:
+def execute(self, input_commands=None, fuzz_config=None):
+ Execute a test.
+
+The method creates backing and test images, runs test
The Saturday 02 Aug 2014 à 01:49:18 (+0200), Max Reitz wrote :
The only really time-consuming operation potentially performed by
qcow2_amend_options() is zero cluster expansion when downgrading qcow2
images from compat=1.1 to compat=0.10, so report status of that
operation and that operation
On Mon, Aug 04, 2014 at 09:50:20AM +0100, Peter Maydell wrote:
On 4 August 2014 08:12, Hu Tao hu...@cn.fujitsu.com wrote:
When optarg happens to contain \n like:
../configure --target-list='i386-softmmu
x86_64-softmmu'
make will fail with message:
config-host.mak:45: *** missing
The Saturday 02 Aug 2014 à 01:49:19 (+0200), Max Reitz wrote :
Reading the refcount of a cluster is an operation which can be useful in
all of the qcow2 code, so make that function globally available.
While touching this function, amend the comment describing the addend
parameter: It is (no
The Saturday 02 Aug 2014 à 01:49:20 (+0200), Max Reitz wrote :
Currently, we have a bitmap for keeping track of which clusters have
been created during the zero cluster expansion process. This was
necessary because we need to properly increase the refcount for shared
L2 tables.
However, now
Hello,
This patchset updates the vmstate static checker to handle fields that
got marked 'unused' in qemu versions. Also update the whitelist based
on a few false-positives.
The following changes since commit 35858955e6c6f9ef41c199d15457c13426ac6434:
Merge remote-tracking branch
On (Mon) 04 Aug 2014 [15:06:05], Amit Shah wrote:
Hello,
This patchset updates the vmstate static checker to handle fields that
got marked 'unused' in qemu versions. Also update the whitelist based
on a few false-positives.
The following changes since commit
Am 02.08.2014 um 00:33 hat Peter Maydell geschrieben:
On 1 August 2014 17:41, Michael Roth mdr...@linux.vnet.ibm.com wrote:
On behalf of the QEMU Team, I'd like to announce the availability of
the QEMU 2.1.0 release. This release contains 2200+ commits from 180
authors.
Thank you to
On Fri, Aug 01, 2014 at 07:27:57PM -0400, John Snow wrote:
On 07/31/2014 10:01 AM, Stefan Hajnoczi wrote:
On Mon, Jul 07, 2014 at 02:18:07PM -0400, John Snow wrote:
+/*** IO macros for the AHCI memory registers. ***/
+#define void_incr(vptr, OFST) ((void *)((char *)(vptr) + (OFST)))
I'm
Gonglei (Arei) arei.gong...@huawei.com writes:
Hi,
Subject: Re: [Qemu-devel] [PATCH v2 5/7] qmp: add set-bootindex command
On Fri, Jul 25, 2014 at 02:52:50PM +0800, arei.gong...@huawei.com wrote:
From: Gonglei arei.gong...@huawei.com
Adds set-bootindex
On Wed, Jul 30, 2014 at 07:39:33PM +0800, Ming Lei wrote:
These patches bring up below 4 changes:
- introduce selective coroutine bypass mechanism
for improving performance of virtio-blk dataplane with
raw format image
- introduce object allocation pool and
On Fri, Aug 01, 2014 at 03:42:05PM +0800, Ming Lei wrote:
On Thu, Jul 31, 2014 at 5:18 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 31/07/2014 05:22, Ming Lei ha scritto:
The problem is that g_slice here is not using the slab-style allocator
because the object is larger than roughly
On Thu, Jul 31, 2014 at 01:20:22AM +0800, Ming Lei wrote:
On Wed, Jul 30, 2014 at 10:00 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 30/07/2014 13:39, Ming Lei ha scritto:
This patch increases max event to 256 for the comming
virtio-blk multi virtqueue support.
Signed-off-by: Ming
On Fri, 25 Jul 2014 14:10:45 +0200
Cornelia Huck cornelia.h...@de.ibm.com wrote:
Currently, qemu will take a hard exit if it fails to set up guest or
host notifiers, giving no real clue as to what went wrong (e.g., when
out of file descriptors).
This patchset tries to make this more
Peter Maydell writes:
On 1 August 2014 17:06, Peter Maydell peter.mayd...@linaro.org wrote:
I'm taking the first two of these into target-arm.next because
they're obvious standalone bugfixes. I need to think about the
last three a bit more: I dislike just dropping the ARMv5 CPUs
from
Paolo Bonzini writes:
Il 30/07/2014 17:20, Alex Bennée ha scritto:
Hi,
snip
The most important thing is I've measured a 25-30% improvement in
kernel and android boot time.
snip
Hi Alex, have you seen this patch? Perhaps you're interested in
reviving it.
On 08/04/2014 06:48 PM, Peter Maydell wrote:
On 4 August 2014 05:55, Alexey Kardashevskiy a...@ozlabs.ru wrote:
Our lawyers refused to provide any public advise on this :-/
Is that it, end of story? :)
Well, I 'd still like to get the license fixed to at least my
personal satisfaction, but
On Mon, Aug 4, 2014 at 6:16 PM, Stefan Hajnoczi stefa...@redhat.com wrote:
On Wed, Jul 30, 2014 at 07:39:33PM +0800, Ming Lei wrote:
These patches bring up below 4 changes:
- introduce selective coroutine bypass mechanism
for improving performance of virtio-blk dataplane with
On 4 August 2014 11:23, Alex Bennée alex.ben...@linaro.org wrote:
Peter Maydell writes:
So:
* there's clearly a big perf win to be had here
* this patchset gives us that for 4K pages on AArch64
* but it doesn't help for 4K pages on AArch32 (really
common)
Well for the AArch32
Add some trace events to virtio-rng for easier debugging
Signed-off-by: Amit Shah amit.s...@redhat.com
---
v2:
- requested_size trace event now shows proper values
---
hw/virtio/virtio-rng.c | 6 ++
trace-events | 5 +
2 files changed, 11 insertions(+)
diff --git
Hi, Markus
Subject: Re: [Qemu-devel] [PATCH v2 5/7] qmp: add set-bootindex
command
On Fri, Jul 25, 2014 at 02:52:50PM +0800, arei.gong...@huawei.com
wrote:
From: Gonglei arei.gong...@huawei.com
Adds set-bootindex id=xx,bootindex=xx,suffix=xx QMP command.
Example QMP
On Thu, Jul 31, 2014 at 08:29:00PM +0800, arei.gong...@huawei.com wrote:
From: Gonglei arei.gong...@huawei.com
Signed-off-by: Gonglei arei.gong...@huawei.com
---
hw/net/vmxnet3.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
Reviewed-by: Stefan Hajnoczi
Alexey Kardashevskiy a...@ozlabs.ru writes:
On 08/04/2014 06:48 PM, Peter Maydell wrote:
On 4 August 2014 05:55, Alexey Kardashevskiy a...@ozlabs.ru wrote:
Our lawyers refused to provide any public advise on this :-/
Is that it, end of story? :)
Well, I 'd still like to get the license
On Mon, Aug 04, 2014 at 04:25:43PM +0800, zhanghailiang wrote:
In function virtio_blk_handle_request, it may freed memory pointed by req,
So do not access member of req after calling this function.
Signed-off-by: zhanghailiang zhang.zhanghaili...@huawei.com
---
hw/block/virtio-blk.c | 5
On 4 August 2014 12:12, Markus Armbruster arm...@redhat.com wrote:
I'm afraid we need rip out the problematic patches. Here's Paolo's
list:
Fabrice Bellard fabr...@bellard.org
1d6bda356153c82e100680d9f2165e32c8fb1330
750afe93fd15fafc20b6c34d30f339547d15c2d1
Jocelyn Mayer
Amit Shah amit.s...@redhat.com writes:
To ensure two virtserialports don't get added to the system with the
same 'name' parameter, we need to access all the ports on all the
devices added, and compare the names.
We currently don't have a list of all VirtIOSerial devices added to the
system.
Alex Bennée writes:
Paolo Bonzini writes:
Il 30/07/2014 17:20, Alex Bennée ha scritto:
Hi,
snip
The most important thing is I've measured a 25-30% improvement in
kernel and android boot time.
snip
Hi Alex, have you seen this patch? Perhaps you're interested in
reviving it.
On Mon, Aug 4, 2014 at 6:21 PM, Stefan Hajnoczi stefa...@redhat.com wrote:
On Fri, Aug 01, 2014 at 03:42:05PM +0800, Ming Lei wrote:
On Thu, Jul 31, 2014 at 5:18 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 31/07/2014 05:22, Ming Lei ha scritto:
The problem is that g_slice here is not
On (Mon) 04 Aug 2014 [13:33:56], Markus Armbruster wrote:
Amit Shah amit.s...@redhat.com writes:
To ensure two virtserialports don't get added to the system with the
same 'name' parameter, we need to access all the ports on all the
devices added, and compare the names.
We currently
Gonglei (Arei) arei.gong...@huawei.com writes:
Hi, Markus
Subject: Re: [Qemu-devel] [PATCH v2 5/7] qmp: add set-bootindex
command
On Fri, Jul 25, 2014 at 02:52:50PM +0800, arei.gong...@huawei.com
wrote:
From: Gonglei arei.gong...@huawei.com
Adds set-bootindex
As the name suggests this lets you load a ROM/disk image that is
gzipped. It is uncompressed before storing it in guest memory.
Signed-off-by: Richard W.M. Jones rjo...@redhat.com
---
hw/core/loader.c| 48
include/hw/loader.h | 1 +
2 files
On aarch64 it is the bootloader's job to uncompress the kernel. UEFI
and u-boot bootloaders do this automatically when the kernel is
gzip-compressed.
However the qemu -kernel option does not do this. The following
command does not work:
qemu-system-aarch64 [...] -kernel /boot/vmlinuz
Changes since v4:
- Split the patch into a generic loader part, and specific arm64
support.
- There is now a specific limit for the gzip loader, plus a comment
to indicate that it's just there to stop an excessive malloc. The
limit is now decoupled (and larger) than the u-boot limit,
On 1 August 2014 17:00, Stefano Stabellini
stefano.stabell...@eu.citrix.com wrote:
The following changes since commit 541bbb07eb197a870661ed702ae1f15c7d46aea6:
Update version for v2.1.0 release (2014-08-01 13:31:29 +0100)
are available in the git repository at:
Hi, Markus
Subject: Re: [Qemu-devel] [PATCH v2 5/7] qmp: add set-bootindex
command
On Fri, Jul 25, 2014 at 02:52:50PM +0800, arei.gong...@huawei.com
wrote:
From: Gonglei arei.gong...@huawei.com
Adds set-bootindex id=xx,bootindex=xx,suffix=xx QMP command.
Il 04/08/2014 10:37, Morty Andersen ha scritto:
Hi
I'm working on an extension to QEMU (target i386). This involves adding
new MSR's. I've got it working in non-KVM mode by adding these MSR's to
the state and adding extra cases to helper_wrmsr(), helper_rdmsr(). The
guest can now read/write
On Mon, Aug 04, 2014 at 04:16:06PM +0800, Hu Tao wrote:
See each patch for the detail.
Hu Tao (3):
hw:i386: typo fix: MEMORY_HOPTLUG_DEVICE - MEMORY_HOTPLUG_DEVICE
pc-dimm: check if node property exceeds available numa nodes
numa: show hex number in error message for consistency and
On Mon, Aug 04, 2014 at 04:16:08PM +0800, Hu Tao wrote:
If user specifies a node number that exceeds the available numa nodes in
emulated system for pc-dimm device, the device will reports an invalid _PXM
to OSPM. Fix it by checking the node value.
Signed-off-by: Hu Tao hu...@cn.fujitsu.com
On 2 August 2014 00:41, Peter Crosthwaite crosthwaitepe...@gmail.com wrote:
ARMv7M has it's own bootloader (separate from the regular ARM
bootloader) that is elf aware. It is able to load elfs but it does
not set the program counter to the elf entry point. Make it more
consistent with the
On 10 July 2014 16:49, Alex Bennée alex.ben...@linaro.org wrote:
We have a number of program state saving functions (pstate, cpsr, xpsr)
which are dependant on the mode the CPU is in. This commit adds a little
documentation to each function and asserts to defend against incorrect
use.
On Mon, Aug 04, 2014 at 04:25:44PM +0800, zhanghailiang wrote:
The function fstat() may fail, so check its return value.
Signed-off-by: zhanghailiang zhang.zhanghaili...@huawei.com
---
hw/misc/ivshmem.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git
On Sat, Aug 02, 2014 at 07:54:30AM +0200, Jan Kiszka wrote:
On 2014-08-01 19:16, Alex Williamson wrote:
Also, it may let some of our device
models deviate from their real versions (I suppose, e.g., none of the
e1000 devices we currently emulate exposed FLR).
Of course, but what are
On 10 July 2014 16:49, Alex Bennée alex.ben...@linaro.org wrote:
This adds a universal program state save and restore function. This is
intended to simplify the migration serialisation functionality and avoid
special casing depending on the mode of the CPU at serialisation time.
From: Gonglei arei.gong...@huawei.com
When we want to change one device's bootindex, we
should lookup the device and change the bootindex.
it is simply that remove it from the global boot list,
then re-add it, sorted by new bootindex.
If the new bootindex has already used by another device
just
From: Gonglei arei.gong...@huawei.com
Introduce del_boot_device_path() to clean up fw_cfg content when
hot-unplugging a device that refers to a bootindex.
Signed-off-by: Gonglei arei.gong...@huawei.com
Signed-off-by: Chenliang chenlian...@huawei.com
---
include/sysemu/sysemu.h | 1 +
vl.c
From: Gonglei arei.gong...@huawei.com
Adds set-bootindex id=xx,bootindex=xx,suffix=xx QMP command.
Example QMP command:
- { execute: set-bootindex,
arguments: { id: ide0-0-1, bootindex: 1, suffix: /disk@0}}
- { return: {} }
Signed-off-by: Gonglei arei.gong...@huawei.com
Signed-off-by:
From: Gonglei arei.gong...@huawei.com
Device should be removed from global boot list when
it is hot-unplugged.
Signed-off-by: Gonglei arei.gong...@huawei.com
Signed-off-by: Chenliang chenlian...@huawei.com
---
hw/core/qdev.c | 4
1 file changed, 4 insertions(+)
diff --git a/hw/core/qdev.c
From: Gonglei arei.gong...@huawei.com
We must assure that the changed bootindex can take effect
when guest is rebooted. So we introduce fw_cfg_machine_reset(),
which change the fw_cfg file's bootindex data using the new
global fw_boot_order list.
Signed-off-by: Chenliang chenlian...@huawei.com
From: Gonglei arei.gong...@huawei.com
Sometimes, we want to modify boot order of a guest, but no need to
shutdown it. We can call dynamic changing bootindex of a guest, which
can be assured taking effect just after the guest rebooting.
For example, in P2V scene, we boot a guest and then attach a
On 10 July 2014 16:50, Alex Bennée alex.ben...@linaro.org wrote:
This enables the saving and restoring of machine state by including the
current program state (*psr) and xregs. The save_state_to_spsr hides the
details of if the processor is in 32 or 64 bit mode at the time.
Signed-off-by:
From: Gonglei arei.gong...@huawei.com
Adds query-bootindex QMP command.
Example QMP command:
- { execute: query-bootindex}
- {
return:[
{
id:ide0-0-0,
bootindex:1,
suffix:/disk@0
},
{
id:nic1,
Hi,
I' am so sorry for missing cc'ed Eduardo.
Best regards,
-Gonglei
-Original Message-
From: Gonglei (Arei)
Sent: Monday, August 04, 2014 8:46 PM
To: qemu-devel@nongnu.org
Subject: [PATCH v5 0/8] modify boot order of guest, and take effect after
rebooting
From: Gonglei
From: Gonglei arei.gong...@huawei.com
Add HMP info-bootindex command to getting
devcie's bootindex via monitor.
Signed-off-by: Gonglei arei.gong...@huawei.com
Signed-off-by: Chenliang chenlian...@huawei.com
---
hmp-commands.hx | 2 ++
hmp.c | 20
hmp.h
From: Gonglei arei.gong...@huawei.com
Add HMP set-bootindex wrapper to allow setting
devcie's bootindex via monitor.
Signed-off-by: Gonglei arei.gong...@huawei.com
Signed-off-by: Chenliang chenlian...@huawei.com
---
hmp-commands.hx | 15 +++
hmp.c | 13 +
hmp.h
1 - 100 of 290 matches
Mail list logo