On Fri, Feb 15, 2013 at 05:12:37PM -0600, Simon Kirby wrote: [...] > Signed-off-by: Simon Kirby <s...@hostway.ca>
Thanks! 2 comments below. Reviewed-by: David Sterba <dste...@suse.cz> > @@ -2919,8 +2923,9 @@ int btrfs_write_out_ino_cache(struct btrfs_root *root, > if (ret) { > btrfs_delalloc_release_metadata(inode, inode->i_size); > #ifdef DEBUG > - printk(KERN_ERR "btrfs: failed to write free ino cache " > - "for root %llu\n", root->root_key.objectid); > + btrfs_err(root->fs_info, > + "btrfs %s: failed to write free ino cache for root > %llu", > + root->root_key.objectid); "failed to write free ino cache for root %llu", > #endif > } > > @@ -2454,8 +2456,8 @@ int btrfs_orphan_cleanup(struct btrfs_root *root) > ret = PTR_ERR(trans); > goto out; > } > - printk(KERN_ERR "auto deleting %Lu\n", > - found_key.objectid); > + btrfs_err(root->fs_info, "auto deleting %Lu", > + found_key.objectid); That's probably only a debugging message, so btrfs_debug would be more appropriate here. > ret = btrfs_del_orphan_item(trans, root, > found_key.objectid); > BUG_ON(ret); /* -ENOMEM or corruption (JDM: Recheck) */ -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html