Received: by 2002:a05:6902:102b:0:0:0:0 with SMTP id x11csp824091ybt; Wed, 8 Jul 2020 12:36:32 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz3xG4UGd0+WPtSwr95NL8+4aljc5pntW4k77eteHhoI8R0lRDRmT7RfYzS2kHxJdM01wV6 X-Received: by 2002:a50:fa07:: with SMTP id b7mr66943800edq.298.1594236992652; Wed, 08 Jul 2020 12:36:32 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1594236992; cv=none; d=google.com; s=arc-20160816; b=w2TkVteMdX7R4YRbE5uboVoqAOm524ZYnOu6TwUR06j15QRopZIueCwziSkLBI6CNx b0IKlYtSPE2/oPFJBgNHbjaDTGvAM3eORLsn3fA07DWENR5eJ9cl+gZayRbfwlbnMxMK wsH3xV8rDacOGJew761i6R7EgtRYVCj+sSp3r8uEHFE2cvSU1daaI3FbUbPpJGyz3q+A R52zbm4tyY6GmwvFZBUeN8fcuqXPU1d9Jue3jeIJp4V4ytELAUKAlsfNwROwEWjXiYT9 77Oa6pHAFkdOQmCe4CL3FEdRQD3R93wwHjEOql5oS1oXI68JJ8E7Idaz8Da2PaVVmAmX MDng== 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=slImY14f0rNlEWIkm1VGpH6ry5mubIzN1YXFRSQUYYw=; b=hHaqu9GjPX0II1BWOjHv/2b3T0SclybUNHO/hxOiejwXi/wMT4eAkfoM+lRZDoZ8Se ZIffTWR4llwL27mVutCNV/d1FQ1tumJBfwLHiSzpOsRvJ5EHnwAjfkuzx9il2sJsKB0l 7bZ2ox+AZQ2+jr9/MCepK3ZoJktGWp8yR93bhLvNNQZE1Hxwts3yM6Qf8A18QmlJzb+4 yZH1Zk5pt4NqqjQkdtNKl18YuHAMg/xR7faHXjBs3GFg8R4XsWL9g/OnBrAEhYC+4ur3 9oTtVfQO8hR7JSvp5X14glDaO06NDG3++zg5KnozK6OwNm7uOTwYrOSti28SJzP35CI4 PbxQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@google.com header.s=20161025 header.b=EIzmcXfU; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id v12si574986eds.85.2020.07.08.12.36.10; Wed, 08 Jul 2020 12:36:32 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@google.com header.s=20161025 header.b=EIzmcXfU; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726698AbgGHTdL (ORCPT + 99 others); Wed, 8 Jul 2020 15:33:11 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40780 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726129AbgGHTdF (ORCPT ); Wed, 8 Jul 2020 15:33:05 -0400 Received: from mail-io1-xd42.google.com (mail-io1-xd42.google.com [IPv6:2607:f8b0:4864:20::d42]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 643FAC061A0B for ; Wed, 8 Jul 2020 12:33:05 -0700 (PDT) Received: by mail-io1-xd42.google.com with SMTP id q8so48285415iow.7 for ; Wed, 08 Jul 2020 12:33:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=slImY14f0rNlEWIkm1VGpH6ry5mubIzN1YXFRSQUYYw=; b=EIzmcXfUjvt2TpgoLMvmOT/1wauSIofM4ahX4qssit2mGp/aZJvZpH0CRIzLFtLSzj 6kOGwZNSPgSre7jg2e413VgkxMXoVHdCL2ISAnaA7VDlGOgsOXE67V7PkS+g2IrVAPpM 4RDM2PSqPHKBq7sucj1es79e+9iuSaSUercAkwFBm+PTQ/TWXGBAfqkJWSkC7i5fA8EX pdC8XtamtPprKBVLrjxBiPVyfIKH9tNEPD1PeHISshRyle6SIZnT9Aa8V131VZ5g48zs 59t53fKWZnIRKv8irTmRNGVXDECNokzdTN08hCWuJgj9+wO5jjZcv85b8NjobvZWHKce xrpA== 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=slImY14f0rNlEWIkm1VGpH6ry5mubIzN1YXFRSQUYYw=; b=cWlMkZ9mNfAz3Tp0m2WlJp2fX54AY2fqSrqjejSTB4znAkMbsKbpt7esWejg18pLin f3l9ov658/kMegLeejydNXJd0r5Ds3XK276UictrEJXkUflFzdAfQZ0rlvlY4F1o/gBz Lij7eneWqfF/KDAPL6X+rWqDf44JFfriyc+Gwn77mzH7K2OwvWDOODZbN9HvV5A0EIKk GMfiab2c/YND18oxmHuPqYt9YUPbkRNh+FZwh18vYnKExLuV2AVxniUWoeaUwskY8Nup j2VKkryx4hdr+QPBmbU1+ruKsHmoHwE5QhdiNWphm74c7VI23kq+D/edF/G2TBJ1CJWB yBfw== X-Gm-Message-State: AOAM533XSKF/QoMFdyYNuIC+6xmNfTP/swajTWj3GzxQZHbJZMfFnMr/ 51uO4iBQsWUIy5eqp9dkmLB+dJu9DHfjr/caU5uBBg== X-Received: by 2002:a5e:cb42:: with SMTP id h2mr37795277iok.43.1594236784442; Wed, 08 Jul 2020 12:33:04 -0700 (PDT) MIME-Version: 1.0 References: <20200702221237.2517080-1-abhishekbh@google.com> <20200703114037.GD2999146@linux.ibm.com> <20200705152304.GE2999146@linux.ibm.com> <5d2ccf3d-b473-cf30-b863-e29bb33b7284@redhat.com> <48f82669-f1ff-0f5e-e531-ebbd151205f9@redhat.com> In-Reply-To: From: Abhishek Bhardwaj Date: Wed, 8 Jul 2020 12:32:27 -0700 Message-ID: Subject: Re: [PATCH v3] x86/speculation/l1tf: Add KConfig for setting the L1D cache flush mode To: Waiman Long Cc: Mike Rapoport , Doug Anderson , Anthony Steinhauser , LKML , Borislav Petkov , "H. Peter Anvin" , Ingo Molnar , Jim Mattson , Joerg Roedel , Josh Poimboeuf , Mark Gross , Paolo Bonzini , Pawan Gupta , Peter Zijlstra , Sean Christopherson , Thomas Gleixner , Tony Luck , Vitaly Kuznetsov , Wanpeng Li , kvm@vger.kernel.org, x86 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 Mon, Jul 6, 2020 at 8:14 AM Waiman Long wrote: > > On 7/5/20 5:51 PM, Abhishek Bhardwaj wrote: > > That is why I said a comment will have to be added to highlight this > dependency. For instance, > > +/* > + * Three of the enums are explicitly assigned as the KVM_VMENTRY_L1D_FLUSH > + * config entry in arch/x86/kvm/Kconfig depends on these values. > + */ > enum vmx_l1d_flush_state { > VMENTER_L1D_FLUSH_AUTO, > - VMENTER_L1D_FLUSH_NEVER, > - VMENTER_L1D_FLUSH_COND, > - VMENTER_L1D_FLUSH_ALWAYS, > + VMENTER_L1D_FLUSH_NEVER = 1, > + VMENTER_L1D_FLUSH_COND = 2, > + VMENTER_L1D_FLUSH_ALWAYS = 3, > VMENTER_L1D_FLUSH_EPT_DISABLED, > VMENTER_L1D_FLUSH_NOT_REQUIRED, > }; > > Of course, this is just a suggestion. > > I'd rather avoid this dependency. However, if people are okay with the > CONFIG option then I am happy to oblige with whatever people agree on. > Can a maintainer chime in ? Waiman if you're the final authority on > this, will you accept the patch if I incorporated your suggestion ? > > It is fine if you think this is not what you want. > > BTW, I am not a maintainer. However, no maintainer will give pre-approval. At least, you have to give the reason why this patch is needed in the patch itself. Before that happens, I don't see how it will get merged upstream. I just updated the patch with the reasoning in the commit message - https://lkml.org/lkml/2020/7/8/1325 > > Cheers, > Longman -- Abhishek