Differential D48289

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

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

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Aug 19, 2:25 PM

	Unknown Object (File)
	Thu, Aug 7, 3:34 AM

	Unknown Object (File)
	Wed, Aug 6, 10:03 PM

	Unknown Object (File)
	Mon, Aug 4, 5:43 AM

	Unknown Object (File)
	Wed, Jul 30, 3:07 AM

	Unknown Object (File)
	Jun 15 2025, 12:17 PM

	Unknown Object (File)
	Jun 5 2025, 4:54 PM

	Unknown Object (File)
	Jun 3 2025, 8:31 PM

View All 38 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.Jan 2 2025, 4:37 PM

Herald added a reviewer: manu. · View Herald TranscriptJan 2 2025, 4:37 PM

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

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

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

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

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

jhb added inline comments.Jan 2 2025, 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.Jan 3 2025, 10:26 PM

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

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

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

Closed by commit rG16f0d01f9ca1: arm64: apple: fix aic for !SMP configurations (authored by kevans). · Explain WhyJan 10 2025, 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...