Page MenuHomeFreeBSD
Differential D17941

acpica : move SRAT/SLIT parsing to sys/dev/acpica
ClosedPublic
Actions

Authored by jchandra on Nov 11 2018, 2:37 AM.
Tags
None
Referenced Files
F107214149: D17941.diff
Sat, Jan 11, 3:43 PM
Unknown Object (File)
Tue, Jan 7, 10:32 PM
Unknown Object (File)
Sat, Dec 14, 2:48 AM
Unknown Object (File)
Nov 25 2024, 10:00 AM
Unknown Object (File)
Oct 20 2024, 9:49 AM
Unknown Object (File)
Sep 15 2024, 9:05 PM
Unknown Object (File)
Sep 15 2024, 9:04 PM
Unknown Object (File)
Sep 15 2024, 9:04 PM
View All 76 Files
Subscribers
imp

Details

Reviewers
andrew
jhb
markj
imp
Group Reviewers
arm64
Commits
rS341742: acpica : move SRAT/SLIT parsing to sys/dev/acpica
Summary

This moves the architecture independent parts of sys/x86/acpica/srat.c to sys/dev/acpica/acpi_pxm.c, to be used later on arm64. The function declarations are moved to sys/dev/acpica/acpivar.h

We also need to update sys/conf/files.{i386,amd64} to use the new file. No functional changes.

Test Plan

No effect on x86 (numa/non-numa)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jchandra created this revision.Nov 11 2018, 2:37 AM
Herald added a subscriber: imp. · View Herald TranscriptNov 11 2018, 2:37 AM
Harbormaster completed remote builds in B20747: Diff 50276.Nov 11 2018, 2:37 AM
andrew added a reviewer: markj.Nov 13 2018, 10:38 AM
markj added inline comments.Nov 15 2018, 10:22 PM
sys/x86/acpica/srat.c
51 ↗(On Diff #50276)

Are all of these includes still needed?

jchandra planned changes to this revision.Nov 16 2018, 1:58 AM
jchandra marked an inline comment as done.

Thanks for the review - will post new version dropping a few includes.

sys/x86/acpica/srat.c
51 ↗(On Diff #50276)

I can drop a few includes.

jchandra updated this revision to Diff 50578.Nov 19 2018, 1:40 AM

Addressed markj's comment - removed unnecessary includes.

Harbormaster completed remote builds in B20883: Diff 50578.Nov 19 2018, 1:40 AM
markj accepted this revision.Dec 3 2018, 2:51 PM
This revision is now accepted and ready to land.Dec 3 2018, 2:51 PM
imp accepted this revision.Dec 3 2018, 3:26 PM
Closed by commit rS341742: acpica : move SRAT/SLIT parsing to sys/dev/acpica (authored by jchandra). · Explain WhyDec 8 2018, 7:11 PM
This revision was automatically updated to reflect the committed changes.
jchandra mentioned this in rS352496: MFC r341741-r341742.Sep 18 2019, 10:45 PM

Revision Contents
Changeset List

PathSize
head/
sys/
conf/
1 line
1 line
dev/
acpica/
649 lines
9 lines
x86/
acpica/
612 lines

Diff 51763

View Options

head/sys/conf/files.amd64

Loading...
View Options

head/sys/conf/files.i386

Loading...
View Options

head/sys/dev/acpica/acpi_pxm.c

Loading...
View Options

head/sys/dev/acpica/acpivar.h

Loading...
View Options

head/sys/x86/acpica/srat.c

Loading...