Page MenuHomeFreeBSD

Retain offset compatibility with pre-12.0 dumps
ClosedPublic

Authored by cem on Jun 15 2018, 6:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Sep 25, 10:27 AM
Unknown Object (File)
Sep 15 2025, 1:13 AM
Unknown Object (File)
Aug 15 2025, 12:28 AM
Unknown Object (File)
Jul 27 2025, 6:04 AM
Unknown Object (File)
Jun 24 2025, 11:41 PM
Unknown Object (File)
Jun 24 2025, 1:58 PM
Unknown Object (File)
Jun 23 2025, 3:23 PM
Unknown Object (File)
Jun 22 2025, 6:00 PM
Subscribers

Details

Summary

As a follow-up to r324965, which adds support for compressed kernel dumps,
readjust dump header members slightly to mostly preserve ABI (somewhat) with
earlier (11.x and older) dumps.

Test Plan

This change does burn 4 bytes of panicstr, but I think that is a worthwhile
tradeoff. An alternative approach that doesn't burn 4 bytes of panicstr would
be to misalign the 8-byte dumpextent field, which would also be fine with me.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I think this is fine. The panic string buffer is bigger than it needs to be.

We already have the compile-time assert in all the MD dump files.

This revision is now accepted and ready to land.Jun 15 2018, 6:51 PM

I think this is fine. The panic string buffer is bigger than it needs to be.

We already have the compile-time assert in all the MD dump files.

Thanks! I'll remove the assert before commit.

This revision was automatically updated to reflect the committed changes.