Page MenuHomeFreeBSD

LinuxKPI: add more linux-specific errno
ClosedPublic

Authored by bz on Mar 23 2021, 4:49 PM.
Tags
None
Referenced Files
F102989109: D29396.id86279.diff
Tue, Nov 19, 1:13 PM
F102981410: D29396.diff
Tue, Nov 19, 11:03 AM
Unknown Object (File)
Wed, Nov 6, 8:51 PM
Unknown Object (File)
Fri, Nov 1, 4:04 PM
Unknown Object (File)
Thu, Oct 24, 11:00 PM
Unknown Object (File)
Oct 17 2024, 9:57 PM
Unknown Object (File)
Oct 17 2024, 9:56 PM
Unknown Object (File)
Oct 17 2024, 9:56 PM
Subscribers

Details

Summary

Add ERFKILL and EBADE found in iwlwifi and brcmfmac wireless drivers.

Obtained-from: bz_iwlwifi
Sponsored-by: The FreeBSD Foundation
MFC-after: 2 weeks
Reviewed-by: ...
Differential Revision:

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bz requested review of this revision.Mar 23 2021, 4:49 PM
sys/compat/linuxkpi/common/include/linux/errno.h
65–66

Do the values match some Linux definitions, or are they arbitrary?

This revision is now accepted and ready to land.Mar 23 2021, 7:40 PM
manu added inline comments.
sys/compat/linuxkpi/common/include/linux/errno.h
65–66

They do not match linux definitions which could cause problems if they are expected to be returned to userland.

sys/compat/linuxkpi/common/include/linux/errno.h
65–66

I'd assume (without looking) that none of the errnos in the 500s is matching Linux. Maybe we should add a comment if my assumption is correct?

If they'd be returned to userland then it'd be a linux binary compat issue only I'd assume as native FreeBSD apps will normally never use those. In my case they currently should not be.

Add a comment above the block of error numbers above 500 to
document expectations as asked by @emaste and clarified by @manu .

This revision now requires review to proceed.Mar 24 2021, 7:12 PM
bz marked 2 inline comments as done.Mar 24 2021, 7:13 PM
This revision is now accepted and ready to land.Mar 24 2021, 7:14 PM
This revision was automatically updated to reflect the committed changes.