Commit b901892b authored by Ahmed S. Darwish's avatar Ahmed S. Darwish Committed by Peter Zijlstra

netfilter: nft_set_rbtree: Use sequence counter with associated rwlock

A sequence counter write side critical section must be protected by some
form of locking to serialize writers. A plain seqcount_t does not
contain the information of which lock must be held when entering a write
side critical section.

Use the new seqcount_rwlock_t data type, which allows to associate a
rwlock with the sequence counter. This enables lockdep to verify that
the rwlock used for writer serialization is held when the write side
critical section is entered.

If lockdep is disabled this lock association is compiled out and has
neither storage size nor runtime overhead.
Signed-off-by: default avatarAhmed S. Darwish <a.darwish@linutronix.de>
Signed-off-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20200720155530.1173732-16-a.darwish@linutronix.de
parent 8201d923
...@@ -18,7 +18,7 @@ ...@@ -18,7 +18,7 @@
struct nft_rbtree { struct nft_rbtree {
struct rb_root root; struct rb_root root;
rwlock_t lock; rwlock_t lock;
seqcount_t count; seqcount_rwlock_t count;
struct delayed_work gc_work; struct delayed_work gc_work;
}; };
...@@ -523,7 +523,7 @@ static int nft_rbtree_init(const struct nft_set *set, ...@@ -523,7 +523,7 @@ static int nft_rbtree_init(const struct nft_set *set,
struct nft_rbtree *priv = nft_set_priv(set); struct nft_rbtree *priv = nft_set_priv(set);
rwlock_init(&priv->lock); rwlock_init(&priv->lock);
seqcount_init(&priv->count); seqcount_rwlock_init(&priv->count, &priv->lock);
priv->root = RB_ROOT; priv->root = RB_ROOT;
INIT_DEFERRABLE_WORK(&priv->gc_work, nft_rbtree_gc); INIT_DEFERRABLE_WORK(&priv->gc_work, nft_rbtree_gc);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment