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

	F137005110: D21947.id63149.diff
	Thu, Nov 20, 11:28 PM

	F137004280: D21947.id63149.diff
	Thu, Nov 20, 11:26 PM

	F137004230: D21947.id63082.diff
	Thu, Nov 20, 11:26 PM

	F137004218: D21947.id63082.diff
	Thu, Nov 20, 11:26 PM

	F137003439: D21947.id63089.diff
	Thu, Nov 20, 11:23 PM

	F137003322: D21947.id63057.diff
	Thu, Nov 20, 11:23 PM

	F137002876: D21947.id63089.diff
	Thu, Nov 20, 11:21 PM

	F137002398: D21947.id63057.diff
	Thu, Nov 20, 11:20 PM

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...