[PATCH] ALSA: timer: fix another ioctl compatibility bug

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

In ALSA timer interface, I found another compatibility bug to different
ABIs. This patch is to fix it. I note that the bug is in really,
really-minor functionality and no drivers use it, therefore it has little
impact to this subsystem. This patch is just for manners between
kernel/userspace interfaces.


To check the compatibility, I wrote a series of easy and small programs for
each ALSA interface. It's available in my repository on github.
https://github.com/takaswie/alsa-ioctl-test

As long as I tested with these programs, this patch is the last fix for
the compatibility issues which comes from differences of data model. On
the other hand, I confirmed that each compatibility layer is not fully
compatible to core implementation. For example, ioctl() with
SNDRV_PCM_IOCTL_WRITEI_FRAMES and argument filled by zero for PCM capture
character device (not for playback device) returns different error codes
for each of x86/x86_64 binary. I think this should be fixed, but don't
start to work for it yet.


Regards

Takashi Sakamoto (1):
  ALSA: timer: fix ioctl compatibility for different data models

 sound/core/timer_compat.c | 29 ++++++++++++++++++++++++++++-
 1 file changed, 28 insertions(+), 1 deletion(-)

-- 
2.7.3

_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux