Re: I request inclusion of reiser4 in the mainline kernel

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

 



Denis Vlasenko wrote:
On Sunday 18 September 2005 03:34, Chris White wrote:

CC-List trimmed

On Saturday 17 September 2005 20:15, Denis Vlasenko wrote:

At least reiser4 is smaller. IIRC xfs is older than reiser4 and had more
time to optimize code size, but:

reiser4        2557872 bytes
xfs            3306782 bytes

And modules sizes:

reiser4.ko        442012 bytes
xfs.ko            494337 bytes

All this is fine and dandy, but saying "My code is better than yours!!" still doesn't solve the issue this thread hopes to achieve, that being "I'd like to get reiser4 into the kernel". There seems to be a lot of (historical?) tension present here, but all that seems to be doing is making things worse. PLEASE keep this thing a tad on par. Keeping this up is hurting everyone more than helping. I wish I could say something as simple as "let's just be friends", but that's saying a lot. I can say this though: this is open source, and that means that our source is open, and we should be too.


I am trying to say that I think that Hans is being treated a bit unfairly.
His fs is new and has fairly complex on-disk structure and complex journalling
machinery, yet his source and object code is smaller than xfs which already
is accepted. This is no easy feat I guess.

Maybe xfs shouldn't be accepted too, this may be an answer.

That argument is specious, and raises the chance that someone will suggest that we learn from our mistakes.

Let's look at the code. Hans' code is not _that_ awful. Yet people
(not all of them, but some) do not point to specific things which they
want to be fixed/improved. I see blanket arguments like "your code is hard
to read". Well. Maybe spend a minute on what exactly is hard to read,
or do we require Hans to be able to read minds from the distance?

Was this code first written in some language which compiles into C, like the original C++ compiler? It just "feels funny," which clearly is not an objective or quantitative thing, just an impression. Your comment "not _that_ awful" is a perfect example of "damned by faint praise." However, unless someone is willing to make the argument that the code is too weird to maintain, style isn't an issue, is it?

Christoph has made some clear technical points, those should be addressed, whether you like the code style or not.

--
   -bill davidsen ([email protected])
"The secret to procrastination is to put things off until the
 last possible moment - but no longer"  -me
-
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