[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20100628.233817.68140182.davem@davemloft.net>
Date: Mon, 28 Jun 2010 23:38:17 -0700 (PDT)
From: David Miller <davem@...emloft.net>
To: kaber@...sh.net
Cc: fw@...len.de, jpirko@...hat.com, netdev@...r.kernel.org,
bazsi@...abit.hu, hidden@....bme.hu
Subject: Re: [PATCH net-next-2.6] netfilter: allow nf_tproxy_core module to
be removed
From: Patrick McHardy <kaber@...sh.net>
Date: Thu, 24 Jun 2010 17:29:29 +0200
> David Miller wrote:
>> From: Florian Westphal <fw@...len.de>
>> Date: Wed, 23 Jun 2010 20:46:11 +0200
>>
>>
>>> tproxy assigns skb->destructor, what prevents module unload while such
>>> skbs may
>>> still be around?
>>>
>>
>> The only reference to nf_tproxy_core.ko is for the symbol,
>> "nf_tproxy_assign_sock".
>> xt_TPROXY.c, which references this symbol, thus creates a symbol
>> dependency on this
>> module, so xt_TPROXY.o needs to unload before nf_tproxy_core.ko can
>> unload, and
>> xt_TPROXY.o has it's own manner for handling module references
>> properly.
>>
>
> I don't see anything waiting for skbs in flight using the tproxy
> destructor in either xt_TPROXY or nf_tproxy_core though, so I think
> Florian is correct.
Ok.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists