> struct fiemap_ctx { > - unsigned int fc_flags; > + unsigned int fc_flags; /* Flags as passed from user */ If you really need a comment here maybe it should be: unsigned int fc_flags; /* FIEMAP_FLAG_* */ Otherwise this generally looks fine to me.
> struct fiemap_ctx { > - unsigned int fc_flags; > + unsigned int fc_flags; /* Flags as passed from user */ If you really need a comment here maybe it should be: unsigned int fc_flags; /* FIEMAP_FLAG_* */ Otherwise this generally looks fine to me.