Attention is currently required from: Aarya, Alexander Goncharov.

View Change

1 comment:

• File erasure_layout.c:

  • Patch Set #1, Line 177: msg_cinfo("Region start 0x%08x not sector aligned! Extending start boundaries by 0x%08x.\n",

    maybe write "Extending start boundaries by 0x%08x bytes. […]

    I added "bytes" thank you, good idea.
    This patch came up in process of debugging https://ticket.coreboot.org/issues/494 , you are very welcome to have a look and say what you think about it!
    If you look my last comment, with 1-line change in existing test, the issue can repro (test fails). So you don't even need images or any hardware.
    At the end I will add a proper test to cover this case, but for debugging phase, changing 1 line of code should be fine :)

To view, visit change 78492. To unsubscribe, or for help writing mail filters, visit settings.