On Tue, Mar 27, 2007 at 03:32:16PM -0700, Luck, Tony wrote: > > FWIW, slab.c has a "struct array_cache *new[NR_CPUS]" on the > > stack. > > I don't think so. That line is part of the "struct ccupdate_struct" > definition. We only declare a pointer to one of those on the stack. You are correct (of course). I was looking in a tree basd on SLES10. That is also true for the other reference in the network code. > Debugging kernel stack overflow is no fun at all, so you might > want to take the better-safe-than-sorry approach here. Probably true. I'll repost a modified patch (hopefully) tommorrow. Does anyone see any other issues? - To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
|