Received: by 2002:a25:8b12:0:0:0:0:0 with SMTP id i18csp554928ybl; Thu, 22 Aug 2019 01:01:15 -0700 (PDT) X-Google-Smtp-Source: APXvYqw3J20i+ds22OIYp2izYGPNF/jtgN0WSy8z92rXHScX5zM7KUZWoKoH3bG+cY7qpDy2Dnoc X-Received: by 2002:a17:902:169:: with SMTP id 96mr36528747plb.297.1566460875160; Thu, 22 Aug 2019 01:01:15 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1566460875; cv=none; d=google.com; s=arc-20160816; b=bftbbPRSrOGfBN4cDV2AR6OZ8AlkIIjv/3dNwavjSEOxqx8vZMSPeg/XkPLMWnUxGb tF7yTjmRqdp5ap0ReJ2bLgnhQK5FLxxIqfE/iPkNMNiRSYjijUhf+/9yV73fz1wAxXWn PQKLWei1g4ZjFsLfMKJzMF09TW26BKVkZAIP9zjoEorjlI/d7+gvkKfVy9ppG8P98zR4 9cq+CHeRfYUY1HVxlNr/z+f8WNQPUQLrXuwNfu5ksBp1wtaUe9RyDj6AiJF9TqhBermP goUnv4P1IKtXtaoLhHhzQbVR37kX2FbIs53QdgqQJBSmKwI3Lzfm+UpbdHHeXAODaT43 oqZA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=hn6rxeyfQPsYnbNUt5MSgWQtbToPh9tKf9jba7m/Mb0=; b=O1rdw/t5LMZh8WNIcjFFPbPvitul6OxVMQj7fEOpjAvibraWdPxxhdV+xWdCq0gQVD k2mNI0UKuGzNkAu8JooULufVm5YdRmJu+g9Ti5bX9uRg+HyV52hqhSxb5nIXH8S/Tui9 nj2rqbg0PDOFePOdumNB61YB8KQzkNp8NcCFOJ0guLpgWZl683WkiiDweIfxninRVFFZ 2oSYSYAgsw3RsTt0Cp28vG3h9ljbAYsFje56X5oDNGV8GHX+auR5XVjyL9IBITR6DzaO VCihMC05eIqS8dc8N9hRrma5EH5j1kkrLrGM6uCAo8il1XLrESwimoWpwcSEDAmZyPOy 3J9g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=qh9NTAxS; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id d9si15970069pgv.577.2019.08.22.01.00.59; Thu, 22 Aug 2019 01:01:15 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=qh9NTAxS; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730771AbfHVFla (ORCPT + 99 others); Thu, 22 Aug 2019 01:41:30 -0400 Received: from mail-io1-f67.google.com ([209.85.166.67]:44064 "EHLO mail-io1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726857AbfHVFl3 (ORCPT ); Thu, 22 Aug 2019 01:41:29 -0400 Received: by mail-io1-f67.google.com with SMTP id j4so1137985iog.11; Wed, 21 Aug 2019 22:41:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=hn6rxeyfQPsYnbNUt5MSgWQtbToPh9tKf9jba7m/Mb0=; b=qh9NTAxShXjNLbP7aaKIkgSdqONWOprq9JZrxoqc5kVXFAF5XJO5QWJ5FTba1FYzh3 y+OtXi6NNfMAbQe96n5PTZ2GLN8Ybf570q7QcCkXQwkR3A2jqYHPWQTtLJDyCxxQ8vpd FzSSFLB6V/D5k3rOIAg7WfBgNX8of0+VO6VwIiUvUCxse5vr2s4F3MMKuILFAbFrOqNJ lS+1sHQf8TtPAnMc9+mI5kOv/1frTd1/wywNNNTIIz42s5GGVf1QxgFPRhvXI6dIUv/7 myvqsmGFBPQngRg+CXIaw4wu72oAay3IjQyGlN2B4sBh6xhEnbbur+SeKGMAfsOPCbsP 1RXg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=hn6rxeyfQPsYnbNUt5MSgWQtbToPh9tKf9jba7m/Mb0=; b=nUhf9Phac08CA/jnkLXcJoi/sEPWvbc+5LpZUcUq+7bk2Up3hmSGWRVvOPQucx+MVD o4wnb8TAcupcOYClvAmrpemduSnNTPnAoP/34rddiZBIMqh82oz/5rGe5wEOYXx2CCbB QQe6fIwQQucfEIimFhzDPMjOsB9i+00edLTmi3GjYdP16K6LJ4q8k/BxpbDq0vl0kvoW DMsooqligz/M4fr9nfrPk7yKhcgbF5CU+c47yYqnXzFyJ4fozfjCcY5NzOHtzKtD2YHI IS511i++fXETUhLa1KHl9B6/UHkh8hesd4yJCUAnItuXua3mjorhOcCkaQbv3pGGaR/P dJXw== X-Gm-Message-State: APjAAAWbAwT+YZMtjrGt9acXxzEIARfkOBVE5X02ghPpEpS9ITZACmf1 PcMcFvt9uOqvhFDFbI7oEe1vr6j+N23a/RjDl7w= X-Received: by 2002:a6b:6e0e:: with SMTP id d14mr25453232ioh.18.1566452488810; Wed, 21 Aug 2019 22:41:28 -0700 (PDT) MIME-Version: 1.0 References: <1566400103-18201-1-git-send-email-nayna@linux.ibm.com> <1566400103-18201-2-git-send-email-nayna@linux.ibm.com> In-Reply-To: From: "Oliver O'Halloran" Date: Thu, 22 Aug 2019 15:41:17 +1000 Message-ID: Subject: Re: [PATCH v2 1/4] powerpc/powernv: Add OPAL API interface to access secure variable To: Nayna Jain , linuxppc-dev , linux-efi@vger.kernel.org, linux-integrity@vger.kernel.org Cc: Linux Kernel Mailing List , Michael Ellerman , Benjamin Herrenschmidt , Paul Mackerras , Ard Biesheuvel , Jeremy Kerr , Matthew Garret , Mimi Zohar , Greg Kroah-Hartman , Claudio Carvalho , George Wilson , Elaine Palmer , Eric Ricther Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Aug 22, 2019 at 3:02 PM Oliver O'Halloran wrote: > > On Wed, 2019-08-21 at 11:08 -0400, Nayna Jain wrote: > > diff --git a/arch/powerpc/platforms/powernv/opal.c b/arch/powerpc/platforms/powernv/opal.c > > index aba443be7daa..ffe6f1cf0830 100644 > > --- a/arch/powerpc/platforms/powernv/opal.c > > +++ b/arch/powerpc/platforms/powernv/opal.c > > @@ -32,6 +32,8 @@ > > #include > > #include > > #include > > +#include > > +#include > > > > #include "powernv.h" > > > > @@ -988,6 +990,9 @@ static int __init opal_init(void) > > /* Initialise OPAL Power control interface */ > > opal_power_control_init(); > > > > + if (is_powerpc_secvar_supported()) > > + secvar_init(); > > + > > The usual pattern here is to have the init function check for support > internally. > > Also, is_powerpc_secvar_supported() doesn't appear to be defined > anywhere. Is that supposed to be is_opal_secvar_supported()? Or is this > series supposed to be applied on top of another series? To answer my own question, yes it depends on the series at [1] which adds IMA support. Turns out actually reading the cover letter helps, who knew. That said, I'm still not entirely sure about this. The implementation of is_powerpc_secvar_supported() in [2] parses the DT and seems to assume the DT bindings that OPAL produces. Are those common with the DT bindings produced by OF when running on pseries? [1] http://patchwork.ozlabs.org/project/linuxppc-dev/list/?series=125961 [2] http://patchwork.ozlabs.org/patch/1149257/ > > > return 0; > > } > > machine_subsys_initcall(powernv, opal_init); >