On Sun, Dec 15, 2024 at 02:47:15PM +0530, Ojaswin Mujoo wrote: > - return (statfsbuf.f_type == 0x58465342); /* XFSB */ > + return (statfsbuf.f_type == XFS_SUPER_MAGIC); Might be worth dropping the superfluous braces here while you're at it. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>