On Thu, Aug 15, 2024 at 01:37:07PM +0200, Pablo Neira Ayuso wrote:
> Hi,
>
> The following patchset relaxes cache requirements, this is based on the
> observation that objects are fetched to report errors and provide hints.
>
> This is a new attempt to speed up incremental updates following a
> different approach, after reverting:
>
> e791dbe109b6 ("cache: recycle existing cache with incremental updates")
>
> which is fragile because cache consistency checking needs more, it should
> be still possible to explore in the future, but this seems a more simple
> approach at this stage.
>
> This is passing tests/shell and tests/py.
>
> Pablo Neira Ayuso (5):
> cache: rule by index requires full cache
> cache: populate chains on demand from error path
> cache: populate objecs on demand from error path
> cache: populate flowtable on demand from error path
> cache: do not fetch set inconditionally on delete
>
> include/cache.h | 1 -
> src/cache.c | 23 ++++++-----------------
> src/cmd.c | 23 +++++++++++++++++++++++
> 3 files changed, 29 insertions(+), 18 deletions(-)
I applied this series to nft master and tested it against the latest
net-next and RHEL-9 kernels. No issues or regressions found.
Thanks Pablo!
Tested-by: Eric Garver <eric@xxxxxxxxxxx>
