From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753158AbbJPBqK (ORCPT ); Thu, 15 Oct 2015 21:46:10 -0400 Received: from mga11.intel.com ([192.55.52.93]:5109 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752921AbbJPBpp (ORCPT ); Thu, 15 Oct 2015 21:45:45 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.17,687,1437462000"; d="scan'208";a="827886158" Message-ID: <1444959919.28005.26.camel@ranerica-desk01> Subject: Re: [PATCH v2] x86/mm: warn on W+x mappings From: Ricardo Neri To: Borislav Petkov Cc: Matt Fleming , Andy Lutomirski , Ingo Molnar , Stephen Smalley , X86 ML , "linux-kernel@vger.kernel.org" , Kees Cook , Thomas Gleixner , "H. Peter Anvin" , Peter Zijlstra , Andy Lutomirski , Denys Vlasenko , Brian Gerst , "linux-efi@vger.kernel.org" , Ard Biesheuvel , luv@ml01.01.org Date: Thu, 15 Oct 2015 18:45:19 -0700 In-Reply-To: <20151015103332.GE4267@pd.tnic> References: <20151012113605.GB7384@pd.tnic> <20151012124113.GD2579@codeblueprint.co.uk> <20151012124936.GA6260@gmail.com> <20151012125548.GE2579@codeblueprint.co.uk> <20151012141754.GA6621@gmail.com> <20151012144928.GF2579@codeblueprint.co.uk> <20151014151807.GA27013@gmail.com> <20151014153522.GC8218@pd.tnic> <20151015101016.GB2975@codeblueprint.co.uk> <20151015103332.GE4267@pd.tnic> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.4-0ubuntu2 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2015-10-15 at 12:33 +0200, Borislav Petkov wrote: > Yeah, that's actually a good idea. Why not upstream it for the wider > audience so that people can actually start reporting b0rked UEFIs? > With > a big and nice FW_BUG splat in there... We attempted to upstream in the past but later I discovered that my implementation in particular is causing warnings due to SMP. Also, I need to implement an alternative or extension to the current efi_map_region, which have the __init qualifier. This is because mappings might happen after the __inits have been discarded. I have this work currently in my scope. Thanks and BR, Ricardo