From: Andrew Morton <akpm@linux-foundation.org>
To: "Gang He" <ghe@suse.com>
Cc: <ocfs2-devel@oss.oracle.com>, "Mark Fasheh" <MFasheh@suse.com>,
<rgoldwyn@suse.de>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v3 0/4] Add online file check feature
Date: Tue, 29 Dec 2015 19:14:55 -0800 [thread overview]
Message-ID: <20151229191455.562f5f42.akpm@linux-foundation.org> (raw)
In-Reply-To: <5683B938020000F900023B48@relay2.provo.novell.com>
On Tue, 29 Dec 2015 20:00:08 -0700 "Gang He" <ghe@suse.com> wrote:
> > This feature should be documented, please. That means all pseudo-file
> > locations, all inputs, all outputs, expected behaviour etc etc. Enough
> > info so that our users can usefully and fully use this feature in the
> > minimum time. Documentation/filesystems/ocfs2.txt is the place for that.
> I ever sent out a document which describes this feature, I will modify that document sightly since sysfs interfaces have been adjusted,
> then send out to the list as a separated patch. Is it OK for you?
Sounds good, thanks. The documentation is important because it permits
reviewers to understand the proposed interfaces, which need to be
nailed down before there's any point in looking at the implementation.
prev parent reply other threads:[~2015-12-30 3:15 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-25 7:16 [PATCH v3 0/4] Add online file check feature Gang He
2015-12-25 7:16 ` [PATCH v3 1/4] ocfs2: export ocfs2_kset for online file check Gang He
2015-12-29 21:38 ` Andrew Morton
2015-12-30 6:13 ` Gang He
2016-01-13 23:02 ` Mark Fasheh
2016-01-13 23:07 ` Andrew Morton
2015-12-25 7:16 ` [PATCH v3 2/4] ocfs2: sysfile interfaces " Gang He
2016-01-13 23:35 ` Mark Fasheh
2016-01-14 3:13 ` Gang He
2015-12-25 7:16 ` [PATCH v3 3/4] ocfs2: create/remove sysfile " Gang He
2016-01-13 23:36 ` Mark Fasheh
2015-12-25 7:16 ` [PATCH v3 4/4] ocfs2: check/fix inode block " Gang He
2016-01-14 1:40 ` [Ocfs2-devel] " Mark Fasheh
2016-01-14 6:30 ` Gang He
2015-12-29 21:41 ` [PATCH v3 0/4] Add online file check feature Andrew Morton
2015-12-30 3:00 ` Gang He
2015-12-30 3:14 ` Andrew Morton [this message]
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=20151229191455.562f5f42.akpm@linux-foundation.org \
--to=akpm@linux-foundation.org \
--cc=MFasheh@suse.com \
--cc=ghe@suse.com \
--cc=linux-kernel@vger.kernel.org \
--cc=ocfs2-devel@oss.oracle.com \
--cc=rgoldwyn@suse.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox