> +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. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>