From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Date: Tue, 21 Apr 2015 22:57:32 +0000 Subject: Re: ioremap_uc() followed by set_memory_wc() - burrying MTRR Message-Id: <20150421225732.GA17356@obsidianresearch.com> List-Id: References: <20150421224601.GY5622@wotan.suse.de> In-Reply-To: <20150421224601.GY5622@wotan.suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "Luis R. Rodriguez" Cc: Andy Lutomirski , mike.marciniszyn@intel.com, infinipath@intel.com, linux-rdma@vger.kernel.org, awalls@md.metrocast.net, Toshi Kani , "H. Peter Anvin" , Ingo Molnar , "linux-kernel@vger.kernel.org" , Hal Rosenstock , Sean Hefty , Suresh Siddha , Rickard Strandqvist , Roland Dreier , Juergen Gross , Mauro Carvalho Chehab , Borislav Petkov , Mel Gorman , Vlastimil Babka , Davidlohr Bueso , Dave Hansen , Jean-Christophe Plagniol-Villard , Thomas Gleixner , Ville =?iso-8859-1?Q?Syrj=E4l=E4?= , Linux Fbdev development list , linux-media@vger.kernel.org, X86 ML , mcgrof@do-not-panic.com On Wed, Apr 22, 2015 at 12:46:01AM +0200, Luis R. Rodriguez wrote: > are talking about annotating the qib driver as "known to be broken without PAT" > and since the ipath driver needs considerable work to be ported to > use PAT (the This only seems to be true for one of the chips that driver supports, not all possibilities. > userspace register is just one area) I wanted to review if we can just remove > MTRR use on the ipath driver and annotate write-combining with PAT as a TODO > item. AFAIK, dropping MTRR support will completely break the performance to the point the driver is unusable. If we drop MTRR we may as well remove the driver. Mike, do you think the time is right to just remove the iPath driver? Jason