From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=100.103.45.18 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1792516616; b=VFcfX6GO+WzDMFLh8A1e7OHurhutKuOBQlZPd7SRo+TEVQF/IYjsdQRRuYx5GPrHQFKi peyMo3WjHo3dQT89BSUph6mpV7HJVfaAKyWaf0ePV1POhsXGlKxsARyTCYdjcoRoGKjb/ RjkIQ8t8vBLP6XsvDVmOXX7vBVUTl7C5a/yPfvIUHih0wV1mzObLq7fZE7UM4pAMkiYcy 2+XHCaZqDOJiKKx6gVWqwLM98LmMWOA82GjzUfdHpqfyQmIL9/hfV1PWDXv4jTxYwjkty VDi51RgCxIvUAFkdHCyN/GhUdD9/sn3EJpIsbc6MV8rcvDV3vML/2AtldPDmMAAiU+A== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1792516616; h=Received:Received:Received:X-ME-Sender:X-ME-Proxy-Cause:X-ME-Proxy: Feedback-ID:Received:X-Mailer:MIME-Version:Date:From:To:Cc:Message-Id: In-Reply-To:References:Subject:Content-Type: Content-Transfer-Encoding; bh=FPjAFqoQ3uCxhlJR+neifQbJwXA/vEfAzYz3BKKcjZY=; b=sUgP8bf0SV9kF+oyjBV6LdBQ1c27PW3Vtx03XDTyJ3DSFtDcH8CMYDIswOc44hwpKq5c u4454b8hzd4JCDgBOpQriLk6uL+wzZRbH1klVLhDSB554ypfQuKpa4gsmRznX2J4egWCp svRu0UPwrncuD0RSvp/XIxqxXfx02YSuxuCvR6qGPN4hAFJ81fYmFHlF4I6r7c8p1xmEO rIMUCnx0AtEg/rdux6OQWLHRMT5ZogytHPR6klly1vDjnnvHAysk5OwiNMQPY4xwjvi79 QnmPDzZ8jIFy7NQ+pGZZqC2eily5FnemUln1dVwIPxel+Omz4CB02dbVnQ18iQnxGUA== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; arc=none smtp.remote-ip=100.103.45.18 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5A4F4606A330 for ; Tue, 20 Oct 2026 17:16:56 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id EA94E1F000CA; Tue, 20 Oct 2026 17:16:55 +0000 (UTC) Received: from stl-compute-04.internal (stl-compute-04.internal [10.204.2.64]) by mailfauth.stl.internal (Postfix) with ESMTP id DCDA9780043; Tue, 20 Oct 2026 13:16:24 -0400 (EDT) Received: from stl-imap-11 ([10.204.2.102]) by stl-compute-04.internal (MEProxy); Tue, 20 Oct 2026 13:16:24 -0400 X-ME-Sender: X-ME-Proxy-Cause: dmFkZTGotKvzjS3n3TRZ0v50m6h3cI6EI0s9z7DT82iBKKE6Q15vLsJzsPvrUeyFhM+Osf gQCGSN/0gacl8Z4ETxw3tPKML9dTO4lVuF/F1klvXVv8yGO3r/A3uaav3XzkLOIXYLTMZz FFsqF7wAcYpunmwtjqbgXO0l1J4vjGUPW6oukba4cH286gSRIKuXCdz5R98K1OWeaG2m+k MQ5HP8jlxADzy9DwGV3VVFP595AkljBUghI3tobW5bE+wnH3T+cxf5UYbM2ZjuXpOuS5bo V+Qg4/JbI5PY79uavQTDd/cYy4kHuix8Glymj2Al+JPFRjDOnkjOLZwG9ETDzscip1bYOY boWxRTt+GnnWCgRyKBro300WrNS4yllkq7ZKa6W6h0Krfze99kR9JzADcwZ4HiFzM9MvXk gdlIsp10OKuK7h0B4l6Lsd6D5IdrwYy2teubhp4ZYsB8ke9VRntGr6uyrRA/O3fA2iV3FW Yq1kfPcYST2wK+XfBgxH3qpcHK/xlCgl6APcu1M8SWveBuoA1Gy89zuQ90MiSfiw0mIBsU X5dQ1zUbmUCRuvYprsMEfe4l+dnZyJ824lFTaa8BcDRRR3kz4tEviUxn6mS1SPl5tHjq+j w9XhYuXSMm3szsIbxNLwfxDLxoUALkZOiu5JN79H+ofWPCDS7pOX/VRTVbMQ X-ME-Proxy: Feedback-ID: i927946fb:Fastmail Received: by mailuser.stl.internal (Postfix, from userid 501) id 7780619C0067; Tue, 20 Oct 2026 13:16:24 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface X-Mailing-List: linux-hyperv@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Date: Tue, 20 Oct 2026 20:16:04 +0300 From: "Leon Romanovsky" To: "Logan Gunthorpe" , "Popov, Pavel E" , "Bjorn Helgaas" Cc: "Bjorn Helgaas" , "Lukas Wunner" , "Jim Chow" , "Radu Rugina" , "Alexey Makhalov" , "Wei Liu" , "Michael Kelley" , "Nathan Ciobanu" , "bcm-kernel-feedback-list@broadcom.com" , "virtualization@lists.linux.dev" , "linux-hyperv@vger.kernel.org" , "linux-pci@vger.kernel.org" , "linux-kernel@vger.kernel.org" Message-Id: <2db83aed-1137-49a2-99a6-1369f163c273@app.fastmail.com> In-Reply-To: References: <20261020161447.GN64621@unreal> <20261020163758.GA2342476@bhelgaas> Subject: Re: [PATCH v3] PCI/P2PDMA: Allow P2PDMA on Intel Xeon server CPUs Content-Type: text/plain Content-Transfer-Encoding: 7bit On Tue, Oct 20, 2026, at 19:54, Logan Gunthorpe wrote: > On 2026-10-20 10:47, Popov, Pavel E wrote: >> If preferred, we could use for cpu_supports_p2pdma() the pattern Logan >> suggested for hypervisor_supports_p2pdma() in v1: the whole function >> under #ifdef CONFIG_X86 with a stub returning false for !CONFIG_X86. >> The Intel CPU list then moves into the same #ifdef block, which saves >> one #ifdef, at the cost of also touching the existing AMD part. > > I personally would prefer cleaning up the #ifdef style but that's an > issue that's already in the code. I don't feel the need to make it > conditional on my Reviewed-by. If you are able to clean that up, I would > appreciate it. Sooner or later that cleanup will be done, and I don't see any reason to delay it while we are touching that code anyway. Thanks > > Thanks, > > Logan