On Fri, 2016-08-19 at 13:36 +0200, Florian Westphal wrote:
> Conntrack gc worker to evict stale entries.

...

> +
> +             hlist_nulls_for_each_entry_rcu(h, n, &ct_hash[i], hnnode) {
> +                     tmp = nf_ct_tuplehash_to_ctrack(h);
> +
> +                     if (nf_ct_is_expired(tmp)) {
> +                             nf_ct_gc_expired(tmp);
> +                             expired_count++;
> +                             continue;

Same remark about hlist_nulls_for_each_entry_rcu() not 'safe' here

> +                     }
> +             }
> +
> +             rcu_read_unlock();
> +             cond_resched();

This could use cond_resched_rcu_qs() 

> +     } while (++buckets < goal &&
> +              expired_count < GC_MAX_EVICTS);
> +
> +     if (gc_work->exiting)
> +             return;
> +

--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to