All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Jan Beulich" <jbeulich@novell.com>
To: Allen M Kay <allen.m.kay@intel.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	Weidong Han <weidong.han@intel.com>,
	Dexuan Cui <dexuan.cui@intel.com>
Subject: Re: [PATCH 1/2][VTD] pci mmconfig support to be used for ATS
Date: Thu, 20 Nov 2008 08:56:26 +0000	[thread overview]
Message-ID: <4925344A.76E4.0078.0@novell.com> (raw)
In-Reply-To: <57C9024A16AD2D4C97DC78E552063EA35C4BC7A4@orsmsx505.amr.corp.intel.com>

>>> "Kay, Allen M" <allen.m.kay@intel.com> 20.11.08 02:42 >>>
>ATS (Address Translation Service) is a PCI specification used for synchronizing iotlb translations between chipset iommu such as vt-d with iotlb on board a PCI device.
>
>This patch enables PCI mmconfig for Intel64 systems.  Most of the code were copied from Linux.  This functionality is need for parsing ATS capability in PCIe extended configuration space.

If this is x86-64 only anyway, why do you need to use fixmap space for this (and limit things arbitrarily to 4 segments) rather than using a 256Mb block from one of the unused virtual address regions in hypervisor space?

Also, using obj-$(CONFIG_X86_64) in the Makefile changes would avoid building dead code for x86-32...

Jan

  reply	other threads:[~2008-11-20  8:56 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-20  1:42 [PATCH 1/2][VTD] pci mmconfig support to be used for ATS Kay, Allen M
2008-11-20  8:56 ` Jan Beulich [this message]
2008-11-20 20:10   ` Kay, Allen M
2008-11-20 20:20     ` Keir Fraser
2008-11-21 10:32       ` [PATCH 1/2][VTD] pci mmconfig support to be usedfor ATS Jan Beulich
2008-11-20 11:54 ` [PATCH 1/2][VTD] pci mmconfig support to be used for ATS Espen Skoglund
2008-11-20 20:04   ` Kay, Allen M

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4925344A.76E4.0078.0@novell.com \
    --to=jbeulich@novell.com \
    --cc=allen.m.kay@intel.com \
    --cc=dexuan.cui@intel.com \
    --cc=weidong.han@intel.com \
    --cc=xen-devel@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.