search for: __btrfs_unlink_inode

...code inspection implied that this would be possible in unlucky error cases. I finally got around to really reading up and reproducing it. diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index ecd9c4c..11fc4b1 100644 --- a/fs/btrfs/inode.c +++ b/fs/btrfs/inode.c @@ -3592,8 +3592,11 @@ static int __btrfs_unlink_inode(struct btrfs_trans_handle *trans, goto err; } - ret = btrfs_del_dir_entries_in_log(trans, root, name, name_len, - dir, index); + if (!strncmp(name, "fail-please", min(name_len, 11))) + ret = -ENOM...

...fo ffff8805d5b42000, task ffff8805b4acec00) Stack: 0000000000000085 ffff880bf0984870 000000000033bef7 0000000000008560 0000000000000000 00000000000007bf ffff880bf0984870 00000000000009d9 0000000000000000 ffff880bf560b1b0 ffff8805c29a5e88 000000000033bef7 Call Trace: [<ffffffffa03210e5>] ? __btrfs_unlink_inode+0x172/0x25e [btrfs] [<ffffffffa032158c>] ? btrfs_rename+0x38b/0x55b [btrfs] [<ffffffff81124e8c>] ? vfs_rename+0x202/0x382 [<ffffffff81127312>] ? sys_renameat+0x169/0x1e5 [<ffffffff81051164>] ? do_page_fault+0x39b/0x3af [<ffffffff8112c35e>] ? dput+0x3f/0x126 [<...

Hi gang, Here''s some trivial cleanups that I''ve built up while reading through the code. They''ve been run through xfstests -g quick. - z -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

...pos; ins.type = BTRFS_EXTENT_DATA_KEY; - ret = btrfs_insert_empty_item(trans, root, path, &ins, sizeof(*fi)); + ret = btrfs_insert_empty_item_for_inode(trans, root, inode, path, + &ins, sizeof(*fi)); if (ret) goto out; leaf = path->nodes[0]; @@ -2780,8 +2781,8 @@ static int __btrfs_unlink_inode(struct btrfs_trans_handle *trans, } path->leave_spinning = 1; - di = btrfs_lookup_dir_item(trans, root, path, dir_ino, - name, name_len, -1); + di = btrfs_lookup_dir_item(trans, root, dir, path, dir_ino, + name, name_len, -1); if (IS_ERR(di)) { ret = PTR_ERR(di); goto...