From owner-freebsd-arm@FreeBSD.ORG Mon Aug 19 16:26:14 2013 Return-Path: Delivered-To: freebsd-arm@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 5DE14D18 for ; Mon, 19 Aug 2013 16:26:14 +0000 (UTC) (envelope-from zbb@semihalf.com) Received: from mail-ea0-f172.google.com (mail-ea0-f172.google.com [209.85.215.172]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id D29B92910 for ; Mon, 19 Aug 2013 16:26:13 +0000 (UTC) Received: by mail-ea0-f172.google.com with SMTP id r16so2401119ead.17 for ; Mon, 19 Aug 2013 09:26:06 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-gm-message-state:message-id:date:from:organization:user-agent :mime-version:to:cc:subject:references:in-reply-to:content-type :content-transfer-encoding; bh=PiMQz8OCuOwxVhtKS5C4QpMtr25K1VIuUJV8g2qUycA=; b=FTyj/haWDGJjiLfj7GeP1WimP8mIypHiA5Ypik3vBMPctmeyQadzdA91YzaGHUamZW TBQlgQk7JyTmGmx0U2GllbdSakLc5F48fIQCgE1+LxLDdsk5L4xkDfrB5ouRKjdFTL6k sm1iwsjYQrlgTyJWf/Ygsp61zCM4Qdi0sFwmcg6XqY/kwtf/aWXVf216+BZQ3rL8w6GX evc0aEDnyZXOGNttPrZWkfWOLHprEEnLYPYPP4z9V8Fm3HLyazP7acrmCRJJNPLh45Lm zffqPntzAEsrTmysLEQWkb2Qi0i9ucqlxJK8NodQe3OEhYhzqjachfv03SeU4RHn8l1E zjpg== X-Gm-Message-State: ALoCoQkRBgXIqa3/SZ0DSUWTDDl6Ac0YUwnxopC3TCu5p2EpLSe2gCHcLxRJKCdTHYdU4dU3clGA X-Received: by 10.14.246.11 with SMTP id p11mr24256552eer.9.1376929566168; Mon, 19 Aug 2013 09:26:06 -0700 (PDT) Received: from [10.0.2.117] (cardhu.semihalf.com. [213.17.239.108]) by mx.google.com with ESMTPSA id n48sm18457700eeg.17.1969.12.31.16.00.00 (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Mon, 19 Aug 2013 09:26:05 -0700 (PDT) Message-ID: <52124719.5000104@semihalf.com> Date: Mon, 19 Aug 2013 18:26:01 +0200 From: Zbyszek Bodek Organization: Semihalf User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130623 Thunderbird/17.0.7 MIME-Version: 1.0 To: "freebsd-arm@freebsd.org" Subject: Re: New pmap-v6.c features and improvements References: <519B6B1C.9060008@semihalf.com> <20130522184232.GA437@jail.io> <519E0D62.5030708@semihalf.com> <51CC4CC1.4020509@semihalf.com> <51D57456.9080504@semihalf.com> <51F68F58.8060600@semihalf.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: ray@freebsd.org, Alan Cox 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: Mon, 19 Aug 2013 16:26:14 -0000 On 01.08.2013 16:19, Zbigniew Bodek wrote: > 2013/7/29 Zbyszek Bodek > > > On 04.07.2013 19:34, Warner Losh wrote: > > > > On Jul 4, 2013, at 7:10 AM, Zbyszek Bodek wrote: > > > >> On 27.06.2013 16 :31, Zbyszek Bodek wrote: > >>> On 23.05.2013 14:36, Zbyszek Bodek wrote: > >>>> On 22.05.2013 20:42, Ruslan Bukin wrote: > >>>>> On Tue, May 21, 2013 at 02:39:56PM +0200, Zbyszek Bodek wrote: > >>>>>> Hello Everyone, > >>>>>> > >>>>>> I would like to introduce another pack of patches for > pmap-v6.c and > >>>>>> related, that we created as a part of Semihalf work on Superpages > >>>>>> support. > >>>>>> > >>>>>> The patches include some major changes like: > >>>>>> > >>>>>> - Switch to AP[1:0] access permissions model > >>>>>> - Transition of the mapping related flags to PTE (stop using > PVF_ flags > >>>>>> in pv_entry) > >>>>>> - Rework of the pmap_enter_locked() function > >>>>>> - pmap code optimizations > >>>>>> > >>>>>> And some minor clean-ups: > >>>>>> > >>>>>> - Get rid of the VERBOSE_INIT_ARM option > >>>>>> - Clean-ups, style and naming improvements to pmap > >>>>>> > >>>>>> Please check out the attachment for details. > >>>>>> > >>>>>> I will be happy to answer your questions and doubts if any. > >>>>>> > >>>>>> Best regards > >>>>>> Zbyszek Bodek > >>>>> > >>>>> I tested new patches with exynos5 and everything is OK. > >>>>> (I mean all works as usual) > >>>>> > >>>> > >>>> Hello. > >>>> > >>>> I'm happy to announce that code has been integrated to the > FreeBSD HEAD. > >>>> Great thanks your help! > >>>> > >>> > >>> Hello Everyone, > >>> > >>> We have two micro patches for pmap-v6.c containing fix for > 'modified' > >>> bit emulation and removal of the redundant PGA_WRITEABLE clearing. > >>> > >>> Please check out the attachment. > >>> > >>> These two are minimal changes and we would like to commit them > soon, so > >>> we would be grateful if you could test them on your ARMv6/v7 > platforms > >>> and give us your remarks. > >>> > >> > >> Hello, > >> > >> Because there were no objections, we've integrated the patches: > >> > >> http://svnweb.freebsd.org/base?view=revision&revision=252694 > >> http://svnweb.freebsd.org/base?view=revision&revision=252695 > > > Hello Everyone, > > I'm sending another set of fixes for pmap-v6.c > Please see attachment. > > 0012 - Removal of the costly, frequent sweeping through the > pv_entries whenever pmap_nuke_pv(), pmap_modify_pv(), etc. > are called. This also makes order with clearing PGA_WRITEABLE > aflag as well as with the pmap_statistics related to the > wired_count. > 0013 - Fix for pamp_set_prot() where not all of the protection bits were > cleared before the actual configuration. L2_XN is not icluded to > the L2_S_PROT mask to maintain consistency between PROT_MASKS for > L2 and L1 descriptors - contain only kernel/user and > read/write permissions bits. > 0014 - Fix for pmap_change_wiring() where "wired" indicator of type > boolean was being used as a "wired flag" passed to > pmap_modify_pv() which was obviously not a valid PVF_WIRED flag. > > Please test those patches on your ARM-based machines and send your > feedback. We will also appreciate your reviews and remarks. > > Thank you in advance for your help. > > > Hello, > > Are there any updates in this topic? > In addition, I'm attaching two more minor improvements, this time: > > 0015 - small clean-ups in pmap_clearbit() that in general gives one relevant > improvement which is to skip redundant "dirty" setting for a > page. > 0016 - removal of the unused pv_kva field from the md_page structure > > If there are no objections to that and earlier presented changes we > would like to > integrate them to the HEAD soon. > Hello, The earlier mentioned patches have been integrated to the FreeBSD HEAD. Please check out below: http://svnweb.freebsd.org/base?view=revision&revision=254531 http://svnweb.freebsd.org/base?view=revision&revision=254532 http://svnweb.freebsd.org/base?view=revision&revision=254533 http://svnweb.freebsd.org/base?view=revision&revision=254535 http://svnweb.freebsd.org/base?view=revision&revision=254536 Best regards Zbyszek Bodek