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)
Wed, Apr 17, 1:33 AM
Unknown Object (File)
Jan 20 2024, 3:29 AM
Unknown Object (File)
Jan 11 2024, 7:27 PM
Unknown Object (File)
Dec 22 2023, 10:19 PM
Unknown Object (File)
Oct 29 2023, 7:54 PM
Unknown Object (File)
Aug 22 2023, 9:25 PM
Unknown Object (File)
Aug 14 2023, 11:43 PM
Unknown Object (File)
Jul 21 2023, 5:44 PM
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 Skipped
Unit
Tests Skipped

Event Timeline

sys/powerpc/powerpc/sigcode64.S
73

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