Re: [PATCH] CFR: use refcount(9) in mcast

2009-11-23 Thread Bruce Simpson
After discussion with jhb@, no real need for it. The atomics add instructions which aren't needed as all accesses are covered by a mutex. cheers, BMS ___ freebsd-net@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-net To unsu

Re: [PATCH] CFR: use refcount(9) in mcast

2009-11-19 Thread Bruce Simpson
Bruce Simpson wrote: Look OK? All accesses are covered by a mutex, so the atomic ops aren't really needed -- but it makes for clearer source. Missed in mld6.c. ___ freebsd-net@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freeb

[PATCH] CFR: use refcount(9) in mcast

2009-11-19 Thread Bruce Simpson
Look OK? All accesses are covered by a mutex, so the atomic ops aren't really needed -- but it makes for clearer source. Index: netinet/in_mcast.c === --- netinet/in_mcast.c (revision 199528) +++ netinet/in_mcast.c (working copy)