Re: [PATCH] - Optional method to purge the TLB on SN systems

From: Jack Steiner <steiner_at_sgi.com>
Date: 2007-03-28 08:46:43
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@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Wed Mar 28 08:47:24 2007

This archive was generated by hypermail 2.1.8 : 2007-03-28 08:47:35 EST