Differential D29396

LinuxKPI: add more linux-specific errno
ClosedPublic
Actions

Authored by bz on Mar 23 2021, 4:49 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, May 3, 4:33 AM

	Unknown Object (File)
	Thu, May 2, 10:17 PM

	Unknown Object (File)
	Thu, May 2, 10:16 PM

	Unknown Object (File)
	Thu, May 2, 10:03 PM

	Unknown Object (File)
	Thu, May 2, 8:52 PM

	Unknown Object (File)
	Apr 2 2024, 10:43 PM

	Unknown Object (File)
	Feb 20 2024, 4:15 PM

	Unknown Object (File)
	Jan 5 2024, 5:42 PM

View All 28 Files

Subscribers

Details

Reviewers

• hselasky
emaste

Group Reviewers

Commits

rG36d6711bd68f: LinuxKPI: add more linux-specific errno
rG4b0632cfc5c8: LinuxKPI: add more linux-specific errno

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 created this revision.Mar 23 2021, 4:49 PM

Herald added subscribers: emaste, imp. · View Herald TranscriptMar 23 2021, 4:49 PM

bz requested review of this revision.Mar 23 2021, 4:49 PM

Harbormaster completed remote builds in B38021: Diff 86207.Mar 23 2021, 4:49 PM

emaste added inline comments.Mar 23 2021, 7:06 PM

sys/compat/linuxkpi/common/include/linux/errno.h
65–66	Do the values match some Linux definitions, or are they arbitrary?

• hselasky accepted this revision.Mar 23 2021, 7:40 PM

This revision is now accepted and ready to land.Mar 23 2021, 7:40 PM

manu added a subscriber: manu.Mar 24 2021, 11:08 AM

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.

bz added inline comments.Mar 24 2021, 12:21 PM

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

Harbormaster completed remote builds in B38056: Diff 86268.Mar 24 2021, 7:13 PM

bz marked 2 inline comments as done.Mar 24 2021, 7:13 PM

emaste accepted this revision.Mar 24 2021, 7:14 PM

This revision is now accepted and ready to land.Mar 24 2021, 7:14 PM

Closed by commit rG4b0632cfc5c8: LinuxKPI: add more linux-specific errno (authored by bz). · Explain WhyMar 24 2021, 10:42 PM

This revision was automatically updated to reflect the committed changes.

bz added a commit: rG4b0632cfc5c8: LinuxKPI: add more linux-specific errno.

bz added a commit: rG36d6711bd68f: LinuxKPI: add more linux-specific errno.Jul 18 2021, 12:36 AM

Revision Contents
Changeset List

Path

Size

sys/

compat/

linuxkpi/

common/

include/

linux/

6 lines

Diff 86279

sys/compat/linuxkpi/common/include/linux/errno.h

Loading...