From mboxrd@z Thu Jan  1 00:00:00 1970
From: Suravee Suthikulanit <suravee.suthikulpanit@amd.com>
Subject: Re: [PATCH v2 2/2] AMD IOMMU: use ioremap()
Date: Thu, 11 Jul 2013 09:05:01 -0500
Message-ID: <51DEBB8D.2040405@amd.com>
References: <51DD555102000078000E3D11@nat28.tlf.novell.com>
	<51DD585702000078000E3D40@nat28.tlf.novell.com>
	<51DE875502000078000E40A7@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: <51DE875502000078000E40A7@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/11/2013 3:22 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>
> Acked-by: Keir Fraser <keir@xen.org>
Acked-by: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>

> ---
> v2: Fix polarity of error check (thanks for noticing, Suravee).
>
> --- 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 )
>           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);
>   
> --- a/xen/include/asm-x86/fixmap.h
> +++ b/xen/include/asm-x86/fixmap.h
> @@ -60,8 +60,6 @@ enum fixed_addresses {
>       FIX_KEXEC_BASE_0,
>       FIX_KEXEC_BASE_END = FIX_KEXEC_BASE_0 \
>         + ((KEXEC_XEN_NO_PAGES >> 1) * KEXEC_IMAGE_NR) - 1,
> -    FIX_IOMMU_MMIO_BASE_0,
> -    FIX_IOMMU_MMIO_END = FIX_IOMMU_MMIO_BASE_0 + IOMMU_PAGES -1,
>       FIX_TBOOT_SHARED_BASE,
>       FIX_MSIX_IO_RESERV_BASE,
>       FIX_MSIX_IO_RESERV_END = FIX_MSIX_IO_RESERV_BASE + FIX_MSIX_MAX_PAGES -1,
> --- a/xen/include/asm-x86/hvm/svm/amd-iommu-defs.h
> +++ b/xen/include/asm-x86/hvm/svm/amd-iommu-defs.h
> @@ -464,10 +464,7 @@
>   #define IOMMU_CONTROL_DISABLED	0
>   #define IOMMU_CONTROL_ENABLED	1
>   
> -#define MMIO_PAGES_PER_IOMMU        (IOMMU_MMIO_REGION_LENGTH / PAGE_SIZE_4K)
> -#define IOMMU_PAGES                 (MMIO_PAGES_PER_IOMMU * MAX_AMD_IOMMUS)
>   #define DEFAULT_DOMAIN_ADDRESS_WIDTH    48
> -#define MAX_AMD_IOMMUS                  32
>   
>   /* interrupt remapping table */
>   #define INT_REMAP_INDEX_DM_MASK         0x1C00
>
>
>