From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 21E8EC6FA99 for ; Mon, 6 Mar 2023 13:57:41 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230074AbjCFN5k (ORCPT ); Mon, 6 Mar 2023 08:57:40 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51438 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229808AbjCFN5h (ORCPT ); Mon, 6 Mar 2023 08:57:37 -0500 Received: from mail-qv1-xf2f.google.com (mail-qv1-xf2f.google.com [IPv6:2607:f8b0:4864:20::f2f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C9CB9301AB for ; Mon, 6 Mar 2023 05:57:36 -0800 (PST) Received: by mail-qv1-xf2f.google.com with SMTP id m4so6641893qvq.3 for ; Mon, 06 Mar 2023 05:57:36 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ziepe.ca; s=google; t=1678111056; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=GmA2ePEzRRQc73jhPODT2wkiXWYmm8Zye+szvA29uao=; b=UAuT69Iva6Y5Q9vXW0yORioDG50uwICCfsAQ6iCSfwASQGzx7S8/SDEbtX7HL9mEC1 kXFA69dsqd9CspV6FwZo3lIxi6RBUh87k+8tHWBH42RFsB4XN2Ez+t64v5pMgYtJZk76 Obdkhfslj6+yvs4i8rQmo9hw7meTTH+/2dHTFayd7Kbq17LhNVYxb+AK5sIG2h+5DRk3 DmAzGrWcr2oBerZlBcYaOgrKFfy0yBHanZoZ3Ao2d+H4W7ZbVAAWxg+tje2jsXceibom epuEWVgCQFuzdjpamjKA/Xt9ySnszKl8vu/Lphd167yCtPki4r8QfdLW8FBoPj9lm5t6 X6AA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1678111056; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=GmA2ePEzRRQc73jhPODT2wkiXWYmm8Zye+szvA29uao=; b=YrCSLKX4KNP9tZNC1INWSqNnBX/Ge5NyN6cr5K9Wdrt/yOkqP7g2Hm/Az1fPjlH1SB 4Y7EsAPCYaaWdznz35SkbQBMD6en+3ETTLlZQaDjGB5+sELAYqQY48uCzlwcyBFCkHtO g5qjfUNrEwCVs1CUy4/ndT3VBT4p7jBkm5TPuMLYMrCUlTVLY4o/unju2PgrMt6BG24N VdYcRK4gXTTg+aJS12C7Fbd3Mb0sk3pIO0kG1Q5PsU0tEgc2Oxli1JqnV5u0lc2v3+TA 1XkeLq+2V4w0qk4GkNoD34MTPC/JaocAo0cWST709jfqLWOvfZasmgmAF3O9b3NE2AK+ dejw== X-Gm-Message-State: AO0yUKUKlE7OWTbm6Ef4BtybpkqogWCAYDntBAFAX51bMPtl/97jb95l cZACH/MWmAxokQv/+ztrvN/duQ== X-Google-Smtp-Source: AK7set+xu3a5y5TdFxFDxJnv0QKElNGgr/wZ8g0z/9UIRGa/GPpvhPKo8/RjGkS7BQXZrFfmFpoewg== X-Received: by 2002:a05:6214:5089:b0:56f:8ef:693 with SMTP id kk9-20020a056214508900b0056f08ef0693mr18088668qvb.0.1678111055926; Mon, 06 Mar 2023 05:57:35 -0800 (PST) Received: from ziepe.ca (hlfxns017vw-142-68-25-194.dhcp-dynamic.fibreop.ns.bellaliant.net. [142.68.25.194]) by smtp.gmail.com with ESMTPSA id u19-20020a05620a121300b007424376ca4bsm7456920qkj.18.2023.03.06.05.57.34 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 06 Mar 2023 05:57:34 -0800 (PST) Received: from jgg by wakko with local (Exim 4.95) (envelope-from ) id 1pZBLK-00CcHk-6V; Mon, 06 Mar 2023 09:57:34 -0400 Date: Mon, 6 Mar 2023 09:57:34 -0400 From: Jason Gunthorpe To: "K V P, Satyanarayana" Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, alex.williamson@redhat.com, cohuck@redhat.com Subject: Re: [PATCH] vfio/pci: Add DVSEC PCI Extended Config Capability to user visible list. Message-ID: References: <20230303055426.2299006-1-satyanarayana.k.v.p@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230303055426.2299006-1-satyanarayana.k.v.p@intel.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Mar 03, 2023 at 05:54:26AM +0000, K V P, Satyanarayana wrote: > Intel Platform Monitoring Technology (PMT) support is indicated by presence > of an Intel defined PCIe Designated Vendor Specific Extended Capabilities > (DVSEC) structure with a PMT specific ID.However DVSEC structures may also > be used by Intel to indicate support for other features. The Out Of Band Management > Services Module (OOBMSM) uses DVSEC to enumerate several features, including PMT. > > The current VFIO driver does not pass DVSEC capabilities to virtual machine (VM) > which makes intel_vsec driver not to work in the VM. This series adds DVSEC > capability to user visible list to allow its use with VFIO. > > Signed-off-by: K V P Satyanarayana > --- > drivers/vfio/pci/vfio_pci_config.c | 8 ++++++++ > 1 file changed, 8 insertions(+) Wasn't the IDXD/SIOV team proposing to use the fact that DVSEC doesn't propogate to indicate that IMS doesn't work? Did this plan get abandoned? It seems at odds with this patch. Why would you use a "Platform Monitoring Technology" device with VFIO anyhow? Honestly I'm a bit reluctant to allow arbitary config space, some of the stuff people put there can be dangerous. Jason