From owner-svn-src-head@FreeBSD.ORG Sat Mar 8 15:13:49 2014 Return-Path: Delivered-To: svn-src-head@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 037C7698; Sat, 8 Mar 2014 15:13:49 +0000 (UTC) Received: from mho-01-ewr.mailhop.org (mho-03-ewr.mailhop.org [204.13.248.66]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id C84A5B9F; Sat, 8 Mar 2014 15:13:48 +0000 (UTC) Received: from c-24-8-230-52.hsd1.co.comcast.net ([24.8.230.52] helo=damnhippie.dyndns.org) by mho-01-ewr.mailhop.org with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.72) (envelope-from ) id 1WMIwd-000G1u-Fn; Sat, 08 Mar 2014 15:13:47 +0000 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 s28FDj66053521; Sat, 8 Mar 2014 08:13:45 -0700 (MST) (envelope-from ian@FreeBSD.org) X-Mail-Handler: Dyn Standard SMTP by Dyn X-Originating-IP: 24.8.230.52 X-Report-Abuse-To: abuse@dyndns.com (see http://www.dyndns.com/services/sendlabs/outbound_abuse.html for abuse reporting information) X-MHO-User: U2FsdGVkX1+9A82oFzZS/8ThnBKlvrTL Subject: Re: svn commit: r262411 - head/sys/arm/arm From: Ian Lepore To: Konstantin Belousov In-Reply-To: <20140307152111.GL24664@kib.kiev.ua> References: <201402232252.s1NMqmI5075701@svn.freebsd.org> <20140305115402.GC24664@kib.kiev.ua> <1394025767.1149.327.camel@revolution.hippie.lan> <20140307152111.GL24664@kib.kiev.ua> Content-Type: text/plain; charset="us-ascii" Date: Sat, 08 Mar 2014 08:13:45 -0700 Message-ID: <1394291625.1149.405.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, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.17 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: Sat, 08 Mar 2014 15:13:49 -0000 On Fri, 2014-03-07 at 17:21 +0200, Konstantin Belousov wrote: > On Wed, Mar 05, 2014 at 06:22:47AM -0700, Ian Lepore wrote: > > On Wed, 2014-03-05 at 13:54 +0200, Konstantin Belousov wrote: > > > On Sun, Feb 23, 2014 at 10:52:48PM +0000, Ian Lepore wrote: > > > > Author: ian > > > > Date: Sun Feb 23 22:52:48 2014 > > > > New Revision: 262411 > > > > URL: http://svnweb.freebsd.org/changeset/base/262411 > > > > > > > > Log: > > > > If the L2 cache type is PIPT, pass a physical address for a flush. > > > > > > > > While this is technically more correct, I don't think it much matters, > > > > because the only thing in the tree that calls cpu_flush_dcache() is md(4) > > > > and I'm > 99% sure it's bogus that it does so; md has no ability to do > > > > anything that can perturb data cache coherency. > > > > > > Yes, md(4) does not break data cache coherency, but I think that > > > Marcel added the flush to ensure instruction cache coherency. The > > > intent was to ensure that harward-architecture machines would > > > see up-to-date memory content when fetching instructions after > > > read on md(4). > > > > Oh. If that's necessary on ia64, it seems like ia64/elf_machdep.c would > > be the place to do the flush. > > I am not sure about ia64, it was needed for PowerPC, I think. > The issue is not limited to the module loads, so elf_machdep.c cannot > solve the problem. The part of the commit message for r192323 about ARM is just wrong. I don't know enough about the other platforms mentioned in there to comment on those, but the flush call which that commit message bills as avoiding pessimization just maximially pessimizes things on ARM. -- Ian