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