public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: hooanon05@yahoo.co.jp
To: Dave Quigley <dpquigl@tycho.nsa.gov>
Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/67] aufs document
Date: Sat, 17 May 2008 00:45:32 +0900	[thread overview]
Message-ID: <18952.1210952732@jrobl> (raw)
In-Reply-To: <1210949976.14706.28.camel@moss-terrapins.epoch.ncsc.mil>


Hello Dave,
Thank you for your comments.

About diffstat, actually I send it but the subject was not 0/67.
Please read the thread whose subject is
"[RFC n/2] AUFS: merging/stacking several filesystems"
The last message in the thread which I sent just before the patch series
has diffstat.
Do you want me to re-send it after changing the subject line?

About the number of patches, I have to agree it is large. I chose the
way to make a patch per a new file, and in case of modifying the
existing multiple files I made a single patch. I guess the latter is the
case you wrote "can be folded."

About the description, simple one for the new files because they are
brand new... I wrote a little more in case of modifying the existing
files (see the last patch). Of course a more detail description will be
necessary in future modification.
If you think the functionality is obscure, please read the aufs
documents included in the patch set (the first patches).


Junjiro Okajima


Dave Quigley:
> A couple of comments about this. First off you might want to reconsider
> how you are breaking up the patches. 67 patches seems a bit much to me
> and while the separation between patches is clear it seems that some of
> them can be folded. Second I'm not sure if I haven't gotten it yet or if
> it wasn't sent but I am missing an Email 0/67. Typically this contains a
> description of the patch set, an index of the patches (possibly with
> descriptions), and a diffstat for the entire patch set. The last thing
> is that each patch should have a worth wile description attached to it.
> All of your comments are two lines and that may make sense based on how
> you have it broken up but if you start breaking up the patch set
> differently you will need better descriptions (possibly even now for
> some of the more obscure functionality).

  parent reply	other threads:[~2008-05-16 15:45 UTC|newest]

