]> git.ipfire.org Git - thirdparty/iptables.git/commit
nft-cache: Fetch cache per table
authorPhil Sutter <phil@nwl.cc>
Wed, 18 Mar 2020 16:08:31 +0000 (17:08 +0100)
committerPhil Sutter <phil@nwl.cc>
Mon, 11 May 2020 12:28:28 +0000 (14:28 +0200)
commit13b351c9ba01751c693df68f657d19e8c3d267ce
treed3710e31d4ab080bee1d3dc0ee658989401b60dc
parent6d1d5aa5c93eca890e28b508ef426b7844caf2b7
nft-cache: Fetch cache per table

Restore per-table operation of cache routines as initially implemented
in commit e2883c5531e6e ("nft-cache: Support partial cache per table").

As before, this doesn't limit fetching of tables (their number is
supposed to be low) but instead limits fetching of sets, chains and
rules to the specified table.

For this to behave correctly when restoring without flushing over
multiple tables, cache must be freed fully after each commit - otherwise
the previous table's cache level is reused for the current one. The
exception being fake cache, used for flushing restore: NFT_CL_FAKE is
set just once at program startup, so it must stay set otherwise
consecutive tables cause pointless cache fetching.

The sole use-case requiring a multi-table cache, iptables-save, is
indicated by req->table being NULL. Therefore, req->table assignment is
a bit sloppy: All calls to nft_cache_level_set() are assumed to set the
same table value, collision detection exists merely to catch programming
mistakes.

Make nft_fini() call nft_release_cache() instead of flush_chain_cache(),
the former does a full cache deinit including cache_req contents.

Signed-off-by: Phil Sutter <phil@nwl.cc>
iptables/nft-cache.c
iptables/nft-cache.h
iptables/nft-cmd.c
iptables/nft.c
iptables/nft.h
iptables/xtables-restore.c
iptables/xtables-save.c