From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965181AbWD1FXw (ORCPT ); Fri, 28 Apr 2006 01:23:52 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S965182AbWD1FXw (ORCPT ); Fri, 28 Apr 2006 01:23:52 -0400 Received: from 216-99-217-87.dsl.aracnet.com ([216.99.217.87]:59010 "EHLO sorel.sous-sol.org") by vger.kernel.org with ESMTP id S965181AbWD1FXv (ORCPT ); Fri, 28 Apr 2006 01:23:51 -0400 Date: Thu, 27 Apr 2006 22:23:28 -0700 From: Chris Wright To: Andi Kleen Cc: Linux Kernel Mailing List , zach@vmware.com, torvalds@osdl.org Subject: Re: [PATCH] x86/PAE: Fix pte_clear for the >4GB RAM case Message-ID: <20060428052328.GA4409@sorel.sous-sol.org> References: <200604272001.k3RK1dmX007637@hera.kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.4.2.1i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org * Andi Kleen (ak@suse.de) wrote: > > +static inline void pmd_clear(pmd_t *pmd) > > +{ > > + u32 *tmp = (u32 *)pmd; > > + *tmp = 0; > > + smp_wmb(); > > + *(tmp + 1) = 0; > > +} > > I think that's still wrong - it should be wmb() not smp_wmb because this > problem can happen on a UP kernel already. I thought the barrier is to keep compiler from reordering not processor.