Linux Sound subsystem development
 help / color / mirror / Atom feed
From: Maciej Strozek <mstrozek@opensource.cirrus.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: Jaroslav Kysela <perex@perex.cz>,
	alsa-devel@alsa-project.org, patches@opensource.cirrus.com,
	linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ALSA: sound: Increase max size of components field
Date: Tue, 28 Oct 2025 11:14:57 +0000	[thread overview]
Message-ID: <f6596d39f91115567741add153eec5c476a85656.camel@opensource.cirrus.com> (raw)
In-Reply-To: <20251028004252.GA553484@workstation.local>

W dniu wto, 28.10.2025 o godzinie 09∶42 +0900, użytkownik Takashi
Sakamoto napisał:
> Hi,
> 
> On Mon, Oct 27, 2025 at 11:50:32AM +0000, Maciej Strozek wrote:
> > diff --git a/include/uapi/sound/asound.h
> > b/include/uapi/sound/asound.h
> > index c302698fb685..7d53f6da59e2 100644
> > --- a/include/uapi/sound/asound.h
> > +++ b/include/uapi/sound/asound.h
> > @@ -1069,7 +1069,12 @@ struct snd_ctl_card_info {
> >         unsigned char longname[80];
> >         unsigned char reserved_[16];
> >         unsigned char mixername[80];
> > -       unsigned char components[256];
> > +       unsigned char components[128];
> > +};
> 
> Any approach to change the result of 'sizeof(struct
> snd_ctl_card_info)'
> breaks ABI to userspace application. This is the reason that Jaroslav
> addresses to the new PCM ioctl command. Unless, the userspace
> application build with the UAPI header provided by the existing
> version
> of Linux kernel does not work as expected in your version of Linux
> kernel, in respect to the sound subsystem.
> 
Apologies for the confusion, the draft was based on the initial patch
that increased the size of the components, so this snippet is actually
bringing back the original size. This will not be present in V2.

> 
> Regards
> 
> Takashi Sakamoto

-- 
Regards,
Maciej

  reply	other threads:[~2025-10-28 11:15 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-23  9:27 [PATCH] ALSA: sound: Increase max size of components field Maciej Strozek
2025-10-23 11:56 ` Jaroslav Kysela
2025-10-27  9:04   ` Takashi Iwai
2025-10-27 11:50     ` Maciej Strozek
2025-10-27 12:32       ` Richard Fitzgerald
2025-10-28  0:42       ` Takashi Sakamoto
2025-10-28 11:14         ` Maciej Strozek [this message]
2025-10-28 11:39       ` Jaroslav Kysela

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f6596d39f91115567741add153eec5c476a85656.camel@opensource.cirrus.com \
    --to=mstrozek@opensource.cirrus.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=patches@opensource.cirrus.com \
    --cc=perex@perex.cz \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox