Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp19231145rwd; Wed, 28 Jun 2023 06:51:14 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ5r2KChYtxALNdFzPETOGwlIWBRfri1m0I1W8k3w+MnnM4L0DcekxnZ8OklekcLEWgUb8ud X-Received: by 2002:a17:907:94c7:b0:992:527:8466 with SMTP id dn7-20020a17090794c700b0099205278466mr5067128ejc.76.1687960274168; Wed, 28 Jun 2023 06:51:14 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1687960274; cv=none; d=google.com; s=arc-20160816; b=Pjqxowt0cAj0yT65DM4F6Lo8BOpyNJ8dDRw3znG5Ub2fSXZ5a4Ru3d+VkxFQWaLy/V iIYUZAW7Exd046dUhgbomol53f7zqG66o9ZbdMxgLG9RwsXQR9BrNdeFAvBmGYgNvUn/ dt1SIICtkrIZQGV9TrAQVmyafK+JvO0eW8MNhInsJdnjXKz3LVPSxNrKWm2NNRpS/Iz7 S0LqWsWBwPJpRfF1YEymZPok+h5SRXfb8VVkGwQhJGEwtZiFMYTDBMZazVNYhbny/Hmf 06DT89L8Jf6aj618ocMhD5F+zOj7mhtvUwZxQyVAAymkVE+PQFcMg77ftT3WbHWoMA3p D/ZQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date; bh=ceIGr8N+0VQv1BgnhO6ODavs9CL7tOTlhfPRHO3FRAI=; fh=/opo3kd2va6k9MvKpV4Z/VWmeTOBpb/aceHeHLKTafY=; b=p5otCq8DJxDzGucQahWvNzkYKPXfb0qBmkRlxJNlJgpc3aVTilXDo51ZowXh5X9qK/ knqHv7taXOYVOvpXM59U4XihACtlm1aB0q1Jdbg5Xnalrkoscl9dzD8fcCImmWpRp6rx 7MVaRCyiHsIIuy9QtbwKrgGLFojfZoycPDzck9y6EeKYKb476fAIqr/OtsLc14Fqc+OF vC+8bn48xqd2etajSIqfBHntbzTiIL7/10OtLG3M/uZ8j97zDxICtUto1i9BYw8IMNvQ 9HzozYncdxEskojCtrWc3PCfJOiOMdLZ1EM9A++Hz4QbpYnkjHhvcyVqKkAP6Mi3ywqo YJEA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id qx9-20020a170906fcc900b0098874379199si5355028ejb.163.2023.06.28.06.50.48; Wed, 28 Jun 2023 06:51:14 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232130AbjF1N2N (ORCPT + 99 others); Wed, 28 Jun 2023 09:28:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46786 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232129AbjF1N1C (ORCPT ); Wed, 28 Jun 2023 09:27:02 -0400 Received: from bmailout2.hostsharing.net (bmailout2.hostsharing.net [IPv6:2a01:37:3000::53df:4ef0:0]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BE5083582; Wed, 28 Jun 2023 06:25:28 -0700 (PDT) Received: from h08.hostsharing.net (h08.hostsharing.net [IPv6:2a01:37:1000::53df:5f1c:0]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "*.hostsharing.net", Issuer "RapidSSL Global TLS RSA4096 SHA256 2022 CA1" (verified OK)) by bmailout2.hostsharing.net (Postfix) with ESMTPS id DB72B2800B3CD; Wed, 28 Jun 2023 15:25:26 +0200 (CEST) Received: by h08.hostsharing.net (Postfix, from userid 100393) id C3BA343F1C9; Wed, 28 Jun 2023 15:25:26 +0200 (CEST) Date: Wed, 28 Jun 2023 15:25:26 +0200 From: Lukas Wunner To: Smita Koralahalli Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Bjorn Helgaas , oohall@gmail.com, Mahesh J Salgaonkar , Kuppuswamy Sathyanarayanan , Yazen Ghannam , Fontenot Nathan , Jay Cornwall , Felix Kuehling Subject: Re: [PATCH v3 2/2] PCI: pciehp: Clear the optional capabilities in DEVCTL2 on a hot-plug Message-ID: <20230628132526.GA14276@wunner.de> References: <20230621185152.105320-1-Smita.KoralahalliChannabasappa@amd.com> <20230621185152.105320-3-Smita.KoralahalliChannabasappa@amd.com> <20230622063105.GA624@wunner.de> <20230622214247.GB11993@wunner.de> <4599d885-219f-3ee0-f425-62746f31cc67@amd.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4599d885-219f-3ee0-f425-62746f31cc67@amd.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-Spam-Status: No, score=-1.7 required=5.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,SPF_HELO_NONE,SPF_NONE, T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 27, 2023 at 10:38:54AM -0700, Smita Koralahalli wrote: > Okay, I see there are no objections except for Bjorn/Jay's comments on > > "But there could be devices where AtomicOps are nominally supported but > untested or broken.." > > Would this be an issue? I think you said that BIOS enables AtomicOps on certain AMD machines? Or did that observation only apply to 10 Bit tags? If BIOS does enable AtomicOps, it would be interesting to know which logic BIOS follows, i.e. how does it determine whether to set AtomicOp Requester Enable on Endpoints? It would also be interesting to know how far that BIOS has proliferated, i.e. how much experience with various Endpoint devices exists in the real world. If it turns out that BIOS has enabled the feature for years on a wide range of Endpoints without any issues, I think that would render concerns mute that enabling it in the kernel might cause regressions. I don't know why the spec says that "discovery of AtomicOp Requester capabilities is outside the scope of this specification". I imagine it would be possible to set AtomicOp Requester Enable, then read it to determine whether the bit is now indeed 1 or hard-wired to 0. In the latter case, AtomicOp Requester capabilities can be assumed to be absent. So that would be a way to make do without any other specific discovery of AtomicOp Requester capabilities. Thanks, Lukas