Re: Trivial Comment Patch: 2.6.6-mm3

From: Danny Cox
Date: Wed May 19 2004 - 06:24:44 EST


Chris,

On Tue, 2004-05-18 at 16:07, Chris Wedgwood wrote:
> On Tue, May 18, 2004 at 09:08:57AM -0400, Danny Cox wrote:
>
> > This fixes a comment in the stack overflow check that's wrong with
> > 4K stacks.
>
> > #ifdef CONFIG_DEBUG_STACKOVERFLOW
> > - /* Debugging check for stack overflow: is there less than 1KB free? */
> > + /* Debugging check for stack overflow: is there less than 512B free? */
> > {
> > long esp;
>
> akpm, how about just yank the line? the comment's sanity depends on a
> CONFIG option and also a value in another file from a far off land.

IMHO, I'd say keep the line, but remove the offensive part (everything
past the ':', inclusive).

--
Daniel S. Cox
Electronic Commerce Systems

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/