On 18-02-2009 06:19, Stephen Hemminger wrote:
> Now that we are using mod_timer_noact() for timer updates there's no need to
Hmm... so where exactly we are using this mod_timer_noact() now?
Jarek P.
> hold the global lock during the timer update since the actual timeout update
> is now protected by the timer locking.
>
> Signed-off-by: Martin Josefsson <gandalf@xxxxxxxxxxxxxx>
>
> ---
> net/netfilter/nf_conntrack_core.c | 9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)
>
> --- a/net/netfilter/nf_conntrack_core.c 2009-02-17 10:55:33.370882059 -0800
> +++ b/net/netfilter/nf_conntrack_core.c 2009-02-17 13:48:25.080060712 -0800
> @@ -793,13 +793,12 @@ void __nf_ct_refresh_acct(struct nf_conn
> NF_CT_ASSERT(ct->timeout.data == (unsigned long)ct);
> NF_CT_ASSERT(skb);
>
> - spin_lock_bh(&nf_conntrack_lock);
> -
> /* Only update if this is not a fixed timeout */
> if (test_bit(IPS_FIXED_TIMEOUT_BIT, &ct->status))
> goto acct;
>
> - /* If not in hash table, timer will not be active yet */
> + /* If not in hash table, timer will not be active yet,
> + we are the only one able to see it. */
> if (!nf_ct_is_confirmed(ct)) {
> ct->timeout.expires = extra_jiffies;
> event = IPCT_REFRESH;
> @@ -821,16 +820,16 @@ acct:
> if (do_acct) {
> struct nf_conn_counter *acct;
>
> + spin_lock_bh(&nf_conntrack_lock);
> acct = nf_conn_acct_find(ct);
> if (acct) {
> acct[CTINFO2DIR(ctinfo)].packets++;
> acct[CTINFO2DIR(ctinfo)].bytes +=
> skb->len - skb_network_offset(skb);
> }
> + spin_unlock_bh(&nf_conntrack_lock);
> }
>
> - spin_unlock_bh(&nf_conntrack_lock);
> -
> /* must be unlocked when calling event cache */
> if (event)
> nf_conntrack_event_cache(event, ct);
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
