linux-pci.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [PATCH] pci: More PRI/PASID cleanup
       [not found]           ` <1320776406.19116.42.camel@bling.home>
@ 2011-11-09  9:59             ` Roedel, Joerg
  0 siblings, 0 replies; only message in thread
From: Roedel, Joerg @ 2011-11-09  9:59 UTC (permalink / raw)
  To: Alex Williamson
  Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org,
	jbarnes@virtuousgeek.org, bhelgaas@google.com

On Tue, Nov 08, 2011 at 11:20:06AM -0700, Alex Williamson wrote:

> My guess is that someone asked why a device would ever expose this table
> if it didn't support enable as a capability.  So, I wouldn't be
> surprised if your hardware sets it, but it's probably just as safe to
> assume enable is supported if the PASID table exists.

That is probably true. I suggest that you split the functional change
out of the patch and do only the renaming. I look over the code again
then and change it in a way that works with reality and the spec. Does
that work for you?


	Joerg

-- 
AMD Operating System Research Center

Advanced Micro Devices GmbH Einsteinring 24 85609 Dornach
General Managers: Alberto Bozzo, Andrew Bowd
Registration: Dornach, Landkr. Muenchen; Registerger. Muenchen, HRB Nr. 43632


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2011-11-09  9:59 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20111103034359.10286.19062.stgit@bling.home>
     [not found] ` <20111108162826.GD5182@amd.com>
     [not found]   ` <1320770670.19116.23.camel@bling.home>
     [not found]     ` <20111108171725.GE5182@amd.com>
     [not found]       ` <1320773518.19116.30.camel@bling.home>
     [not found]         ` <20111108175410.GH5182@amd.com>
     [not found]           ` <1320776406.19116.42.camel@bling.home>
2011-11-09  9:59             ` [PATCH] pci: More PRI/PASID cleanup Roedel, Joerg

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).