Page MenuHomeFreeBSD
Differential D1086

Use the BSD_TO_LINUX_SIGNAL() wherever there is no need to check the ABI as it is known.
ClosedPublic
Actions

Authored by dchagin on Nov 3 2014, 7:03 AM.
Tags
None
Referenced Files
F101319958: D1086.diff
Sun, Oct 27, 6:30 PM
Unknown Object (File)
Wed, Oct 23, 11:34 AM
Unknown Object (File)
Sep 22 2024, 10:56 AM
Unknown Object (File)
Sep 20 2024, 1:14 AM
Unknown Object (File)
Sep 19 2024, 11:47 AM
Unknown Object (File)
Sep 18 2024, 6:17 PM
Unknown Object (File)
Sep 14 2024, 8:29 PM
Unknown Object (File)
Sep 8 2024, 2:34 PM
View All 93 Files
Subscribers
trasz

Details

Reviewers
None
Group Reviewers
Linux Emulation
Commits
rS283436: Use the BSD_TO_LINUX_SIGNAL() wherever there is no need

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

dchagin updated this revision to Diff 2234.Nov 3 2014, 7:03 AM
dchagin retitled this revision from to Use the BSD_TO_LINUX_SIGNAL() wherever there is no need to check the ABI as it is known..
dchagin added a reviewer: Linux Emulation.Nov 3 2014, 9:06 AM
dchagin updated this revision to Diff 2316.Nov 6 2014, 9:26 PM

More BSD_TO_LINUX..

Updating D1086: Use the BSD_TO_LINUX_SIGNAL() wherever there is no need

to check the ABI as it is known.

trasz added a subscriber: trasz.Mar 25 2015, 1:34 PM

Where is BSD_TO_LINUX_SIGNAL() defined?

dchagin added a comment.Mar 25 2015, 7:10 PM
In D1086#5, @trasz wrote:

Where is BSD_TO_LINUX_SIGNAL() defined?

/compat/linux/linux_signal.h

trasz added inline comments.Mar 27 2015, 11:48 AM
sys/amd64/linux32/linux32_sysvec.c
347

Doesn't this macro miss check for sig > 0? If sig == 0, we end up dereferencing bsd_to_linux_signal[-1], right?

dchagin added inline comments.Mar 27 2015, 3:35 PM
sys/amd64/linux32/linux32_sysvec.c
347

in this context sig is always valid

Closed by commit rS283436: Use the BSD_TO_LINUX_SIGNAL() wherever there is no need (authored by dchagin). · Explain WhyMay 24 2015, 4:30 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
sys/
amd64/
linux32/
8 lines
compat/
linux/
11 lines
i386/
linux/
8 lines

Diff 2316

View Options

sys/amd64/linux32/linux32_sysvec.c

Loading...
View Options

sys/compat/linux/linux_signal.c

Loading...
View Options

sys/i386/linux/linux_sysvec.c

Loading...