[coreboot] [PATCH] Factor out common CAR asm snippets
Myles Watson
mylesgw at gmail.com
Sat Oct 2 00:38:34 CEST 2010
> IMHO the patch makes it really hard to actually see what the code does.
> I think Idwer was recently running into a problem where the use of
> post_code would trash %eax were hard to comprehend. Now we seem have a
> lot more code that comes from somewhere else and touches registers
> without it being obvious to the reader of the code.
I agree for Save/Restore BIST, since it's only one line. I like the macros
for longer snippets, though. Would a comment like
/* Clobbers %eax, %edx, ... */
be the right way to fix it? Would it be better to have the clobber list in
the name?
disable_sse_with_eax()?
Thanks,
Myles
More information about the coreboot
mailing list