diff options
| author | David Sterba <dsterba@suse.com> | 2019-10-01 19:57:39 +0200 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2019-11-18 12:46:52 +0100 |
| commit | e1f60a6580c04d0d2492bb6034e968b8c29c78cf (patch) | |
| tree | 56ef88afaeba63621dafb6b83d8cef6760fd9f35 /fs/btrfs/dev-replace.c | |
| parent | 4143cb8b6f00910e73a7503fd922211b9f08cf48 (diff) | |
| download | lwn-e1f60a6580c04d0d2492bb6034e968b8c29c78cf.tar.gz lwn-e1f60a6580c04d0d2492bb6034e968b8c29c78cf.zip | |
btrfs: add __pure attribute to functions
The attribute is more relaxed than const and the functions could
dereference pointers, as long as the observable state is not changed. We
do have such functions, based on -Wsuggest-attribute=pure .
The visible effects of this patch are negligible, there are differences
in the assembly but hard to summarize.
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/dev-replace.c')
| -rw-r--r-- | fs/btrfs/dev-replace.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/dev-replace.c b/fs/btrfs/dev-replace.c index 48890826b5e6..f639dde2a679 100644 --- a/fs/btrfs/dev-replace.c +++ b/fs/btrfs/dev-replace.c @@ -986,7 +986,7 @@ static int btrfs_dev_replace_kthread(void *data) return 0; } -int btrfs_dev_replace_is_ongoing(struct btrfs_dev_replace *dev_replace) +int __pure btrfs_dev_replace_is_ongoing(struct btrfs_dev_replace *dev_replace) { if (!dev_replace->is_valid) return 0; |