[Libreboot-dev] Why do you carry 0042-cpu-amd-car-Initialize-entire-CAR-space-instead-of-o.patch ?

[xdrudis]

Hello, and thank you very much for your work.

I'm looking at the code and I see that libreboot is patching coreboot
with a change that was abandoned in coreboot: 

resources/libreboot/patch/kgpe-d16/0042-cpu-amd-car-Initialize-entire-CAR-space-instead-of-o.patch

http://review.coreboot.org/#/c/11974/

I'm not good at assembler but from what I've read I think the 
original code was right and the patch introduces a bug, because
rep lodsl or rep stosl copy 4 bytes, so we only have to repeat
this CacheSize/4 times (= CacheSize / 2^2 = CacheSize >> 2).

I can't test now but I'll reply if/ when I can test it. 
But I thought I'd ask because I may be missing something. 

Thanks.

Xavi