From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH] convert init_rmode_tss to slots_lock Date: Wed, 19 Mar 2008 14:18:37 +0200 Message-ID: <47E1049D.7010407@qumranet.com> References: <20080318204234.GA18022@dmt> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm-devel To: Marcelo Tosatti Return-path: In-Reply-To: <20080318204234.GA18022@dmt> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: kvm-devel-bounces@lists.sourceforge.net Errors-To: kvm-devel-bounces@lists.sourceforge.net List-Id: kvm.vger.kernel.org Marcelo Tosatti wrote: > init_rmode_tss was forgotten during the conversion from mmap_sem to > slots_lock. > > Applied, thanks. > There are several sites reading the slots data without taking > slots_lock, including tdp_page_fault. I'll be sending a patch > to move acquision to vcpu_run as suggested by Avi later today. > You'll need to acquire it for KVM_SET_MSRS (since that can trigger a write, if writing to the clock base). -- error compiling committee.c: too many arguments to function ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/