net: keep refcount warning in reqsk_free()
authorGuillaume Nault <gnault@redhat.com>
Sat, 9 Mar 2019 09:26:53 +0000 (10:26 +0100)
committerDavid S. Miller <davem@davemloft.net>
Sun, 10 Mar 2019 03:51:44 +0000 (19:51 -0800)
As Eric Dumazet said, "We do not have a way to tell if the req was ever
inserted in a hash table, so better play safe.".
Let's remove this comment, so that nobody will be tempted to drop the
WARN_ON_ONCE() line.

Signed-off-by: Guillaume Nault <gnault@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/request_sock.h

index 347015515a7de68a09d935e2b4a1baad3479cc03..21a5243fecd160825deb50fbe8e5bdfc1d4d39dc 100644 (file)
@@ -108,7 +108,6 @@ reqsk_alloc(const struct request_sock_ops *ops, struct sock *sk_listener,
 
 static inline void reqsk_free(struct request_sock *req)
 {
-       /* temporary debugging */
        WARN_ON_ONCE(refcount_read(&req->rsk_refcnt) != 0);
 
        req->rsk_ops->destructor(req);