On Tue, Jul 02, 2024 at 07:09:20AM +0200, Christoph Hellwig wrote: > > +extern void jdm_new_filehandle_v5(jdm_filehandle_t **handlep, size_t *hlen, > > + jdm_fshandle_t *fshandlep, struct xfs_bulkstat *sp); > > No need for the extern here. Same for the others below. Ok, will remove. > Otherwise looks good: > > Reviewed-by: Christoph Hellwig <hch@xxxxxx> Thanks! --D