There’s a discussion going on right now on lkml about the evils (read: inherent raciness) of the sleep_on() interface. Looking at trident.c, I see two uses of interruptible_sleep_on_timeout(). I have to confess I can’t immediately see the race there, but moving to the wait_event() interface is still probably in order. Unfortunately, Ranny borrowed tea, the laptop with the trident card in it, for his flight to the US and won’t return it until he comes back in a week and a half. Not being able to test your driver changes sucks.
January 18, 2004
2 Comments »
RSS feed for comments on this post. TrackBack URI
Which particular trident card were you using?
Comment by ideawerkz — January 18, 2004 @ 12:18 PM |
ALi 5451, IIRC.
Comment by mulix — January 19, 2004 @ 12:32 AM |