From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Fleming Subject: Re: [PATCH] efi: Disable interrupts around EFI calls, not in the epilog/prolog calls Date: Tue, 10 Mar 2015 13:36:18 +0000 Message-ID: <20150310133618.GB17573@codeblueprint.co.uk> References: <20150303025804.GA24587@kt-Inspiron-3542> <20150303063433.GA30468@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20150303063433.GA30468-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: linux-efi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Ingo Molnar Cc: Tapasweni Pathak , matt.fleming-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org, mingo-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, hpa-YMNOUZJC4hwAvxtiuMwx3w@public.gmane.org, x86-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-efi@vger.kernel.org On Tue, 03 Mar, at 07:34:33AM, Ingo Molnar wrote: > > So why are interrupts disabled around page table operations to begin > with? It's not like any of this can execute on two CPUs at once, nor > can this be executed from interrupt context. > > So, shouldn't we only protect the EFI calls themselves? If we did that > we could save the efi_flags global variable ugliness as well. > > I.e. something like the attached patch. (not tested and all that) Looks good to me Ingo. I'll run it through the test farm. Thanks! -- Matt Fleming, Intel Open Source Technology Center From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752872AbbCJNgY (ORCPT ); Tue, 10 Mar 2015 09:36:24 -0400 Received: from mail-wi0-f180.google.com ([209.85.212.180]:36228 "EHLO mail-wi0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750848AbbCJNgV (ORCPT ); Tue, 10 Mar 2015 09:36:21 -0400 Date: Tue, 10 Mar 2015 13:36:18 +0000 From: Matt Fleming To: Ingo Molnar Cc: Tapasweni Pathak , matt.fleming@intel.com, tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, x86@kernel.org, linux-efi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] efi: Disable interrupts around EFI calls, not in the epilog/prolog calls Message-ID: <20150310133618.GB17573@codeblueprint.co.uk> References: <20150303025804.GA24587@kt-Inspiron-3542> <20150303063433.GA30468@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150303063433.GA30468@gmail.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 03 Mar, at 07:34:33AM, Ingo Molnar wrote: > > So why are interrupts disabled around page table operations to begin > with? It's not like any of this can execute on two CPUs at once, nor > can this be executed from interrupt context. > > So, shouldn't we only protect the EFI calls themselves? If we did that > we could save the efi_flags global variable ugliness as well. > > I.e. something like the attached patch. (not tested and all that) Looks good to me Ingo. I'll run it through the test farm. Thanks! -- Matt Fleming, Intel Open Source Technology Center