Do not remove inodes from the lfru list until the inode is destroyed

vitalif · vitalif · commit 9ca5fc5e0002 · 2022-03-15T23:17:34.000+03:00
Previously, when inodes were removed from lfru on resetCache, some
buffers could be leaked and lead to OOM
diff --git a/internal/file.go b/internal/file.go
@@ -1584,8 +1584,6 @@ func (inode *Inode) resetCache() {
 	inode.SetCacheState(ST_CACHED)
 	// Invalidate metadata entry
 	inode.AttrTime = time.Time{}
-	// Remove from LFRU tracker
-	inode.fs.lfru.Forget(inode.Id)
 }
 
 func (inode *Inode) FlushSmallObject() {

Original file line number	Diff line number	Diff line change
`@@ -1584,8 +1584,6 @@ func (inode *Inode) resetCache() {`
`1584`	`1584`	`inode.SetCacheState(ST_CACHED)`
`1585`	`1585`	`// Invalidate metadata entry`
`1586`	`1586`	`inode.AttrTime = time.Time{}`
`1587`		`- // Remove from LFRU tracker`
`1588`		`- inode.fs.lfru.Forget(inode.Id)`
`1589`	`1587`	`}`
`1590`	`1588`
`1591`	`1589`	`func (inode *Inode) FlushSmallObject() {`