mbox series

[nft,v2,0/5] Reduce cache overhead a bit

Message ID 20211202131136.29242-1-phil@nwl.cc
Headers show
Series Reduce cache overhead a bit | expand

Message

Phil Sutter Dec. 2, 2021, 1:11 p.m. UTC 
Second try after a quick review and some testing:

- Tested with stable kernels v4.4.293 and v4.9.291: This series does not
  change any of the shell tests' results. The changes are supposedly
  bug- and feature-compatible.

- Upon error return from kernel, check errno to make sure it is really
  ENOENT which is expected instead of ignoring any error with non-dump
  requests.

Phil Sutter (5):
  cache: Filter tables on kernel side
  cache: Filter rule list on kernel side
  cache: Filter chain list on kernel side
  cache: Filter set list on server side
  cache: Support filtering for a specific flowtable

 include/cache.h                               |   1 +
 include/mnl.h                                 |  14 +-
 include/netlink.h                             |   3 +-
 src/cache.c                                   | 188 ++++++++++--------
 src/mnl.c                                     |  93 +++++++--
 src/netlink.c                                 |  15 +-
 tests/shell/testcases/listing/0020flowtable_0 |  51 ++++-
 7 files changed, 248 insertions(+), 117 deletions(-)

Comments

Pablo Neira Ayuso Dec. 2, 2021, 9:39 p.m. UTC | #1 
On Thu, Dec 02, 2021 at 02:11:31PM +0100, Phil Sutter wrote:
> Second try after a quick review and some testing:
> 
> - Tested with stable kernels v4.4.293 and v4.9.291: This series does not
>   change any of the shell tests' results. The changes are supposedly
>   bug- and feature-compatible.

LGTM, thanks