[brussels-dev] [networking-discuss] ::nce mdb walker
James Carlson
james.d.carlson at Sun.COM
Tue Jan 29 09:41:50 PST 2008
Sowmini.Varadhan at Sun.COM writes:
> Meanwhile, here's the updated webrev, in case you (or anyone
> else!) want to take another look:
>
> http://cr.opensolaris.org/~sowmini/nce_walker/
The new code looks pretty good. There are a couple of very minor nits
that I'd pick (e.g., modules/ip/ip.c:1408,1419: why is the return
inside the braces for one clause but not the other?), but I'm happy
enough with it.
--
James Carlson, Solaris Networking <james.d.carlson at sun.com>
Sun Microsystems / 35 Network Drive 71.232W Vox +1 781 442 2084
MS UBUR02-212 / Burlington MA 01803-2757 42.496N Fax +1 781 442 1677
More information about the brussels-dev
mailing list