Messages in this thread | | | From | Ingo Oeser <> | Subject | Re: [PATCH 0/4] add task handling notifier | Date | Thu, 20 Dec 2007 23:25:14 +0100 |
| |
Hi Jan,
I like and support your idea!
On Thursday 20 December 2007, Jan Beulich wrote: > With more and more sub-systems/sub-components leaving their footprint > in task handling functions, it seems reasonable to add notifiers that > these components can use instead of having them all patch themselves > directly into core files.
Yes, but why export variables? Wouldn't it be better to export an API?
That simplifies the callers (they all pass "current" as task and "task_notifier_list" as arguments).
It also prevents exposing internal variables (notifier lists ARE internal variables) to modules.
What do you think?
Best Regards
Ingo Oeser
| |