Brett Mastbergen <bmastbergen@xxxxxxxxxxxx> wrote: > +const struct datatype ct_id_type = { > + .type = TYPE_CT_ID, > + .name = "ct_id", > + .desc = "conntrack id", > + .byteorder = BYTEORDER_BIG_ENDIAN, > + .size = 4 * BITS_PER_BYTE, > + .basetype = &integer_type, > +}; Pablo, I know we've discussed this in the past but I think the ct id would be a good candidate to add a generic uint32_t integer type. AFAICS the ct_id_type is unneeded per this patch, but IIRC to make use of this for set definitions we can't use generic integer type as it has no fixed size. This is the first data type that doesn't need any specific parsing/printing, AFAICS its just needed to get a fixed 32bit size.