On Mon, Feb 12, 2024 at 11:06:04AM +0200, Mika Westerberg wrote: > @@ -22,6 +24,7 @@ > /** > * struct tb_ctl - Thunderbolt control channel > * @nhi: Pointer to the NHI structure > + * @index: Domain number. This will be output with the trace record. > * @tx: Transmit ring > * @rx: Receive ring > * @frame_pool: DMA pool for control messages > @@ -35,6 +38,7 @@ > */ > struct tb_ctl { > struct tb_nhi *nhi; > + int index; > struct tb_ring *tx; > struct tb_ring *rx; Did you mean to put a "hole" in the structure like this? If not, please move it around a bit. thanks, greg k-h