cvs commit: src/sys/kern kern_shutdown.c vfs_subr.c
Alfred Perlstein
alfred at freebsd.org
Thu Jul 15 14:28:37 PDT 2004
* John Baldwin <jhb at FreeBSD.org> [040715 11:56] wrote:
> On Thursday 15 July 2004 01:29 am, Alfred Perlstein wrote:
> > > Also, why do you copy the proc name to the stack lower in
> > > kern_shutdown.c? Do you fear that the proc might change from
> > > under you when you're at the bottom and you can't therefore
> > > only keep a reference to the proc instead?
> >
> > Because I don't know if the proc will be there after it nukes itself.
>
> They don't nuke themselves, they just go to sleep. A quick look at the code
> would have revealed this.
y'know what, in the context of fixing a printf I wasn't really
interested in looking that far into the scheduler. A 20 char stack
variable and a string copy at shutdown isn't such a big deal. If it
bugs you guys that much, why don't you just fix it?
Or are you worried about getting 15 emails about how you inefficiently
printing a non-critical string like I have? :)
hugs,
--
- Alfred Perlstein
- Research Engineering Development Inc.
- email: bright at mu.org cell: 408-480-4684
More information about the cvs-src
mailing list