diff options
author | Julia Lawall <Julia.Lawall@lip6.fr> | 2012-11-03 06:58:34 -0400 |
---|---|---|
committer | Josef Bacik <jbacik@fusionio.com> | 2012-12-12 17:15:23 -0500 |
commit | 31b1a2bd758f439fc945b3ac5899d890cb7e2dc6 (patch) | |
tree | a5659a8e51a780eb02a51c2f79c8e3c25d29bb92 /fs/btrfs/volumes.c | |
parent | 5269b67e3d809dcaa4c6763a343423bb1b7b3fe6 (diff) |
fs/btrfs: use WARN
Use WARN rather than printk followed by WARN_ON(1), for conciseness.
A simplified version of the semantic patch that makes this transformation
is as follows: (http://coccinelle.lip6.fr/)
// <smpl>
@@
expression list es;
@@
-printk(
+WARN(1,
es);
-WARN_ON(1);
// </smpl>
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Reviewed-by: David Sterba <dsterba@suse.cz>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
Diffstat (limited to 'fs/btrfs/volumes.c')
-rw-r--r-- | fs/btrfs/volumes.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c index eeed97d19dee..3f4bfee66d7b 100644 --- a/fs/btrfs/volumes.c +++ b/fs/btrfs/volumes.c | |||
@@ -3323,9 +3323,8 @@ static int __btrfs_alloc_chunk(struct btrfs_trans_handle *trans, | |||
3323 | cur = cur->next; | 3323 | cur = cur->next; |
3324 | 3324 | ||
3325 | if (!device->writeable) { | 3325 | if (!device->writeable) { |
3326 | printk(KERN_ERR | 3326 | WARN(1, KERN_ERR |
3327 | "btrfs: read-only device in alloc_list\n"); | 3327 | "btrfs: read-only device in alloc_list\n"); |
3328 | WARN_ON(1); | ||
3329 | continue; | 3328 | continue; |
3330 | } | 3329 | } |
3331 | 3330 | ||