Page MenuHomeFreeBSD

mountroot: On failure return ENOENT, not ENODEV
AbandonedPublic

Authored by trasz on May 18 2022, 1:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 10, 2:39 PM
Unknown Object (File)
Oct 22 2024, 2:20 PM
Unknown Object (File)
Oct 9 2024, 5:30 PM
Unknown Object (File)
Sep 27 2024, 2:58 PM
Unknown Object (File)
Sep 27 2024, 1:49 AM
Unknown Object (File)
Sep 24 2024, 7:23 PM
Unknown Object (File)
Sep 18 2024, 10:07 AM
Unknown Object (File)
Sep 5 2024, 11:42 PM
Subscribers

Details

Reviewers
kib
Summary

As it is now, if we wait for a device node to appear
and it never happens, we return ENODEV ("failed with error 19").
Use ENOENT instead; ENODEV might be quite misleading.

Sponsored By: EPSRC

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 45619
Build 42507: arc lint + arc unit

Event Timeline

trasz requested review of this revision.May 18 2022, 1:19 PM

I do not object, mostly because I cannot make an opinion. You mentioned that 'ENODEV might be quite misleading'. Can you provide more info, and explain what this change gives us?

My first thought about ENODEV was something about GEOM. ENOENT, on the other hand, would make it obvious what's going on: the root device node is simply not there.

As I said, I do not see a point, but I do not object.

This revision is now accepted and ready to land.May 19 2022, 1:55 PM