Limit MAXCPU <= MAX_APIC_ID assertion to i386
AbandonedPublic
Actions

Authored by emaste on Oct 19 2022, 7:32 PM.

Tags

None

Referenced Files

	F171474919: D37060.id.diff
	Fri, Sep 11, 8:08 AM

	F171386161: D37060.diff
	Thu, Sep 10, 8:37 PM

	Unknown Object (File)
	Thu, Sep 10, 7:46 AM

	Unknown Object (File)
	Tue, Sep 8, 12:54 AM

	Unknown Object (File)
	Tue, Sep 8, 12:07 AM

	Unknown Object (File)
	Sun, Sep 6, 9:36 AM

	Unknown Object (File)
	Sat, Sep 5, 5:22 PM

	Unknown Object (File)
	Sat, Sep 5, 3:42 PM

View All Files

Subscribers

kib

Details

Reviewers

jhb
royger

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

From @jhb in D36838: the assertion about MAXCPU being <= MAX_APIC_ID is no longer relevant with x2APIC and can be removed I think. I'm not sure if we support x2APIC on i386, so maybe the assertion has to stay for i386 still

emaste mentioned this in D36838: amd64: Bump MAXCPU to 1024 (from 256).Oct 19 2022, 7:33 PM

No, MAX_APIC_ID is also relevant for amd64, as it's used by madt_parse_cpu(), it needs to be bumped to twice the value of MAXCPU I think.

I guess you could have a i386 system with APIC IDs > 0x200, but it's unlikely such system doesn't support amd64.

i386 also uses x2APIC when supported by hardware and not disabled by BIOS.

emaste abandoned this revision.Oct 20 2022, 11:57 AM

Revision Contents
Changeset List

Path

Size

sys/

x86/

mp_x86.c

2 lines

Diff 112006

View Options

Limit MAXCPU <= MAX_APIC_ID assertion to i386AbandonedPublicActions