On 13/07/2023 14:47, Linu Cherian wrote:
Changelog from v1:
- V2 is a complete patchset with kernel panic trace tested on Linux 6.4. Details on testing with relevant console logs has been added for reference.
- Two additional patches(patch 6 & 7) has been included to manage stopping of trace at the time of kernel panic.
- Few bug fixes.
TODO:
- Add support to prevent overwriting of trace data captured in previous boot. (Suggested by James)
- DTS properties for reserved memory might need some refinements, since Linux arm64 kernel has limitation on the number of reserved regions it supports(ie. 64).
I assume you already thought about this given the note, but you could have a single region for both metadata and trace, as presumably the metadata is a known fixed size so it can be placed at the beginning or end.
But from a quick look at the other device tree files I don't see anything close to 64 regions being used anywhere. It doesn't seem like it would be an issue as this feature may not be widely used either.