From owner-freebsd-arm@FreeBSD.ORG Tue Apr 30 16:21:19 2013 Return-Path: Delivered-To: freebsd-arm@FreeBSD.org Received: from mx1.freebsd.org (mx1.FreeBSD.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id 223602DC for ; Tue, 30 Apr 2013 16:21:19 +0000 (UTC) (envelope-from ian@FreeBSD.org) Received: from mho-02-ewr.mailhop.org (mho-02-ewr.mailhop.org [204.13.248.72]) by mx1.freebsd.org (Postfix) with ESMTP id F2A5C1D2C for ; Tue, 30 Apr 2013 16:21:18 +0000 (UTC) Received: from c-24-8-230-52.hsd1.co.comcast.net ([24.8.230.52] helo=damnhippie.dyndns.org) by mho-02-ewr.mailhop.org with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.72) (envelope-from ) id 1UXDIs-0007IW-1P; Tue, 30 Apr 2013 16:21:18 +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 r3UGLFWr007533; Tue, 30 Apr 2013 10:21:15 -0600 (MDT) (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+ETJSeO7NeJLqciw8tUK2I Subject: Re: RFC: Patches with AXP support and pmap&smp fixes. From: Ian Lepore To: Grzegorz Bernacki In-Reply-To: <517E8610.5050204@semihalf.com> References: <517E8610.5050204@semihalf.com> Content-Type: text/plain; charset="us-ascii" Date: Tue, 30 Apr 2013 10:21:15 -0600 Message-ID: <1367338875.1180.44.camel@revolution.hippie.lan> Mime-Version: 1.0 X-Mailer: Evolution 2.32.1 FreeBSD GNOME Team Port Content-Transfer-Encoding: 7bit Cc: freebsd-arm@FreeBSD.org X-BeenThere: freebsd-arm@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Porting FreeBSD to the StrongARM Processor List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 30 Apr 2013 16:21:19 -0000 On Mon, 2013-04-29 at 16:39 +0200, Grzegorz Bernacki wrote: > Hi, > > I am going to submit some changes related to Armada XP support and some > general ARM fixes. You can find them at: > http://people.freebsd.org/~gber/armada > > It would be good if someone could review changes in generic ARM code i.e.: > 1) > http://people.freebsd.org/~gber/armada/0004-arm-smp-Fix-AP-processors-initialization-procedure.patch > > This patch fixes race condition in pcpu_init function. pcpu_init > performs operation on signly-linked tail queue and the queue can be > corrupted by secondary cpus initialization. > > 2) > http://people.freebsd.org/~gber/armada/0007-arm-Fix-L2-PTE-access-permissions-management.patch > http://people.freebsd.org/~gber/armada/0008-arm-Fix-page-reference-emulation-on-ARMv6-and-v7.patch > > These are changes which fixes reference simulation and access > permissions in pmap v6. > > It would be great if you could also review armada patches. > We will appreciate all comments and remarks. If there will be no > objections I am going to submit these changes at the beginning of the > next week. > > thanks, > greg I've reviewed them, and see no problems. It might not be a bad idea to paste the protections truth table from the commit message as a comment block in pmap_set_prot(); I had to keep referring to it while convincing myself the changes were right for every path through the routine. -- Ian