Bug 4194 - Can't read directories with many files+directories
Summary: Can't read directories with many files+directories
Status: RESOLVED FIXED
Alias: None
Product: CifsVFS
Classification: Unclassified
Component: kernel fs (show other bugs)
Version: 3.x
Hardware: x86 Linux
: P3 normal
Target Milestone: ---
Assignee: Samba Bugzilla Account
QA Contact: cifs QA contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-10-29 07:09 UTC by Walkie
Modified: 2022-05-15 20:19 UTC (History)
1 user (show)

See Also:


Attachments
trace of an ls /mount/baddirectory operation (27.96 KB, text/plain)
2007-01-15 11:19 UTC, Timothee Besset
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Walkie 2006-10-29 07:09:19 UTC
ls returns
ls: reading directory .: Invalid argument
on cifs mounted volume on directories with many files+directories (tested 105 number is ok, more gets error)
and orter methods to stat that directory also non-funcctionsl
server uses FC5 samba3.0.21b-2 client is FC6 samba 3.0.23c-2
when client was FC5 samba 3.0.23c-2 everything works fine
Comment 1 Timothee Besset 2007-01-15 10:52:30 UTC
I am experiencing something very similar, with directories that have respectively  159, 124 and 175 files in them. Likely all the directories in the tree that have a  higher file count.

The client is running up to date FC5 (kernel version: 2.6.18-1.2200.fc5), I have no idea what the server is (a Windows box probably). man mount.cifs says cifs version 1.39

Going to see about upgrading to latest.

Comment 2 Timothee Besset 2007-01-15 11:00:07 UTC
I mistakenly reported version 1.39, looks like it's 1.45:

# modinfo cifs
filename:       /lib/modules/2.6.18-1.2200.fc5/kernel/fs/cifs/cifs.ko
author:         Steve French <sfrench@us.ibm.com>
license:        GPL
description:    VFS to access servers complying with the SNIA CIFS Specification e.g. Samba and Windows
version:        1.45
vermagic:       2.6.18-1.2200.fc5 mod_unload 686 REGPARM 4KSTACKS gcc-4.1
depends:
srcversion:     535782E74FBE6506A31138A
parm:           cifs_max_pending:Simultaneous requests to server. Default: 50 Range: 2 to 256 (int)
parm:           cifs_min_small:Small network buffers in pool. Default: 30 Range: 2 to 256 (int)
parm:           cifs_min_rcv:Network buffers in pool. Default: 4 Range: 1 to 64 (int)
parm:           CIFSMaxBufSize:Network buffer size (not including header). Default: 16384 Range: 8192 to 130048 (int)
Comment 3 Timothee Besset 2007-01-15 11:19:54 UTC
Created attachment 2257 [details]
trace of an ls /mount/baddirectory operation

A trace of an ls performed on one of those bad directories. It loops through the files, the only thing I can notice is this at the end:

 fs/cifs/readdir.c: filldir rc = -22
 fs/cifs/readdir.c: new entry ea1f51ac old entry ea1f514c
 fs/cifs/readdir.c: CIFS VFS: leaving cifs_readdir (xid = 16530490) rc = -22
Comment 4 Timothee Besset 2007-01-15 11:28:30 UTC
ok, found stuff by looking in the mailing list archives
http://lists.samba.org/archive/linux-cifs-client/2006-December/001657.html
Comment 5 Spammer 2007-04-28 04:05:57 UTC
Created attachment 2496
Comment 6 Björn Jacke 2022-05-12 05:31:08 UTC
moving to cifsvfs. I guess this is long fixed and should be closed, right?
Comment 7 Ronnie Sahlberg 2022-05-12 22:30:00 UTC
This was fixed more than a decade ago. Lets close it.