On Wed, Jan 19, 2011 at 08:02:28PM +0100, Edgar E. Iglesias wrote:
> On Wed, Jan 19, 2011 at 03:02:26PM -0200, Marcelo Tosatti wrote:
> > On Tue, Jan 18, 2011 at 11:00:57AM +0100, Jan Kiszka wrote:
> > > On 2011-01-18 01:19, Edgar E. Iglesias wrote:
> > > > On Mon, Jan 17, 2011 at 11:03:08AM +0100,
On Sat, Jan 22, 2011 at 03:40:34PM +0100, Aurelien Jarno wrote:
> On Sat, Jan 22, 2011 at 03:32:36PM +0100, Alexander Graf wrote:
> >
> > On 22.01.2011, at 15:14, Edgar E. Iglesias wrote:
> >
> > > On Sat, Jan 22, 2011 at 02:13:03PM +0100, Aurelien Jarno wrote:
> > >> On Tue, Jan 18, 2011 at 01:5
On Sat, Jan 22, 2011 at 02:43:36PM +, Blue Swirl wrote:
> helper_addkc() in op_helper.c looks buggy, GCC 4.6.0 complains:
> CCmicroblaze-softmmu/op_helper.o
> /src/qemu/target-microblaze/op_helper.c: In function 'helper_addkc':
> /src/qemu/target-microblaze/op_helper.c:133:14: error: vari
On Sat, Jan 22, 2011 at 5:45 PM, Robert Reif wrote:
> Blue Swirl wrote:
>>
>> On Sat, Jan 22, 2011 at 3:30 PM, Mateusz Loskot
>> wrote:
>>
>>>
>>> On 18/01/11 21:51, Blue Swirl wrote:
>>>
On Tue, Jan 18, 2011 at 6:00 PM, Mateusz Loskot
wrote:
>
> On 18/01/11 17:36, Bl
Thanks ble,
The addk. Helper is now there only there to compute carry. I
Feel free to remove the unused statement and unusrd vars. Otherwise ill do
when im back again. Cheers
On Jan 22, 2011 3:43 PM, "Blue Swirl" wrote:
>
> helper_addkc() in op_helper.c looks buggy, GCC 4.6.0 complains:
> CC
2011/1/21 Raphaël Lefèvre :
> On Wed, Jan 19, 2011 at 2:13 AM, Stefano Bonifazi
> wrote:
>> On 01/18/2011 06:17 PM, Blue Swirl wrote:
>>>
>>> On Tue, Jan 18, 2011 at 9:29 AM, Stefano Bonifazi
>>> wrote:
Hi all!
I am working on qemu-user (qemu-ppc).
I'd like to edit the value
Blue Swirl wrote:
On Sat, Jan 22, 2011 at 3:30 PM, Mateusz Loskot wrote:
On 18/01/11 21:51, Blue Swirl wrote:
On Tue, Jan 18, 2011 at 6:00 PM, Mateusz Loskot wrote:
On 18/01/11 17:36, Blue Swirl wrote:
On Tue, Jan 18, 2011 at 3:27 PM, Mateusz Loskot
wrote:
On Sat, Jan 22, 2011 at 3:30 PM, Mateusz Loskot wrote:
> On 18/01/11 21:51, Blue Swirl wrote:
>> On Tue, Jan 18, 2011 at 6:00 PM, Mateusz Loskot wrote:
>>> On 18/01/11 17:36, Blue Swirl wrote:
On Tue, Jan 18, 2011 at 3:27 PM, Mateusz Loskot
wrote:
>
> Hi,
>
> Recen
On 18/01/11 21:51, Blue Swirl wrote:
> On Tue, Jan 18, 2011 at 6:00 PM, Mateusz Loskot wrote:
>> On 18/01/11 17:36, Blue Swirl wrote:
>>>
>>> On Tue, Jan 18, 2011 at 3:27 PM, Mateusz Loskot
>>> wrote:
Hi,
Recently, I have reported mysterious issues on NetBSD 5.1
emulated
helper_addkc() in op_helper.c looks buggy, GCC 4.6.0 complains:
CCmicroblaze-softmmu/op_helper.o
/src/qemu/target-microblaze/op_helper.c: In function 'helper_addkc':
/src/qemu/target-microblaze/op_helper.c:133:14: error: variable 'd'
set but not used [-Werror=unused-but-set-variable]
On Sat, Jan 22, 2011 at 03:32:36PM +0100, Alexander Graf wrote:
>
> On 22.01.2011, at 15:14, Edgar E. Iglesias wrote:
>
> > On Sat, Jan 22, 2011 at 02:13:03PM +0100, Aurelien Jarno wrote:
> >> On Tue, Jan 18, 2011 at 01:58:25PM +0100, Jan Kiszka wrote:
> >>> On 2011-01-18 13:05, Alexander Graf wr
> From: qemu-devel-bounces+wschi=dolby@nongnu.org
> [mailto:qemu-devel-bounces+wschi=dolby@nongnu.org] On
> Behalf Of Schildbach, Wolfgang
>
> > -Original Message-
> > From: qemu-devel-bounces+wschi=dolby@nongnu.org
> > [mailto:qemu-devel-bounces+wschi=dolby@nongnu.org] O
On Sat, Jan 22, 2011 at 03:28:06PM +0100, Alexander Graf wrote:
>
> On 22.01.2011, at 14:13, Aurelien Jarno wrote:
>
> > On Tue, Jan 18, 2011 at 01:58:25PM +0100, Jan Kiszka wrote:
> >> On 2011-01-18 13:05, Alexander Graf wrote:
> >>>
> >>> On 18.01.2011, at 10:08, Gerd Hoffmann wrote:
> >>>
>
On 22.01.2011, at 15:14, Edgar E. Iglesias wrote:
> On Sat, Jan 22, 2011 at 02:13:03PM +0100, Aurelien Jarno wrote:
>> On Tue, Jan 18, 2011 at 01:58:25PM +0100, Jan Kiszka wrote:
>>> On 2011-01-18 13:05, Alexander Graf wrote:
On 18.01.2011, at 10:08, Gerd Hoffmann wrote:
> Hi
On 22.01.2011, at 14:13, Aurelien Jarno wrote:
> On Tue, Jan 18, 2011 at 01:58:25PM +0100, Jan Kiszka wrote:
>> On 2011-01-18 13:05, Alexander Graf wrote:
>>>
>>> On 18.01.2011, at 10:08, Gerd Hoffmann wrote:
>>>
Hi,
>> Worse might also be that unknown issue that force you to inj
On Fri, Jan 21, 2011 at 11:25:05PM +, Peter Maydell wrote:
> On 21 January 2011 10:07, Christophe Lyon wrote:
> > I have developed some tests for ARM-Neon in the form of C sources files
> > calling ARM Neon intrinsics, and comparing the results of the resulting
> > program with a known referen
On Sat, Jan 22, 2011 at 02:13:03PM +0100, Aurelien Jarno wrote:
> On Tue, Jan 18, 2011 at 01:58:25PM +0100, Jan Kiszka wrote:
> > On 2011-01-18 13:05, Alexander Graf wrote:
> > >
> > > On 18.01.2011, at 10:08, Gerd Hoffmann wrote:
> > >
> > >> Hi,
> > >>
> > Worse might also be that unknown
On Tue, Jan 18, 2011 at 01:58:25PM +0100, Jan Kiszka wrote:
> On 2011-01-18 13:05, Alexander Graf wrote:
> >
> > On 18.01.2011, at 10:08, Gerd Hoffmann wrote:
> >
> >> Hi,
> >>
> Worse might also be that unknown issue that force you to inject an IRQ
> here. We don't know. That's probab
Check if the backend option is missing before searching the backend
table. This fixes a NULL pointer dereference when QEMU is invoked with
the following invalid command-line:
$ qemu -chardev id=foo,path=/tmp/socket
Previously QEMU would segfault, now it produces this error message:
chardev:
QEMU already has a visible container_of() macro definition. Avoid the
compiler error.
Signed-off-by: Stefan Hajnoczi
---
continuation.h |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/continuation.h b/continuation.h
index 585788e..112df65 100644
--- a/continuation.h
++
Copied from a fix to the original code by Anthony Liguori
.
Signed-off-by: Stefan Hajnoczi
---
coroutine_ucontext.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/coroutine_ucontext.c b/coroutine_ucontext.c
index 25e8687..f76da94 100644
--- a/coroutine_ucontext.c
+++ b/
Signed-off-by: Stefan Weil
---
hw/ppc405_boards.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/hw/ppc405_boards.c b/hw/ppc405_boards.c
index b1d2b94..912757b 100644
--- a/hw/ppc405_boards.c
+++ b/hw/ppc405_boards.c
@@ -560,6 +560,7 @@ static void taihu_405ep_init(ram_a
Signed-off-by: Stefan Weil
---
hw/pci.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/hw/pci.c b/hw/pci.c
index 63f476a..38a60ae 100644
--- a/hw/pci.c
+++ b/hw/pci.c
@@ -1877,6 +1877,7 @@ static int pci_add_option_rom(PCIDevice *pdev, bool
is_default_rom)
if (size
Signed-off-by: Stefan Weil
---
hw/s390-virtio.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/hw/s390-virtio.c b/hw/s390-virtio.c
index f29b624..850422f 100644
--- a/hw/s390-virtio.c
+++ b/hw/s390-virtio.c
@@ -204,6 +204,7 @@ static void s390_init(ram_addr_t ram_size,
Hello,
For those who still use kqemu (not all cpu's support kvm;), here is a
small patch that re-enable the init_MUTEX() macro, whose removal
was planed in 2009: see https://lkml.org/lkml/2009/7/26/10
Signed-off-by: christophe pernod
---
diff -up kqemu/kqemu-linux.c.ORG kqemu/kqemu-linux.c
---
Asynchronous image format code is becoming very complex. Let's try
using coroutines to write sequential code without callbacks but use
coroutines to switch stacks under the hood.
Signed-off-by: Stefan Hajnoczi
---
Makefile.objs|2 +-
continuation.c | 87 +
QCOW2 with coroutines is not safe because synchronous code paths are no
longer guaranteed to execute without interference from pending requests.
A blocking call like bdrv_pread() causes the coroutine to yield and
another request can be processed during that time, causing to race
conditions or inter
Add a function to get the current coroutine. There is always a current
coroutine, either the "leader" (default main coroutine) or a specific
coroutine created with qemu_coroutine_create().
Signed-off-by: Stefan Hajnoczi
---
qemu-coroutine.c |5 +
qemu-coroutine.h |5 +
2 files c
Implement transparent support for the following synchronous I/O
functions to be called from coroutines:
bdrv_read()
bdrv_write()
bdrv_pread()
bdrv_pwrite()
bdrv_pwrite_sync()
bdrv_write_sync()
bdrv_flush()
These functions will use asynchronous I/O behind the scenes but
otherwise behave the same a
Converting qcow2 to use coroutines is fairly simple since most of qcow2
is synchronous. The synchronous I/O functions likes bdrv_pread() now
transparently work when called from a coroutine, so all the synchronous
code just works.
The explicitly asynchronous code is adjusted to repeatedly call
qco
The qemu_in_coroutine() function checks whether or not we are currently
in a user coroutine. This can be used to distinguish between executing
normal vcpu or iothread code from executing a coroutine.
Signed-off-by: Stefan Hajnoczi
---
qemu-coroutine.c |5 +
qemu-coroutine.h |7 +
Add coroutine versions of the bdrv_aio_readv() and bdrv_aio_writev()
functions. The coroutine version doesn't take a callback and simply
returns the error value. Behind the scenes they are implemented using
bdrv_aio_readv() and bdrv_aio_writev().
Signed-off-by: Stefan Hajnoczi
---
block.c |
Add functions to create coroutines and transfer control into a coroutine
and back out again.
Signed-off-by: Stefan Hajnoczi
---
Makefile.objs|2 +-
qemu-coroutine.c | 40
qemu-coroutine.h | 76 ++
3 file
This patch series prototypes making QCOW2 fully asynchronous to eliminate the
timing jitter and poor performance that has been observed. QCOW2 has
asynchronous I/O code paths for some of the read/write common cases but
metadata access is always synchronous.
One solution is to rewrite QCOW2 to be
Each pthread should have its own current coroutine. This ensures
coroutines are thread-safe.
Signed-off-by: Stefan Hajnoczi
---
coroutine_ucontext.c |5 -
1 files changed, 0 insertions(+), 5 deletions(-)
diff --git a/coroutine_ucontext.c b/coroutine_ucontext.c
index f76da94..289e5bd 10
Make it possible to check whether a coroutine is the default main
coroutine (the "leader") or not.
Signed-off-by: Stefan Hajnoczi
---
coroutine.h |2 ++
coroutine_ucontext.c |5 +
2 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/coroutine.h b/coroutine.h
index
On 20 January 2011 17:08, Stefan Weil wrote:
> Yes, that's a problem with some parts of the old code.
> For files which you want to modify, you could remove
> the spaces with your script before applying your other
> modifications and create a separate patch which only
> removes the superfluous spa
37 matches
Mail list logo