* [PATCH] ALSA: use __devexit_p
@ 2007-06-23 21:40 Randy Dunlap
2007-06-23 22:50 ` Jeff Garzik
2007-06-25 10:45 ` Takashi Iwai
0 siblings, 2 replies; 4+ messages in thread
From: Randy Dunlap @ 2007-06-23 21:40 UTC (permalink / raw)
To: perex, akpm; +Cc: lkml
From: Randy Dunlap <randy.dunlap@oracle.com>
Change __devexit to __devexit_p:
sound/isa/opl3sa2.c:956: error: expected expression before '__attribute__'
Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
---
sound/isa/opl3sa2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- linux-2.6.22-rc5-git8.orig/sound/isa/opl3sa2.c
+++ linux-2.6.22-rc5-git8/sound/isa/opl3sa2.c
@@ -953,7 +953,7 @@ static int snd_opl3sa2_isa_resume(struct
static struct isa_driver snd_opl3sa2_isa_driver = {
.match = snd_opl3sa2_isa_match,
.probe = snd_opl3sa2_isa_probe,
- .remove = __devexit( snd_opl3sa2_isa_remove),
+ .remove = __devexit_p(snd_opl3sa2_isa_remove),
#ifdef CONFIG_PM
.suspend = snd_opl3sa2_isa_suspend,
.resume = snd_opl3sa2_isa_resume,
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ALSA: use __devexit_p
2007-06-23 21:40 [PATCH] ALSA: use __devexit_p Randy Dunlap
@ 2007-06-23 22:50 ` Jeff Garzik
2007-06-25 10:46 ` Takashi Iwai
2007-06-25 10:45 ` Takashi Iwai
1 sibling, 1 reply; 4+ messages in thread
From: Jeff Garzik @ 2007-06-23 22:50 UTC (permalink / raw)
To: perex; +Cc: Randy Dunlap, akpm, lkml
This reminds me...
Someone needs to go through ALSA and audit all delays executed via
$FOO_interruptible().
Several delays within ALSA wait for hardware conditions, and do not
check for signals pending, which means that the wait-for-condition loop
becomes a busy loop:
while (1) {
foo = read_hardware()
if (foo & interesting_bits)
break;
schedule_timeout_interruptible(1);
}
The proper fix is (a) remove "_interruptible" [recommended] or (b) check
for signals.
grep'ing for "_interruptible" quickly finds several such ALSA bugs.
Jeff
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ALSA: use __devexit_p
2007-06-23 22:50 ` Jeff Garzik
@ 2007-06-25 10:46 ` Takashi Iwai
0 siblings, 0 replies; 4+ messages in thread
From: Takashi Iwai @ 2007-06-25 10:46 UTC (permalink / raw)
To: Jeff Garzik; +Cc: perex, Randy Dunlap, akpm, lkml
At Sat, 23 Jun 2007 18:50:57 -0400,
Jeff Garzik wrote:
>
> This reminds me...
>
> Someone needs to go through ALSA and audit all delays executed via
> $FOO_interruptible().
>
> Several delays within ALSA wait for hardware conditions, and do not
> check for signals pending, which means that the wait-for-condition loop
> becomes a busy loop:
>
> while (1) {
> foo = read_hardware()
> if (foo & interesting_bits)
> break;
> schedule_timeout_interruptible(1);
> }
>
> The proper fix is (a) remove "_interruptible" [recommended] or (b) check
> for signals.
>
> grep'ing for "_interruptible" quickly finds several such ALSA bugs.
Yep... I fixed now on ALSA tree. Thanks for hints!
Takashi
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ALSA: use __devexit_p
2007-06-23 21:40 [PATCH] ALSA: use __devexit_p Randy Dunlap
2007-06-23 22:50 ` Jeff Garzik
@ 2007-06-25 10:45 ` Takashi Iwai
1 sibling, 0 replies; 4+ messages in thread
From: Takashi Iwai @ 2007-06-25 10:45 UTC (permalink / raw)
To: Randy Dunlap; +Cc: perex, akpm, lkml
At Sat, 23 Jun 2007 14:40:22 -0700,
Randy Dunlap wrote:
>
> From: Randy Dunlap <randy.dunlap@oracle.com>
>
> Change __devexit to __devexit_p:
> sound/isa/opl3sa2.c:956: error: expected expression before '__attribute__'
>
> Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Thanks, I applied them to ALSA tree now.
Takashi
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2007-06-25 10:46 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-06-23 21:40 [PATCH] ALSA: use __devexit_p Randy Dunlap
2007-06-23 22:50 ` Jeff Garzik
2007-06-25 10:46 ` Takashi Iwai
2007-06-25 10:45 ` Takashi Iwai
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).