Jan Kiszka a écrit :
> From: Jan Kiszka
>
> Since 0b57e287, cpu_memory_rw_debug already triggers a TB invalidation.
> As it doesn't (and cannot) set is_cpu_write_access=1 but "consumes" the
> currently executed TB, the tb_invalidate_phys_page_range call from
> patch_instruction didn't work anymor
From: Jan Kiszka
Since 0b57e287, cpu_memory_rw_debug already triggers a TB invalidation.
As it doesn't (and cannot) set is_cpu_write_access=1 but "consumes" the
currently executed TB, the tb_invalidate_phys_page_range call from
patch_instruction didn't work anymore.
Fix this by open-coding the r
From: Jan Kiszka
Cirrus is triggering this, e.g. during Win2k boot: Changes only on
disabled regions require no topology update when transaction depth drops
to 0 again.
Signed-off-by: Jan Kiszka
---
memory.c | 17 +++--
1 files changed, 15 insertions(+), 2 deletions(-)
diff --gi
On Sun, 4 Nov 2012, Jan Kiszka wrote:
I cannot comment on why it was done like this, just that this patch
breaks the Linux vmware X driver. Please fix or revert.
I'll look into it. If I (or someone else) don't come up with a fix today
you can revert.
Regards,
BALATON Zoltan
Hello,
Clean XP install cores with SCSI LSI 53C89A disk when copying files.
Isn't on the same file, so looks like a timing problem. Reproduceable.
Driver used is sym_hi. Details are below.
See also:
https://lists.gnu.org/archive/html/qemu-devel/2012-03/msg00523.html
Looks like problem is f
Ping?
On 01.11.2012 21:06, Gerhard Wiesinger wrote:
Fix crash with VNC under NT 4.0 and VMWare VGA and window which is
outside of the visible area.
Backtrace:
#0 set_bit (addr=, nr=-3) at ./bitops.h:122
#1 vnc_dpy_update (ds=, x=-48, y=145, w=57, h=161) at
ui/vnc.c:452
#2 0x7f1ce057e2e
Versions before gcc-4.6 don't support unnamed fields in initializers
(see http://gcc.gnu.org/bugzilla/show_bug.cgi?id=10676).
Offset and OffsetHigh belong to an unnamed struct which is part of an
unnamed union. Therefore the original code does not work with older
versions of gcc.
Signed-off-by: S
Am 03.11.2012 21:15, schrieb Blue Swirl:
On Sat, Nov 3, 2012 at 7:02 PM, Peter Maydell wrote:
On 3 November 2012 19:47, Blue Swirl wrote:
--- a/Makefile
+++ b/Makefile
@@ -122,7 +122,7 @@ subdir-pixman: pixman/Makefile
$(call quiet-command,$(MAKE) $(SUBDIR_MAKEFLAGS) -C pixman V="$(V
> -Original Message-
> From: Jan Kiszka [mailto:jan.kis...@web.de]
> Sent: Saturday, November 03, 2012 6:55 PM
> To: Hao, Xudong
> Cc: qemu-devel@nongnu.org; a...@redhat.com; k...@vger.kernel.org
> Subject: Re: [PATCH 1/2] qemu-kvm/cpuid: fix a emulation of guest physical
> address space
>
On 2012-11-04 13:15, Hao, Xudong wrote:
>> -Original Message-
>> From: Jan Kiszka [mailto:jan.kis...@web.de]
>> Sent: Saturday, November 03, 2012 6:55 PM
>> To: Hao, Xudong
>> Cc: qemu-devel@nongnu.org; a...@redhat.com; k...@vger.kernel.org
>> Subject: Re: [PATCH 1/2] qemu-kvm/cpuid: fix a
Keep saving display surface parameters at init and using these cached
values instead of getting them when needed. Not sure why this is
needed (maybe due to the interaction with the vga device) but not
doing this broke the Xorg vmware driver at least.
Signed-off-by: BALATON Zoltan
---
hw/vmware_v
On Sun, 4 Nov 2012, Jan Kiszka wrote:
breaks the Linux vmware X driver. Please fix or revert.
Couldn't really figure out what's happening but sent a patch which reverts
just these parts which seems to fix it for me.
Regards,
BALATON Zoltan
Ping? Original patch still applies on master.
On Mon, Oct 29, 2012 at 4:30 AM, Catalin Patulea wrote:
> On Mon, Oct 29, 2012 at 4:04 AM, Peter Maydell
> wrote:
>> I'm not fantastically enthused about bitfields, but since this is a test
>> program and not part of QEMU proper I don't think rewrit
David Gibson wrote:
> On Fri, Nov 02, 2012 at 11:58:32AM +0100, Juan Quintela wrote:
>> David Gibson wrote:
>> > On Wed, Oct 31, 2012 at 01:08:16PM +0200, Orit Wasserman wrote:
>> >> On 10/31/2012 05:43 AM, David Gibson wrote:
>>
>> Reviewed-by: Juan Quintela
>>
>> Good catch, I missunderstoo
On 11/04/2012 10:30 AM, Jan Kiszka wrote:
> From: Jan Kiszka
>
> Cirrus is triggering this, e.g. during Win2k boot: Changes only on
> disabled regions require no topology update when transaction depth drops
> to 0 again.
817dcc5368988b0 (pci: give each device its own address space) mad this
much
Paolo,
On current git master, if I do:
./configure --prefix=/home/somlo/KVM-OSX/SCRATCH --target-list=x86_64-softmmu
make clean
make
I end up with:
CCqemu-timer.o
qemu-timer.c: In function `init_timer_alarm':
qemu-timer.c:780:5: error: implicit declaration of function `pthread_atfork'
This compiler error is fixed by including pthread.h:
qemu-timer.c: In function ‘init_timer_alarm’:
qemu-timer.c:782: error: implicit declaration of function ‘pthread_atfork’
Another compiler warning is caused by the unused local function
reinit_timers on non-POSIX hosts. Include that function onl
Those constants are larger than 32 bits and need a suffix to avoid
warnings from some versions of gcc.
Signed-off-by: Stefan Weil
---
target-mips/dsp_helper.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/target-mips/dsp_helper.c b/target-mips/dsp_helper.c
index b591
"Gabriel L. Somlo" writes:
> Paolo,
>
> On current git master, if I do:
>
> ./configure --prefix=/home/somlo/KVM-OSX/SCRATCH --target-list=x86_64-softmmu
> make clean
> make
I just pushed a fix for this.
Regards,
Anthony Liguori
>
>
> I end up with:
>
> CCqemu-timer.o
> qemu-timer.c: I
A compiler warning is caused by the unused local function reinit_timers
on non-POSIX hosts. Include that function only for POSIX hosts.
Signed-off-by: Stefan Weil
---
qemu-timer.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/qemu-timer.c b/qemu-timer.c
index 8d9cf38..0d2bb94 100644
-
Avi Kivity writes:
> On 10/22/2012 09:04 AM, Philipp Hahn wrote:
>> Hello Doug,
>>
>> On Saturday 20 October 2012 00:46:43 Doug Goldstein wrote:
>>> I'm using libvirt 0.10.2 and I had qemu-kvm 1.1.1 running all my VMs.
>> ...
>>> I had upgraded to qemu-kvm 1.1.2
>> ...
>>> qemu: warning: error
On Fri, Nov 02, 2012 at 02:07:45PM +0100, Juan Quintela wrote:
> David Gibson wrote:
> > Asking for some advice on the list.
> >
> > I have prorotype savevm and migration support ready for the pseries
> > machine. They seem to work under simple circumstances (idle guest).
> > To test them more ex
On Fri, Nov 02, 2012 at 02:12:25PM +0200, Orit Wasserman wrote:
> On 11/02/2012 05:10 AM, David Gibson wrote:
> > Asking for some advice on the list.
> >
> > I have prorotype savevm and migration support ready for the pseries
> > machine. They seem to work under simple circumstances (idle guest).
On Sun, Nov 04, 2012 at 08:17:29PM +0100, Juan Quintela wrote:
> David Gibson wrote:
> > On Fri, Nov 02, 2012 at 11:58:32AM +0100, Juan Quintela wrote:
> >> David Gibson wrote:
> >> > On Wed, Oct 31, 2012 at 01:08:16PM +0200, Orit Wasserman wrote:
> >> >> On 10/31/2012 05:43 AM, David Gibson wrot
> -Original Message-
> From: Jan Kiszka [mailto:jan.kis...@web.de]
> Sent: Sunday, November 04, 2012 8:55 PM
> To: Hao, Xudong
> Cc: qemu-devel@nongnu.org; a...@redhat.com; k...@vger.kernel.org
> Subject: Re: [PATCH 1/2] qemu-kvm/cpuid: fix a emulation of guest physical
> address space
>
>
On Fri, Nov 02, 2012 at 08:31:23AM +0100, Gerd Hoffmann wrote:
> Hi,
>
> > +static inline void xhci_dma_write_u32s(XHCIState *xhci, dma_addr_t addr,
> > + uint32_t *buf, size_t len)
> > +{
> > +int i;
> > +
> > +for (i = 0; i < (len / sizeof(uint32_t
On Mon, Oct 29, 2012 at 4:51 PM, Paolo Bonzini wrote:
> Il 29/10/2012 00:48, Liu Ping Fan ha scritto:
>> Signed-off-by: Liu Ping Fan
>> ---
>> cpus.c|3 ++
>> exec.c| 58
>> +
>> qemu-thread.h |8 +++
>> vl.c
On Fri, Nov 2, 2012 at 6:39 PM, Jan Kiszka wrote:
> On 2012-10-29 00:48, Liu Ping Fan wrote:
>> Signed-off-by: Liu Ping Fan
>> ---
>> cpus.c|3 ++
>> exec.c| 58
>> +
>> qemu-thread.h |8 +++
>> vl.c
From: Liu Ping Fan
v1:
https://lists.gnu.org/archive/html/qemu-devel/2012-07/msg03312.html
v2:
http://lists.gnu.org/archive/html/qemu-devel/2012-08/msg01275.html
v3:
http://lists.nongnu.org/archive/html/qemu-devel/2012-09/msg01474.html
v4:
http://lists.gnu.org/archive/html/qemu-devel/2012-10/m
From: Liu Ping Fan
If out of global lock, we will be challenged by SMP in low level,
so need atomic ops.
This file is a wrapper of GCC atomic builtin.
Signed-off-by: Liu Ping Fan
---
include/qemu/atomic.h | 63 +
1 files changed, 63 insertions
From: Liu Ping Fan
Signed-off-by: Liu Ping Fan
---
include/qemu/object.h |3 ++-
qom/object.c | 11 +--
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/include/qemu/object.h b/include/qemu/object.h
index cc75fee..0c02614 100644
--- a/include/qemu/object.h
+
From: Liu Ping Fan
Without biglock, we try to protect the mr by increase refcnt.
If we can inc refcnt, go backward and resort to biglock.
Another point is memory radix-tree can be flushed by another
thread, so we should get the copy of terminal mr to survive
from such issue.
Signed-off-by: Liu
On Sun, Nov 4, 2012 at 3:51 PM, Anthony Liguori wrote:
> Avi Kivity writes:
>
>> On 10/22/2012 09:04 AM, Philipp Hahn wrote:
>>> Hello Doug,
>>>
>>> On Saturday 20 October 2012 00:46:43 Doug Goldstein wrote:
I'm using libvirt 0.10.2 and I had qemu-kvm 1.1.1 running all my VMs.
>>> ...
I
On Sun, Nov 04, 2012 at 12:09:34PM +0100, Stefan Weil wrote:
> Versions before gcc-4.6 don't support unnamed fields in initializers
> (see http://gcc.gnu.org/bugzilla/show_bug.cgi?id=10676).
>
> Offset and OffsetHigh belong to an unnamed struct which is part of an
> unnamed union. Therefore the or
From: Liu Ping Fan
To anti the recursive big lock, introduce separate interfaces to allow
address space dispatcher called with/without big lock.
Signed-off-by: Liu Ping Fan
---
cpu-common.h |3 +++
exec.c | 22 ++
kvm-all.c|6 +-
3 files changed, 26
From: Liu Ping Fan
When device unplug has been ack by guest, we first remove it from memory
to prevent incoming access from dispatcher. Then we isolate it from
device composition tree
Signed-off-by: Liu Ping Fan
---
hw/qdev.c | 26 ++
hw/qdev.h |3 ++-
2 files cha
From: Liu Ping Fan
After breaking down big lock, nested MMIO request which not targeting
at RAM can cause deadlock issue. Supposing the scene: dev_a,b with
fine-grain locks lockA/B, then ABBA dealock issue can be triggered.
We fix this by tracing and rejecting such request.
Signed-off-by: Liu Pi
From: Liu Ping Fan
For those address spaces which want to be able out of big lock, they
will be protected by their own local.
Signed-off-by: Liu Ping Fan
---
memory.c |1 +
memory.h |3 +++
2 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/memory.c b/memory.c
index 2f68d67
From: Liu Ping Fan
Signed-off-by: Liu Ping Fan
---
hw/acpi_piix4.c |2 +-
hw/pci.c| 13 -
hw/pci.h|1 +
3 files changed, 14 insertions(+), 2 deletions(-)
diff --git a/hw/acpi_piix4.c b/hw/acpi_piix4.c
index 15275cf..b45a016 100644
--- a/hw/acpi_piix4.c
+++
On 2012-11-05 03:42, Hao, Xudong wrote:
>> -Original Message-
>> From: Jan Kiszka [mailto:jan.kis...@web.de]
>> Sent: Sunday, November 04, 2012 8:55 PM
>> To: Hao, Xudong
>> Cc: qemu-devel@nongnu.org; a...@redhat.com; k...@vger.kernel.org
>> Subject: Re: [PATCH 1/2] qemu-kvm/cpuid: fix a em
On 2012-11-04 20:21, Avi Kivity wrote:
> On 11/04/2012 10:30 AM, Jan Kiszka wrote:
>> From: Jan Kiszka
>>
>> Cirrus is triggering this, e.g. during Win2k boot: Changes only on
>> disabled regions require no topology update when transaction depth drops
>> to 0 again.
>
> 817dcc5368988b0 (pci: give
On 2012-11-04 18:41, BALATON Zoltan wrote:
> Keep saving display surface parameters at init and using these cached
> values instead of getting them when needed. Not sure why this is
> needed (maybe due to the interaction with the vga device) but not
> doing this broke the Xorg vmware driver at leas
On 2012-11-05 06:38, Liu Ping Fan wrote:
> From: Liu Ping Fan
>
> Without biglock, we try to protect the mr by increase refcnt.
> If we can inc refcnt, go backward and resort to biglock.
s/can/cannot/ - could create confusion.
Jan
signature.asc
Description: OpenPGP digital signature
On 2012-11-05 06:38, Liu Ping Fan wrote:
> From: Liu Ping Fan
>
> After breaking down big lock, nested MMIO request which not targeting
> at RAM can cause deadlock issue. Supposing the scene: dev_a,b with
> fine-grain locks lockA/B, then ABBA dealock issue can be triggered.
> We fix this by traci
On 2012-11-05 06:38, Liu Ping Fan wrote:
> From: Liu Ping Fan
>
> v1:
> https://lists.gnu.org/archive/html/qemu-devel/2012-07/msg03312.html
>
> v2:
> http://lists.gnu.org/archive/html/qemu-devel/2012-08/msg01275.html
>
> v3:
> http://lists.nongnu.org/archive/html/qemu-devel/2012-09/msg01474.htm
On Sat, Nov 03, 2012 at 06:48:35PM +, Blue Swirl wrote:
> Fix build on a 32 bit host:
> CCmips-softmmu/target-mips/dsp_helper.o
> /src/qemu/target-mips/dsp_helper.c: In function 'helper_dextr_rs_w':
> /src/qemu/target-mips/dsp_helper.c:3556: error: integer constant is too large
> for 'lo
On Sun, Nov 04, 2012 at 09:29:35PM +0100, Stefan Weil wrote:
> Those constants are larger than 32 bits and need a suffix to avoid
> warnings from some versions of gcc.
>
> Signed-off-by: Stefan Weil
> ---
> target-mips/dsp_helper.c |4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
This patch series fixes the TCG arm backend for the MIPS target, as well
as for big endian targets when not using the ARMv6+ instructions set.
The corresponding patches are candidate for a stable release.
--
Changes v2 -> v3:
- patch 1:
- The new code allow up to 20 bits to be loaded (and not 2
The TCG arm backend considers likely that the offset to the TLB
entries does not exceed 12 bits for mem_index = 0. In practice this is
not true for at least the MIPS target.
The current patch fixes that by loading the bits 23-12 with a separate
instruction, and using loads with address writeback,
On an ARM host, the registers definitions from cpu.h clash
with /usr/include/sys/ucontext.h. As there are unused, just remove
them.
Cc: Jia Liu
Cc: qemu-sta...@nongnu.org
Reviewed-by: Peter Maydell
Signed-off-by: Aurelien Jarno
---
target-openrisc/cpu.h | 18 --
1 file change
The bswap16 TCG opcode assumes that the high bytes of the temp equal
to 0 before calling it. The ARM backend implementation takes this
assumption to slightly optimize the generated code.
The same implementation is called for implementing the cross-endian
qemu_st16 opcode, where this assumption is
在 2012-11-02五的 08:16 +0100，Stefan Hajnoczi写道：
> On Fri, Nov 2, 2012 at 6:11 AM, liguang wrote:
> > diff --git a/qemu-img.c b/qemu-img.c
> > index b41e670..d4ea800 100644
> > --- a/qemu-img.c
> > +++ b/qemu-img.c
> > @@ -337,10 +337,15 @@ static int img_create(int argc, char **argv)
> >
> > /*
> -Original Message-
> From: Blue Swirl [mailto:blauwir...@gmail.com]
> Sent: Saturday, November 03, 2012 6:44 PM
> To: Hao, Xudong
> Cc: qemu-devel@nongnu.org; a...@redhat.com; k...@vger.kernel.org
> Subject: Re: [Qemu-devel] [PATCH 2/2] qemu-kvm/pci-assign: 64 bits bar
> emulation
>
> On
qemu-img will complain when qcow or qcow2
size overflow for 64 bits, report the right
message in this condition.
Signed-off-by: liguang
---
qemu-img.c |7 ++-
1 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/qemu-img.c b/qemu-img.c
index b41e670..d9434ad 100644
--- a/qemu-i
On Fri, Nov 02, 2012 at 12:00:53PM +, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange"
>
> Over time various systemtap reserved words have been blacklisted
> in the trace backend generator. The list is not complete though,
> so there is continued risk of problems in the future. Preempt
On Fri, Oct 26, 2012 at 01:46:34PM +0200, Gerd Hoffmann wrote:
> Disable trace events prefixed with a '-'. Useful
> to enable a group of tracepoints with exceptions,
> like this:
>
> usb_xhci_port_*
> -usb_xhci_port_read
>
> which will enable all xhci port tracepoints except reads.
>
> Sign
Signed-off-by: Stefan Hajnoczi
---
docs/tracing.txt | 4
1 file changed, 4 insertions(+)
diff --git a/docs/tracing.txt b/docs/tracing.txt
index c541133..7901409 100644
--- a/docs/tracing.txt
+++ b/docs/tracing.txt
@@ -139,6 +139,10 @@ having a common prefix in a batch. For example, virtio-b
On Fri, Nov 2, 2012 at 10:43 PM, Paolo Bonzini wrote:
> Weakrefs only tell you if the symbol was defined elsewhere, so you
> need a further check at runtime to pick the default definition
> when needed.
>
> This could be automated by the compiler, but it does not do it.
>
> Signed-off-by: Paolo Bo
58 matches
Mail list logo
