From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.parisc-linux.org (palinux.external.hp.com [192.25.206.14]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "mail.parisc-linux.org", Issuer "CAcert Class 3 Root" (verified OK)) by ozlabs.org (Postfix) with ESMTPS id 14C26DE2F8 for ; Wed, 4 Jun 2008 04:55:56 +1000 (EST) Date: Tue, 3 Jun 2008 12:55:41 -0600 From: Matthew Wilcox To: Trent Piepho Subject: Re: MMIO and gcc re-ordering issue Message-ID: <20080603185541.GB3549@parisc-linux.org> References: <1211852026.3286.36.camel@pasglop> <20080602072403.GA20222@flint.arm.linux.org.uk> <200806031416.18195.nickpiggin@yahoo.com.au> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: Cc: linux-arch@vger.kernel.org, Nick Piggin , Russell King , linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org, scottwood@freescale.com, Linus Torvalds , David Miller , alan@lxorguk.ukuu.org.uk List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, Jun 03, 2008 at 11:47:00AM -0700, Trent Piepho wrote: > On Tue, 3 Jun 2008, Linus Torvalds wrote: > >On Tue, 3 Jun 2008, Nick Piggin wrote: > >> > >>Linus: on x86, memory operations to wc and wc+ memory are not ordered > >>with one another, or operations to other memory types (ie. load/load > >>and store/store reordering is allowed). Also, as you know, store/load > >>reordering is explicitly allowed as well, which covers all memory > >>types. So perhaps it is not quite true to say readl/writel is strongly > >>ordered by default even on x86. You would have to put in some > >>mfence instructions in them to make it so. > > So on x86, these could be re-ordered? > > writel(START_OPERATION, CONTROL_REGISTER); > status = readl(STATUS_REGISTER); You wouldn't ask for write-combining memory mapping for control or status registers. > >Well, you have to ask for WC/WC+ anyway, so it's immaterial. A driver that > >does that needs to be aware of it. IOW, it's a non-issue, imnsho. > > You need to ask for coherent DMA memory too. Different case. Coherent DMA memory is *host* memory that the *device* accesses. We're talking about *device* memory that the *cpu* accesses. -- Intel are signing my paycheques ... these opinions are still mine "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step."