Thread overview: 80+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-16 14:32 [PATCH 1/67] aufs document hooanon05
2008-05-16 14:32 ` [PATCH 2/67] aufs manual hooanon05
2008-05-16 14:32   ` [PATCH 3/67] aufs global header file hooanon05
2008-05-16 14:32     ` [PATCH 4/67] aufs configuration hooanon05
2008-05-16 14:32       ` [PATCH 5/67] aufs Makefile hooanon05
2008-05-16 14:32         ` [PATCH 6/67] aufs main header file hooanon05
2008-05-16 14:32           ` [PATCH 7/67] aufs module initialization and module-global hooanon05
2008-05-16 14:32             ` [PATCH 8/67] aufs module global variables and operations hooanon05
2008-05-16 14:32               ` [PATCH 9/67] aufs super_block operations hooanon05
2008-05-16 14:32                 ` [PATCH 10/67] aufs mount and " hooanon05
2008-05-16 14:32                   ` [PATCH 11/67] aufs superblock private data hooanon05
2008-05-16 14:32                     ` [PATCH 12/67] aufs branch filesystems and xino for them hooanon05
2008-05-16 14:32                       ` [PATCH 13/67] aufs branch management hooanon05
2008-05-16 14:32                         ` [PATCH 14/67] aufs external inode number translation table and bitmap hooanon05
2008-05-16 14:32                           ` [PATCH 15/67] aufs special handling for inode attributes on FUSE branch hooanon05
2008-05-16 14:32                             ` [PATCH 16/67] aufs lookup functions for NFS branch in linux-2.6.19 and later hooanon05
2008-05-16 14:32                               ` [PATCH 17/67] aufs special handling inode attributes on XFS branch in linux-2.6.24 " hooanon05
2008-05-16 14:32                                 ` [PATCH 18/67] aufs sysfs interface and lifetime management, header hooanon05
2008-05-16 14:32                                   ` [PATCH 19/67] aufs sysfs interface and lifetime management, source hooanon05
2008-05-16 14:32                                     ` [PATCH 20/67] aufs mount options/flags, header hooanon05
2008-05-16 14:32                                       ` [PATCH 21/67] aufs mount options/flags, source hooanon05
2008-05-16 14:32                                         ` [PATCH 22/67] aufs workqueue for asynchronous/super-io/delegated operations, header hooanon05
2008-05-16 14:32                                           ` [PATCH 23/67] aufs workqueue for asynchronous/super-io/delegated operations, source hooanon05
2008-05-16 14:32                                             ` [PATCH 24/67] aufs sub-VFS, header hooanon05
2008-05-16 14:32                                               ` [PATCH 25/67] aufs sub-VFS, source hooanon05
2008-05-16 14:32                                                 ` [PATCH 26/67] aufs sub-dcache, header hooanon05
2008-05-16 14:32                                                   ` [PATCH 27/67] aufs sub-dcache, source hooanon05
2008-05-16 14:32                                                     ` [PATCH 28/67] aufs copy-up/down functions hooanon05
2008-05-16 14:32                                                       ` [PATCH 29/67] aufs copy-up functions, see wbr_policy.c for copy-down hooanon05
2008-05-16 14:32                                                         ` [PATCH 30/67] aufs whiteout for logical deletion and opaque directory, header hooanon05
2008-05-16 14:32                                                           ` [PATCH 31/67] aufs whiteout for logical deletion and opaque directory, source hooanon05
2008-05-16 14:32                                                             ` [PATCH 32/67] aufs pseudo-link hooanon05
2008-05-16 14:32                                                               ` [PATCH 33/67] aufs policies for selecting one among multiple writable branches hooanon05
2008-05-16 14:32                                                                 ` [PATCH 34/67] aufs lookup and dentry operations, header hooanon05
2008-05-16 14:32                                                                   ` [PATCH 35/67] aufs lookup and dentry operations, source hooanon05
2008-05-16 14:32                                                                     ` [PATCH 36/67] aufs dentry private data hooanon05
2008-05-16 14:32                                                                       ` [PATCH 37/67] aufs file operations hooanon05
2008-05-16 14:32                                                                         ` [PATCH 38/67] aufs handling file/dir, and address_space operation hooanon05
2008-05-16 14:32                                                                           ` [PATCH 39/67] aufs file private data hooanon05
2008-05-16 14:32                                                                             ` [PATCH 40/67] aufs file and vm operations hooanon05
2008-05-16 14:32                                                                               ` [PATCH 41/67] aufs directory operations, header hooanon05
2008-05-16 14:32                                                                                 ` [PATCH 42/67] aufs directory operations, source hooanon05
2008-05-16 14:32                                                                                   ` [PATCH 43/67] aufs virtual or vertical directory hooanon05
2008-05-16 14:32                                                                                     ` [PATCH 44/67] aufs inode operations hooanon05
2008-05-16 14:32                                                                                       ` [PATCH 45/67] aufs inode functions hooanon05
2008-05-16 14:33                                                                                         ` [PATCH 46/67] aufs inode private data hooanon05
2008-05-16 14:33                                                                                           ` [PATCH 47/67] aufs inode operations (except add/del/rename) hooanon05
2008-05-16 14:33                                                                                             ` [PATCH 48/67] aufs inode operations (add entry) hooanon05
2008-05-16 14:33                                                                                               ` [PATCH 49/67] aufs inode operations (del entry) hooanon05
2008-05-16 14:33                                                                                                 ` [PATCH 50/67] aufs inode operation (rename entry) hooanon05
2008-05-16 14:33                                                                                                   ` [PATCH 51/67] aufs lower (branch filesystem) inode and setting inotify hooanon05
2008-05-16 14:33                                                                                                     ` [PATCH 52/67] aufs inotify handler hooanon05
2008-05-16 14:33                                                                                                       ` [PATCH 53/67] aufs sub-routines for vfs in hinotify or dlgt mode hooanon05
2008-05-16 14:33                                                                                                         ` [PATCH 54/67] aufs lookup functions in 'delegate' mode hooanon05
2008-05-16 14:33                                                                                                           ` [PATCH 55/67] aufs export via nfs hooanon05
2008-05-16 14:33                                                                                                             ` [PATCH 56/67] aufs 'robr', aufs as readonly branch of another aufs hooanon05
2008-05-16 14:33                                                                                                               ` [PATCH 57/67] aufs sysfs interface hooanon05
2008-05-16 14:33                                                                                                                 ` [PATCH 58/67] aufs misc functions, header hooanon05
2008-05-16 14:33                                                                                                                   ` [PATCH 59/67] aufs misc functions, source hooanon05
2008-05-16 14:33                                                                                                                     ` [PATCH 60/67] aufs debug print functions, header hooanon05
2008-05-16 14:33                                                                                                                       ` [PATCH 61/67] aufs debug print functions, source hooanon05
2008-05-16 14:33                                                                                                                         ` [PATCH 62/67] aufs magic sysrq handler hooanon05
2008-05-16 14:33                                                                                                                           ` [PATCH 63/67] aufs mount helper hooanon05
2008-05-16 14:33                                                                                                                             ` [PATCH 64/67] aufs pseudo-link helper hooanon05
2008-05-16 14:33                                                                                                                               ` [PATCH 65/67] aufs pseudo-link helper for symlink hooanon05
2008-05-16 14:33                                                                                                                                 ` [PATCH 66/67] aufs umount helper hooanon05
2008-05-16 14:33                                                                                                                                   ` [PATCH 67/67] merge aufs hooanon05
2008-05-16 15:36               ` [PATCH 8/67] aufs module global variables and operations Jan Engelhardt
2008-05-16 15:33           ` [PATCH 6/67] aufs main header file Jan Engelhardt
2008-05-16 17:25         ` [PATCH 5/67] aufs Makefile Sam Ravnborg
2008-05-19  2:27           ` hooanon05
2008-05-16 15:28       ` [PATCH 4/67] aufs configuration Jan Engelhardt
2008-05-19  2:23         ` hooanon05
2008-05-16 14:59 ` [PATCH 1/67] aufs document Dave Quigley
2008-05-16 15:32   ` Jan Engelhardt
2008-05-16 21:07     ` Josef 'Jeff' Sipek
2008-05-19  2:25     ` hooanon05
2008-05-16 15:45   ` hooanon05 [this message]
2008-05-16 16:09     ` Dave Quigley
2008-05-17  2:06       ` hooanon05

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=18952.1210952732@jrobl \
    --to=hooanon05@yahoo.co.jp \
    --cc=dpquigl@tycho.nsa.gov \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    /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