From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=53790 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PulEF-0004TV-Vl for qemu-devel@nongnu.org; Wed, 02 Mar 2011 07:32:33 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PulEE-00047u-Kx for qemu-devel@nongnu.org; Wed, 02 Mar 2011 07:32:31 -0500 Received: from mx1.redhat.com ([209.132.183.28]:38309) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PulEE-00047d-9t for qemu-devel@nongnu.org; Wed, 02 Mar 2011 07:32:30 -0500 Date: Wed, 2 Mar 2011 14:32:23 +0200 From: Alon Levy Subject: Re: [Qemu-devel] Re: kvm crashes with spice while loading qxl Message-ID: <20110302123223.GG18908@playa.tlv.redhat.com> References: <4D68F20D.2020401@web.de> <20110227190312.GA14445@playa.redhat.com> <4D6AA1DE.6020102@web.de> <20110301125846.GA19545@playa.tlv.redhat.com> <4D6DFE4B.6040805@web.de> <20110302105617.GB18908@playa.tlv.redhat.com> <4D6E2B40.8020507@web.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4D6E2B40.8020507@web.de> List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jan Kiszka Cc: xming , Gerd Hoffmann , kvm@vger.kernel.org, qemu-devel On Wed, Mar 02, 2011 at 12:34:24PM +0100, Jan Kiszka wrote: > On 2011-03-02 11:56, Alon Levy wrote: > > On Wed, Mar 02, 2011 at 09:22:35AM +0100, Jan Kiszka wrote: > >> On 2011-03-01 13:58, 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. > >>> > >>> So I didn't test with qemu not having this patch, but according to the discussion in the > >>> launchpad bug the problem only happens with qemu-kvm. This doesn't rule out it being a > >>> bug, perhaps it is just triggered much less frequently I guess. > >> > >> Again: qemu-kvm has the instrumentation to detect the bug, qemu is > >> lacking this, but both trees will break subtly if cpu_current_env is not > >> properly restored. > > > > ok, so what do you want to be done further before this patch is applied? > > The patch posted to qemu-devel just requires a changelog that correctly > reflects what it addresses (and where). Just sent, Alon > > Jan >