Page MenuHomeFreeBSD
Differential D21072

Remove support for kernel.tramp and kernel.tramp.gz
ClosedPublic
Actions

Authored by imp on Jul 26 2019, 4:50 AM.
Tags
None
Referenced Files
F86185274: D21072.id60198.diff
Sun, Jun 16, 5:02 PM
Unknown Object (File)
Thu, May 23, 7:59 PM
Unknown Object (File)
Tue, May 21, 10:14 PM
Unknown Object (File)
Mon, May 20, 2:58 AM
Unknown Object (File)
May 13 2024, 10:53 PM
Unknown Object (File)
May 8 2024, 8:15 PM
Unknown Object (File)
May 3 2024, 4:10 PM
Unknown Object (File)
May 2 2024, 3:05 AM
View All 68 Files
Subscribers
andrew
delphij
ota_j.email.ne.jp
yamori813_yahoo.co.jp

Details

Reviewers
gonzo
• ian
Commits
rS350377: Remove support for kernel.tramp and kernel.tramp.gz
Summary

Nothing uses these anymore. They were for super small armv4 boards. Today, all
armv5 kernels boot via uboot which has its own compression scheme.

Diff Detail

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

Event Timeline

imp created this revision.Jul 26 2019, 4:50 AM
Herald added a subscriber: andrew. · View Herald TranscriptJul 26 2019, 4:50 AM
Harbormaster completed remote builds in B25512: Diff 60158.Jul 26 2019, 4:50 AM
imp added reviewers: gonzo, • ian.Jul 26 2019, 4:52 AM
delphij added a subscriber: delphij.Jul 26 2019, 7:32 AM
delphij added inline comments.
sys/conf/Makefile.arm
128 ↗(On Diff #60158)

Maybe unifdef -UKZIP and have elf_trampoline.c changed too?

andrew added inline comments.Jul 26 2019, 8:49 AM
sys/conf/Makefile.arm
128 ↗(On Diff #60158)

elf_trampoline.c should just be removed. With this change it will no longer be built.

• ian added inline comments.Jul 26 2019, 2:57 PM
sys/conf/Makefile.arm
91 ↗(On Diff #60158)

This reference to KERNPHYSADDR can go too, it doesn't show up in ldscript.arm so the subsitution is a no-op here. The subsitution for KERNVIRTADDR is still vaild though.

ota_j.email.ne.jp added a subscriber: ota_j.email.ne.jp.Jul 26 2019, 10:37 PM
yamori813_yahoo.co.jp added a subscriber: yamori813_yahoo.co.jp.Jul 26 2019, 11:29 PM

I agree this change.

gonzo accepted this revision.Jul 26 2019, 11:34 PM

No objections

This revision is now accepted and ready to land.Jul 26 2019, 11:34 PM
ota_j.email.ne.jp added inline comments.Jul 27 2019, 12:24 AM
sys/conf/Makefile.arm
128 ↗(On Diff #60158)

It looks sys/arm/arm/inckern.S is also no longer used with this removal.

imp updated this revision to Diff 60185.Jul 27 2019, 5:43 AM

Update per reviews

This revision now requires review to proceed.Jul 27 2019, 5:43 AM
Harbormaster completed remote builds in B25532: Diff 60185.Jul 27 2019, 5:43 AM
This revision was not accepted when it landed; it landed in state Needs Review.Jul 27 2019, 5:24 PM
Closed by commit rS350377: Remove support for kernel.tramp and kernel.tramp.gz (authored by imp). · Explain Why
This revision was automatically updated to reflect the committed changes.
imp added a commit: rS350377: Remove support for kernel.tramp and kernel.tramp.gz.
ota_j.email.ne.jp mentioned this in D19706: Allow Kernel to link in both legacy libkern/zlib and new sys/contrib/zlib..Jul 28 2019, 8:20 PM

Revision Contents
Changeset List

PathSize
head/
sys/
arm/
arm/
42 lines
conf/
1 line
mv/
discovery/
1 line
orion/
1 line
1 line
ralink/
1 line
conf/
83 lines

Diff 60198

View Options

head/sys/arm/arm/inckern.S

Loading...
View Options

head/sys/arm/conf/NOTES.armv5

Loading...
View Options

head/sys/arm/mv/discovery/std.db78xxx

Loading...
View Options

head/sys/arm/mv/orion/std.db88f5xxx

Loading...
View Options

head/sys/arm/mv/orion/std.ts7800

Loading...
View Options

head/sys/arm/ralink/std.ralink

Loading...
View Options

head/sys/conf/Makefile.arm

Loading...