> +#define min_t(type, x, y) ( ((type)(x)) > ((type)(y)) ? ((type)(y)) : ((type)(x)) ) Should this really be a hidden in a .c file in repair vs in a common header where we can use it everywhere? Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>