build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/27358 )
Change subject: riscv: temporarily block multiple-threads ......................................................................
Patch Set 28:
(4 comments)
https://review.coreboot.org/#/c/27358/28/src/arch/riscv/include/arch/smp/smp... File src/arch/riscv/include/arch/smp/smp.h:
https://review.coreboot.org/#/c/27358/28/src/arch/riscv/include/arch/smp/smp... PS28, Line 62: if (arrived > 1) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/27358/28/src/arch/riscv/include/arch/smp/smp... PS28, Line 66: while (bar->wait[gen] == 0) ; space prohibited before semicolon
https://review.coreboot.org/#/c/27358/28/src/arch/riscv/include/arch/smp/smp... PS28, Line 66: while (bar->wait[gen] == 0) ; trailing statements should be on next line
https://review.coreboot.org/#/c/27358/28/src/arch/riscv/include/arch/smp/smp... PS28, Line 68: if (atomic_fetch_add(&(bar->entered[gen]), -1) == 1) { braces {} are not necessary for single statement blocks
-
build bot (Jenkins) (Code Review)