Commit Graph
128 Commits
Author SHA1 Message Date
Łukasz Magiera 0d1174cb81 Merge pull request #2924 from filecoin-project/feat/mpool-cfg-overesitm
Add GasLimitOverestimation to mpool config
2020-08-10 11:33:05 +02:00
Jakub Sztandera 3710f33e14 Don't reject messages that don't pass RBF, ignore them
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
2020-08-09 03:17:40 +02:00
Jakub Sztandera 5ea9df1824 Use 25% overesitm
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
2020-08-09 02:22:37 +02:00
Jakub Sztandera a826877ee3 Reduce overesitm to 10%, remove pledgeSectors test rate limit
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
2020-08-09 00:37:24 +02:00
Jakub Sztandera 46fd9d5e80 Quiet down pending messages, fix GasLimit estimation, reduce fee cap
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
2020-08-09 00:29:59 +02:00
vyzo bcb33d75d0 Merge pull request #2918 from filecoin-project/feat/mpool-tests
Some more mpool tests
2020-08-08 22:03:07 +03:00
Jakub Sztandera 5f157affdd Add GasLimitOverestimation to mpool config
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
2020-08-08 17:09:47 +02:00
vyzo 9bc5d9489d also log chain creation timings 2020-08-08 17:07:37 +03:00
vyzo b469277046 use structured logging for timings 2020-08-08 16:59:08 +03:00
vyzo 809e7d5390 add timings to various components of message selection 2020-08-08 16:51:30 +03:00
vyzo 4cdb5f7b81 add unit test for priority message selection 2020-08-08 11:22:47 +03:00
vyzo 2d9f2c7e72 add unit test for message skipping in chain creation 2020-08-08 11:16:50 +03:00
Łukasz Magiera e65db33d77 Merge pull request #2909 from filecoin-project/fix/mpool-bug
fix mpool bugs
2020-08-08 01:32:28 +02:00
vyzo 4c8c8f48f1 skip messages with nonce before the current nonce in chain creation 2020-08-08 02:16:56 +03:00
vyzo 01b8a6de94 remove bug line 2020-08-08 01:45:34 +03:00
Łukasz Magiera af15273f71 gas: Corruntly do float math 2020-08-08 00:41:57 +02:00
vyzo 7e16e3e220 satisfy linter 2020-08-07 20:20:22 +03:00
vyzo f6e8f02fd2 add persistence for mpool config 2020-08-07 20:10:09 +03:00
vyzo 1b56b88132 add cooldown to message pruning 2020-08-07 19:51:49 +03:00
vyzo c735d0da0e fix test compilation issue 2020-08-07 19:51:49 +03:00
vyzo b40346e1ea never prune priority actor messages 2020-08-07 19:51:49 +03:00
vyzo 56c2894c80 add priority actor message selection 2020-08-07 19:51:49 +03:00
vyzo 42e1338ffe mpool config 2020-08-07 19:51:49 +03:00
Łukasz Magiera 60545e6722 mempool: Allow 0 gasPerf chains 2020-08-07 14:43:32 +02:00
Łukasz Magiera 5a50d293b9 Merge pull request #2891 from filecoin-project/feat/mpool-negative-chain-pruning
some small improvements in the handling of non-performing chains
2020-08-07 10:14:09 +02:00
vyzo b31b322e77 also check for 0 in early gasPerf check 2020-08-07 10:55:43 +03:00
vyzo 26f025bc96 some small improvements in the handling of non-performing chains
1. We break the message accumulation the first time we see a chain with gasPerf <= 0
   as it is not profitable to include them in the block
2. When trimming, we remove also trim negative performing messages to make the trimmed
   chain profitable.
2020-08-07 10:33:00 +03:00
vyzo e977649f29 add regression test for selection trimming 2020-08-07 10:10:54 +03:00
Łukasz Magiera 0612c2eab9 Merge remote-tracking branch 'origin/next' into feat/dyn-base-fee 2020-08-07 04:15:42 +02:00
Jakub Sztandera 884d03f7f1 address feedback
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
2020-08-07 04:09:01 +02:00
Jakub Sztandera 05c527b681 Hold tipset lock while selecting messages
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
2020-08-07 02:48:53 +02:00
Łukasz Magiera 402b4b6c70 mpool: Fix SelectMessages hanging 2020-08-07 01:01:45 +02:00
Jakub Sztandera f45970a0d5 Fix FeeCap estimation
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
2020-08-07 00:54:58 +02:00
Jakub Sztandera 2b2b632cd6 Make mpool select only profitable messages
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
2020-08-07 00:40:25 +02:00
Jakub Sztandera 94a38cd97d Fix mpool test
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
2020-08-06 23:14:09 +02:00
Jakub Sztandera ab08858b45 Delete GasPrice from this world
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
2020-08-06 23:08:42 +02:00
Jakub Sztandera d6765847df Fix some messagepool tests
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
2020-08-06 21:25:30 +02:00
vyzo 2166e9a0ff account for message Value in balance calculation 2020-08-06 20:54:27 +03:00
vyzo e87d8f9543 add basic selection tests 2020-08-06 20:15:15 +03:00
vyzo 1e80e3ae65 create maximal chains
Merge chains when gasPerf is non-descreasing (instead of increasing) to create maximal chains.
This also obviates the need for stable sort, as the chains are guaranteed to be in decreasing
order for the same actor.
2020-08-06 18:37:28 +03:00
vyzo 7a8c74fee8 use stable sort for chain sorting
the default non-stable sort may reorder chains with the same gasPerf, which might change the order
of messages per nonce; facepalm.
2020-08-06 17:40:22 +03:00
vyzo 16a1f9b682 message chain creation tests 2020-08-06 17:08:02 +03:00
vyzo 61d027db2a use supplied tipset when looking up actor state 2020-08-06 17:07:37 +03:00
vyzo 30ec4e7524 import cosmetics 2020-08-06 11:17:11 +03:00
vyzo a08d780bef enhance SelectMessages method to take a target message 2020-08-05 23:16:09 +03:00
vyzo 5f599dcaf5 remove duplicate time log 2020-08-05 21:48:29 +03:00
vyzo 829c7953d5 fix dereference bug 2020-08-05 18:28:04 +03:00
vyzo a8b0db91e8 implement pruning using the chain selection logic 2020-08-05 13:28:14 +03:00
vyzo b4be1472c0 use RequiredFunds in chain inclusion balance sanity check 2020-08-05 13:08:07 +03:00
vyzo 06a13681ae satisfy stupid linter 2020-08-05 10:43:49 +03:00