From owner-svn-src-head@FreeBSD.ORG Tue Jan 8 17:01:22 2013 Return-Path: Delivered-To: svn-src-head@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id 77918E7A; Tue, 8 Jan 2013 17:01:22 +0000 (UTC) (envelope-from freebsd@damnhippie.dyndns.org) Received: from duck.symmetricom.us (duck.symmetricom.us [206.168.13.214]) by mx1.freebsd.org (Postfix) with ESMTP id 480A0D79; Tue, 8 Jan 2013 17:01:21 +0000 (UTC) Received: from damnhippie.dyndns.org (daffy.symmetricom.us [206.168.13.218]) by duck.symmetricom.us (8.14.5/8.14.5) with ESMTP id r08H1FZQ028033; Tue, 8 Jan 2013 10:01:15 -0700 (MST) (envelope-from freebsd@damnhippie.dyndns.org) Received: from [172.22.42.240] (revolution.hippie.lan [172.22.42.240]) by damnhippie.dyndns.org (8.14.3/8.14.3) with ESMTP id r08H1BRN095165; Tue, 8 Jan 2013 10:01:11 -0700 (MST) (envelope-from freebsd@damnhippie.dyndns.org) Subject: Re: svn commit: r245147 - head/sys/arm/include From: Ian Lepore To: Konstantin Belousov In-Reply-To: <20130108155641.GG82219@kib.kiev.ua> References: <201301080240.r082eKVq080302@svn.freebsd.org> <20130108030022.GC82219@kib.kiev.ua> <50EBA947.1030902@freebsd.org> <20130108155641.GG82219@kib.kiev.ua> Content-Type: text/plain; charset="us-ascii" Date: Tue, 08 Jan 2013 10:01:11 -0700 Message-ID: <1357664471.1088.131.camel@revolution.hippie.lan> Mime-Version: 1.0 X-Mailer: Evolution 2.32.1 FreeBSD GNOME Team Port Content-Transfer-Encoding: 7bit Cc: svn-src-head@FreeBSD.org, Oleksandr Tymoshenko , svn-src-all@FreeBSD.org, src-committers@FreeBSD.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 08 Jan 2013 17:01:22 -0000 On Tue, 2013-01-08 at 17:56 +0200, Konstantin Belousov wrote: > On Mon, Jan 07, 2013 at 09:06:15PM -0800, Oleksandr Tymoshenko wrote: > > On 1/7/2013 7:00 PM, Konstantin Belousov wrote: > > > On Tue, Jan 08, 2013 at 02:40:20AM +0000, Oleksandr Tymoshenko wrote: > > >> Author: gonzo > > >> Date: Tue Jan 8 02:40:20 2013 > > >> New Revision: 245147 > > >> URL: http://svnweb.freebsd.org/changeset/base/245147 > > >> > > >> Log: > > >> Switch default cache type for ARMv6/ARMv7 from write-through to > > >> writeback-writeallocate > > > Could you, please, describe how this is supposed to work. > > > > > > Assume that a process mapped the same file page at two different > > > addresses, both read-write, and writes to both mappings. Usermode code > > > does not consider the need to flush caches or synchronize writes into > > > non-overlapping regions, usually. Would it break, i.e. could the values > > > appear in the page which were never written to it ? > > > > I might misunderstood question so let me rephrase it: > > One physical page P, virtual addresses A and B both mapped to it. Two > > conditions should > > be true: > > > > - if I write word to A+0x200 same value should appear at B+0x200 next > > time it is read > > - If there are no writes to P either through A or B each next read > > should yield same result. > I am more concerned with the following: > assume that current content in the page is 0x200:u, 0x201:v, and a byte > x was written at A+0x200, byte y at B+0x201. Is it possible that > future read of the bytes at A+0x200, A+0x201 (on the same core) > returns (x, v) ? > > > > > These conditions are met for ARMv7 devices for both WT and WBWA caches. > > They're PIPT > > so no aliasing in this case. Up until now I believed that "no aliasing" > > is true for all ARM CPUs > > we target but quick check proved me wrong: ARM1176 on which Raspberry Pi > > is based is prone to cache aliasing problem. Which might explain memory > > corruption > > easily reproducible under load. Again the problem is not related to > > cache type itself but > > to the lack of handling of this situation in pmap module. > I am trying to find a way through the ARM ARM and some additional texts. > Could it be that cache aliasing is only limited to the I-cache on ARM, > at least for recent cores ? My concern is hopefully not valid than. > > Hm, it seems, according to the link below, that ARMv6 is affected. > > > > > Some info on subject: > > http://blogs.arm.com/software-enablement/716-page-colouring-on-armv6-and-a-bit-on-armv7/ > > This seems to support the idea that mmap does not work on ARM, at least > for ARMv6. It seems that sf buffers do not obey the arch requirements > for aliasing as well. > > Thank you for the link. > > > > > Thank you for raising this topic. I hope people more ARM-savvy then me > > can confirm or refute > > my point of view. > > > > > > > > Another similar question, sf buffers on ARM flush the cache on sf_buf_free() > > > (sometimes). If the page, for which the sf buffer is created, mapped into > > > the userspace read-write, could it be that a value appears that was never > > > written ? Note that sf buffers are used e.g. by uiomove_fromphys(). > > I believe the stuff above is applicable to this question too. I'm just learning the armv6/v7 stuff myself right now, but I can answer your question for our current armv4/v5 implementation... When there is more than one mapping of a page in v4/v5 and any one of those mappings is writable, then the pmap.c code changes all existing mappings to be uncacheable to maintain coherency. If the writable mapping is removed and all that remains are read/exec mappings, the existing mappings are made cacheable again. Yes, that's just as inefficient and expensive as it sounds. :) -- Ian