[alsa-devel] [PATCH] Show processing coefficients in codec proc file
david.henningsson at canonical.com
Fri Aug 20 16:46:25 CEST 2010
2010-08-19 22:24, Takashi Iwai skrev:
> At Thu, 19 Aug 2010 20:48:56 +0200,
> David Henningsson wrote:
>> 2010-08-19 20:28, Takashi Iwai skrev:
>>> At Thu, 19 Aug 2010 20:17:42 +0200,
>>> David Henningsson wrote:
>>>> This patch is helpful for tracking down bugs without having all
>>>> information about the chip.
>>> I don't want to change coef index in reading a proc file in general.
>> Good point. But I think the solution to that problem would be to
>> - read current index
>> - do the loop
>> - restore current index
>> ...and perhaps protect that with an appropriate mutex (which one)?
>>> If any, you should implement a codec-specific hook.
>> I'm sorry, but I don't really see how that would help...?
> If it's specific to a codec, then you know whether reading the coef
> in that way can be harmful or not. It's not only about the race
> via proc file but also the influence of coef on the actual codec
Oh, so there are chips that are *that* broken...
Yet it could be a lifesaver so I wouldn't want to give up on the idea
Do you know off the top of your head approximately what codec chips this
is about, where reading a coef (or setting its index) can cause unwanted
David Henningsson, Canonical Ltd.
More information about the Alsa-devel