``` + TRACE("iface %p, refcount %lu.\n", iface, refcount); ```
The usual message here is "%p increasing/decreasing refcount to %lu." Otherwise it's ambiguous whether that's the previous or current refcount.
I think the rest of it looks fine though.