20 Nov
2021
20 Nov
'21
10:03 a.m.
Attention is currently required from: Raul Rangel, Paul Menzel, Julius Werner, Rob Barnes, Karthik Ramasubramanian. Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59322 )
Change subject: arch/x86: Remove spinlock dependency on threading ......................................................................
Patch Set 8:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/59322/comment/0e8cc5e6_d27955cc PS8, Line 13: commit a98d302fe9dc ("x86/smp/spinlock: Disable thread coop when taking spinlock") IMO you should do a clean revert first, and not have printk() as a special case with spinlock, but use mutex there too.
--
To view, visit https://review.coreboot.org/c/coreboot/+/59322
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I55d0cd2bcd82721098ca24ae2368bb6a16a07df4
Gerrit-Change-Number: 59322
Gerrit-PatchSet: 8
Gerrit-Owner: Raul Rangel rrangel@chromium.org
Gerrit-Reviewer: Julius Werner jwerner@chromium.org
Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com
Gerrit-Reviewer: Kyösti Mälkki kyosti.malkki@gmail.com
Gerrit-Reviewer: Paul Menzel paulepanter@mailbox.org
Gerrit-Reviewer: Rob Barnes robbarnes@google.com
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-Attention: Raul Rangel rrangel@chromium.org
Gerrit-Attention: Paul Menzel paulepanter@mailbox.org
Gerrit-Attention: Julius Werner jwerner@chromium.org
Gerrit-Attention: Rob Barnes robbarnes@google.com
Gerrit-Attention: Karthik Ramasubramanian kramasub@google.com
Gerrit-Comment-Date: Sat, 20 Nov 2021 18:03:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment