From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH RFC] kvm: write protect memory after slot swap Date: Mon, 25 Oct 2010 13:50:40 +0200 Message-ID: <4CC56F10.8040803@redhat.com> References: <20101025012124.GA31262@redhat.com> <4CC54EB8.9020604@redhat.com> <4CC56CC6.4020703@siemens.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: "Michael S. Tsirkin" , Marcelo Tosatti , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , "x86@kernel.org" , Gleb Natapov , Sheng Yang , "kvm@vger.kernel.org" , "linux-kernel@vger.kernel.org" To: Jan Kiszka Return-path: Received: from mx1.redhat.com ([209.132.183.28]:57729 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751244Ab0JYLu7 (ORCPT ); Mon, 25 Oct 2010 07:50:59 -0400 In-Reply-To: <4CC56CC6.4020703@siemens.com> Sender: kvm-owner@vger.kernel.org List-ID: On 10/25/2010 01:40 PM, Jan Kiszka wrote: > > > > Excellent catch, I stared at this code for a while and didn't see the > > bug. Patch applied. > > > > BTW, while this was an annoying one for graphic emulation, wasn't it > potentially lethal for live migration? Deadly. Yes autofs passed it happily. We need a unit test that bangs on pages and the bitmap with tighter timing. > The issue looks like is was introduced with the switch to SRCU, so every > kernel since 2.6.34 should be affected, correct? Yes. -- error compiling committee.c: too many arguments to function