Apologies for the late reply!!
Never mind, we figured it out,
Since we are doing cloning of VM, at the source(from where cloning is
initiated) we are redirecting the disk driver of the VM to a new
incremental image (qcow2). Also making the previous disk image as read
only, since the cloned VM wil
The "info mtree" command in QEMU console prints only "memory" and "I/O"
address spaces while there are actually a lot more other AddressSpace
structs created by PCI and VIO devices. Those devices do not normally
have names and therefore not present in "info qtree" output.
The patch fixes this.
Si
> "Jean-Christophe" == Jean-Christophe DUBOIS writes:
Jean-Christophe> This patch is providing a complete version of the
Jean-Christophe> EPIT timer. Note, however that the GPT timer in the
Jean-Christophe> same file is still not complete.
Looks good. You can add my
Reviewed-by: Peter Chu
On 28 April 2013 19:26, Maksim Ratnikov wrote:
> Previous realization doesn't consider flags in the status register.
> Add DS and INTR bits of HST_STS register set after transaction execution.
> Update bits resetting in HST_STS register. Update error processing: if
> DEV_ERR bit are set
> transact
Stefan Hajnoczi writes:
> On Fri, Apr 26, 2013 at 02:15:46PM +0200, Lluís Vilanova wrote:
>> Stefan Hajnoczi writes:
>>
>> > On Wed, Apr 24, 2013 at 02:17:22PM +0200, Lluís Vilanova wrote:
>> > Given that the group of users for this feature is so small while the
>> > burden of supporting this is
Previous realization doesn't consider flags in the status register.
Add DS and INTR bits of HST_STS register set after transaction execution.
Update bits resetting in HST_STS register. Update error processing: if
DEV_ERR bit are set
transaction isn't execution.
Signed-off-by: Maksim_Ratnikov
---
The following error occurs when building dtc module:
CHK version_gen.h
CC libfdt/fdt.o
cc1: error: dtc: No such file or directory [-Werror]
cc1: all warnings being treated as errors
make[1]: *** [libfdt/fdt.o] Error 1
make: *** [subdir-dtc] Error 2
In rules.mak, "-I$(
Cc: Blue Sw
The fw_cfg ABI is Little Endian, so byte-swap the generically read
byte array to host endianness.
This unbreaks the fw_cfg tests on ppc.
Signed-off-by: Andreas Färber
---
tests/libqos/fw_cfg.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/tests/libqos/fw_cfg.c b/tes
Now KVM can support a flash memory. This feature depends on
KVM_CAP_READONLY_MEM, which was introduced in Linux 3.7.
Signed-off-by: Jordan Justen
---
hw/block/pc_sysfw.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/block/pc_sysfw.c b/hw/block/pc_sysfw.c
index e88d67e
A slot that uses KVM_MEM_READONLY can be read from and code
can execute from the region, but writes will trap.
For regions that are readonly and also not writeable, we
force the slot to be removed so reads or writes to the region
will trap. (A memory region in this state is not executable
within k
This requires the KVM READONLY memory capability.
Signed-off-by: Jordan Justen
---
hw/block/pc_sysfw.c |3 +++
1 file changed, 3 insertions(+)
diff --git a/hw/block/pc_sysfw.c b/hw/block/pc_sysfw.c
index aad8614..e88d67e 100644
--- a/hw/block/pc_sysfw.c
+++ b/hw/block/pc_sysfw.c
@@ -237,6 +
We should sync params->ram_size after we fixup memory size on
a alignment boundary. Otherwise Guest would exceed the actual
memory region.
Signed-off-by: Tiejun Chen
---
hw/ppc/e500.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/hw/ppc/e500.c b/hw/ppc/e500.c
index c1bdb6b..145da0e 10
This causes any writes to the memory region to trap to the
device handler.
This is also important for KVM, because this allows the memory
region to be set using KVM_MEM_READONLY, which allows the memory
region to be read & executed. (Without this, KVM will not support
executing from the memory reg
On a Linux 3.8.0 based kernel, I occasionally saw a situation
where the memory region would continue to trap on memory
read even though KVM_MEM_READONLY was set.
I found that if I set the slot to a size of 0, and before
setting the slot, it would then behave as expected.
Signed-off-by: Jordan Jus
Signed-off-by: Jordan Justen
---
include/sysemu/kvm.h | 10 ++
kvm-all.c|6 ++
kvm-stub.c |1 +
3 files changed, 17 insertions(+)
diff --git a/include/sysemu/kvm.h b/include/sysemu/kvm.h
index 75bd7d9..c83f51c 100644
--- a/include/sysemu/kvm.h
+++ b/in
git://github.com/jljusten/qemu.git kvm-flash-v1
Utilize KVM_CAP_READONLY_MEM to support PC system flash emulation
with KVM.
Jordan Justen (6):
kvm: add kvm_readonly_mem_enabled
kvm: support using KVM_MEM_READONLY flag for readonly regions
kvm: workaround a possible KVM bug when using KVM_ME
Am 16.04.2013 16:45, schrieb Anthony Liguori:
> This validates some basic characteristics of fw_cfg.
>
> Signed-off-by: Anthony Liguori
> ---
> tests/Makefile | 2 +
> tests/fw_cfg-test.c | 141
>
> 2 files changed, 143 insertions(+)
>
openSUSE 12.3 has seccomp.h in /usr/include/libseccomp-1.0.1,
so add `pkg-config --cflags libseccomp` output to QEMU_CFLAGS.
Cc: qemu-sta...@nongnu.org
Signed-off-by: Andreas Färber
---
configure | 1 +
1 file changed, 1 insertion(+)
diff --git a/configure b/configure
index 21438d4..643153f 100
On 2013-04-28 15:53, Andreas Färber wrote:
> Am 28.04.2013 15:03, schrieb Jan Kiszka:
>> On 2013-04-28 13:22, Andreas Färber wrote:
>>> Necessary to change the name of ICCDevice's parent object field.
>>>
>>> Signed-off-by: Andreas Färber --- Could any of
>>> the APIC experts please review whether
Am 26.04.2013 20:21, schrieb Alexander Graf:
> From: Fabien Chouteau
>
> According to the different user's manuals, the vector offset for system
> reset (both /HRESET and /SRESET) is 0x00100.
>
> This patch may break support of some executables, as the power-on start
> address may change. For a
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Am 28.04.2013 15:03, schrieb Jan Kiszka:
> On 2013-04-28 13:22, Andreas Färber wrote:
>> Necessary to change the name of ICCDevice's parent object field.
>>
>> Signed-off-by: Andreas Färber --- Could any of
>> the APIC experts please review whether I
Hi ! i am trying to emulate cavium octeon's mips64 linux kernel on Qemu.I
am currently having some issues with init code and want to debug init .i am
starting the Qemu using -s -S option in the command line and the attaching
the gdb using target remote localhost:1234
What i want to ask is there any
On 2013-04-28 13:22, Andreas Färber wrote:
> Necessary to change the name of ICCDevice's parent object field.
>
> Signed-off-by: Andreas Färber
> ---
> Could any of the APIC experts please review whether I'm touching any hot
> path?
> Not sure if this was a mix of pre- and post-QOM code or int
On Sat, Apr 27, 2013 at 01:13:16PM +0800, Jason Wang wrote:
> On 04/26/2013 10:27 PM, Petr Matousek wrote:
> > On Fri, Apr 26, 2013 at 04:34:02PM +0800, Jason Wang wrote:
> >> There are several several issues in the current checking:
> >>
> >> - The check was based on the minus of unsigned values w
Necessary to change the name of ICCDevice's parent object field.
Signed-off-by: Andreas Färber
---
Could any of the APIC experts please review whether I'm touching any hot path?
Not sure if this was a mix of pre- and post-QOM code or intentional... Thanks.
hw/i386/kvm/apic.c | 4 ++--
Am 28.04.2013 12:22, schrieb Alexander Graf:
>
> On 28.04.2013, at 12:16, Andreas Färber wrote:
>
>> P.S. I am happy about your review comments, but you were not
>> intentionally CC'ed on a PReP patch - this is apparently the result of
>> Paolo having used hw/ppc/ pattern in the ppc TCG guest cor
On 28.04.2013, at 12:16, Andreas Färber wrote:
> Am 28.04.2013 11:44, schrieb Alexander Graf:
>>
>> On 28.04.2013, at 02:32, Andreas Färber wrote:
>>
>>> This prepares for switching from OpenHack'Ware to OpenBIOS.
>>>
>>> Signed-off-by: Andreas Färber
>>> ---
>>> hw/ppc/prep.c | 21 ++
Am 28.04.2013 11:44, schrieb Alexander Graf:
>
> On 28.04.2013, at 02:32, Andreas Färber wrote:
>
>> This prepares for switching from OpenHack'Ware to OpenBIOS.
>>
>> Signed-off-by: Andreas Färber
>> ---
>> hw/ppc/prep.c | 21 +
>> 1 file changed, 13 insertions(+), 8 deletions
On Tue, 04/23 10:24, Jeff Cody wrote:
> if (flags & BDRV_O_RDWR) {
> -ret = -ENOTSUP;
> -goto fail;
> +vhdx_update_headers(bs, s, false);
Do we really have to update the header immediately on RW open? I assume
when implementing vhdx_co_writev, this is guaranteed to get
On Tue, 04/23 10:24, Jeff Cody wrote:
> +/* opens the specified header block from the VHDX file header section */
> +static int vhdx_parse_header(BlockDriverState *bs, BDRVVHDXState *s)
> +{
> +int ret = 0;
> +vhdx_header *header1;
> +vhdx_header *header2;
> +uint64_t h1_seq = 0;
>
similiar -> similar
recieve -> receive
transfered -> transferred
preperation -> preparation
Most changes are in comments, one modifies a parameter name in a function
prototype.
The spelling fixes were made using codespell.
Signed-off-by: Stefan Weil
---
More improvements are needed for include
On 28.04.2013, at 02:32, Andreas Färber wrote:
> This prepares for switching from OpenHack'Ware to OpenBIOS.
>
> Signed-off-by: Andreas Färber
> ---
> hw/ppc/prep.c | 21 +
> 1 file changed, 13 insertions(+), 8 deletions(-)
>
> diff --git a/hw/ppc/prep.c b/hw/ppc/prep.c
> in
On 04/28/2013 04:25 PM, Michael S. Tsirkin wrote:
> On Sun, Apr 28, 2013 at 03:51:32PM +0800, Jason Wang wrote:
>> On 04/28/2013 03:32 AM, Michael S. Tsirkin wrote:
>>> On Sat, Apr 27, 2013 at 01:11:16PM +0800, Jason Wang wrote:
On 04/26/2013 08:26 PM, Michael S. Tsirkin wrote:
> On Fri, A
On 04/28/2013 04:32 PM, Michael S. Tsirkin wrote:
> On Fri, Apr 26, 2013 at 04:34:03PM +0800, Jason Wang wrote:
>> virtio-rng-ccw has zero config length, so we need validate the config length
>> before trying to access it. Otherwise we may crash since vdev->config is
>> NULL.
>>
>> Cc: Cornelia Hu
On 04/28/2013 04:31 PM, Michael S. Tsirkin wrote:
> On Fri, Apr 26, 2013 at 04:34:04PM +0800, Jason Wang wrote:
>> virtio-rng-s390 has zero config length, so no need to sync its config
>> otherwise
>> qemu will crash since vdev->config is NULL.
>>
>> Cc: Alexander Graf
>> Cc: Richard Henderson
>
On Sun, Apr 28, 2013 at 03:54:20PM +0800, Jason Wang wrote:
> On 04/28/2013 03:26 AM, Michael S. Tsirkin wrote:
> > On Fri, Apr 26, 2013 at 04:34:02PM +0800, Jason Wang wrote:
> >> There are several several issues in the current checking:
> >>
> >> - The check was based on the minus of unsigned val
On Fri, Apr 26, 2013 at 04:34:03PM +0800, Jason Wang wrote:
> virtio-rng-ccw has zero config length, so we need validate the config length
> before trying to access it. Otherwise we may crash since vdev->config is NULL.
>
> Cc: Cornelia Huck
> Cc: Richard Henderson
> Cc: Alexander Graf
> Signed
On Fri, Apr 26, 2013 at 04:34:04PM +0800, Jason Wang wrote:
> virtio-rng-s390 has zero config length, so no need to sync its config
> otherwise
> qemu will crash since vdev->config is NULL.
>
> Cc: Alexander Graf
> Cc: Richard Henderson
> Signed-off-by: Jason Wang
Actully, it validates get_co
On Sun, Apr 28, 2013 at 03:51:32PM +0800, Jason Wang wrote:
> On 04/28/2013 03:32 AM, Michael S. Tsirkin wrote:
> > On Sat, Apr 27, 2013 at 01:11:16PM +0800, Jason Wang wrote:
> >> On 04/26/2013 08:26 PM, Michael S. Tsirkin wrote:
> >>> On Fri, Apr 26, 2013 at 06:27:40PM +0800, Jason Wang wrote:
>
On 04/28/2013 03:26 AM, Michael S. Tsirkin wrote:
> On Fri, Apr 26, 2013 at 04:34:02PM +0800, Jason Wang wrote:
>> There are several several issues in the current checking:
>>
>> - The check was based on the minus of unsigned values which can overflow
>> - It was done after .{set|get}_config() whic
On 04/28/2013 03:32 AM, Michael S. Tsirkin wrote:
> On Sat, Apr 27, 2013 at 01:11:16PM +0800, Jason Wang wrote:
>> On 04/26/2013 08:26 PM, Michael S. Tsirkin wrote:
>>> On Fri, Apr 26, 2013 at 06:27:40PM +0800, Jason Wang wrote:
Commit 32993698 (vhost: disable on tap link down) tries to disabl
> +static void cpu_to_leguids(ms_guid *guid)
> +{
> +cpu_to_le32s(&guid->data1);
> +cpu_to_le16s(&guid->data2);
> +cpu_to_le16s(&guid->data3);
> +}
This one seems used in 5/5 only, so this patch fails compiling (defined
but not used with -Werror=unused-function). Maybe move to 5/5?
-
42 matches
Mail list logo
