Differential D48289

arm64: apple: fix aic for !SMP configurations
ClosedPublic
Actions

Authored by kevans on Thu, Jan 2, 4:37 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Jan 10, 12:03 AM

	Unknown Object (File)
	Thu, Jan 9, 5:11 PM

	Unknown Object (File)
	Mon, Jan 6, 5:16 PM

	Unknown Object (File)
	Mon, Jan 6, 2:21 PM

	Unknown Object (File)
	Mon, Jan 6, 10:53 AM

	Unknown Object (File)
	Mon, Jan 6, 9:56 AM

	Unknown Object (File)
	Mon, Jan 6, 9:25 AM

	Unknown Object (File)
	Fri, Jan 3, 10:41 PM

View All 9 Files

Subscribers

Details

Reviewers

jhb
andrew
manu

Group Reviewers

Commits

rG16f0d01f9ca1: arm64: apple: fix aic for !SMP configurations

Summary

Record the BSP CPU identification in attach for later use in our
bus_setup_intr implementation.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 61497
Build 58381: arc lint + arc unit

Event Timeline

kevans created this revision.Thu, Jan 2, 4:37 PM

Herald added a reviewer: manu. · View Herald TranscriptThu, Jan 2, 4:37 PM

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

kevans requested review of this revision.Thu, Jan 2, 4:37 PM

Harbormaster completed remote builds in B61460: Diff 148679.Thu, Jan 2, 4:37 PM

andrew accepted this revision.Thu, Jan 2, 5:43 PM

This revision is now accepted and ready to land.Thu, Jan 2, 5:43 PM

jhb added inline comments.Thu, Jan 2, 8:04 PM

sys/arm64/apple/apple_aic.c
141	I would be tempted to make #ifdef's a bit smaller by keeping sc_cpuids around always so that only sc_ipimasks and sc_ipi_srcs are under the #ifdef. It means you will end up allocating a single-element array for sc_cpuids, but that's fine. No one will probably use a UP kernel anyway so I'd optimize for minimizing diffs.

Reverse course, just move sc_cpuids out from underneath SMP and let it allocatee
a single-entry sc_cpuids. The overhead of still doing intr_irq_next_cpu() is
minimal since it's simply a PCPU_GET(cpuid) on !SMP kernels.

This revision now requires review to proceed.Fri, Jan 3, 10:26 PM

Harbormaster completed remote builds in B61497: Diff 148756.Fri, Jan 3, 10:27 PM

jhb accepted this revision.Thu, Jan 9, 9:17 PM

This revision is now accepted and ready to land.Thu, Jan 9, 9:17 PM

Closed by commit rG16f0d01f9ca1: arm64: apple: fix aic for !SMP configurations (authored by kevans). · Explain WhyFri, Jan 10, 4:28 AM

This revision was automatically updated to reflect the committed changes.

kevans added a commit: rG16f0d01f9ca1: arm64: apple: fix aic for !SMP configurations.

Revision Contents
Changeset List

Path

Size

sys/

arm64/

apple/

5 lines

Diff 148756

sys/arm64/apple/apple_aic.c

Loading...