Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: Problem with unregister_ata_idle_func and one-shot events

Re: Problem with unregister_ata_idle_func and one-shot events

From: Jonathan Gordon <jdgordy_at_gmail.com>
Date: Thu, 29 May 2008 11:06:52 +1000

2008/5/29 Bertrik Sikken <bertrik_at_sikken.nl>:
> Hi all,
>
> I think there's a problem with the unregister_ata_idle_func function,
> which is probably responsible for "event xxx not found" panics, like
> the one in http://www.rockbox.org/tracker/task/8993
>
> The problem is that ata_idle events are registered as one-shot events
> which clear themselves once they've fired. A subsequent call to
> unregister_ata_idle_func may fail the find the event (because it
> cleared itself) and cause the panic.
>
> Completely removing unregister_ata_idle_func is an option but may be
> a bit too drastic. I can also see a potential problem if a plugin
> registers an ata_idle callback and then gets unloaded leaving a
> stray function pointer. An example of a plugin that could use the
> ata_idle callback is the battery bench plugin.
> Maybe the simplest way is just to remove the panic.
>
> What do you think?
>
> Kind regards,
> Bertrik
>
>

why not just remove the panicf? (actually why panic at all if the
event isnt found?)
Received on 2008-05-29


Page was last modified "Jan 10 2012" The Rockbox Crew
aaa