[systemd-devel] [PATCH] libudev: replace name_to_handle_at with normal sscanf

Lennart Poettering lennart at poettering.net
Wed Apr 23 22:35:15 PDT 2014


On Thu, 24.04.14 07:28, Zbigniew Jędrzejewski-Szmek (zbyszek at in.waw.pl) wrote:
> > Have you checked that EOPNOTSUPP is really the error that is returned by
> > name_to_handle_at() if the kernel has the entire syscall disabled? Note
> > that there are two different cases to distuingish here: a file system
> > not supporting the operation, and the kernel not supporting the
> > syscal...
> It's warning on everything except EOPNOTSUPP. My reasoning is:
> 
> EOPNOTSUPP -> syscall supported, but wrong fs type, -> return false
> ENOSYS -> syscall not supported -> warn and return false
> any other error -> memory error or other major screw-up -> warn and return false

Looks good to me then, Kay? 

Though I still wonder about the difference between ENOTSUP and
EOPNOTSUPP. path_is_mount_point() expectes the former, your patch the
latter?

Lennart

-- 
Lennart Poettering, Red Hat


More information about the systemd-devel mailing list