Attention is currently required from: Arthur Heymans.
Raul Rangel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68913 )
Change subject: cpu/x86: Set up cpu_info storage using MSR
......................................................................
Patch Set 8:
(1 comment)
File src/arch/x86/include/arch/cpu.h:
https://review.coreboot.org/c/coreboot/+/68913/comment/3c9962a2_b90a4a17
PS8, Line 168: if (long_mode_supported()) {
: msr_t msr = rdmsr(IA32_GS_BASE);
: struct per_cpu_segment_data *segment_data =
: (struct per_cpu_segment_data *)(uintptr_t)msr.raw;
: return segment_data->cpu_info;
: }
You can drop this now can't you?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/68913
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Id38d3c84ef757c0b322ec0d387f57043147cb447
Gerrit-Change-Number: 68913
Gerrit-PatchSet: 8
Gerrit-Owner: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Reviewer: Marc Jones
marc@marcjonesconsulting.com
Gerrit-Reviewer: Raul Rangel
rrangel@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Comment-Date: Mon, 14 Nov 2022 02:13:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment