Re: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Isaku Yamahata
On Wed, Oct 22, 2008 at 02:11:52PM +0800, Xu, Anthony wrote: Isaku Yamahata wrote: On Wed, Oct 22, 2008 at 01:56:05PM +0800, Xu, Anthony wrote: Yes, it is not SMP-safe there is lock for p2m. Modifying p2m is not a frequent operation, why not add a lock for it? It is frequent to read p2m

RE: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Xu, Anthony
Isaku Yamahata wrote: On Wed, Oct 22, 2008 at 02:11:52PM +0800, Xu, Anthony wrote: Isaku Yamahata wrote: On Wed, Oct 22, 2008 at 01:56:05PM +0800, Xu, Anthony wrote: Yes, it is not SMP-safe there is lock for p2m. Modifying p2m is not a frequent operation, why not add a lock for it? It is

Re: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Isaku Yamahata
On Wed, Oct 22, 2008 at 02:30:50PM +0800, Xu, Anthony wrote: Isaku Yamahata wrote: On Wed, Oct 22, 2008 at 02:11:52PM +0800, Xu, Anthony wrote: Isaku Yamahata wrote: On Wed, Oct 22, 2008 at 01:56:05PM +0800, Xu, Anthony wrote: Yes, it is not SMP-safe there is lock for p2m. Modifying p2m

RE: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Xu, Anthony
Isaku Yamahata wrote: On Wed, Oct 22, 2008 at 02:30:50PM +0800, Xu, Anthony wrote: Isaku Yamahata wrote: On Wed, Oct 22, 2008 at 02:11:52PM +0800, Xu, Anthony wrote: Isaku Yamahata wrote: On Wed, Oct 22, 2008 at 01:56:05PM +0800, Xu, Anthony wrote: Yes, it is not SMP-safe there is lock for

Re: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Isaku Yamahata
On Wed, Oct 22, 2008 at 02:50:40PM +0800, Xu, Anthony wrote: Isaku Yamahata wrote: On Wed, Oct 22, 2008 at 02:30:50PM +0800, Xu, Anthony wrote: Isaku Yamahata wrote: On Wed, Oct 22, 2008 at 02:11:52PM +0800, Xu, Anthony wrote: Isaku Yamahata wrote: On Wed, Oct 22, 2008 at 01:56:05PM

RE: [Xen-ia64-devel][PATCH][VTD] add hypercalls needed for VTD

2008-10-22 Thread Xu, Anthony
The new one, I applies manually #Cset 18649 -- 18653 in xen-devel tree on #Cset 18637. Tested by booting rhel5.1 guest. Please pull xen-devel tree and apply this patch. I'll send out other patches soon. Thanks Anthony Isaku Yamahata wrote: Hi Anthony. With the following 3 patches applied,

Re: [Xen-ia64-devel][PATCH][VTD] add head files and helper functions for VTD/ia64

2008-10-22 Thread Isaku Yamahata
Applied, thanks. On Tue, Oct 21, 2008 at 06:17:00PM +0800, Xu, Anthony wrote: Updated one per your comments Thanks, Anthony Isaku Yamahata wrote: On Tue, Oct 21, 2008 at 11:02:08AM +0800, Xu, Anthony wrote: Add head files and helper functions for VTD/ia64. Signed-off-by; Anthony

Re: [Xen-ia64-devel][PATCH] IA64: update ia64 px info transfer corresponding to x86 px info update.

2008-10-22 Thread Isaku Yamahata
Applied, thanks. On Tue, Oct 21, 2008 at 10:49:27AM +0800, Liu, Jinsong wrote: IA64: update ia64 px info transfer corresponding to x86 px info update. In x86 side, there is a new patch to solve platform hypercall compatible issue result from px info transfer. This patch update ia64 px info

Re: [Xen-ia64-devel][PATCH][VTD] add hypercalls needed for VTD

2008-10-22 Thread Isaku Yamahata
Applied, thanks. On Wed, Oct 22, 2008 at 03:12:29PM +0800, Xu, Anthony wrote: The new one, I applies manually #Cset 18649 -- 18653 in xen-devel tree on #Cset 18637. Tested by booting rhel5.1 guest. Please pull xen-devel tree and apply this patch. I'll send out other patches soon. Thanks

[Xen-ia64-devel] Please pull ia64 trees

2008-10-22 Thread Isaku Yamahata
Hi Keir, Please pull the ia64 trees: http://xenbits.xensource.com/ext/ia64/xen-unstable.hg http://xenbits.xensource.com/ext/ia64/linux-2.6.18-xen.hg This includes the ones which fixes compilation breakage and VT-d preliminary patches from Anthony Xu. Thanks, -- yamahata

[Xen-ia64-devel][PATCH][VTD][dom0] dom0 vtd patch

2008-10-22 Thread Xu, Anthony
dom0/ia64/vtd patch Signed-off-by: Anthony Xu [EMAIL PROTECTED] dom0_vtd.patch Description: dom0_vtd.patch ___ Xen-ia64-devel mailing list Xen-ia64-devel@lists.xensource.com http://lists.xensource.com/xen-ia64-devel

