Acking all the old comments on the hack PS1. Since I added .near_reset_vector as a real patch, I thought it would be better to use the same Change-ID vs. using a fresh one.
4 comments:
Patch Set #1, Line 35: fffffe00 <_start16bit>:
Oh, that's right. […]
Acking but not sure it's important for the commit message anymore. As I mentioned in the newer version, it's reproducible by bumping C_ENV_BOOTBLOCK_SIZE over 64K.
File src/cpu/x86/16bit/entry16.inc:
Patch Set #1, Line 44: .type _start16bit, @function
Sure. But that's incorrect which is what I think the point of your comment. […]
Address was changed by how high I'd put it originally and that's changed now. Also CB:30855 and CB:37154 have since touched the alignment.
File src/cpu/x86/16bit/reset16.ld:
Patch Set #1, Line 20: _ROMTOP
If we are going to define this symbol here we should remove it from line 30?
Ack
Patch Set #1, Line 21: _NEAR_RESET_VECTOR
> […]
Ack
To view, visit change 36418. To unsubscribe, or for help writing mail filters, visit settings.