trie: fix size accounting in cleaner (ethereum#25007)

aaronbuchwald · qinglin89 · commit fafc29f92201 · 2022-10-21T12:42:18.000+08:00
Decrease children size instead of dirties size when marking dirties as cleaned up in trie cleaner
diff --git a/trie/database.go b/trie/database.go
@@ -838,7 +838,7 @@ func (c *cleaner) Put(key []byte, rlp []byte) error {
 	delete(c.db.dirties, hash)
 	c.db.dirtiesSize -= common.StorageSize(common.HashLength + int(node.size))
 	if node.children != nil {
-		c.db.dirtiesSize -= common.StorageSize(cachedNodeChildrenSize + len(node.children)*(common.HashLength+2))
+		c.db.childrenSize -= common.StorageSize(cachedNodeChildrenSize + len(node.children)*(common.HashLength+2))
 	}
 	// Move the flushed node into the clean cache to prevent insta-reloads
 	if c.db.cleans != nil {

Original file line number	Diff line number	Diff line change
`@@ -838,7 +838,7 @@ func (c *cleaner) Put(key []byte, rlp []byte) error {`
`838`	`838`	`delete(c.db.dirties, hash)`
`839`	`839`	`c.db.dirtiesSize -= common.StorageSize(common.HashLength + int(node.size))`
`840`	`840`	`if node.children != nil {`
`841`		`- c.db.dirtiesSize -= common.StorageSize(cachedNodeChildrenSize + len(node.children)*(common.HashLength+2))`
	`841`	`+ c.db.childrenSize -= common.StorageSize(cachedNodeChildrenSize + len(node.children)*(common.HashLength+2))`
`842`	`842`	`}`
`843`	`843`	`// Move the flushed node into the clean cache to prevent insta-reloads`
`844`	`844`	`if c.db.cleans != nil {`