Page MenuHomeFreeBSD

[PowerPC] Save a dword in the powerpc64 signal trampoline
ClosedPublic

Authored by bdragon on Jan 6 2020, 6:26 PM.
Referenced Files
Unknown Object (File)
Mon, Dec 23, 6:27 PM
Unknown Object (File)
Sat, Dec 21, 1:54 PM
Unknown Object (File)
Wed, Dec 18, 12:27 PM
Unknown Object (File)
Dec 5 2024, 8:57 PM
Unknown Object (File)
Nov 27 2024, 1:46 AM
Unknown Object (File)
Nov 23 2024, 9:51 AM
Unknown Object (File)
Nov 21 2024, 10:20 PM
Unknown Object (File)
Nov 20 2024, 12:28 AM
Subscribers

Details

Summary

In rS291668, an instruction was added to sigcode64.S without the nop pad at the end being taken out.

Due to alignment, this means that a dword is being wasted on the shared page for no reason.

Take out this nop, and add some comments while I'm here.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

sys/powerpc/powerpc/sigcode64.S
70 ↗(On Diff #66417)

Actually this comment is incorrect now that I remember what I was thinking at the time regarding it being exec_copyout_strings() that cares.

When I was tracing through this, I was confused about how it worked. Apparently the code in exec_copyout_strings() is only used when a unified shared page is not used.

The rest of the comment should be accurate though.

OK, so my comments are incorrect in general. The nop still needs to go though.

Remove incorrect comments from original patch, add comment to describe why we're exiting instead.

This revision is now accepted and ready to land.Jan 14 2020, 9:28 PM