> What errno values do you suggest? For me it's 'checksum is correct:
> yes/no', hence return 1/0.

Oh, I have no strong preerence here.

> I see an -EIO below, but that does not seem right here. There's a call
> to btrfs_read_dev_super that would indicate an unreadable block. We
> could use EFSCORRUPTED as xfs does (with value of EUCLEAN = 117) though.
> 
> As implemented now, EINVAL will make 'mount' print the instructive
> message "look into the syslog", I'm not sure if we wouldn't need to
> update userspace to reflect the fine grained error codes.

Yeah, EINVAL seems reasonable.  mount(2):

       EINVAL source had an invalid superblock.

Documented (ish) and everything!

- z
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to