Attention is currently required from: Kane Chen, Tim Wawrzynczak, Arthur Heymans.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/64467 )
Change subject: cpu/x86/mtrr: Print apic id in when set up MTRRs for BSP/APs
......................................................................
Patch Set 4:
(1 comment)
File src/cpu/x86/mtrr/mtrr.c:
https://review.coreboot.org/c/coreboot/+/64467/comment/cb65a882_844ed9bb
PS3, Line 358: lapicid
i checked cpu_index function, there is loop inside to compare apic id. i feel it's too much.
The apic_id here is to indicate what cpu is running the task, so i guess apic_id should be good enough?
thanks
CB:69509 changes that so it's fine.
I agree with Arthur that, you can wait and land your CL on top of CB:69509
--
To view, visit
https://review.coreboot.org/c/coreboot/+/64467
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ifd2e1e411f86fa3ea42ed50546facec31b89c3e1
Gerrit-Change-Number: 64467
Gerrit-PatchSet: 4
Gerrit-Owner: Kane Chen
kane.chen@intel.corp-partner.google.com
Gerrit-Reviewer: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Reviewer: Eric Lai
eric_lai@quanta.corp-partner.google.com
Gerrit-Reviewer: Subrata Banik
subratabanik@google.com
Gerrit-Reviewer: Tim Wawrzynczak
inforichland@gmail.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Kane Chen
kane.chen@intel.com
Gerrit-Attention: Kane Chen
kane.chen@intel.corp-partner.google.com
Gerrit-Attention: Tim Wawrzynczak
inforichland@gmail.com
Gerrit-Attention: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Attention: Kane Chen
kane.chen@intel.com
Gerrit-Comment-Date: Tue, 29 Nov 2022 09:01:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Subrata Banik
subratabanik@google.com
Comment-In-Reply-To: Arthur Heymans
arthur@aheymans.xyz
Comment-In-Reply-To: Kane Chen
kane.chen@intel.com
Gerrit-MessageType: comment