On 16.12.20 г. 18:22 ч., Josef Bacik wrote: > While testing other things I was noticing that sometimes my VM would > fail to load the btrfs module because the self test failed like this > > BTRFS: selftest: fs/btrfs/tests/inode-tests.c:963 miscount, wanted 1, got 0 > > This turned out to be because sometimes the btrfs ino would be the btree > inode number, and thus we'd skip calling the set extent delalloc bit > helper, and thus not adjust ->outstanding_extents. Fix this by making > sure we init test inodes with a valid inode number so that we don't get > random failures during self tests. This warrants slightly more explanation why this initialization is required, namely that newly allocated indoes are initialized by passing a set callback, since we are acquiring inodes for tests we need to simulate this behavior by, effectively, open coding btrfs_init_locked_inode. <snip>
- [PATCH 00/13] Serious fixes for different error paths Josef Bacik
- [PATCH 01/13] btrfs: don't get an EINTR during drop_s... Josef Bacik
- Re: [PATCH 01/13] btrfs: don't get an EINTR durin... David Sterba
- [PATCH 02/13] btrfs: initialize test inodes location Josef Bacik
- Re: [PATCH 02/13] btrfs: initialize test inodes l... Nikolay Borisov
- Re: [PATCH 02/13] btrfs: initialize test inod... David Sterba
- [PATCH 03/13] btrfs: fix reloc root leak with 0 ref r... Josef Bacik
- [PATCH 11/13] btrfs: keep track of the root owner for... Josef Bacik
- Re: [PATCH 11/13] btrfs: keep track of the root o... David Sterba
- Re: [PATCH 11/13] btrfs: keep track of the ro... David Sterba
- [PATCH 08/13] btrfs: abort the transaction if we fail... Josef Bacik
- Re: [PATCH 08/13] btrfs: abort the transaction if... David Sterba
- [PATCH 06/13] btrfs: add ASSERT()'s for deleting back... Josef Bacik
- Re: [PATCH 06/13] btrfs: add ASSERT()'s for delet... David Sterba
- [PATCH 07/13] btrfs: do not double free backref nodes... Josef Bacik