Page MenuHomeFreeBSD
Differential D45214

kernel: Add defination of .init_array and .fini_array for all other platforms
ClosedPublic
Actions

Authored by zlei on May 16 2024, 7:54 AM.
Tags
None
Referenced Files
F174469869: D45214.id142689.diff
Sat, Oct 3, 12:21 PM
Unknown Object (File)
Fri, Oct 2, 9:25 PM
Unknown Object (File)
Wed, Sep 23, 3:02 AM
Unknown Object (File)
Tue, Sep 22, 4:58 AM
Unknown Object (File)
Thu, Sep 10, 3:22 AM
Unknown Object (File)
Wed, Sep 9, 12:37 PM
Unknown Object (File)
Fri, Sep 4, 9:10 PM
Unknown Object (File)
Fri, Sep 4, 9:10 PM
View All Files
Subscribers
emaste
imp
mjg

Details

Reviewers
andrew
Commits
rGbb5f70161454: kernel: Add defination of .init_array and .fini_array for all other platforms
rG3e76d05231b0: kernel: Add defination of .init_array and .fini_array for all other platforms
Summary

In memoriam: hselasky
Obtained from: https://reviews.freebsd.org/D40467
MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

zlei created this revision.May 16 2024, 7:54 AM
Herald added a reviewer: andrew. · View Herald TranscriptMay 16 2024, 7:54 AM
Herald added a reviewer: andrew. · View Herald Transcript
Herald added subscribers: emaste, imp. · View Herald Transcript
zlei requested review of this revision.May 16 2024, 7:54 AM
zlei mentioned this in D40467: ld(1): Correctly define .init_array and .fini_array for all platforms..May 16 2024, 7:57 AM
zlei added a comment.May 16 2024, 7:59 AM

This is split from D40467 with fixing the order of .fini_array section.

mjg added a subscriber: mjg.EditedMay 16 2024, 8:35 AM

This needs to retain proper authorship, so the Author field still needs to be hps. Add the end of the commit message you should mention this was extracted by you from a bigger patch of his. So add that, drop "in memoriam".

I have no comments on the patch itself.

zlei added a comment.Sep 2 2024, 4:30 AM

@imp

In D45214#1031437, @mjg wrote:

This needs to retain proper authorship, so the Author field still needs to be hps. Add the end of the commit message you should mention this was extracted by you from a bigger patch of his. So add that, drop "in memoriam".

I have no comments on the patch itself.

Thanks for the hint! Landed.

This revision was not accepted when it landed; it landed in state Needs Review.Sep 2 2024, 4:30 AM
Closed by commit rG3e76d05231b0: kernel: Add defination of .init_array and .fini_array for all other platforms (authored by zlei). · Explain Why
This revision was automatically updated to reflect the committed changes.
zlei added a commit: rG3e76d05231b0: kernel: Add defination of .init_array and .fini_array for all other platforms.
zlei added a commit: rGbb5f70161454: kernel: Add defination of .init_array and .fini_array for all other platforms.Sep 22 2024, 2:49 PM

Revision Contents
Changeset List

Diff 142689

View Options

sys/conf/ldscript.arm

Loading...
View Options

sys/conf/ldscript.arm64

Loading...
View Options

sys/conf/ldscript.powerpc

Loading...
View Options

sys/conf/ldscript.powerpc64

Loading...
View Options

sys/conf/ldscript.powerpc64le

Loading...
View Options

sys/conf/ldscript.powerpcspe

Loading...
View Options

sys/conf/ldscript.riscv

Loading...