Differential D3107

Fix ARM64 stack calculation
ClosedPublic
Actions

Authored by wma_semihalf.com on Jul 16 2015, 8:23 AM.

Tags

None

Referenced Files

	F132556190: D3107.id7046.diff
	Fri, Oct 17, 10:58 PM

	Unknown Object (File)
	Wed, Oct 15, 1:45 AM

	Unknown Object (File)
	Thu, Sep 25, 10:56 PM

	Unknown Object (File)
	Sun, Sep 21, 1:43 AM

	Unknown Object (File)
	Thu, Sep 18, 10:47 AM

	Unknown Object (File)
	Sep 10 2025, 5:42 AM

	Unknown Object (File)
	Sep 5 2025, 10:34 PM

	Unknown Object (File)
	Sep 5 2025, 12:20 PM

Subscribers

freebsd-arm-list

Details

Reviewers

zbb
emaste
andoriyu_gmail.com

Commits

rS285654: Fix secondary stacks calculation on ARM64

Summary

Fixes in locore.s:
 - opt_kstack_pages.h include was added to ensure the proper
   KSTACK_PAGES value is used
 - secondary stack calculation is modified to provide
   stack_top = secondary_stacks + (cpu_id)*PAGE_SIZE*KSTACK_PAGES
   because on ARMv8 the stack grows into lower memory addresses

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

wma_semihalf.com updated this revision to Diff 7001.Jul 16 2015, 8:23 AM

wma_semihalf.com retitled this revision from to Fix ARM64 stack calculation.

wma_semihalf.com updated this object.

wma_semihalf.com edited the test plan for this revision. (Show Details)

wma_semihalf.com added reviewers: andoriyu_gmail.com, emaste, zbb.

wma_semihalf.com set the repository for this revision to rS FreeBSD src repository - subversion.

wma_semihalf.com added a subscriber: freebsd-arm-list.

Herald added subscribers: emaste, andrew, imp. · View Herald TranscriptJul 16 2015, 8:23 AM

wma_semihalf.com updated this revision to Diff 7002.Jul 16 2015, 8:27 AM

zbb accepted this revision.Jul 17 2015, 9:52 AM

zbb edited edge metadata.

This revision is now accepted and ready to land.Jul 17 2015, 9:52 AM

Closed by commit rS285654: Fix secondary stacks calculation on ARM64 (authored by zbb). · Explain WhyJul 17 2015, 2:08 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

arm64/

arm64/

3 lines

Diff 7046

head/sys/arm64/arm64/locore.S

Loading...