Review benchmarking

[bkchr]

We are still seeing the following log lines on production:

[Parachain] Benchmarked storage weight smaller than consumed storage weight. benchmarked: 7140 consumed: 10710 unspent: 0 

Node-side PoV size higher than runtime proof size weight. node-side: 66824 extrinsic_len: 11138 runtime: 64767, missing: 2057. Setting to node-side proof size.

We should check which transaction are responsible for these and see if the benchmarking needs to be improved. The second log line is probably related to the BASE_EXTRINSIC weight which is being worked on right now.

[Moliholy]

@bkchr is paritytech/polkadot-sdk#5594 expected to be released reasonably soon? Otherwise we're blind, as we don't know which exact call is causing the problem and it's pretty hard to find that out.

[bkchr]

It will be backported to the stable branches

[Moliholy]

@bkchr does this log entry appear like +1200 times in a single block? Because if so, it could very likely be the migration pallet.

[bkchr]

No, not that often. It happens way less.