From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: [PATCH v2 11/23] MIPS: KVM: Fix timer race modifying guest CP0_Cause Date: Thu, 29 May 2014 13:31:41 +0200 Message-ID: <53871A9D.2080309@redhat.com> References: <1401355005-20370-1-git-send-email-james.hogan@imgtec.com> <1401355005-20370-12-git-send-email-james.hogan@imgtec.com> <53870D99.3030900@redhat.com> <5387122C.2080203@imgtec.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Cc: Andreas Herrmann , Gleb Natapov , kvm@vger.kernel.org, Ralf Baechle , linux-mips@linux-mips.org, Sanjay Lal To: James Hogan Return-path: Received: from mx1.redhat.com ([209.132.183.28]:49491 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751924AbaE2LdD (ORCPT ); Thu, 29 May 2014 07:33:03 -0400 In-Reply-To: <5387122C.2080203@imgtec.com> Sender: kvm-owner@vger.kernel.org List-ID: Il 29/05/2014 12:55, James Hogan ha scritto: >> > Shouldn't you have a loop too around the ll/sc? > Yes, it has a do {} while () look around the inline asm, although I > didn't mention it in the commit message. It's modelled on > arch/mips/include/asm/bitops.h. Ugh, sorry---I misread that as a do/while(0). But it is an inline function, not a macro. Paolo