Bug 2672 - include filename in "cifs_get_inode_info in lookup" warning
Summary: include filename in "cifs_get_inode_info in lookup" warning
Status: RESOLVED FIXED
Alias: None
Product: CifsVFS
Classification: Unclassified
Component: kernel fs (show other bugs)
Version: 2.6
Hardware: x86 Linux
: P3 enhancement
Target Milestone: ---
Assignee: Steve French
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-04-29 06:38 UTC by Trevor Cordes
Modified: 2005-06-04 18:09 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Trevor Cordes 2005-04-29 06:38:35 UTC
When I used to use smbmount, I used to get lots of "could not access" warnings
when the backups I run over smb to Windows machines got to pagefile.sys, etc. 
That warning always let me know the offending file so I know not to worry.

Now that I use mount -t cifs instead of smbmount, I get what I believe are
replacement errors when it hits the same files.  These errors are all "... on
cifs_get_inode_info in lookup".  Unfortunately, these errors do not tell me the
filename like smbmount did so I'm not sure whether or not I should be worried
about these errors.

I propose a patch to include the filename that caused the error in the warning
output so people can know whether to worry or not:

instead of:
("Error 0x%x or (%d decimal) on cifs_get_inode_info in lookup", rc, rc));

do:
("Error 0x%x or (%d decimal) on cifs_get_inode_info in lookup of %s", rc, rc,
full_path));
Comment 1 Steve French 2005-05-16 10:20:42 UTC
I don't know if this helps your example of open of pagefile.sys but it does make
sense to dump the path name here and I have changed the code in dir.c and
updated the project bk tree (will update the cogito/bit tree on kernel.org later
today).
Comment 2 Trevor Cordes 2005-06-04 18:09:53 UTC
Thanks!