From mboxrd@z Thu Jan  1 00:00:00 1970
From: Suravee Suthikulanit <suravee.suthikulpanit@amd.com>
Subject: Re: [PATCH 2/2] AMD IOMMU: use ioremap()
Date: Wed, 10 Jul 2013 12:12:26 -0500
Message-ID: <51DD95FA.9060106@amd.com>
References: <51DD555102000078000E3D11@nat28.tlf.novell.com>
	<51DD585702000078000E3D40@nat28.tlf.novell.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <51DD585702000078000E3D40@nat28.tlf.novell.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Jan Beulich <JBeulich@suse.com>
Cc: Jacob Shin <jacob.shin@amd.com>, xiantao.zhang@intel.com, xen-devel <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

On 7/10/2013 5:49 AM, Jan Beulich wrote:
> There's no point in using the fixmap here, and it gets
> map_iommu_mmio_region() in line with unmap_iommu_mmio_region(), which
> was already using iounmap() (thus crashing if actually used).
>
> Signed-off-by: Jan Beulich<jbeulich@suse.com>
>
> --- a/xen/drivers/passthrough/amd/iommu_init.c
> +++ b/xen/drivers/passthrough/amd/iommu_init.c
> @@ -48,19 +48,10 @@ static int iommu_has_ht_flag(struct amd_
>   
>   static int __init map_iommu_mmio_region(struct amd_iommu *iommu)
>   {
> -    unsigned long mfn;
> -
> -    if ( nr_amd_iommus > MAX_AMD_IOMMUS )
> -    {
> -        AMD_IOMMU_DEBUG("nr_amd_iommus %d > MAX_IOMMUS\n", nr_amd_iommus);
> +    iommu->mmio_base = ioremap(iommu->mmio_base_phys,
> +                               IOMMU_MMIO_REGION_LENGTH);
> +    if ( iommu->mmio_base )
This should have been "if ( !iommu->mmio_base )".

>           return -ENOMEM;
> -    }
> -
> -    iommu->mmio_base = (void *)fix_to_virt(
> -        FIX_IOMMU_MMIO_BASE_0 + nr_amd_iommus * MMIO_PAGES_PER_IOMMU);
> -    mfn = (unsigned long)(iommu->mmio_base_phys >> PAGE_SHIFT);
> -    map_pages_to_xen((unsigned long)iommu->mmio_base, mfn,
> -                     MMIO_PAGES_PER_IOMMU, PAGE_HYPERVISOR_NOCACHE);
>   
>       memset(iommu->mmio_base, 0, IOMMU_MMIO_REGION_LENGTH);
Once I changed the above logic, IOMMU is initialized correctly. I have 
tested pass-though a NIC and thing looks fine.

Suravee