Page MenuHomeFreeBSD
Differential D54398

Make .align match comment in code
Needs ReviewPublic
Actions

Authored by cperciva on Mon, Dec 29, 5:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 16, 11:51 AM
Unknown Object (File)
Fri, Jan 9, 10:12 PM
Unknown Object (File)
Thu, Jan 8, 10:08 AM
Unknown Object (File)
Wed, Jan 7, 7:10 PM
Unknown Object (File)
Wed, Jan 7, 4:45 PM
Unknown Object (File)
Wed, Jan 7, 2:20 AM
Unknown Object (File)
Mon, Jan 5, 1:19 PM
Unknown Object (File)
Sat, Jan 3, 8:28 PM
View All 12 Files
Subscribers
emaste
imp

Details

Reviewers
andrew
manu
Summary

I have no idea whether the comment is wrong or the .align is wrong. Also, no idea if this can cause any problems; it just caught my eye as I was looking at the code.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

cperciva created this revision.Mon, Dec 29, 5:12 PM
Herald added a reviewer: manu. · View Herald TranscriptMon, Dec 29, 5:12 PM
Herald added subscribers: emaste, imp. · View Herald Transcript
cperciva requested review of this revision.Mon, Dec 29, 5:12 PM
andrew added inline comments.Mon, Dec 29, 5:15 PM
sys/arm64/arm64/sigtramp.S
51–52

This and the comment should be removed, this isn't copied to the stack as we use the shared page, and the stack is non-executable.

Revision Contents
Changeset List

PathSize
sys/
arm64/
arm64/
2 lines

Diff 168713

View Options

sys/arm64/arm64/sigtramp.S

Loading...