> +int fiemap_fill_usr_extent(struct fiemap_ctx *f_ctx, u64 logical, Please spell out user. > +int fiemap_fill_next_extent(struct fiemap_ctx *f_ctx, u64 logical, > + u64 phys, u64 len, u32 flags) > +{ > + return f_ctx->fc_cb(f_ctx, logical, phys, len, flags); > +} > +EXPORT_SYMBOL(fiemap_fill_next_extent); Do we really need this wrapper? Which reminds me that we usually pass the callbacks as direct function arguments. Any good reason it is in the context here?