Commits · 68598d2ea886322f9b4b0058e5b288418622de95 · Kirill Smelkov / linux

07 Mar, 2017 1 commit

btrfs: remove btrfs_err_str function from uapi/linux/btrfs.h · 68598d2e

Dmitry V. Levin authored Mar 01, 2017

btrfs_err_str function is not called from anywhere and is replicated
in the userspace headers for btrfs-progs.

It's removal also fixes the following linux/btrfs.h userspace
compilation error:

/usr/include/linux/btrfs.h: In function 'btrfs_err_str':
/usr/include/linux/btrfs.h:740:11: error: 'NULL' undeclared (first use in this function)
    return NULL;
Suggested-by: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: Dmitry V. Levin <ldv@altlinux.org>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

68598d2e

28 Feb, 2017 39 commits

Merge branch 'for-chris-4.11-part2' of... · e9f467d0

Chris Mason authored Feb 28, 2017

Merge branch 'for-chris-4.11-part2' of git://git.kernel.org/pub/scm/linux/kernel/git/kdave/linux into for-linus-4.11

e9f467d0

btrfs: add dummy callback for readpage_io_failed and drop checks · 20a7db8a

David Sterba authored Feb 17, 2017

Make extent_io_ops::readpage_io_failed_hook callback mandatory and
define a dummy function for btrfs_extent_io_ops. As the failed IO
callback is not performance critical, the branch vs extra trade off does
not hurt.
Signed-off-by: David Sterba <dsterba@suse.com>

20a7db8a

btrfs: drop checks for mandatory extent_io_ops callbacks · 20c9801d

David Sterba authored Feb 17, 2017

We know that eadpage_end_io_hook, submit_bio_hook and merge_bio_hook are
always defined so we can drop the checks before we call them.
Signed-off-by: David Sterba <dsterba@suse.com>

20c9801d

btrfs: document existence of extent_io ops callbacks · 4d53dddb

David Sterba authored Feb 17, 2017

Some of the callbacks defined in btree_extent_io_ops and
btrfs_extent_io_ops do always exist so we don't need to check the
existence before each call. This patch just reorders the definition and
documents which are mandatory/optional.
Signed-off-by: David Sterba <dsterba@suse.com>

4d53dddb

btrfs: let writepage_end_io_hook return void · c3988d63

David Sterba authored Feb 17, 2017

There's no error path in any of the instances, always return 0.
Reviewed-by: Liu Bo <bo.li.liu@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.com>

c3988d63

btrfs: do proper error handling in btrfs_insert_xattr_item · b9d04c60

David Sterba authored Feb 17, 2017

The space check in btrfs_insert_xattr_item is duplicated in it's caller
(do_setxattr) so we won't hit the BUG_ON. Continuing without any check
could be disasterous so turn it to a proper error handling.
Reviewed-by: Liu Bo <bo.li.liu@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.com>

b9d04c60

btrfs: handle allocation error in update_dev_stat_item · fa252992
David Sterba authored Feb 15, 2017
```
Reviewed-by: Liu Bo <bo.li.liu@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
fa252992

btrfs: remove BUG_ON from __tree_mod_log_insert · 047e5e17

David Sterba authored Feb 14, 2017

All callers dereference the 'tm' parameter before it gets to this
function, the NULL check does not make much sense here.
Reviewed-by: Liu Bo <bo.li.liu@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.com>

047e5e17

btrfs: derive maximum output size in the compression implementation · e5d74902

David Sterba authored Feb 14, 2017

The value of max_out can be calculated from the parameters passed to the
compressors, which is number of pages and the page size, and we don't
have to needlessly pass it around.
Signed-off-by: David Sterba <dsterba@suse.com>

e5d74902

btrfs: use predefined limits for calculating maximum number of pages for compression · 069eac78
David Sterba authored Feb 14, 2017
```
Signed-off-by: David Sterba <dsterba@suse.com>
```
069eac78

btrfs: export compression buffer limits in a header · ff763866

David Sterba authored Feb 14, 2017

Move the buffer limit definitions out of compress_file_range.
Reviewed-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
Signed-off-by: David Sterba <dsterba@suse.com>

ff763866

btrfs: merge nr_pages input and output parameter in compress_pages · 4d3a800e

David Sterba authored Feb 14, 2017

The parameter saying how many pages can be allocated at maximum can be
merged with the output page counter, to save some stack space.  The
compression implementation will sink the parameter to a local variable
so everything works as before.

The nr_pages variables can also be simply merged in compress_file_range
into one.
Signed-off-by: David Sterba <dsterba@suse.com>

4d3a800e

btrfs: merge length input and output parameter in compress_pages · 38c31464

David Sterba authored Feb 14, 2017

The length parameter is basically duplicated for input and output in the
top level caller of the compress_pages chain. We can simply use one
variable for that and reduce stack consumption. The compression
implementation will sink the parameter to a local variable so everything
works as before.
Signed-off-by: David Sterba <dsterba@suse.com>

38c31464

btrfs: constify name of subvolume in creation helpers · 52f75f4f
David Sterba authored Feb 14, 2017
```
Signed-off-by: David Sterba <dsterba@suse.com>
```
52f75f4f
btrfs: constify buffers used by compression helpers · 14a3357b
David Sterba authored Feb 14, 2017
```
Signed-off-by: David Sterba <dsterba@suse.com>
```
14a3357b

btrfs: constify input buffer of btrfs_csum_data · 9ed57367

David Sterba authored Feb 14, 2017

The function does not modify the input buffer, also update a typecast in
one caller.
Signed-off-by: David Sterba <dsterba@suse.com>

9ed57367

btrfs: constify device path passed to relevant helpers · da353f6b
David Sterba authored Feb 14, 2017
```
Signed-off-by: David Sterba <dsterba@suse.com>
```
da353f6b
btrfs: make btrfs_inode_resume_unlocked_dio take btrfs_inode · 0b581701
Nikolay Borisov authored Feb 20, 2017
```
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
0b581701
btrfs: make btrfs_inode_block_unlocked_dio take btrfs_inode · abcefb1e
Nikolay Borisov authored Feb 20, 2017
```
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
abcefb1e

btrfs: Make btrfs_add_nondir take btrfs_inode · cef415af

Nikolay Borisov authored Feb 20, 2017

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

cef415af

btrfs: Make btrfs_add_link take btrfs_inode · db0a669f

Nikolay Borisov authored Feb 20, 2017

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

db0a669f

btrfs: Make btrfs_del_delalloc_inode take btrfs_inode · 9e3e97f4
Nikolay Borisov authored Feb 20, 2017
```
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
9e3e97f4

