Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933307AbbDWJwu (ORCPT ); Thu, 23 Apr 2015 05:52:50 -0400 Received: from smtp02.citrix.com ([66.165.176.63]:5729 "EHLO SMTP02.CITRIX.COM" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751340AbbDWJwr (ORCPT ); Thu, 23 Apr 2015 05:52:47 -0400 X-IronPort-AV: E=Sophos;i="5.11,630,1422921600"; d="scan'208";a="257850260" Message-ID: <5538C0D4.5050400@citrix.com> Date: Thu, 23 Apr 2015 10:52:20 +0100 From: David Vrabel User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.4.0 MIME-Version: 1.0 To: Stefano Stabellini , CC: , Konrad Rzeszutek Wilk , , Subject: Re: [PATCH] xen: Add __GFP_DMA flag when xen_swiotlb_init gets free pages on ARM References: In-Reply-To: Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-DLP: MIA2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3222 Lines: 95 On 23/04/15 10:47, Stefano Stabellini wrote: > Make sure that xen_swiotlb_init allocates buffers that are DMA capable > when at least one memblock is available below 4G. Otherwise we assume > that all devices on the SoC can cope with >4G addresses. This commit message must explain why the PFNs (IPAs) are relevant for DMA. David > No functional changes on x86. > > From: Chen Baozi > > Signed-off-by: Chen Baozi > Signed-off-by: Stefano Stabellini > Tested-by: Chen Baozi > > diff --git a/arch/arm/include/asm/xen/page.h b/arch/arm/include/asm/xen/page.h > index 2f7e6ff..0b579b2 100644 > --- a/arch/arm/include/asm/xen/page.h > +++ b/arch/arm/include/asm/xen/page.h > @@ -110,5 +110,6 @@ static inline bool set_phys_to_machine(unsigned long pfn, unsigned long mfn) > bool xen_arch_need_swiotlb(struct device *dev, > unsigned long pfn, > unsigned long mfn); > +unsigned long xen_get_swiotlb_free_pages(unsigned int order); > > #endif /* _ASM_ARM_XEN_PAGE_H */ > diff --git a/arch/arm/xen/mm.c b/arch/arm/xen/mm.c > index 793551d..4983250 100644 > --- a/arch/arm/xen/mm.c > +++ b/arch/arm/xen/mm.c > @@ -4,6 +4,7 @@ > #include > #include > #include > +#include > #include > #include > #include > @@ -21,6 +22,20 @@ > #include > #include > > +unsigned long xen_get_swiotlb_free_pages(unsigned int order) > +{ > + struct memblock_region *reg; > + gfp_t flags = __GFP_NOWARN; > + > + for_each_memblock(memory, reg) { > + if (reg->base < (phys_addr_t)0xffffffff) { > + flags |= __GFP_DMA; > + break; > + } > + } > + return __get_free_pages(flags, order); > +} > + > enum dma_cache_op { > DMA_UNMAP, > DMA_MAP, > diff --git a/arch/x86/include/asm/xen/page.h b/arch/x86/include/asm/xen/page.h > index 358dcd3..c44a5d5 100644 > --- a/arch/x86/include/asm/xen/page.h > +++ b/arch/x86/include/asm/xen/page.h > @@ -269,4 +269,9 @@ static inline bool xen_arch_need_swiotlb(struct device *dev, > return false; > } > > +static inline unsigned long xen_get_swiotlb_free_pages(unsigned int order) > +{ > + return __get_free_pages(__GFP_NOWARN, order); > +} > + > #endif /* _ASM_X86_XEN_PAGE_H */ > diff --git a/drivers/xen/swiotlb-xen.c b/drivers/xen/swiotlb-xen.c > index 810ad41..4c54932 100644 > --- a/drivers/xen/swiotlb-xen.c > +++ b/drivers/xen/swiotlb-xen.c > @@ -235,7 +235,7 @@ retry: > #define SLABS_PER_PAGE (1 << (PAGE_SHIFT - IO_TLB_SHIFT)) > #define IO_TLB_MIN_SLABS ((1<<20) >> IO_TLB_SHIFT) > while ((SLABS_PER_PAGE << order) > IO_TLB_MIN_SLABS) { > - xen_io_tlb_start = (void *)__get_free_pages(__GFP_NOWARN, order); > + xen_io_tlb_start = (void *)xen_get_swiotlb_free_pages(order); > if (xen_io_tlb_start) > break; > order--; > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/