Messages in this thread | | | Subject | Re: [PATCH 2/16] fs/hfsplus/bnode.c: use WARN | From | Vyacheslav Dubeyko <> | Date | Sun, 4 Nov 2012 14:49:00 +0300 |
| |
On Nov 3, 2012, at 1:58 PM, Julia Lawall wrote:
> From: Julia Lawall <Julia.Lawall@lip6.fr> > > 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> > > --- > fs/hfsplus/bnode.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/fs/hfsplus/bnode.c b/fs/hfsplus/bnode.c > index 5c125ce..7a92c2c 100644 > --- a/fs/hfsplus/bnode.c > +++ b/fs/hfsplus/bnode.c > @@ -588,8 +588,7 @@ struct hfs_bnode *hfs_bnode_create(struct hfs_btree *tree, u32 num) > node = hfs_bnode_findhash(tree, num); > spin_unlock(&tree->hash_lock); > if (node) { > - printk(KERN_CRIT "new node %u already hashed?\n", num); > - WARN_ON(1); > + WARN(1, KERN_CRIT "new node %u already hashed?\n", num); > return node; > } > node = __hfs_bnode_create(tree, num); >
Looks good.
Reviewed-by: Vyacheslav Dubeyko <slava@dubeyko.com>
Thanks, Vyacheslav Dubeyko.
> -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/
| |