From mboxrd@z Thu Jan 1 00:00:00 1970 From: Borislav Petkov Subject: Re: [PATCH 4/4] x86, efi: Map runtime services 1:1 Date: Thu, 6 Jun 2013 15:29:08 +0200 Message-ID: <20130606132908.GE20972@pd.tnic> References: <1370177770-26661-1-git-send-email-bp@alien8.de> <1370177770-26661-5-git-send-email-bp@alien8.de> <20130606131439.GI30420@console-pimps.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: <20130606131439.GI30420-HNK1S37rvNbeXh+fF434Mdi2O/JbrIOy@public.gmane.org> Sender: linux-efi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Matt Fleming Cc: Linux EFI , Matthew Garrett , Jiri Kosina , X86-ML , LKML , Borislav Petkov List-Id: linux-efi@vger.kernel.org On Thu, Jun 06, 2013 at 02:14:39PM +0100, Matt Fleming wrote: > This patch makes my TunnelMountain machine spin in handle_pte_fault() > when triggering one of the "firmware makes references to physical > addresses" code paths. > > I'll try and dig into this tomorrow to figure out what's going on. Hmmm, looks like we have forgotten to map an EFI region and we do a #PF when executing the EFI code. If so, I'm thinking we don't want to touch the #PF code and teach it about EFI so maybe we should make sure we've mapped *everything* EFI runtime would need before we switch PGDs... -- Regards/Gruss, Boris. Sent from a fat crate under my desk. Formatting is fine. --