Re: [PATCH 23/41] union-mount: Changes to the namespace handling

[Erez Zadok]

In message <1256152779-10054-24-git-send-email-vaurora@xxxxxxxxxx>, Valerie Aurora writes:
> From: Jan Blunck <jblunck@xxxxxxx>
> 
> Creates the proper struct union_mount when mounting something into a
> union. If the topmost filesystem isn't capable of handling the white-out
> filetype it could only be mount read-only.
> 
> Signed-off-by: Jan Blunck <jblunck@xxxxxxx>
> Signed-off-by: Valerie Aurora <vaurora@xxxxxxxxxx>
> ---
>  fs/namespace.c        |    7 ++++++
>  fs/union.c            |   57 +++++++++++++++++++++++++++++++++++++++++++++++++
>  include/linux/mount.h |    3 ++
>  include/linux/union.h |   10 +++++++-
>  4 files changed, 75 insertions(+), 2 deletions(-)
> 
> diff --git a/fs/namespace.c b/fs/namespace.c

I'm curious if this fs/namespace.c patch could be merged with another
fs/namespace.c patch.

> diff --git a/include/linux/union.h b/include/linux/union.h
> index b035a82..0b6f356 100644
> --- a/include/linux/union.h
> +++ b/include/linux/union.h
> @@ -30,8 +30,9 @@ struct union_mount {
>  	atomic_t u_count;		/* reference count */
>  	struct mutex u_mutex;
>  	struct list_head u_unions;	/* list head for d_unions */
> -	struct hlist_node u_hash;	/* list head for searching */
> -	struct hlist_node u_rhash;	/* list head for reverse searching */
> +	struct list_head u_list;	/* list head for mnt_unions */
> +	struct hlist_node u_hash;	/* list head for seaching */

s/seaching/searching/

M-x ispell-comments-and-strings to the rescue.

> +	struct hlist_node u_rhash;	/* list head for reverse seaching */

A previous patch introduced struct union_mount; this patch modifies it.  Why
not introduce the final struct union_mount only once in this patchset?  It's
somewhat frustrating to have to read a patch as critical as the one defining
major new data structures, and try to understand it, only to find out that
the data structure is about to undergo major surgery later on.

Erez.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/