From: Eric Sandeen <sandeen@redhat.com>
To: "Theodore Ts'o" <tytso@mit.edu>, Nick Krause <xerofoify@gmail.com>
Cc: linux-ext4@vger.kernel.org
Subject: Re: Work on ext4
Date: Fri, 25 Jul 2014 11:07:35 -0500 [thread overview]
Message-ID: <53D280C7.1080904@redhat.com> (raw)
In-Reply-To: <20140725154142.GU1865@thunk.org>
On 7/25/14, 10:41 AM, Theodore Ts'o wrote:
...
> This will hopefully get you started on the testing side of things,
> which is also a good way to start learning about how ext4 works.
>
> Cheers,
>
> - Ted
The old adage on IRC is "Don't ask to ask; ask." In
development-land, it's more like "Don't ask to help; help."
The trick is knowing what constitutes "help." Asking for
detailed instruction is not "help."
Over on the btrfs-list, Hugo had a good set of suggestions as
well; some are btrfs-specific, but in general, good advice for
anyone trying to get engaged with an upstream project:
>From Hugo -
...
> My recommendations for you, if you want to work on btrfs, are:
>
> * Build and install the latest kernel from Linus's git repo
>
> * Read and understand the user documentation [2]
>
> * Create one or several btrfs filesystems with different
> configurations and learn how they work in userspace -- what are the
> features, what are the problems you see? Actually use at least one
> of the filesystems you created for real data in daily use (with
> backups)
>
> * Build the userspace tools from git
>
> * Pick up one of the userspace projects from [3] and implement that.
> If you pick the right one(s), you'll have to learn about some of
> the internal structures of the FS anyway. Compile and test your
> patch. If you're adding a new feature, write an automated xfstest
> for it as well.
>
> * Get that patch accepted. This will probably involve a sequence of
> revisions to it, multiple versions over a period of several weeks
> or more, with a review process. You should also send your test to
> xfstests and get that accepted.
>
> * Do the above again, until you get used to the processes involved,
> and have demonstrated that you can work well with the other people
> in the subsystem, and are generally producing useful and sane code.
> It's all about trust -- can you be trusted to mostly do the right
> thing? (So far on linux-kernel, you've rather demonstrated the
> opposite: your intentions are good, but your execution leaves a lot
> to be desired)
>
> * Use the documentation at [4], and the output of btrfs-debug-tree to
> understand the internal structure of the FS
>
> * Pick up one of the smaller, more self-contained ideas from the
> projects page [5] (say, [6] or [7]) and try to implement it. Again:
> build, write test code, test thoroughly, submit patch for review,
> modify as suggested by reviewers, and repeat as often as necessary
>
> Hugo.
next prev parent reply other threads:[~2014-07-25 16:07 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-25 2:29 Work on ext4 Nick Krause
2014-07-25 14:59 ` Theodore Ts'o
2014-07-25 15:22 ` Nick Krause
2014-07-25 15:41 ` Theodore Ts'o
2014-07-25 16:07 ` Eric Sandeen [this message]
2014-07-28 4:01 ` Nick Krause
2014-07-28 12:45 ` Theodore Ts'o
2014-07-28 15:17 ` Nick Krause
2014-07-25 15:24 ` Wang Shilong
2014-07-25 15:59 ` Nick Krause
2014-07-25 16:06 ` Nick Krause
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=53D280C7.1080904@redhat.com \
--to=sandeen@redhat.com \
--cc=linux-ext4@vger.kernel.org \
--cc=tytso@mit.edu \
--cc=xerofoify@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.