Disable polkavm logging in pallet-revive#10385
Conversation
|
/cmd prdoc --audience runtime_dev --bump minor |
…time_dev --bump minor'
koute
left a comment
There was a problem hiding this comment.
While you're at it can you bump PolkaVM across the whole codebase? It shouldn't be too much trouble.
Differential Tests Results (PolkaVM)Specified Tests
Counts
FailuresThe test specifiers seen in this section have the format 'path::case_idx::compilation_mode' and they're compatible with the revive differential tests framework and can be specified to it directly in the same way that they're provided through the The failures are provided in an expandable section to ensure that the PR does not get polluted with information. Please click on the section below for more information Detailed Differential Tests Failure Information
|
Differential Tests Results (REVM)Specified Tests
Counts
FailuresThe test specifiers seen in this section have the format 'path::case_idx::compilation_mode' and they're compatible with the revive differential tests framework and can be specified to it directly in the same way that they're provided through the The failures are provided in an expandable section to ensure that the PR does not get polluted with information. Please click on the section below for more information Detailed Differential Tests Failure Information
|
xermicus
left a comment
There was a problem hiding this comment.
Do I get it right: This disables the interpreter logs in all runtimes? And we don't yet have any other method for debugging PVM contracts. If I'm not missing something can we add this as a const bool to Config instead?
@xermicus do you think we can use already existing |
|
Not sure what's preferred here - adding a (Not sure why those are not just const bools but are in storage instead - but I guess at least we get some consistency by adding it there) |
|
@athei @pgherveou WDYT? What option do you prefer? |
|
/cmd prdoc --audience runtime_dev --bump minor |
@xermicus done |
|
Command "prdoc --audience runtime_dev --bump minor" has failed ❌! See logs here |
|
/cmd prdoc --audience runtime_dev --bump major --force |
…time_dev --bump major --force'
|
All GitHub workflows were cancelled due to failure one of the required jobs. |
This PR adds configurable control over PolkaVM logging in `pallet-revive` to address performance degradation (details: #8760 (comment)) - Upgrades PolkaVM to v0.30.0 which provides `set_imperfect_logger_filtering_workaround()` - Adds `pvm_logs` flag to `DebugSettings` to control PolkaVM interpreter logging - Disables PolkaVM logs by default (when `pvm_logs=false`), enabling them only when explicitly configured - Fixes performance issue where excessive PolkaVM logging was impacting block proposal times The logging can be re-enabled via debug settings when needed for troubleshooting. Additionally: - PolkaVM has been bumped globally across whole codebase. --------- Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Backport all pallet-revive related changes into `unstable2507`. These are all the changes we want to get onto the next Kusama release. Main changes include - EVM backend - Ethereum block storage - Generalized gas mapping The complete list of PRs in this backport is - #9482 - #9455 - #9454 - #9501 - #9177 - #9285 - #9606 - #9414 - #9557 - #9617 - #9385 - #9679 - #9705 - #9561 - #9744 - #9736 - #9701 - #9517 - #9771 - #9683 - #9791 - #9717 - #9759 - #9823 - #9768 - #9853 - #9801 - #9780 - #9796 - #9878 - #9841 - #9670 - #9865 - #9803 - #9928 - #9818 - #9911 - #9942 - #9831 - #9945 - #9603 - #9968 - #9939 - #9991 - #9914 - #9997 - #9985 - #10016 - #10027 - #10026 - #9418 - #9988 - #10041 - #10047 - #10032 - #10065 - #10089 - #10080 - #10090 - #10106 - #10020 - #9512 - #10109 - #9699 - #10100 - #9909 - #10120 - #10146 - #10157 - #10168 - #10169 - #10160 - #10129 - #10175 - #10186 - #10192 - #10148 - #10193 - #10220 - #10233 - #10191 - #10225 - #10246 - #10239 - #10159 - #10252 - #10224 - #10267 - #10271 - #10214 - #10297 - #10290 - #10281 - #10272 - #10303 - #10336 - #10244 - #10366 - #10380 - #10383 - #10387 - #10302 - #10309 - #10427 - #10385 - #10451 - #10471 - #10166 - #10510 - #10393 - #10540 - #9587 - #10071 - #10558 - #10554 - #10325 --------- Signed-off-by: xermicus <cyrill@parity.io> Co-authored-by: Pavlo Khrystenko <45178695+pkhry@users.noreply.github.com> Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Javier Viola <363911+pepoviola@users.noreply.github.com> Co-authored-by: Bastian Köcher <git@kchr.de> Co-authored-by: Bastian Köcher <info@kchr.de> Co-authored-by: pgherveou <pgherveou@gmail.com> Co-authored-by: Omar <OmarAbdulla7@hotmail.com> Co-authored-by: 0xRVE <robertvaneerdewijk@gmail.com> Co-authored-by: xermicus <cyrill@parity.io> Co-authored-by: Alexander Samusev <41779041+alvicsam@users.noreply.github.com>
This PR adds configurable control over PolkaVM logging in
pallet-reviveto address performance degradation (details: #8760 (comment))set_imperfect_logger_filtering_workaround()pvm_logsflag toDebugSettingsto control PolkaVM interpreter loggingpvm_logs=false), enabling them only when explicitly configuredThe logging can be re-enabled via debug settings when needed for troubleshooting.
Additionally: