Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S980910AbdDYGlv (ORCPT ); Tue, 25 Apr 2017 02:41:51 -0400 Received: from edison.jonmasters.org ([173.255.233.168]:60639 "EHLO edison.jonmasters.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S980896AbdDYGll (ORCPT ); Tue, 25 Apr 2017 02:41:41 -0400 To: Lorenzo Pieralisi , linux-pci@vger.kernel.org References: <20170419164913.19674-1-lorenzo.pieralisi@arm.com> Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Pratyush Anand , Arnd Bergmann , Jonathan Corbet , Will Deacon , Jingoo Han , Bjorn Helgaas , Mingkai Hu , Tanmay Inamdar , Murali Karicheri , Russell King , Bharat Kumar Gogada , Ray Jui , Wenrui Li , Shawn Lin , Minghuan Lian , Catalin Marinas , Jon Mason , Gabriele Paoloni , Thomas Petazzoni , Joao Pinto , Thierry Reding , "Luis R . Rodriguez" , Michal Simek , Stanimir Varbanov , Zhou Wang , Roy Zang , Benjamin Herrenschmidt , John Garry From: Jon Masters Organization: World Organi{s,z}ation Of Broken Dreams Message-ID: <2e24e205-d761-9172-9463-4a53e1a0de4d@jonmasters.org> Date: Tue, 25 Apr 2017 02:40:57 -0400 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.0 MIME-Version: 1.0 In-Reply-To: <20170419164913.19674-1-lorenzo.pieralisi@arm.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-SA-Exim-Connect-IP: 74.92.29.237 X-SA-Exim-Mail-From: jcm@jonmasters.org Subject: Re: [PATCH v4 00/21] PCI: fix config space memory mappings X-SA-Exim-Version: 4.2.1 (built Sun, 08 Nov 2009 07:31:22 +0000) X-SA-Exim-Scanned: Yes (on edison.jonmasters.org) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 743 Lines: 15 On 04/19/2017 12:48 PM, Lorenzo Pieralisi wrote: > On some platforms (ie ARM/ARM64) ioremap fails to comply with the PCI > configuration non-posted write transactions requirement, because it > provides a memory mapping that issues "bufferable" or, in PCI terms > "posted" write transactions. Likewise, the current pci_remap_iospace() > implementation maps the physical address range that the PCI translates > to I/O space cycles to virtual address space through pgprot_device() > attributes that on eg ARM64 provides a memory mapping issuing > posted writes transactions, which is not PCI specifications compliant. Side note that I've pinged all of the ARM server vendors and asked them to verify this patch series on their platforms. Jon.