kernel thread creation cleanup
Alexander Leidinger
Alexander at Leidinger.net
Wed Apr 6 13:43:33 UTC 2011
Quoting Kostik Belousov <kostikbel at gmail.com> (from Wed, 6 Apr 2011
13:21:43 +0300):
> On Tue, Apr 05, 2011 at 06:16:01PM -0700, Julian Elischer wrote:
>> I was just looking in the thread creation code after most of a decade
>> NOT looking at it..
>> boy we really need to go through there with a broom.. the cobwebs are
>> getting thick.
>> Like we always call the code to put an upcall, even though we don't
>> have upcalls any more,
> cpu_set_upcall() probably could be renamed to cpu_init_thread(),
> and cpu_set_upcall_kse() is better named cpu_init_thread_for_user().
>
> IMO, rename would only add a code churn.
Do not underestimate the impact of misnamed functions.
Do not underestimate the usefulness of correcly named functions.
Does a change help other developers (to understand or extend) or
improves what we have: it is not code churn.
Bye,
Alexander.
--
You can move the world with an idea,
but you have to think of it first.
http://www.Leidinger.net Alexander @ Leidinger.net: PGP ID = B0063FE7
http://www.FreeBSD.org netchild @ FreeBSD.org : PGP ID = 72077137
More information about the freebsd-current
mailing list