[alsa-devel] [PATCH] ALSA: core: fix 64-bit SNDRV_PCM_IOCTL_STATUS ABI breakage
tiwai at suse.de
Mon Oct 29 08:15:46 CET 2012
At Sun, 28 Oct 2012 11:45:35 +0100,
Clemens Ladisch wrote:
> Takashi Iwai wrote:
> > Clemens Ladisch wrote:
> >> Commit 4eeaaeaea (ALSA: core: add hooks for audio timestamps) added the
> >> new audio_tstamp field to struct snd_pcm_status. However, struct
> >> timespec requires 64-bit alignment, so the 64-bit compiler would insert
> >> 32 bits of padding before this field, which broke SNDRV_PCM_IOCTL_STATUS
> >> with error messages like this:
> >> kernel: unknown ioctl = 0x80984120
> >> To solve this, insert the padding explicitly so that it can be taken
> >> into account when calculating the ABI structure size.
> >> Signed-off-by: Clemens Ladisch <clemens at ladisch.de>
> > Oops, that's bad. Thanks for catching this.
> > I applied it now to for-next branch.
> > Could you fix alsa-lib code as well?
> Oops; done.
More information about the Alsa-devel