[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: kvm crashes with spice while loading qxl
From: |
Alon Levy |
Subject: |
Re: [Qemu-devel] Re: kvm crashes with spice while loading qxl |
Date: |
Sun, 27 Feb 2011 21:29:18 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Sun, Feb 27, 2011 at 08:27:01PM +0100, Jan Kiszka wrote:
> On 2011-02-27 20:16, Alon Levy wrote:
> > On Sun, Feb 27, 2011 at 08:11:26PM +0100, Jan Kiszka wrote:
> >> On 2011-02-27 20:03, Alon Levy wrote:
> >>> On Sat, Feb 26, 2011 at 01:29:01PM +0100, Jan Kiszka wrote:
> >>>> On 2011-02-26 12:43, xming wrote:
> >>>>> When trying to start X (and it loads qxl driver) the kvm process just
> >>>>> crashes.
> >>>
> >>> This is fixed by Gerd's attached patch (taken from rhel repository, don't
> >>> know
> >>> why it wasn't pushed to qemu-kvm upstream). I'll send it to kvm list as
> >>> well (separate email).
> >>
> >> Patch looks OK on first glance, but the changelog is misleading: This
> >> was broken for _both_ trees, but upstream didn't detect the bug.
> >>
> >
> > The trees the patch commit message refers to are qemu and qemu-kvm.
>
> The same did I.
>
> > qemu doesn't even have cpu_single_env.
>
> Really? Check again. :)
Sorry, grepped the wrong repo. I'll send this to qemu-devel too then.
>
> > It didn't talk about two qemu-kvm trees.
> >
> >> My concerns regarding other side effects of juggling with global mutex
> >> in spice code remain.
> >
> > I know there used to be a mutex in spice code and during the upstreaming
> > process it
> > got ditched in favor of the qemu global io mutex. I would have rather
> > deferred this
> > to Gerd since he wrote this, but he is not available atm.
>
> It's not necessarily bad to drop the io mutex, but it is more tricky
> than it may appear on first glance.
>
> Jan
>