gic_v3: consolidate per-cpu redistributor information
ClosedPublic
Actions

Authored by jchandra on Apr 8 2019, 2:14 AM.

Details

Reviewers

andrew
manu

Group Reviewers

arm64

Commits

rS352501: MFC r348377-r348379, r349347
rS348377: gic_v3: consolidate per-cpu redistributor information

Summary

Update 'struct gic_redists' to consolidate all per-cpu redistributor information into a new 'struct redist_pcpu'. Provide a new interface (GICV3_IVAR_REDIST) for the GIC driver which retrieves the above per-cpu data.

This per-cpu redistributor struct will be later used to improve GIC ITS setup.

While there, remove some unused fields in gic_v3_var.h interface.
No functional changes.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 24518

Event Timeline

jchandra created this revision.Apr 8 2019, 2:14 AM

Herald added a reviewer: manu. · View Herald TranscriptApr 8 2019, 2:14 AM

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

Harbormaster completed remote builds in B23564: Diff 55930.Apr 8 2019, 2:14 AM

val_packett.cool added a subscriber: val_packett.cool.May 8 2019, 11:52 AM

jchandra updated this revision to Diff 57924.May 26 2019, 11:24 PM

Harbormaster completed remote builds in B24518: Diff 57924.May 26 2019, 11:24 PM

andrew added inline comments.May 28 2019, 9:54 AM

sys/arm64/arm64/gic_v3.c
393–394	If you plan on MFCing this you should keep `GICV3_IVAR_REDIST_VADDR` to preserve KBI. It can be removed in a later commit that doesn't get an MFC.
sys/arm64/arm64/gic_v3_var.h
96	I would make `GICV3_IVAR_REDIST` 1002 with a comment that 1001 was `GICV3_IVAR_REDIST_VADDR` as it's a change in KBI.