| Submitter | Theodore Ts'o |
|---|---|
| Date | March 19, 2010, 12:50 a.m. |
| Message ID | <1268959842-4697-1-git-send-email-tytso@mit.edu> |
| Download | mbox | patch |
| Permalink | /patch/48091/ |
| State | Rejected |
| Headers | show |
Comments
Patch
diff --git a/fs/ext4/namei.c b/fs/ext4/namei.c index 0c070fa..0c0cba4 100644 --- a/fs/ext4/namei.c +++ b/fs/ext4/namei.c @@ -1072,9 +1072,9 @@ static struct dentry *ext4_lookup(struct inode *dir, struct dentry *dentry, stru inode = ext4_iget(dir->i_sb, ino); if (unlikely(IS_ERR(inode))) { if (PTR_ERR(inode) == -ESTALE) { - ext4_error(dir->i_sb, - "deleted inode referenced: %u", - ino); + EXT4_ERROR_INODE(dir, + "deleted inode referenced: %u", + ino); return ERR_PTR(-EIO); } else { return ERR_CAST(inode);
EXT4_ERROR_INODE() tends to provide better error information and in a more consistent format. Addresses-Google-Bug: #2507977 Signed-off-by: "Theodore Ts'o" <tytso@mit.edu> --- Yes, there are more calls to ext4_error that we probably want to convert to use ext4_error_inode; this is just for illustrative purposes, and because it's the one which is most obnoxious if there is a bad entry in a particular inode which is referenced over and over again (see patch 3 in this series). fs/ext4/namei.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-)