From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH 29/67] dma-direct: use node local allocations for coherent memory Date: Tue, 2 Jan 2018 11:29:10 +0100 Message-ID: References: <20171229081911.2802-1-hch@lst.de> <20171229081911.2802-30-hch@lst.de> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: <20171229081911.2802-30-hch@lst.de> Sender: linux-m68k-owner@vger.kernel.org List-Id: linux-m68k@vger.kernel.org To: Christoph Hellwig Cc: iommu@lists.linux-foundation.org, alpha , arcml , linux-arm-kernel@lists.infradead.org, adi-buildroot-devel@lists.sourceforge.net, linux-c6x-dev@linux-c6x.org, Cris , "open list:QUALCOMM HEXAGON..." , linux-ia64@vger.kernel.org, linux-m68k , "open list:METAG ARCHITECTURE" , Michal Simek , Linux MIPS Mailing List , Parisc List , linuxppc-dev@lists.ozlabs.org, patches@groups.riscv.org, linux-s390 , Linux-sh list , sparclinux Missing patch description explaining why this change is desirable. On Fri, Dec 29, 2017 at 9:18 AM, Christoph Hellwig wrote: > --- a/lib/dma-direct.c > +++ b/lib/dma-direct.c > @@ -39,7 +39,7 @@ static void *dma_direct_alloc(struct device *dev, size_t size, > if (gfpflags_allow_blocking(gfp)) > page = dma_alloc_from_contiguous(dev, count, page_order, gfp); > if (!page) > - page = alloc_pages(gfp, page_order); > + page = alloc_pages_node(dev_to_node(dev), gfp, page_order); > if (!page) > return NULL; > Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds