Skip to content

Commit 631ebed

Browse files
josefbacikthangqn-ampere
josefbacik
authored and
thangqn-ampere
committed
btrfs: move lockdep class helpers to locking.c
[ Upstream commit 0a27a04 ] These definitions exist in disk-io.c, which is not related to the locking. Move this over to locking.h/c where it makes more sense. Reviewed-by: Johannes Thumshirn <[email protected]> Signed-off-by: Josef Bacik <[email protected]> Reviewed-by: David Sterba <[email protected]> Signed-off-by: David Sterba <[email protected]> Signed-off-by: Sasha Levin <[email protected]>
1 parent 1f45152 commit 631ebed

File tree

4 files changed

+89
-92
lines changed

4 files changed

+89
-92
lines changed

‎fs/btrfs/disk-io.c‎

Lines changed: 0 additions & 82 deletions
Original file line numberDiff line numberDiff line change
@@ -121,88 +121,6 @@ struct async_submit_bio {
121121
blk_status_t status;
122122
};
123123

124-
/*
125-
* Lockdep class keys for extent_buffer->lock's in this root. For a given
126-
* eb, the lockdep key is determined by the btrfs_root it belongs to and
127-
* the level the eb occupies in the tree.
128-
*
129-
* Different roots are used for different purposes and may nest inside each
130-
* other and they require separate keysets. As lockdep keys should be
131-
* static, assign keysets according to the purpose of the root as indicated
132-
* by btrfs_root->root_key.objectid. This ensures that all special purpose
133-
* roots have separate keysets.
134-
*
135-
* Lock-nesting across peer nodes is always done with the immediate parent
136-
* node locked thus preventing deadlock. As lockdep doesn't know this, use
137-
* subclass to avoid triggering lockdep warning in such cases.
138-
*
139-
* The key is set by the readpage_end_io_hook after the buffer has passed
140-
* csum validation but before the pages are unlocked. It is also set by
141-
* btrfs_init_new_buffer on freshly allocated blocks.
142-
*
143-
* We also add a check to make sure the highest level of the tree is the
144-
* same as our lockdep setup here. If BTRFS_MAX_LEVEL changes, this code
145-
* needs update as well.
146-
*/
147-
#ifdef CONFIG_DEBUG_LOCK_ALLOC
148-
# if BTRFS_MAX_LEVEL != 8
149-
# error
150-
# endif
151-
152-
#define DEFINE_LEVEL(stem, level) \
153-
.names[level] = "btrfs-" stem "-0" #level,
154-
155-
#define DEFINE_NAME(stem) \
156-
DEFINE_LEVEL(stem, 0) \
157-
DEFINE_LEVEL(stem, 1) \
158-
DEFINE_LEVEL(stem, 2) \
159-
DEFINE_LEVEL(stem, 3) \
160-
DEFINE_LEVEL(stem, 4) \
161-
DEFINE_LEVEL(stem, 5) \
162-
DEFINE_LEVEL(stem, 6) \
163-
DEFINE_LEVEL(stem, 7)
164-
165-
static struct btrfs_lockdep_keyset {
166-
u64 id; /* root objectid */
167-
/* Longest entry: btrfs-free-space-00 */
168-
char names[BTRFS_MAX_LEVEL][20];
169-
struct lock_class_key keys[BTRFS_MAX_LEVEL];
170-
} btrfs_lockdep_keysets[] = {
171-
{ .id = BTRFS_ROOT_TREE_OBJECTID, DEFINE_NAME("root") },
172-
{ .id = BTRFS_EXTENT_TREE_OBJECTID, DEFINE_NAME("extent") },
173-
{ .id = BTRFS_CHUNK_TREE_OBJECTID, DEFINE_NAME("chunk") },
174-
{ .id = BTRFS_DEV_TREE_OBJECTID, DEFINE_NAME("dev") },
175-
{ .id = BTRFS_CSUM_TREE_OBJECTID, DEFINE_NAME("csum") },
176-
{ .id = BTRFS_QUOTA_TREE_OBJECTID, DEFINE_NAME("quota") },
177-
{ .id = BTRFS_TREE_LOG_OBJECTID, DEFINE_NAME("log") },
178-
{ .id = BTRFS_TREE_RELOC_OBJECTID, DEFINE_NAME("treloc") },
179-
{ .id = BTRFS_DATA_RELOC_TREE_OBJECTID, DEFINE_NAME("dreloc") },
180-
{ .id = BTRFS_UUID_TREE_OBJECTID, DEFINE_NAME("uuid") },
181-
{ .id = BTRFS_FREE_SPACE_TREE_OBJECTID, DEFINE_NAME("free-space") },
182-
{ .id = 0, DEFINE_NAME("tree") },
183-
};
184-
185-
#undef DEFINE_LEVEL
186-
#undef DEFINE_NAME
187-
188-
void btrfs_set_buffer_lockdep_class(u64 objectid, struct extent_buffer *eb,
189-
int level)
190-
{
191-
struct btrfs_lockdep_keyset *ks;
192-
193-
BUG_ON(level >= ARRAY_SIZE(ks->keys));
194-
195-
/* find the matching keyset, id 0 is the default entry */
196-
for (ks = btrfs_lockdep_keysets; ks->id; ks++)
197-
if (ks->id == objectid)
198-
break;
199-
200-
lockdep_set_class_and_name(&eb->lock,
201-
&ks->keys[level], ks->names[level]);
202-
}
203-
204-
#endif
205-
206124
/*
207125
* Compute the csum of a btree block and store the result to provided buffer.
208126
*/

‎fs/btrfs/disk-io.h‎

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -140,14 +140,4 @@ int btrfs_init_root_free_objectid(struct btrfs_root *root);
140140
int __init btrfs_end_io_wq_init(void);
141141
void __cold btrfs_end_io_wq_exit(void);
142142

143-
#ifdef CONFIG_DEBUG_LOCK_ALLOC
144-
void btrfs_set_buffer_lockdep_class(u64 objectid,
145-
struct extent_buffer *eb, int level);
146-
#else
147-
static inline void btrfs_set_buffer_lockdep_class(u64 objectid,
148-
struct extent_buffer *eb, int level)
149-
{
150-
}
151-
#endif
152-
153143
#endif

‎fs/btrfs/locking.c‎

Lines changed: 80 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,86 @@
1313
#include "extent_io.h"
1414
#include "locking.h"
1515

16+
/*
17+
* Lockdep class keys for extent_buffer->lock's in this root. For a given
18+
* eb, the lockdep key is determined by the btrfs_root it belongs to and
19+
* the level the eb occupies in the tree.
20+
*
21+
* Different roots are used for different purposes and may nest inside each
22+
* other and they require separate keysets. As lockdep keys should be
23+
* static, assign keysets according to the purpose of the root as indicated
24+
* by btrfs_root->root_key.objectid. This ensures that all special purpose
25+
* roots have separate keysets.
26+
*
27+
* Lock-nesting across peer nodes is always done with the immediate parent
28+
* node locked thus preventing deadlock. As lockdep doesn't know this, use
29+
* subclass to avoid triggering lockdep warning in such cases.
30+
*
31+
* The key is set by the readpage_end_io_hook after the buffer has passed
32+
* csum validation but before the pages are unlocked. It is also set by
33+
* btrfs_init_new_buffer on freshly allocated blocks.
34+
*
35+
* We also add a check to make sure the highest level of the tree is the
36+
* same as our lockdep setup here. If BTRFS_MAX_LEVEL changes, this code
37+
* needs update as well.
38+
*/
39+
#ifdef CONFIG_DEBUG_LOCK_ALLOC
40+
#if BTRFS_MAX_LEVEL != 8
41+
#error
42+
#endif
43+
44+
#define DEFINE_LEVEL(stem, level) \
45+
.names[level] = "btrfs-" stem "-0" #level,
46+
47+
#define DEFINE_NAME(stem) \
48+
DEFINE_LEVEL(stem, 0) \
49+
DEFINE_LEVEL(stem, 1) \
50+
DEFINE_LEVEL(stem, 2) \
51+
DEFINE_LEVEL(stem, 3) \
52+
DEFINE_LEVEL(stem, 4) \
53+
DEFINE_LEVEL(stem, 5) \
54+
DEFINE_LEVEL(stem, 6) \
55+
DEFINE_LEVEL(stem, 7)
56+
57+
static struct btrfs_lockdep_keyset {
58+
u64 id; /* root objectid */
59+
/* Longest entry: btrfs-free-space-00 */
60+
char names[BTRFS_MAX_LEVEL][20];
61+
struct lock_class_key keys[BTRFS_MAX_LEVEL];
62+
} btrfs_lockdep_keysets[] = {
63+
{ .id = BTRFS_ROOT_TREE_OBJECTID, DEFINE_NAME("root") },
64+
{ .id = BTRFS_EXTENT_TREE_OBJECTID, DEFINE_NAME("extent") },
65+
{ .id = BTRFS_CHUNK_TREE_OBJECTID, DEFINE_NAME("chunk") },
66+
{ .id = BTRFS_DEV_TREE_OBJECTID, DEFINE_NAME("dev") },
67+
{ .id = BTRFS_CSUM_TREE_OBJECTID, DEFINE_NAME("csum") },
68+
{ .id = BTRFS_QUOTA_TREE_OBJECTID, DEFINE_NAME("quota") },
69+
{ .id = BTRFS_TREE_LOG_OBJECTID, DEFINE_NAME("log") },
70+
{ .id = BTRFS_TREE_RELOC_OBJECTID, DEFINE_NAME("treloc") },
71+
{ .id = BTRFS_DATA_RELOC_TREE_OBJECTID, DEFINE_NAME("dreloc") },
72+
{ .id = BTRFS_UUID_TREE_OBJECTID, DEFINE_NAME("uuid") },
73+
{ .id = BTRFS_FREE_SPACE_TREE_OBJECTID, DEFINE_NAME("free-space") },
74+
{ .id = 0, DEFINE_NAME("tree") },
75+
};
76+
77+
#undef DEFINE_LEVEL
78+
#undef DEFINE_NAME
79+
80+
void btrfs_set_buffer_lockdep_class(u64 objectid, struct extent_buffer *eb, int level)
81+
{
82+
struct btrfs_lockdep_keyset *ks;
83+
84+
BUG_ON(level >= ARRAY_SIZE(ks->keys));
85+
86+
/* Find the matching keyset, id 0 is the default entry */
87+
for (ks = btrfs_lockdep_keysets; ks->id; ks++)
88+
if (ks->id == objectid)
89+
break;
90+
91+
lockdep_set_class_and_name(&eb->lock, &ks->keys[level], ks->names[level]);
92+
}
93+
94+
#endif
95+
1696
/*
1797
* Extent buffer locking
1898
* =====================

‎fs/btrfs/locking.h‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -130,4 +130,13 @@ void btrfs_drew_write_unlock(struct btrfs_drew_lock *lock);
130130
void btrfs_drew_read_lock(struct btrfs_drew_lock *lock);
131131
void btrfs_drew_read_unlock(struct btrfs_drew_lock *lock);
132132

133+
#ifdef CONFIG_DEBUG_LOCK_ALLOC
134+
void btrfs_set_buffer_lockdep_class(u64 objectid, struct extent_buffer *eb, int level);
135+
#else
136+
static inline void btrfs_set_buffer_lockdep_class(u64 objectid,
137+
struct extent_buffer *eb, int level)
138+
{
139+
}
140+
#endif
141+
133142
#endif

0 commit comments

Comments
 (0)