All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH for 2.1] qemu-char: initialize out_lock
@ 2014-06-25  7:04 Paolo Bonzini
  2014-06-25 10:26 ` Stefan Weil
  2014-06-26 12:32 ` Peter Maydell
  0 siblings, 2 replies; 3+ messages in thread
From: Paolo Bonzini @ 2014-06-25  7:04 UTC (permalink / raw)
  To: qemu-devel; +Cc: sw

Otherwise, Windows fails with a deadlock.

Reported-by: Stefan Weil <sw@weilnetz.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 qemu-char.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/qemu-char.c b/qemu-char.c
index 2e50a10..17bd360 100644
--- a/qemu-char.c
+++ b/qemu-char.c
@@ -94,6 +94,7 @@ static QTAILQ_HEAD(CharDriverStateHead, CharDriverState) chardevs =
 CharDriverState *qemu_chr_alloc(void)
 {
     CharDriverState *chr = g_malloc0(sizeof(CharDriverState));
+    qemu_mutex_init(&chr->chr_write_lock);
     return chr;
 }
 
-- 
1.9.3

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [Qemu-devel] [PATCH for 2.1] qemu-char: initialize out_lock
  2014-06-25  7:04 [Qemu-devel] [PATCH for 2.1] qemu-char: initialize out_lock Paolo Bonzini
@ 2014-06-25 10:26 ` Stefan Weil
  2014-06-26 12:32 ` Peter Maydell
  1 sibling, 0 replies; 3+ messages in thread
From: Stefan Weil @ 2014-06-25 10:26 UTC (permalink / raw)
  To: Paolo Bonzini, qemu-devel; +Cc: qemu-trivial

Am 25.06.2014 09:04, schrieb Paolo Bonzini:
> Otherwise, Windows fails with a deadlock.
>
> Reported-by: Stefan Weil <sw@weilnetz.de>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>  qemu-char.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/qemu-char.c b/qemu-char.c
> index 2e50a10..17bd360 100644
> --- a/qemu-char.c
> +++ b/qemu-char.c
> @@ -94,6 +94,7 @@ static QTAILQ_HEAD(CharDriverStateHead, CharDriverState) chardevs =
>  CharDriverState *qemu_chr_alloc(void)
>  {
>      CharDriverState *chr = g_malloc0(sizeof(CharDriverState));
> +    qemu_mutex_init(&chr->chr_write_lock);
>      return chr;
>  }
>  


Thanks, this fixes the problem with QEMU for Windows.

Tested-by: Stefan Weil <sw@weilnetz.de>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Qemu-devel] [PATCH for 2.1] qemu-char: initialize out_lock
  2014-06-25  7:04 [Qemu-devel] [PATCH for 2.1] qemu-char: initialize out_lock Paolo Bonzini
  2014-06-25 10:26 ` Stefan Weil
@ 2014-06-26 12:32 ` Peter Maydell
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Maydell @ 2014-06-26 12:32 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: Stefan Weil, QEMU Developers

On 25 June 2014 08:04, Paolo Bonzini <pbonzini@redhat.com> wrote:
> Otherwise, Windows fails with a deadlock.
>
> Reported-by: Stefan Weil <sw@weilnetz.de>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

Applied to master (with s/out_lock/chr_write_lock/ in the subject);
thanks.

-- PMM

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-06-26 12:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-25  7:04 [Qemu-devel] [PATCH for 2.1] qemu-char: initialize out_lock Paolo Bonzini
2014-06-25 10:26 ` Stefan Weil
2014-06-26 12:32 ` Peter Maydell

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.