]> git.baikalelectronics.ru Git - kernel.git/commit
netfilter: x_tables: Use correct memory barriers.
authorMark Tomlinson <mark.tomlinson@alliedtelesis.co.nz>
Mon, 8 Mar 2021 01:24:13 +0000 (14:24 +1300)
committerPablo Neira Ayuso <pablo@netfilter.org>
Mon, 15 Mar 2021 17:42:01 +0000 (18:42 +0100)
commit6d3c1c94eb45d801e3016fcf55dbc2b6d367021f
tree270384002ea1744d6e39a525d76c274a4feede61
parenta8192b82d396cadee7b0bd1d28c72e76b9490733
netfilter: x_tables: Use correct memory barriers.

When a new table value was assigned, it was followed by a write memory
barrier. This ensured that all writes before this point would complete
before any writes after this point. However, to determine whether the
rules are unused, the sequence counter is read. To ensure that all
writes have been done before these reads, a full memory barrier is
needed, not just a write memory barrier. The same argument applies when
incrementing the counter, before the rules are read.

Changing to using smp_mb() instead of smp_wmb() fixes the kernel panic
reported in d59e3d10acbd (which is still present), while still
maintaining the same speed of replacing tables.

The smb_mb() barriers potentially slow the packet path, however testing
has shown no measurable change in performance on a 4-core MIPS64
platform.

Fixes: a52d52e33bbd ("netfilter: get rid of atomic ops in fast path")
Signed-off-by: Mark Tomlinson <mark.tomlinson@alliedtelesis.co.nz>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
include/linux/netfilter/x_tables.h
net/netfilter/x_tables.c