From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Jan Beulich" Subject: Re: [PATCH] Fix cpu offline bug Date: Tue, 08 Mar 2011 08:52:23 +0000 Message-ID: <4D75FC5702000078000351B2@vpn.id2.novell.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: Content-Disposition: inline List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Keir Fraser Cc: Jinsong Liu , "xen-devel@lists.xensource.com" , Yunhong Jiang , Xin Li List-Id: xen-devel@lists.xenproject.org >>> On 05.03.11, Keir Fraser wrote: >Urk, good point! But your fix still allows what the comment says it is supposed to avoid - simply introducing a variable doesn't help afaict, you also need to insert a barrier() to avoid a second read (or "cpu_state" would need to be volatile). Jan