Re: reiser4 plugins

From: Kyle Moffett
Date: Tue Jun 21 2005 - 22:21:19 EST

On Jun 21, 2005, at 22:47:13, Hans Reiser wrote:
Andi Kleen wrote:
and your child definitely
is in serious need of that to be mergeable. I'm sure Christoph is able
to review inpartially even when he is involved with other FS.
As impartial as a puppy on PCP....

So where else are you planning to get a competent reviewer who is fluent
in the internals of filesystems? Good reviewers don't grow on trees, and
in order to be able to understand filesystem issues, one must generally
have worked with them before... Besides, what good is insulting others
going to do?

Christoph is aggressive about things he does not take the time to
understand or ask about first.
[rant snipped]

From my objective re-reading of his posts, I can see that he is critical
of things that are difficult to understand not just to be critical, but
to provoke additional thought over those portions of the code. In many
cases this leads to better abstractions and simpler code than otherwise.

How about review by benchmark instead?

/dev/sda is a great filesystem with awesome benchmarks, assuming one only
needs to store a single file. Besides, benchmarks aren't the only thing
important about code. If the interface consists of:

void clear_current_filename(void);
void add_char_to_current_filename(char x);
void read_bytes_from_current_file(char *byte, unsigned long size);
void write_bytes_to_current_file(const char *byte, unsigned long size);

then this is clearly not a good API, regardless of how well or poorly it
may perform.

It works, it runs faster than the competition, users like it, we
addressed the core kernel patch complaints, it should go in and receive
the exposure that will result in lots of useful improvements and
suggestions. It seems like we are getting an unusual review process.

If you look over other patches in -mm, you will see that your review
process is not unusual, especially given the number of concerns that other
developers have raised over Reiser4.

[irrelevant algorithm rant snipped]

If you guys want to understand
what I am doing I am happy to talk about it extensively, but please
don't require that I groupthink. I frankly think that with my
benchmarks, I should be allowed to tinker on my own.

Yes, you can tinker on your own all you want. Another project that has
taken that route is GrSecurity, which was alive and well last I checked.

If you don't like others criticisms, take up your marbles and go home,
just don't expect them to accept your work when you've not fixed it to
community standards.

Kyle Moffett

Somone asked my why I work on this free (
software stuff and not get a real job. Charles Shultz had the best answer:

"Why do musicians compose symphonies and poets write poems? They do it
because life wouldn't have any meaning for them if they didn't. That's why
I draw cartoons. It's my life."
-- Charles Shultz

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at