linux-kernel - Re: New filesystem for Linux kernel

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <11287.1235481490@jrobl>
Date:	Tue, 24 Feb 2009 22:18:10 +0900
From:	hooanon05@...oo.co.jp
To:	Miklos Szeredi <miklos@...redi.hu>
Cc:	tomas@...x.org, akpm@...ux-foundation.org,
	linux-fsdevel@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: New filesystem for Linux kernel 


Miklos Szeredi:
> I think the biggest problem is too many features.
> 
>    > git diff master...aufs2 | diffstat
>     ...
>     73 files changed, 23527 insertions(+), 7 deletions(-)
>                       ^^^^^^
> This is an unreviewable amount of code, it would make AUFS one of the
> biggest filesystems on linux.
> 
> The first step would be to separate out the very core functionality,
> which should be a couple thousands of lines max.  And when that has
> been accepted and stabilised, then you can start adding fancy
> features.

I have to admit aufs is big, but actually, as I wrote in the documents,
aufs2 has already dropped several features. And I believe it is the core
feature set. If aufs2 drops some more features, then both of users and
reviewers will say it doesn't work in this case, in that case. I don't
think you would like to review such unusable code in real world.
For those who wants to begin with aufs2 principle (or basic
architecture), I described and posted these documents.

Actually ocfs2 and xfs are much bigger.
If they have been reviewed, I'd ask you to review aufs2 too.
$ cd linux/fs
$ for i in *; do test -d $i && echo -n $i && find $i -type f | xargs wc -l | tail -1; done | tr -s '[[:blank:]]' | sort -n -k 2 | tail
nfs 30363 total
ntfs 31346 total
jfs 33056 total
cifs 34059 total
ubifs 34721 total
btrfs 43417 total
aufs 46325 total
nls 54855 total
ocfs2 71294 total
xfs 102144 total


J. R. Okajima
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/