From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9F9A8C2D0DA for ; Thu, 26 Dec 2019 14:41:38 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 7C747206CB for ; Thu, 26 Dec 2019 14:41:38 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726626AbfLZOlh (ORCPT ); Thu, 26 Dec 2019 09:41:37 -0500 Received: from mx3.molgen.mpg.de ([141.14.17.11]:60597 "EHLO mx1.molgen.mpg.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1725954AbfLZOlh (ORCPT ); Thu, 26 Dec 2019 09:41:37 -0500 Received: from [192.168.1.6] (x590e15ff.dyn.telefonica.de [89.14.21.255]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: pmenzel) by mx.molgen.mpg.de (Postfix) with ESMTPSA id 8BDD5202254E9; Thu, 26 Dec 2019 15:41:34 +0100 (CET) Subject: Re: [Regression drm-tip] Internal audio device missing To: Takashi Iwai Cc: Jaroslav Kysela , Takashi Sakamoto , Takashi Iwai , alsa-devel@alsa-project.org, LKML , dri-devel@lists.freedesktop.org References: <6aa4e167-abb0-6a2c-c00e-558aa79d94be@molgen.mpg.de> From: Paul Menzel Message-ID: <91e1ebe8-270e-d62f-844e-2018fe77145e@molgen.mpg.de> Date: Thu, 26 Dec 2019 15:41:34 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.3.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Dear Takashi, Am 26.12.19 um 11:03 schrieb Takashi Iwai: > On Thu, 26 Dec 2019 10:47:18 +0100, Paul Menzel wrote: >> With >> >> $ git describe --tags drm-tip/drm-tip >> v5.5-rc3-1481-ga20d8cd6901a >> >> the internal audio device is not available, and just a dummy device. >> >> Running `alsa-info.sh` [1], the messages below are shown with the >> problematic Linux kernel. >> >> alsactl: get_controls:567: snd_ctl_open error: Sound protocol is >> not compatible >> cat: /tmp/alsa-info.ateDlDjrZX/alsactl.tmp: No such file or directory > > That's an unexpected side-effect of the recent protocol version bump > in sound.git for-next branch. It seems that we can't change the minor > version unless we really want to break something. > > Below is the fix patch. Please give it a try. Thank you for the quick reply and fix. > -- 8< -- > From: Takashi Iwai > Subject: [PATCH] ALSA: control: Fix incompatible protocol error > > The recent change to bump the ALSA control API protocol version from > 2.0.7 to 2.1.0 caused a regression on user-space; while the user-space > expects both the major and the minor versions to be identical with the > supported numbers, we changed the minor number from 0 to 1. > > For recovering from the incompatibility, this patch changes the > protocol version again to 2.0.8, which is compatible, but yet higher > than the original number 2.0.7, indicating that the protocol change. > > Fixes: bd3eb4e87eb3 ("ALSA: ctl: bump protocol version up to v2.1.0") > Reported-by: Paul Menzel > Signed-off-by: Takashi Iwai > --- > include/uapi/sound/asound.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/uapi/sound/asound.h b/include/uapi/sound/asound.h > index e36dadaf84ba..30ebb2a42983 100644 > --- a/include/uapi/sound/asound.h > +++ b/include/uapi/sound/asound.h > @@ -936,7 +936,7 @@ struct snd_timer_tread { > * * > ****************************************************************************/ > > -#define SNDRV_CTL_VERSION SNDRV_PROTOCOL_VERSION(2, 1, 0) > +#define SNDRV_CTL_VERSION SNDRV_PROTOCOL_VERSION(2, 0, 8) > > struct snd_ctl_card_info { > int card; /* card number */ > Tested-by: Paul Menzel Are there CI systems, which should have caught this problem? Which user-space component should forward this problem to the user (desktop environment displaying a warning)? Kind regards, Paul