mbox series

[0/5] More lockdep annotations

Message ID cover.1559321947.git.dsterba@suse.com (mailing list archive)
Headers show
Series More lockdep annotations | expand

Message

David Sterba May 31, 2019, 5:01 p.m. UTC 
Lockdep annotations are better than comments about necessary locks.

David Sterba (5):
  btrfs: tests: add locks around add_extent_mapping
  btrfs: assert extent map tree lock in add_extent_mapping
  btrfs: assert tree mod log lock in __tree_mod_log_insert
  btrfs: assert delayed ref lock in btrfs_find_delayed_ref_head
  btrfs: assert bio list lock in merge_rbio

 fs/btrfs/ctree.c                  |  4 ++--
 fs/btrfs/delayed-ref.c            |  7 ++++---
 fs/btrfs/extent_map.c             |  2 ++
 fs/btrfs/raid56.c                 |  4 ++--
 fs/btrfs/tests/extent-map-tests.c | 22 ++++++++++++++++++++++
 5 files changed, 32 insertions(+), 7 deletions(-)

Comments

Nikolay Borisov June 5, 2019, 7:34 a.m. UTC | #1 
On 31.05.19 г. 20:01 ч., David Sterba wrote:
> Lockdep annotations are better than comments about necessary locks.
> 
> David Sterba (5):
>   btrfs: tests: add locks around add_extent_mapping
>   btrfs: assert extent map tree lock in add_extent_mapping
>   btrfs: assert tree mod log lock in __tree_mod_log_insert
>   btrfs: assert delayed ref lock in btrfs_find_delayed_ref_head
>   btrfs: assert bio list lock in merge_rbio
> 
>  fs/btrfs/ctree.c                  |  4 ++--
>  fs/btrfs/delayed-ref.c            |  7 ++++---
>  fs/btrfs/extent_map.c             |  2 ++
>  fs/btrfs/raid56.c                 |  4 ++--
>  fs/btrfs/tests/extent-map-tests.c | 22 ++++++++++++++++++++++
>  5 files changed, 32 insertions(+), 7 deletions(-)
> 


For the whole series :

Reviewed-by: Nikolay Borisov <nborisov@suse.com>
David Sterba June 7, 2019, 1:31 p.m. UTC | #2 
On Wed, Jun 05, 2019 at 10:34:24AM +0300, Nikolay Borisov wrote:
> 
> 
> On 31.05.19 г. 20:01 ч., David Sterba wrote:
> > Lockdep annotations are better than comments about necessary locks.
> > 
> > David Sterba (5):
> >   btrfs: tests: add locks around add_extent_mapping
> >   btrfs: assert extent map tree lock in add_extent_mapping
> >   btrfs: assert tree mod log lock in __tree_mod_log_insert
> >   btrfs: assert delayed ref lock in btrfs_find_delayed_ref_head
> >   btrfs: assert bio list lock in merge_rbio
> > 
> >  fs/btrfs/ctree.c                  |  4 ++--
> >  fs/btrfs/delayed-ref.c            |  7 ++++---
> >  fs/btrfs/extent_map.c             |  2 ++
> >  fs/btrfs/raid56.c                 |  4 ++--
> >  fs/btrfs/tests/extent-map-tests.c | 22 ++++++++++++++++++++++
> >  5 files changed, 32 insertions(+), 7 deletions(-)
> > 
> 
> 
> For the whole series :
> 
> Reviewed-by: Nikolay Borisov <nborisov@suse.com>

Thanks, series moved to misc-next.