Re: [nft PATCH] Enable automerge feature for anonymous sets

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Phil,

On Tue, Feb 06, 2018 at 07:18:47PM +0100, Phil Sutter wrote:
> Automatic merging of adjacent/overlapping ranges upon insertion has
> clear benefits performance- and readability-wise. The drawbacks which
> led to disabling it by default don't apply to anonymous sets since they
> are read-only anyway, so enable this feature for them again.

Question is, why someone would be adding elements with overlapping
ranges in an anonymous set? Then, when listing the ruleset you will
get something different to what you've added. This would also be
inconsistent with regards to the existing behaviour in named sets,
where this is turned off by default.

For named sets, that are useful to maintain white/blacklists, I
understand this simplifies complexity for people dealing with them.
But not sure for anonymous sets.

@Jeff: Is this also useful to you in the anonymous set use-case? IIRC
we agreed that this was good for named sets, but not for anonymous
sets.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Netfitler Users]     [LARTC]     [Bugtraq]     [Yosemite Forum]

  Powered by Linux