Page MenuHomeFreeBSD

Fix intr_machdep.c for armv8
ClosedPublic

Authored by wma_semihalf.com on Jul 9 2015, 5:33 AM.
Tags
None
Referenced Files
F139417287: D3029.id6916.diff
Thu, Dec 11, 8:38 PM
Unknown Object (File)
Mon, Nov 24, 9:28 PM
Unknown Object (File)
Fri, Nov 21, 7:14 AM
Unknown Object (File)
Fri, Nov 21, 7:14 AM
Unknown Object (File)
Fri, Nov 21, 7:07 AM
Unknown Object (File)
Fri, Nov 21, 7:05 AM
Unknown Object (File)
Wed, Nov 19, 4:34 AM
Unknown Object (File)
Fri, Nov 14, 4:48 AM
Subscribers

Details

Summary

On armv8 IPIs are mapped to 0-15. Incrementing the number by 16 is wrong, because it sets a reserved bit in the IPI register. This patch removes all "+16" to comply with specs.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

wma_semihalf.com retitled this revision from to Fix intr_machdep.c for armv8.
wma_semihalf.com updated this object.
wma_semihalf.com edited the test plan for this revision. (Show Details)
wma_semihalf.com added reviewers: andrew, emaste, zbb.
wma_semihalf.com set the repository for this revision to rS FreeBSD src repository - subversion.
sys/arm64/arm64/intr_machdep.c
450

Why has this been added? I's unrelated to the rest of the patch.

sys/arm64/arm64/intr_machdep.c
450

It's just a cleanup here. We'll need this function in the near future, so I added it into this patch and use inside arm_unmask_ipi, just not to touch this file again.

sys/arm64/arm64/intr_machdep.c
450

Can you move this part of the change to that review then? I need to think about D3034 a little, but have no problems with the rest of this change.

This revision was automatically updated to reflect the committed changes.