> > I'd rather do the fanotity_fh padding optimization I outlined in another > > email. That would save one long without any packing and the following u8 > > name_len would get packed tightly after the fanotify_fh by the compiler. > > > > OK. I will try that and the non-inherited variant of perm/name event struct > and see how it looks like. > Pushed sample code to branch fanotify_name-wip: b5e56d3e1358 fanotify: fanotify_perm_event inherits from fanotify_path_event 55041285b3b7 fanotify: divorce fanotify_path_event and fanotify_fid_event I opted for fanotify_name_event inherits from fanotify_fid_event, because it felt better this way. I wasn't sure about fanotify_perm_event inherits from fanotify_path_event, so did that is a separate patch so you can judge both variants. IMO, neither variant is that good or bad, so I could go with either. I do like the end result with your suggestions better than fanotify_name-v2. If you like this version, I will work the changes into the series. Thanks, Amir.