Jozsef Kadlecsik wrote: > This is a good question. Probably it is better to add a (general) 'flags' > field. But I have no idea for what else we could use it :-)
My opinion: Just add a bitfield. If other needs to add other small fields in the future they can add it next to yours. The compiler will pack them nicely. Hmm.. btw, isn't there plenty of space in the ->status field? From what I can tell only 3 out of 32 bits is defined in this field. Should be possible to shrink it to a char or short, giving room for several more flag fields without increasing the size of a conntrack entry.. Regards Henrik