Attention is currently required from: Bill XIE, Angel Pons, Arthur Heymans.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69215 )
Change subject: cpu/x86/mp_init.c: Fix the logic to add AP cpus
......................................................................
Patch Set 4: Code-Review+1
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/69215/comment/a8a5ac87_de97a77f
PS1, Line 9: https://review.coreboot.org/c/coreboot/+/64340
As nobody memorizes commit hashes, the summary should be always added in my opinion. That’s also, what is common in the Linux kernel project.
Commit 48c825ebd1be (cpu/x86/mp_init.c: Use linked list data structures) …
--
To view, visit
https://review.coreboot.org/c/coreboot/+/69215
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1075bc52091e619ef79cbeab0a0f6b57cff1d708
Gerrit-Change-Number: 69215
Gerrit-PatchSet: 4
Gerrit-Owner: Bill XIE
persmule@hardenedlinux.org
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Reviewer: Lean Sheng Tan
sheng.tan@9elements.com
Gerrit-Reviewer: Paul Menzel
paulepanter@mailbox.org
Gerrit-Reviewer: Tim Wawrzynczak
inforichland@gmail.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Bill XIE
persmule@hardenedlinux.org
Gerrit-Attention: Angel Pons
th3fanbus@gmail.com
Gerrit-Attention: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Comment-Date: Sun, 06 Nov 2022 08:48:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Bill XIE
persmule@hardenedlinux.org
Comment-In-Reply-To: Angel Pons
th3fanbus@gmail.com
Gerrit-MessageType: comment