> From: Tian, Kevin > Sent: Friday, May 19, 2023 4:42 PM > > +struct iommu_hw_info { > > + __u32 size; > > + __u32 flags; > > + __u32 dev_id; > > + __u32 data_len; > > + __aligned_u64 data_ptr; > > + __u32 out_data_type; > > + __u32 __reserved; > > it's unusual to have reserved field in the end. It makes more sense > to move data_ptr to the end to make it meaningful. > Please ignore this comment. typed too fast...