OSDN Git Service

ALSA: Replace snd_malloc_pages() and snd_free_pages() with standard helpers, take#2
authorTakashi Iwai <tiwai@suse.de>
Fri, 23 Nov 2018 18:38:13 +0000 (19:38 +0100)
committerTakashi Iwai <tiwai@suse.de>
Wed, 27 Mar 2019 16:15:01 +0000 (17:15 +0100)
commit734b5a0bbdf43518e6739c8156a985e385e557fe
treecfadbf71e5579f10a6ae526f31923348a9007ae0
parente42dd3ee3f9c6007c569386a8477a19d3e7503f9
ALSA: Replace snd_malloc_pages() and snd_free_pages() with standard helpers, take#2

snd_malloc_pages() and snd_free_pages() are merely thin wrappers of
the standard page allocator / free functions.  Even the arguments are
compatible with some standard helpers, so there is little merit of
keeping these wrappers.

This patch replaces the all existing callers of snd_malloc_pages() and
snd_free_pages() with the direct calls of the standard helper
functions.  In this version, we use a recently introduced one,
alloc_pages_exact(), which suits better than the old
snd_malloc_pages() implementation for our purposes.  Then we can avoid
the waste of pages by alignment to power-of-two.

Since alloc_pages_exact() does split pages, we need no longer
__GFP_COMP flag; or better to say, we must not pass __GFP_COMP to
alloc_pages_exact().  So the former unconditional addition of
__GFP_COMP flag in snd_malloc_pages() is dropped, as well as in most
other places.

Reviewed-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Acked-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
include/sound/memalloc.h
sound/core/memalloc.c
sound/core/pcm.c
sound/usb/usx2y/usX2Yhwdep.c
sound/usb/usx2y/usbusx2y.c
sound/usb/usx2y/usx2yhwdeppcm.c