From owner-svn-src-head@freebsd.org Mon Dec 18 14:30:09 2017 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 11A25E8ED1F; Mon, 18 Dec 2017 14:30:09 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from mail108.syd.optusnet.com.au (mail108.syd.optusnet.com.au [211.29.132.59]) by mx1.freebsd.org (Postfix) with ESMTP id C66B47CEDE; Mon, 18 Dec 2017 14:30:08 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from [192.168.0.102] (c110-21-101-228.carlnfd1.nsw.optusnet.com.au [110.21.101.228]) by mail108.syd.optusnet.com.au (Postfix) with ESMTPS id F02461A2704; Tue, 19 Dec 2017 01:03:18 +1100 (AEDT) Date: Tue, 19 Dec 2017 01:03:17 +1100 (EST) From: Bruce Evans X-X-Sender: bde@besplex.bde.org To: Bruce Evans cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r326932 - in head/sys: i386/i386 x86/acpica x86/x86 In-Reply-To: <201712181353.vBIDrM1W023033@repo.freebsd.org> Message-ID: <20171219005921.H2160@besplex.bde.org> References: <201712181353.vBIDrM1W023033@repo.freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed X-Optus-CM-Score: 0 X-Optus-CM-Analysis: v=2.2 cv=KeqiiUQD c=1 sm=1 tr=0 a=PalzARQSbocsUSjMRkwAPg==:117 a=PalzARQSbocsUSjMRkwAPg==:17 a=kj9zAlcOel0A:10 a=6I5d2MoRAAAA:8 a=8mRGWCooAZZwRf8n84EA:9 a=CjuIK1q_8ugA:10 a=IjZwj45LgO3ly-622nXo:22 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.25 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: Mon, 18 Dec 2017 14:30:09 -0000 On Mon, 18 Dec 2017, Bruce Evans wrote: > Author: bde > Date: Mon Dec 18 13:53:22 2017 > New Revision: 326932 > URL: https://svnweb.freebsd.org/changeset/base/326932 > > Log: > Remove the permanent double mapping of low physical memory and replace > it by a transient double mapping for the one instruction in ACPI wakeup > where it is needed (and for many surrounding instructions in ACPI resume). > Invalidate the TLB as soon as convenient after undoing the transient > mapping. ACPI resume already has the strict ordering needed for this. Forgot: Reviewed by: kib This central part of my recent fixes involving the permanent double mapping needed review more than the previous parts. Bruce