btrfs: Make get_extent_t take btrfs_inode · fc4f21b1

Nikolay Borisov authored Feb 20, 2017

In addition to changing the signature, this patch also switches
all the functions which are used as an argument to also take btrfs_inode.
Namely those are: btrfs_get_extent and btrfs_get_extent_filemap.
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

fc4f21b1

btrfs: Make check_extent_to_block take btrfs_inode · 1c8c9c52
Nikolay Borisov authored Feb 20, 2017
```
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
1c8c9c52
btrfs: Make clone_update_extent_map take btrfs_inode · a2f392e4
Nikolay Borisov authored Feb 20, 2017
```
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
a2f392e4
btrfs: Make btrfs_clear_bit_hook take btrfs_inode · 6fc0ef68
Nikolay Borisov authored Feb 20, 2017
```
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
6fc0ef68
btrfs: Make btrfs_extent_item_to_extent_map take btrfs_inode · 9cdc5124
Nikolay Borisov authored Feb 20, 2017
```
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
9cdc5124
btrfs: make btrfs_log_inode_parent take btrfs_inode · 19df27a9
Nikolay Borisov authored Feb 20, 2017
```
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
19df27a9
btrfs: Make check_parent_dirs_for_sync take btrfs_inode · aefa6115
Nikolay Borisov authored Feb 20, 2017
```
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
aefa6115

btrfs: Make btrfs_orphan_add take btrfs_inode · 73f2e545

Nikolay Borisov authored Feb 20, 2017

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

73f2e545

btrfs: make btrfs_orphan_del take btrfs_inode · 3d6ae7bb

Nikolay Borisov authored Feb 20, 2017

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

3d6ae7bb

btrfs: make btrfs_free_io_failure_record take btrfs_inode · 7ab7956e
Nikolay Borisov authored Feb 20, 2017
```
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
7ab7956e

btrfs: make clean_io_failure take btrfs_inode · b30cb441

Nikolay Borisov authored Feb 20, 2017

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

b30cb441

btrfs: make repair_io_failure take btrfs_inode · 9d4f7f8a

Nikolay Borisov authored Feb 20, 2017

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

9d4f7f8a

btrfs: make check_compressed_csum take btrfs_inode · f898ac6a
Nikolay Borisov authored Feb 20, 2017
```
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
f898ac6a
btrfs: make btrfs_print_data_csum_error take btrfs_inode · 0970a22e
Nikolay Borisov authored Feb 20, 2017
```
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
0970a22e

btrfs: make free_io_failure take btrfs_inode · 4ac1f4ac

Nikolay Borisov authored Feb 20, 2017

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

4ac1f4ac

btrfs: Make lock_and_cleanup_extent_if_need take btrfs_inode · 2cff578c
Nikolay Borisov authored Feb 20, 2017
```
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
```
2cff578c

btrfs: Make check_can_nocow take btrfs_inode · 85b7ab67

Nikolay Borisov authored Feb 20, 2017

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

85b7ab67