Page MenuHomeFreeBSD
Differential D20897

Use 'retval' label for first error in syscallenter().
ClosedPublic
Actions

Authored by jhb on Jul 9 2019, 11:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 17, 8:57 AM
Unknown Object (File)
Fri, May 17, 8:57 AM
Unknown Object (File)
Fri, May 17, 8:57 AM
Unknown Object (File)
Fri, May 17, 8:45 AM
Unknown Object (File)
Mon, May 13, 8:36 PM
Unknown Object (File)
Thu, Apr 25, 12:56 PM
Unknown Object (File)
Wed, Apr 24, 10:18 PM
Unknown Object (File)
Apr 19 2024, 3:41 PM
View All 28 Files
Subscribers
imp

Details

Reviewers
kib
Commits
rS349871: Use 'retval' label for first error in syscallenter().
Summary

This is more consistent with the rest of the function and lets us
unindent most of the function.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 25271
Build 23938: arc lint + arc unit

Event Timeline

jhb created this revision.Jul 9 2019, 11:02 PM
Harbormaster completed remote builds in B25271: Diff 59583.Jul 9 2019, 11:02 PM
jhb added a child revision: D20898: Always set td_errno to the error value of a system call..Jul 9 2019, 11:03 PM
kib accepted this revision.Jul 9 2019, 11:15 PM
kib added inline comments.
sys/kern/subr_syscall.c
92

I would use the opportunity and style the flags checks, (p_flag & P_TRACED) != 0 etc.

This revision is now accepted and ready to land.Jul 9 2019, 11:15 PM
jhb marked an inline comment as done.Jul 9 2019, 11:50 PM
Closed by commit rS349871: Use 'retval' label for first error in syscallenter(). (authored by jhb). · Explain WhyJul 9 2019, 11:58 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rS349871: Use 'retval' label for first error in syscallenter()..
Herald added a subscriber: imp. · View Herald TranscriptJul 9 2019, 11:58 PM

Revision Contents
Changeset List

PathSize
sys/
kern/
97 lines

Diff 59583

View Options

sys/kern/subr_syscall.c

Loading...