Paths

Table of Contentst

Differential D21947

gcore: Add aarch64 32-bit core support
ClosedPublic
Actions

Authored by jhibbits on Oct 8 2019, 9:30 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Feb 8 2024, 3:52 PM

	Unknown Object (File)
	Feb 8 2024, 2:52 PM

	Unknown Object (File)
	Dec 20 2023, 5:31 AM

	Unknown Object (File)
	Oct 27 2023, 6:52 AM

	Unknown Object (File)
	Sep 3 2023, 3:31 PM

	Unknown Object (File)
	Sep 2 2023, 6:47 PM

	Unknown Object (File)
	Aug 31 2023, 4:31 AM

	Unknown Object (File)
	Aug 25 2023, 11:26 AM

View All 17 Files

Subscribers

Details

Reviewers

manu
andrew

Group Reviewers

Commits

rS353436: gcore: Add aarch64 32-bit core support

Summary

Add trivial 32-bit arm cores on aarch64 support for gcore. This doesn't handle fpregs.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 26936
Build 25239: arc lint + arc unit

Event Timeline

jhibbits created this revision.Oct 8 2019, 9:30 PM

Herald added a reviewer: manu. · View Herald TranscriptOct 8 2019, 9:30 PM

Herald added subscribers: emaste, andrew. · View Herald Transcript

Harbormaster completed remote builds in B26936: Diff 63057.Oct 8 2019, 9:30 PM

stevek added a subscriber: stevek.Oct 8 2019, 9:32 PM

jhibbits retitled this revision from gcore: Add aarch64 support to gcore: Add aarch64 32-bit core support.Oct 8 2019, 9:33 PM

jhibbits edited the summary of this revision. (Show Details)

andrew added inline comments.Oct 9 2019, 8:42 AM

usr.bin/gcore/elf32core.c
40	I think you want to use `rs->x[13]` here. The comment in `fill_regs32` in `sys/arm64/arm64/machdep.c` is useful: `/* For arm32, SP is r13 and LR is r14 */` You probably also want to set `r_lr` to `rs->x14`.

Address andrew's feedback. Thanks!

Harbormaster completed remote builds in B26953: Diff 63082.Oct 9 2019, 3:43 PM

emaste added inline comments.Oct 9 2019, 4:46 PM

usr.bin/gcore/Makefile
8	my usual pedantic comment, keep these sorts of additions in alpha order :)
usr.bin/gcore/elf32core.c
54	fwiw we've used ARM64TODO in the past

Address @emaste's comments

Harbormaster completed remote builds in B26958: Diff 63089.Oct 9 2019, 5:26 PM

bdragon mentioned this in D21954: gcore: Add powerpc64 32-bit gcore support.Oct 11 2019, 1:59 AM

andrew accepted this revision.Oct 11 2019, 7:27 AM

This revision is now accepted and ready to land.Oct 11 2019, 7:27 AM

Closed by commit rS353436: gcore: Add aarch64 32-bit core support (authored by jhibbits). · Explain WhyOct 11 2019, 2:16 PM

This revision was automatically updated to reflect the committed changes.

jhibbits added a commit: rS353436: gcore: Add aarch64 32-bit core support.

Herald added a subscriber: imp. · View Herald TranscriptOct 11 2019, 2:16 PM

Revision Contents
Changeset List

Path

Size

sys/

arm64/

include/

4 lines

usr.bin/

gcore/

2 lines

11 lines

Diff 63057

sys/arm64/include/elf.h

Loading...

usr.bin/gcore/Makefile

Loading...

usr.bin/gcore/elf32core.c

Loading...