ext3_dx_add_entry: Directory index full!

[Bernie Innocenti]

On 2.6.24.4-64.fc8, I createed and mounted a filesystem like this:

 mke2fs -m0 -b 1024 -R stride=64 -I 128 -i 2048 -j  -L mail -O dir_index,sparse_super -v /dev/sdc1
 mount -t ext3 -o noatime,data=writeback,nosuid,usrquota /dev/sdc1 /mail

Then I copied a directory with 200K small files into it:

 cp -a home/simone/Maildir/YouHaveJunkSir /mail/
 [...]
 cp: cannot create regular file `/mnt/YouHaveJunkSir/1174170042.20731_2.trinity.develer.com:2,b': No space left on device
 cp: cannot create regular file `/mnt/YouHaveJunkSir/1186341042.8337_2.trinity.develer.com:2,': No space left on device
 cp: cannot create regular file `/mnt/YouHaveJunkSir/1209101786.3888_2.trinity.develer.com:2,': No space left on device

The kernel logs are also scary:

 EXT3-fs warning (device sdc1): ext3_dx_add_entry: Directory index full!
 EXT3-fs warning (device sdc1): ext3_dx_add_entry: Directory index full!
 [...]

The failing check looks like this:

       if (levels && (dx_get_count(frames->entries) ==
                  dx_get_limit(frames->entries))) {
           ext3_warning(sb, __FUNCTION__,
                    "Directory index full!");
           err = -ENOSPC;
           goto cleanup;
       }

The limit is set in make_indexed_dir():

 dx_set_limit (entries, dx_root_limit(dir, sizeof(root->info)));

with this helper function:

 static inline unsigned dx_root_limit (struct inode *dir, unsigned infosize)
 {
     unsigned entry_space = dir->i_sb->s_blocksize - EXT3_DIR_REC_LEN(1) -
         EXT3_DIR_REC_LEN(2) - infosize;
     return 0? 20: entry_space / sizeof(struct dx_entry);
 }

Am I reading the above code correctly?  Why does it always return
20 no matter what?


Some background: I'm moving users' Maildirs to a separate filesystem tuned
for small files to increase performance.  One of our users intentionally
collected spam for 5 years in one folder and likes it this way.
We could easily work it around, but first I'd like to understand whether
the particular parameters we used trigger a bug in ext3 or if we're just
hitting a (possibly undocumented) limit.

--
  \___/
 _| X |  Bernie Innocenti - http://www.codewiz.org/
 \|_O_|  "It's an education project, not a laptop project!"
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/