RE: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Xu, Anthony
The new one, -if (phy_pte.ma != VA_MATTR_NATPAGE) +/* if a device is assigned to a domain through VTD, the MMIO for this + * device needs to retain to UC attribute + */ +if (phy_pte.ma == VA_MATTR_WC) phy_pte.ma = VA_MATTR_WB; Doesn't this break the

RE: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Kouya Shimura
Hi Anthony, Xu, Anthony writes: -if (phy_pte.ma != VA_MATTR_NATPAGE) +/* if a device is assigned to a domain through VTD, the MMIO for this + * device needs to retain to UC attribute + */ +if (phy_pte.ma == VA_MATTR_WC) phy_pte.ma = VA_MATTR_WB;

Re: [Xen-ia64-devel][PATCH][VTD][dom0] dom0 vtd patch

2008-10-22 Thread Isaku Yamahata
On Wed, Oct 22, 2008 at 05:43:13PM +0800, Xu, Anthony wrote: dom0/ia64/vtd patch Signed-off-by: Anthony Xu [EMAIL PROTECTED] In XEN_DOMCTL_get_device_group case. struct xen_domctl_get_device_group { uint32_t machine_bdf; /* IN */ uint32_t max_sdevs;/* IN */

Re: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Isaku Yamahata
On Wed, Oct 22, 2008 at 05:35:36PM +0800, Xu, Anthony wrote: The new one, -if (phy_pte.ma != VA_MATTR_NATPAGE) +/* if a device is assigned to a domain through VTD, the MMIO for this + * device needs to retain to UC attribute + */ +if (phy_pte.ma == VA_MATTR_WC)

Re: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Isaku Yamahata
On Thu, Oct 23, 2008 at 10:03:55AM +0800, Xu, Anthony wrote: Isaku Yamahata wrote: On Wed, Oct 22, 2008 at 05:35:36PM +0800, Xu, Anthony wrote: The new one, -if (phy_pte.ma != VA_MATTR_NATPAGE) +/* if a device is assigned to a domain through VTD, the MMIO for this + *

RE: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Xu, Anthony
Isaku Yamahata wrote: On Thu, Oct 23, 2008 at 10:03:55AM +0800, Xu, Anthony wrote: Isaku Yamahata wrote: On Wed, Oct 22, 2008 at 05:35:36PM +0800, Xu, Anthony wrote: The new one, -if (phy_pte.ma != VA_MATTR_NATPAGE) +/* if a device is assigned to a domain through VTD, the MMIO for

Re: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Isaku Yamahata
On Thu, Oct 23, 2008 at 10:31:20AM +0800, Xu, Anthony wrote: Iommu_un/map_page does nothing for PV domain. Why is there a race? Oh sorry. I'm not very familiar with iommu code (yet). My understanding is iommu_domain_init() is called unconditionaly by arch_domain_create() so that

[Xen-ia64-devel] How about stub-domain in IPF-Xen now?

2008-10-22 Thread Zhang, Jingke
Hi Isaku, We checked the stub-domain in x86 side, basically it works well for booting, smp, and memory (only some issues on save-restore). I remember we did not support stub-domain for resource limit. Do you have new plan to let IA64-Xen support it? Or is ia64 stub-domain available now?

RE: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Xu, Anthony
Isaku Yamahata wrote: On Thu, Oct 23, 2008 at 10:31:20AM +0800, Xu, Anthony wrote: Iommu_un/map_page does nothing for PV domain. Why is there a race? Oh sorry. I'm not very familiar with iommu code (yet). My understanding is iommu_domain_init() is called unconditionaly by

RE: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Xu, Anthony
Xu, Anthony wrote: Isaku Yamahata wrote: On Thu, Oct 23, 2008 at 10:31:20AM +0800, Xu, Anthony wrote: Iommu_un/map_page does nothing for PV domain. Why is there a race? Oh sorry. I'm not very familiar with iommu code (yet). My understanding is iommu_domain_init() is called unconditionaly

[Xen-ia64-devel][PATCH][VTD]P2M entry may change from _PAGE_IO to real MMIO page

2008-10-22 Thread Xu, Anthony
P2M entry may change from _PAGE_IO to real MMIO page Signed-off-by: Anthony Xu [EMAIL PROTECTED] enhancement_assign_page.patch Description: enhancement_assign_page.patch ___ Xen-ia64-devel mailing list Xen-ia64-devel@lists.xensource.com

RE: [Xen-ia64-devel][PATCH][VTD] small patches for VTD

2008-10-22 Thread Xu, Anthony
Hi shimura I only tested in old version 18134. I can reproduce it. And I find another way to handle it, Will send out patch soon. Thanks, anthony Kouya Shimura wrote: Hi Anthony, Xu, Anthony writes: -if (phy_pte.ma != VA_MATTR_NATPAGE) +/* if a device is assigned to a domain

[Xen-ia64-devel][PATCH][VTD] a small fix in mm.c

2008-10-22 Thread Xu, Anthony
A small fix in mm.c mfn_valid makes sure it is a normal page Signed-off-by: Anthony Xu [EMAIL PROTECTED] small_fix_mm.patch Description: small_fix_mm.patch ___ Xen-ia64-devel mailing list Xen-ia64-devel@lists.xensource.com