Re: [EXT4 set 7][PATCH 1/1]Remove 32000 subdirs limit.

2007-07-22 Thread Ingo Oeser
On Tuesday 17 July 2007, Kalpak Shah wrote: > Index: linux-2.6.22/include/linux/ext4_fs.h > === > --- linux-2.6.22.orig/include/linux/ext4_fs.h > +++ linux-2.6.22/include/linux/ext4_fs.h > @@ -797,12 +797,18 @@ struct ext4_dir_entry_2

Re: [EXT4 set 7][PATCH 1/1]Remove 32000 subdirs limit.

2007-07-17 Thread Kalpak Shah
On Fri, 2007-07-13 at 09:53 -0700, Andrew Morton wrote: > On Fri, 13 Jul 2007 16:00:48 +0530 Kalpak Shah <[EMAIL PROTECTED]> wrote: > > > > > > > > > - if (inode->i_nlink >= EXT4_LINK_MAX) > > > > + if (EXT4_DIR_LINK_MAX(inode)) > > > > return -EMLINK; > > > > > > ar

Re: [EXT4 set 7][PATCH 1/1]Remove 32000 subdirs limit.

2007-07-13 Thread Andrew Morton
On Fri, 13 Jul 2007 16:00:48 +0530 Kalpak Shah <[EMAIL PROTECTED]> wrote: > > > > > > - if (inode->i_nlink >= EXT4_LINK_MAX) > > > + if (EXT4_DIR_LINK_MAX(inode)) > > > return -EMLINK; > > > > argh. WHY_IS_EXT4_FULL_OF_UPPER_CASE_MACROS_WHICH_COULD_BE_IMPLEMENTED > > as_lower_case_in

Re: [EXT4 set 7][PATCH 1/1]Remove 32000 subdirs limit.

2007-07-13 Thread Pekka Enberg
On 7/13/07, Kalpak Shah <[EMAIL PROTECTED]> wrote: > EXT4_DIR_LINK_MAX() is buggy: it evaluates its arg twice. #define EXT4_DIR_LINK_MAX(dir) (!is_dx(dir) && (dir)->i_nlink >= EXT4_LINK_MAX) [snip] Sorry, I didn't understand what is the problem with this macro? The expression represented b

Re: [EXT4 set 7][PATCH 1/1]Remove 32000 subdirs limit.

2007-07-13 Thread Kalpak Shah
The updated patch is attached. comments inline... On Tue, 2007-07-10 at 22:40 -0700, Andrew Morton wrote: > > If we exceed 65000 subdirectories in an htree directory it sets the > > inode link count to 1 and no longer counts subdirectories. The > > directory link count is not actually used when d

Re: [EXT4 set 7][PATCH 1/1]Remove 32000 subdirs limit.

2007-07-11 Thread Andreas Dilger
On Jul 10, 2007 22:40 -0700, Andrew Morton wrote: > On Sun, 01 Jul 2007 03:38:18 -0400 Mingming Cao <[EMAIL PROTECTED]> wrote: > > A EXT4_FEATURE_RO_COMPAT_DIR_NLINK flag has been added and it is set if > > the subdir count for any directory crosses 65000. > > Would I be correct in assuming that

Re: [EXT4 set 7][PATCH 1/1]Remove 32000 subdirs limit.

2007-07-10 Thread Andrew Morton
On Sun, 01 Jul 2007 03:38:18 -0400 Mingming Cao <[EMAIL PROTECTED]> wrote: > >From [EMAIL PROTECTED] Thu May 17 17:21:08 2007 > Hi, > > I have rebased this patch to 2.6.22-rc1 so that it can be added to the > ext4 patch queue. It has been tested by creating more than 65000 subdirs > and then dele

[EXT4 set 7][PATCH 1/1]Remove 32000 subdirs limit.

2007-07-01 Thread Mingming Cao
>From [EMAIL PROTECTED] Thu May 17 17:21:08 2007 Hi, I have rebased this patch to 2.6.22-rc1 so that it can be added to the ext4 patch queue. It has been tested by creating more than 65000 subdirs and then deleting them and checking the nlinks. The e2fsprogs part of this patch was sent earlier by