From owner-svn-src-all@FreeBSD.ORG Mon Sep 16 10:34:44 2013 Return-Path: Delivered-To: svn-src-all@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 C61ACC87; Mon, 16 Sep 2013 10:34:44 +0000 (UTC) (envelope-from zbb@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:1900:2254:2068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id B33E22E40; Mon, 16 Sep 2013 10:34:44 +0000 (UTC) Received: from svn.freebsd.org ([127.0.1.70]) by svn.freebsd.org (8.14.7/8.14.7) with ESMTP id r8GAYiN2078473; Mon, 16 Sep 2013 10:34:44 GMT (envelope-from zbb@svn.freebsd.org) Received: (from zbb@localhost) by svn.freebsd.org (8.14.7/8.14.5/Submit) id r8GAYiuY078472; Mon, 16 Sep 2013 10:34:44 GMT (envelope-from zbb@svn.freebsd.org) Message-Id: <201309161034.r8GAYiuY078472@svn.freebsd.org> From: Zbigniew Bodek Date: Mon, 16 Sep 2013 10:34:44 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r255611 - head/sys/arm/arm X-SVN-Group: head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 16 Sep 2013 10:34:44 -0000 Author: zbb Date: Mon Sep 16 10:34:44 2013 New Revision: 255611 URL: http://svnweb.freebsd.org/changeset/base/255611 Log: Write protect base page after superpage demotion so that it may repromote When clearing the modification status of the superpage, one of the base pages produced during demotion should be marked as write disabled. The intention is that subsequent write access may repromote. In the current implementation this was done wrong as write permission was granted instead of forbidden. Approved by: cognet (mentor) Approved by: re Modified: head/sys/arm/arm/pmap-v6.c Modified: head/sys/arm/arm/pmap-v6.c ============================================================================== --- head/sys/arm/arm/pmap-v6.c Mon Sep 16 10:06:40 2013 (r255610) +++ head/sys/arm/arm/pmap-v6.c Mon Sep 16 10:34:44 2013 (r255611) @@ -987,7 +987,7 @@ pmap_clearbit(struct vm_page *m, u_int m ("pmap_clearbit: no PV " "entry for managed mapping")); pve->pv_flags &= ~PVF_WRITE; - *ptep &= ~L2_APX; + *ptep |= L2_APX; PTE_SYNC(ptep); } }