| Message ID | 20210119122649.187778-2-nborisov@suse.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Make btrfs W=1 clean | expand |
On 1/19/21 7:26 AM, Nikolay Borisov wrote: > Fixes fs/btrfs/extent_map.c:399: warning: Function parameter or member > 'modified' not described in 'add_extent_mapping' > > Signed-off-by: Nikolay Borisov <nborisov@suse.com> Subject should be 'parameter'. Thanks, Josef
On 1/19/21 7:26 AM, Nikolay Borisov wrote: > Fixes fs/btrfs/extent_map.c:399: warning: Function parameter or member > 'modified' not described in 'add_extent_mapping' > > Signed-off-by: Nikolay Borisov <nborisov@suse.com> > --- > fs/btrfs/extent_map.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/fs/btrfs/extent_map.c b/fs/btrfs/extent_map.c > index bd6229fb2b6f..aa1ff6f2ade9 100644 > --- a/fs/btrfs/extent_map.c > +++ b/fs/btrfs/extent_map.c > @@ -388,6 +388,8 @@ static void extent_map_device_clear_bits(struct extent_map *em, unsigned bits) > * add_extent_mapping - add new extent map to the extent tree > * @tree: tree to insert new map in > * @em: map to insert > + * @modified: bool indicating whether the given @em should be added to the > + * modified list, which indicates the extent needs to be logged > * > * Insert @em into @tree or perform a simple forward/backward merge with > * existing mappings. The extent_map struct passed in will be inserted > You can add Reviewed-by: Josef Bacik <josef@toxicpanda.com> to the series once you fix up those two spelling mistakes. Thanks, Josef
diff --git a/fs/btrfs/extent_map.c b/fs/btrfs/extent_map.c index bd6229fb2b6f..aa1ff6f2ade9 100644 --- a/fs/btrfs/extent_map.c +++ b/fs/btrfs/extent_map.c @@ -388,6 +388,8 @@ static void extent_map_device_clear_bits(struct extent_map *em, unsigned bits) * add_extent_mapping - add new extent map to the extent tree * @tree: tree to insert new map in * @em: map to insert + * @modified: bool indicating whether the given @em should be added to the + * modified list, which indicates the extent needs to be logged * * Insert @em into @tree or perform a simple forward/backward merge with * existing mappings. The extent_map struct passed in will be inserted
Fixes fs/btrfs/extent_map.c:399: warning: Function parameter or member 'modified' not described in 'add_extent_mapping' Signed-off-by: Nikolay Borisov <nborisov@suse.com> --- fs/btrfs/extent_map.c | 2 ++ 1 file changed, 2 insertions(+)