On Mon, Jan 22, 2024 at 02:47:54PM -0500, Jamal Hadi Salim wrote: > @@ -39,6 +55,27 @@ struct p4tc_template_ops { > struct p4tc_template_common { > char name[P4TC_TMPL_NAMSZ]; > struct p4tc_template_ops *ops; > + u32 p_id; > + u32 PAD0; Perhaps __pad0 is more common. But, is it really needed? > +}; Only nit. Reviewed-by: Marcelo Ricardo Leitner <marcelo.leitner@xxxxxxxxx>