[email protected] wrote:
> a bit later than promised,  but a webrev with fixes is available
> at
>     http:://cr.opensolaris.org/~sowmini/jim_review

ip_arp.c:

  nit: 849s/mulitcasts/multicasts/

(I'm guessing a "mulitcast" might be used if Jeff Foxworthy fell off of
the stage.  ;-})

The rest of the changes look good.

> There was one other comment was about the possible leakage of
> detach_mp. Meem subsequently pointed out that detach is not needed at
> all for any case, so I've removed it as part of this round of changes.

OK; sounds good.

-- 
James Carlson         42.703N 71.076W         <[email protected]>
_______________________________________________
networking-discuss mailing list
[email protected]

Reply via email to