Re: I request inclusion of reiser4 in the mainline kernel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Alan Cox wrote:

>On Sul, 2005-09-18 at 22:07 -0700, Hans Reiser wrote:
>  
>
>>>the ability to fix some of those bugs fast, but we also all remember
>>>what happened with reiser3 later on despite early fast fixing.
>>> 
>>>
>>>      
>>>
>>What was that?
>>    
>>
>
>Jeff Mahoney added file attributes to reiserfs3, you whined and pointed
>people at the yet to be released reiserfs4.
>
If you benchmarked that code, you might understand why I "whined."  You
can't just create a file per directory and stuff the attributes in it
and expect good performance.  Let's not forget that there was no
documentation, no design document, no design review, no QA process.

It is always a judgment call to decide what should be deferred to the
next major release and what should go into a stable branch.  File
attributes are a significant portion of the bugs that V3 has had.  File
attributes got added so that a marketer would have a bullet point added,
which can be very important and I am genuinely eager to work hard to
make marketers happy, but to the extent I get to decide, it will never
happen at the cost of coding it the wrong way.

Jeff is a great guy, and his bitmap related code is great stuff with
good design and solid empirical work behind it.  You have to really
understand the difference between V3 and V4 to appreciate that it was
not feasible for him to code xattrs for V3 the right way, because it
would be a disk format change and a nightmare to do it.  The code was
doomed by V3's lack of plugins before it was even written.  There is a
reason why V4 came into being....

If added to V4, xattrs would be higher performance and cleaner to
implement.  It would be far better to have spent the programming effort
on adding them to V4 and getting V4 out a little sooner.

I won't convince you of this one but it is also my reason: They are
inelegant semantics.

I don't remember the details of the 4k stack and journaling issues you
describe, so I will say nothing.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]
  Powered by Linux