Page MenuHomeFreeBSD
Differential D31017

loader: Don't reserve space for symbols twice.
ClosedPublic
Actions

Authored by imp on Jul 4 2021, 4:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Sep 5, 12:15 PM
Unknown Object (File)
Sat, Sep 5, 4:06 AM
Unknown Object (File)
Sat, Sep 5, 2:35 AM
Unknown Object (File)
Fri, Sep 4, 10:36 PM
Unknown Object (File)
Fri, Sep 4, 9:50 PM
Unknown Object (File)
Thu, Sep 3, 11:25 AM
Unknown Object (File)
Thu, Sep 3, 10:47 AM
Unknown Object (File)
Thu, Sep 3, 5:10 AM
View All Files
Subscribers
dab

Details

Reviewers
jhb
neel
Commits
rG1e1d3725569f: loader: Don't reserve space for symbols twice.
rG297e9f364b5a: loader: Don't reserve space for symbols twice.
Summary

The current code bumps lastaddr twice for the symbol table
location. However, the first bump is bogus and results in wasted
space. Remove it.

PR: 110995

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 40260
Build 37149: arc lint + arc unit

Event Timeline

imp created this revision.Jul 4 2021, 4:11 PM
Herald added a subscriber: dab. · View Herald TranscriptJul 4 2021, 4:11 PM
imp requested review of this revision.Jul 4 2021, 4:11 PM
Harbormaster completed remote builds in B40260: Diff 91719.Jul 4 2021, 4:11 PM
imp edited the summary of this revision. (Show Details)Jul 4 2021, 4:13 PM
imp added a reviewer: jhb.
imp added inline comments.Jul 4 2021, 4:34 PM
stand/common/load_elf_obj.c
295–297

I think we already do the stuff above here, so we can just do these lines and omit them above.

imp retitled this revision from don't load symbol table into memory to loader: Don't reserve space for symbols twice..Jul 4 2021, 4:40 PM
imp edited the summary of this revision. (Show Details)
imp added a reviewer: neel.
imp updated this revision to Diff 91721.Jul 4 2021, 5:10 PM
imp edited the summary of this revision. (Show Details)

actually remove... no need to comment these redunant lines out.

Harbormaster completed remote builds in B40262: Diff 91721.Jul 4 2021, 5:10 PM
This revision was not accepted when it landed; it landed in state Needs Review.Jul 12 2021, 9:31 PM
Closed by commit rG297e9f364b5a: loader: Don't reserve space for symbols twice. (authored by imp). · Explain Why
This revision was automatically updated to reflect the committed changes.
imp added a commit: rG297e9f364b5a: loader: Don't reserve space for symbols twice..
imp added a commit: rG1e1d3725569f: loader: Don't reserve space for symbols twice..Jul 16 2021, 6:33 PM

Revision Contents
Changeset List

PathSize
stand/
common/
4 lines

Diff 91719

View Options

stand/common/load_elf_obj.c

Loading...