nilfs2: use nilfs_warning() in allocator implementation
authorRyusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Sat, 7 Nov 2015 00:31:43 +0000 (16:31 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sat, 7 Nov 2015 01:50:42 +0000 (17:50 -0800)
This uses nilfs_warning() to replace "printk(KERN_WARNING ...);" in the
bitmap based allocator implementation of nilfs2.  The warning messages are
modified to include the device name and the inode number in each message.
This makes it clear which metadata file of which device has output
warnings such as "entry number xxxx already freed".

Signed-off-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/nilfs2/alloc.c

index 8df0f3b7839b5ee979ee48d63323dde21b637ce2..afe983643fdf979e3831ab53045327ea3dcadb13 100644 (file)
@@ -583,8 +583,10 @@ void nilfs_palloc_commit_free_entry(struct inode *inode,
 
        if (!nilfs_clear_bit_atomic(nilfs_mdt_bgl_lock(inode, group),
                                    group_offset, bitmap))
-               printk(KERN_WARNING "%s: entry number %llu already freed\n",
-                      __func__, (unsigned long long)req->pr_entry_nr);
+               nilfs_warning(inode->i_sb, __func__,
+                             "entry number %llu already freed: ino=%lu\n",
+                             (unsigned long long)req->pr_entry_nr,
+                             (unsigned long)inode->i_ino);
        else
                nilfs_palloc_group_desc_add_entries(inode, group, desc, 1);
 
@@ -620,8 +622,10 @@ void nilfs_palloc_abort_alloc_entry(struct inode *inode,
        bitmap = bitmap_kaddr + bh_offset(req->pr_bitmap_bh);
        if (!nilfs_clear_bit_atomic(nilfs_mdt_bgl_lock(inode, group),
                                    group_offset, bitmap))
-               printk(KERN_WARNING "%s: entry number %llu already freed\n",
-                      __func__, (unsigned long long)req->pr_entry_nr);
+               nilfs_warning(inode->i_sb, __func__,
+                             "entry number %llu already freed: ino=%lu\n",
+                             (unsigned long long)req->pr_entry_nr,
+                             (unsigned long)inode->i_ino);
        else
                nilfs_palloc_group_desc_add_entries(inode, group, desc, 1);
 
@@ -734,10 +738,10 @@ int nilfs_palloc_freev(struct inode *inode, __u64 *entry_nrs, size_t nitems)
                        if (!nilfs_clear_bit_atomic(
                                    nilfs_mdt_bgl_lock(inode, group),
                                    group_offset, bitmap)) {
-                               printk(KERN_WARNING
-                                      "%s: entry number %llu already freed\n",
-                                      __func__,
-                                      (unsigned long long)entry_nrs[j]);
+                               nilfs_warning(inode->i_sb, __func__,
+                                             "entry number %llu already freed: ino=%lu\n",
+                                             (unsigned long long)entry_nrs[j],
+                                             (unsigned long)inode->i_ino);
                        } else {
                                n++;
                        }