ALSA: memalloc: Don't align the size to power-of-two

The size passed to dma_alloc_coherent() doesn't have to be aligned
with power-of-two, rather it should be the raw size.  As a minor
optimization, remove the size adjustment in the current code.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
Takashi Iwai 2018-08-08 16:56:46 +02:00
Родитель b8e131542b
Коммит 03486830c5
1 изменённых файлов: 2 добавлений и 7 удалений

Просмотреть файл

@ -84,29 +84,24 @@ EXPORT_SYMBOL(snd_free_pages);
/* allocate the coherent DMA pages */ /* allocate the coherent DMA pages */
static void *snd_malloc_dev_pages(struct device *dev, size_t size, dma_addr_t *dma) static void *snd_malloc_dev_pages(struct device *dev, size_t size, dma_addr_t *dma)
{ {
int pg;
gfp_t gfp_flags; gfp_t gfp_flags;
if (WARN_ON(!dma)) if (WARN_ON(!dma))
return NULL; return NULL;
pg = get_order(size);
gfp_flags = GFP_KERNEL gfp_flags = GFP_KERNEL
| __GFP_COMP /* compound page lets parts be mapped */ | __GFP_COMP /* compound page lets parts be mapped */
| __GFP_NORETRY /* don't trigger OOM-killer */ | __GFP_NORETRY /* don't trigger OOM-killer */
| __GFP_NOWARN; /* no stack trace print - this call is non-critical */ | __GFP_NOWARN; /* no stack trace print - this call is non-critical */
return dma_alloc_coherent(dev, PAGE_SIZE << pg, dma, gfp_flags); return dma_alloc_coherent(dev, size, dma, gfp_flags);
} }
/* free the coherent DMA pages */ /* free the coherent DMA pages */
static void snd_free_dev_pages(struct device *dev, size_t size, void *ptr, static void snd_free_dev_pages(struct device *dev, size_t size, void *ptr,
dma_addr_t dma) dma_addr_t dma)
{ {
int pg;
if (ptr == NULL) if (ptr == NULL)
return; return;
pg = get_order(size); dma_free_coherent(dev, size, ptr, dma);
dma_free_coherent(dev, PAGE_SIZE << pg, ptr, dma);
} }
#ifdef CONFIG_GENERIC_ALLOCATOR #ifdef CONFIG_GENERIC_ALLOCATOR