Attention is currently required from: Keith Hui, Branden Waldner. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59766 )
Change subject: [Untested]cpu/x86/mp.h: Implement a pre-SSE2 mfence ......................................................................
Patch Set 3: Verified+1
(3 comments)
File src/include/cpu/x86/mp.h:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-139505): https://review.coreboot.org/c/coreboot/+/59766/comment/973bb21d_b3bd7784 PS3, Line 18: __asm__ __volatile__("mfence\t\n": : :"memory"); spaces required around that ':' (ctx:VxW)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-139505): https://review.coreboot.org/c/coreboot/+/59766/comment/b97f5bd8_bcac8942 PS3, Line 18: __asm__ __volatile__("mfence\t\n": : :"memory"); spaces required around that ':' (ctx:WxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-139505): https://review.coreboot.org/c/coreboot/+/59766/comment/a851aac8_37606206 PS3, Line 20: __asm__ __volatile__("lock; addl $0,0(%%esp)": : : "memory"); spaces required around that ':' (ctx:VxW)