August 2021 - Alsa-devel - mailman.alsa-project.org

[PATCH] ALSA: memalloc: Fix regression with SNDRV_DMA_TYPE_CONTINUOUS
by Takashi Iwai 01 Aug '21

01 Aug '21

The recent code refactoring made the mmap of continuous pages to be done via the own helper snd_dma_continuous_mmap() with remap_pfn_range(). There I overlooked that dmab->addr isn't set for the allocation with SNDRV_DMA_TYPE_CONTINUOUS. This resulted always in an error at mmap with this buffer type on the system such as Intel SST Baytrail driver. This patch fixes the regression by passing the correct address. Fixes: 30b7ba6972d5 ("ALSA: core: Add continuous and vmalloc mmap ops") Reported-by: Hans de Goede <hdegoede(a)redhat.com> Link: https://lore.kernel.org/r/8d6674da-7d7b-803e-acc9-7de6cb1223fa@redhat.com Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/core/memalloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/core/memalloc.c b/sound/core/memalloc.c index 83b79edfa52d..439a358ecfe9 100644 --- a/sound/core/memalloc.c +++ b/sound/core/memalloc.c @@ -215,7 +215,7 @@ static int snd_dma_continuous_mmap(struct snd_dma_buffer *dmab, struct vm_area_struct *area) { return remap_pfn_range(area, area->vm_start, - dmab->addr >> PAGE_SHIFT, + page_to_pfn(virt_to_page(dmab->area)), area->vm_end - area->vm_start, area->vm_page_prot); } -- 2.26.2

1 0

[5.14 regression] "ALSA: core: Add continuous and vmalloc mmap ops" breaks Intel SST audio
by Hans de Goede 01 Aug '21

01 Aug '21

Hi All;, As discussed in the "5.14 regression, Intel SST (BYT) audio no longer works" thread, audio on X86 devices using the Intel SST driver no longer works with 5.14. After poking at this for a while I've found the culprit: 30b7ba6972d5 ("ALSA: core: Add continuous and vmalloc mmap ops") If I revert that single commit then everything works fine again with 5.14. This is with a Fedora 34 userspace using pipewire as audiodaemon Regards, Hans

2 3

[PATCH] ALSA: usb-audio: make array static const, makes object smaller
by Colin King 01 Aug '21

01 Aug '21

From: Colin Ian King <colin.king(a)canonical.com> Don't populate array names_to_check on the stack but instead it static. Makes the object code smaller by 56 bytes. Before: text data bss dec hex filename 103512 34380 0 137892 21aa4 ./sound/usb/mixer.o After: text data bss dec hex filename 103264 34572 0 137836 21a6c ./sound/usb/mixer.o gcc version 10.2.0) Signed-off-by: Colin Ian King <colin.king(a)canonical.com> --- sound/usb/mixer.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/sound/usb/mixer.c b/sound/usb/mixer.c index f4cdaf1ba44a..aec2499284a5 100644 --- a/sound/usb/mixer.c +++ b/sound/usb/mixer.c @@ -1572,8 +1572,9 @@ static size_t append_ctl_name(struct snd_kcontrol *kctl, const char *str) static void check_no_speaker_on_headset(struct snd_kcontrol *kctl, struct snd_card *card) { - const char *names_to_check[] = { - "Headset", "headset", "Headphone", "headphone", NULL}; + static const char *names_to_check[] = { + "Headset", "headset", "Headphone", "headphone", NULL + }; const char **s; bool found = false; -- 2.31.1

2 1