part_block: use expanding reporter #4782
Labels
Comments
cnuke
added a commit
to cnuke/genode
that referenced
this issue
Mar 10, 2023
Depending on the number of partitions the current fixed size of 4096 bytes might not be sufficient for the resulting partitions report. The use of the expanding reporter allows for accomodating large reports while still keeping the resource usage down for the common case of few partitions (with potentially short names). Fixes genodelabs#4782.
|
Commit e7d7b46 fixes this issue. |
chelmuth
pushed a commit
that referenced
this issue
Mar 10, 2023
Depending on the number of partitions the current fixed size of 4096 bytes might not be sufficient for the resulting partitions report. The use of the expanding reporter allows for accomodating large reports while still keeping the resource usage down for the common case of few partitions (with potentially short names). Fixes #4782.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As reported on the mailling-list using Sculpt with a disk that contains many partitions does not work. Although
part_blockis able to parse the partition-table (as shown in the LOG) it later complains that no valid partition table was found.This behaviour is due to the fact the component cannot generate the partitions report as the underlying buffer is too small, which results in an exeception that is silently caught and treated as a non-valid partition-table.
The following commit addresses this short-coming by replacing the static
Reporterwith anExpanding_reporter.The text was updated successfully, but these errors were encountered: