Page MenuHomeFreeBSD
Differential D50475

rtld: get syscall implementations from libsys
ClosedPublic
Actions

Authored by brooks on Thu, May 22, 7:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jun 5, 10:30 AM
Unknown Object (File)
Wed, Jun 4, 10:34 PM
Unknown Object (File)
Sun, Jun 1, 9:23 PM
Unknown Object (File)
Wed, May 28, 4:41 PM
Unknown Object (File)
Wed, May 28, 9:50 AM
Unknown Object (File)
Sun, May 25, 6:15 PM
Unknown Object (File)
Sat, May 24, 10:39 AM
Unknown Object (File)
Sat, May 24, 9:41 AM
View All 12 Files
Subscribers
bdragon
imp

Details

Reviewers
dg612_cam.ac.uk
jrtc27
jhb
dim
Commits
rG387783da979e: rtld: get syscall implementations from libsys
Summary

When I added libsys I failed to switch rtld's reuse of object files
from libc to to use ones from libsys instead. This would have turned up
as a broken system in d7847a8d35143, but SHARED_CFLAGS is not being
applied to assembly files.

PR: 286975

Diff Detail

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

Event Timeline

brooks created this revision.Thu, May 22, 7:04 PM
Herald added subscribers: bdragon, imp. · View Herald TranscriptThu, May 22, 7:04 PM
brooks requested review of this revision.Thu, May 22, 7:04 PM
Harbormaster completed remote builds in B64363: Diff 155890.Thu, May 22, 7:04 PM
brooks added a parent revision: D50474: src.libnames.mk: add LIBSYS_PIC.Thu, May 22, 7:04 PM
brooks mentioned this in D50465: libsys: Use PICO_CFLAGS to disable syscall stubs.Thu, May 22, 7:05 PM
brooks added a child revision: D50466: share/mk: pass SHARED_CFLAGS to PIC and PIE from assembly.
jrtc27 added inline comments.Thu, May 22, 7:09 PM
libexec/rtld-elf/rtld-libc/Makefile.inc
95

Duplicated by the later line

99
101

This is never empty? (And were it to be, rtld_libc.a probably shouldn't depend on LIBC_NOSSP_PIC unconditionally)

brooks updated this revision to Diff 155892.Thu, May 22, 7:25 PM

Address feedback from @jrtc27

Harbormaster completed remote builds in B64365: Diff 155892.Thu, May 22, 7:25 PM
brooks marked 3 inline comments as done.Thu, May 22, 7:26 PM
brooks added inline comments.
libexec/rtld-elf/rtld-libc/Makefile.inc
101

Oops, that's a leftover from my initial (broken) implementation where some archs didn't have any libc entries.

jrtc27 accepted this revision.Thu, May 22, 9:22 PM
This revision is now accepted and ready to land.Thu, May 22, 9:22 PM
brooks mentioned this in D50464: share/mk: add {PICO,PIEO,NOSSPPICO}_CFLAGS for C and assembly.Thu, May 22, 10:32 PM
jhb accepted this revision.Fri, May 23, 5:37 PM
Closed by commit rG387783da979e: rtld: get syscall implementations from libsys (authored by brooks). · Explain WhyFri, May 23, 11:48 PM
This revision was automatically updated to reflect the committed changes.
brooks marked an inline comment as done.
brooks added a commit: rG387783da979e: rtld: get syscall implementations from libsys.

Revision Contents
Changeset List

PathSize
libexec/
rtld-elf/
rtld-libc/
30 lines

Diff 155957

View Options

libexec/rtld-elf/rtld-libc/Makefile.inc

Loading...