> On Thu, 2009-07-23 at 14:56 -0400, Sebastien Roy wrote:
 > > On Thu, 2009-07-23 at 11:40 -0700, Peter Memishian wrote:
 > > > > The phyint_t itself is part of a stack which is associated with a
 > > >  > particular zone, and that zone will have the link kstats of interest.
 > > > 
 > > > We store that context in the case of a shared-stack zone today?
 > > 
 > > We store the ip_stack_t in the ill_t.  Doing a netstackid_to_zoneid() on
 > > such a stack will yield the zone that owns the stack.  Shared-stack
 > > zones don't have their own stacks by definition.
 > 
 > Here's the code change:
 > 
 > http://zhadum.east/ws/seb/seb-clearview/webrev/

Don't forget to remove the comment.  Also, I'd prefer `zoneid'
`kstat_zoneid' to make the code format better.

 > Note that I find it strange that we store the ip_stack_t in the ill_t
 > and not in the phyint_t, since one cannot possibly have a phyint_t that
 > has an ill_t in one stack, and another in a different stack.

Yes it's odd.

-- 
meem

Reply via email to