Home » Mailing list archives » linux-kernel » 2008 » May » 16

Re: [PATCH 1/67] aufs document

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Dave Quigley <dpquigl@...>
To: <hooanon05@...>
Cc: <linux-fsdevel@...>, <linux-kernel@...>
Subject: Re: [PATCH 1/67] aufs document
Date: Friday, May 16, 2008 - 10:59 am

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).

Dave

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