From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934906AbcHEKhU (ORCPT ); Fri, 5 Aug 2016 06:37:20 -0400 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:43435 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751799AbcHEKhQ (ORCPT ); Fri, 5 Aug 2016 06:37:16 -0400 Date: Fri, 5 Aug 2016 12:37:13 +0200 From: Pavel Machek To: "Rafael J. Wysocki" Cc: the arch/x86 maintainers , Linux PM list , "Rafael J. Wysocki" , Thomas Garnier , Thomas Gleixner , Ingo Molnar , "H . Peter Anvin" , Kees Cook , Yinghai Lu , LKML , kernel-hardening@lists.openwall.com Subject: Re: [PATCH] x86/power/64: Do not refer to __PAGE_OFFSET from assembly code Message-ID: <20160805103713.GA8106@amd> References: <1470071280-78706-1-git-send-email-thgarnie@google.com> <2464745.UmGP58NeXC@vostro.rjw.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2464745.UmGP58NeXC@vostro.rjw.lan> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed 2016-08-03 01:19:26, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > When CONFIG_RANDOMIZE_MEMORY is set on x86-64, __PAGE_OFFSET becomes > a variable and using it as a symbol in the image memory restoration > assembly code under core_restore_code is not correct any more. On a related note... we should really have page_offset variable in such case, and use that -- having __FOO_BAR not being a constant is ugly/confusing/dangerous. > To avoid that problem, modify set_up_temporary_mappings() to compute > the physical address of the temporary page tables and store it in > temp_level4_pgt, so that the value of that variable is ready to be > written into CR3. Then, the assembly code doesn't have to worry > about converting that value into a physical address and things work > regardless of whether or not CONFIG_RANDOMIZE_MEMORY is set. > > Reported-and-tested-by: Thomas Garnier > Signed-off-by: Rafael J. Wysocki Acked-by: Pavel Machek Is similar patch needed for i386? Best regards, Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html