At Tue, 07 Jun 2011 11:30:28 +0200, Adrian Knoth wrote:
On 06/07/2011 11:28 AM, Takashi Iwai wrote:
Well, it's a wrong lock for snd_hdspm_midi_input_available(). So you'll need to lock twice, either once unlock and lock another or nested. Re-locking would be less messy in this case, I suppose.
ok, here's a revised patch. has been running for 20 or so hours without problems, although the load was light most of the time.
please consider for inclusion.
Looks good. But please give a proper changelog and your sign-off for inclusion.
I've handled this. I take care of everything, commit (with changelog and sign-off) to my git repo and will send the final patch series to the list once it's ready.
OK, thanks.
(Note that usually a patch always requires the sign-off of the original author, no matter who commits. But in this case, it doesn't matter much because the patch is trivial.)
Takashi