Re: [PATCH] Btrfs: fix btrfs_evict_inode to handle abnormal inodes correctly

2018-01-26 Thread Josef Bacik
On Thu, Jan 25, 2018 at 11:02:53AM -0700, Liu Bo wrote: > This regression is introduced in > commit 3d48d9810de4 ("btrfs: Handle uninitialised inode eviction"). > > There are two problems, > > a) it is ->destroy_inode() that does the final free on inode, not >->evict_inode(), > b) clear_inode

Re: [PATCH] Btrfs: fix btrfs_evict_inode to handle abnormal inodes correctly

2018-01-26 Thread Nikolay Borisov
On 25.01.2018 20:02, Liu Bo wrote: > This regression is introduced in > commit 3d48d9810de4 ("btrfs: Handle uninitialised inode eviction"). > > There are two problems, > > a) it is ->destroy_inode() that does the final free on inode, not >->evict_inode(), > b) clear_inode() must be called b

[PATCH] Btrfs: fix btrfs_evict_inode to handle abnormal inodes correctly

2018-01-25 Thread Liu Bo
This regression is introduced in commit 3d48d9810de4 ("btrfs: Handle uninitialised inode eviction"). There are two problems, a) it is ->destroy_inode() that does the final free on inode, not ->evict_inode(), b) clear_inode() must be called before ->evict_inode() returns. This could end up hit