Pablo Neira Ayuso <pablo@xxxxxxxxxxxxx> wrote: > This patch adds a 64-bit conntrack ID extension that allows userspace to > uniquely identify a conntrack object. > > The existing 32-bit ID is not good to uniquely identify a conntrack > object. Long time ago, this used to be an incremental number that could > quickly wrap around. Someone suggested to use 64-bits, back then this > was considered to be too much memory for just an ID. So we usually > suggested to users that they should combine it with the conntrack tuple > to achieve a way to uniquely conntrack objects. This has always > generated a bit of controversy since userspace applications needed to > deal with extra work. > > At some point, someone remove the explicit ct->id field that we used to > have to save memory space. This ID was modified to part of its memory > address. Howeover, this is a problem because objects can be quickly > recycled with the slab-by-rcu approach that we use these days. So even > combining this 32-bit ID with the tuple doesn't ensure that this is > unique. Moreover, this is leaking the pointer to userspace in 32-bit > arches, which is not good. > > So let's introduce a 64-bit unique ID that ensures no overlaps. This is > only allocated once in the first packet, and never ever again from the > hot path, so let's keep this in a separated extension not to grab more > cachelines. > > ID assignment is lockless: this patch divides the 64-bit space between Why do we need an id in the first place? Can you elaborate? As far as I know there is no need for this ID at all, the netlink attribute is only provided for backwards compat. And for that the suggested approach (hash of tuple and memory addresses) will work just fine. -- 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
