Messages in this thread | | | Subject | Re: Trivial Comment Patch: 2.6.6-mm3 | From | Danny Cox <> | Date | Wed, 19 May 2004 07:18:14 -0400 |
| |
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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |