> I think his analysis is alright but he patch is questionable.
Yes. "if (tb) tcp_tw_put(tw)" cannot be right, no doubts.
Seems, it is enough to remove from bind hash _before_ established.
The idea was that bind hash is pure slave of another state, so that
it need not refcounting at all. Note that adding the second increment
does not help: when we verify that leakage (the situation, when
bucket is in bind hash, but has no timer running) is impossible
we immediately arrive to elimination of the refcount.
Raghu, could you check the variant with inverted order of removal?
Do you see holes? From my side... I need to think more. :-)
Alexey
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/