Linux CIFS filesystem development
 help / color / mirror / Atom feed
From: Valerie Aurora <vaurora-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: David Howells <dhowells-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Cc: viro-rfM+Q5joDG/XmaaqVzeoHQ@public.gmane.org,
	jmoyer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org,
	linux-fs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	autofs-CPWUtch7KCBzeIdxy0IIJw@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-afs-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Ian Kent <raven-PKsaG3nR2I+sTnJN9+BGXg@public.gmane.org>
Subject: Re: [PATCH 07/17] Make dentry::d_mounted into a more general field for special function dirs
Date: Fri, 8 Oct 2010 19:57:42 -0400	[thread overview]
Message-ID: <20101008235741.GF30846@shell> (raw)
In-Reply-To: <20100930181531.30939.10438.stgit-S6HVgzuS8uM4Awkfq6JHfwNdhmdF6hFW@public.gmane.org>

On Thu, Sep 30, 2010 at 07:15:31PM +0100, David Howells wrote:
> Make the d_mounted field in struct dentry into a more general field for special
> function directories such as mountpoints and autofs substructures.
> 
> d_mounted is renamed d_managed, and that is split into three fields:
> 
>  (*) #define DMANAGED_MOUNTPOINT	0x0fffffff

No matter how hard I try, I always read this as "DAMAGED".  Which I
can't help but imagine subliminally influences the reader's opinion of
the patches.

Of course I am excellent at naming things, see "chunkfs" and
"relatime".  But some ideas for naming various concepts in this patch:

D_MIGHT_MOUNT
D_CHILL_OUT
D_ITS_COMPLICATED

-VAL
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2010-10-08 23:57 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-30 18:14 [PATCH 00/17] Introduce automounter dentry ops David Howells
     [not found] ` <20100930181455.30939.53914.stgit-S6HVgzuS8uM4Awkfq6JHfwNdhmdF6hFW@public.gmane.org>
2010-09-30 18:15   ` [PATCH 01/17] Add a dentry op to handle automounting rather than abusing follow_link() David Howells
2010-09-30 18:15   ` [PATCH 02/17] AFS: Use d_automount() " David Howells
2010-09-30 18:15   ` [PATCH 03/17] NFS: " David Howells
2010-09-30 18:15   ` [PATCH 07/17] Make dentry::d_mounted into a more general field for special function dirs David Howells
     [not found]     ` <20100930181531.30939.10438.stgit-S6HVgzuS8uM4Awkfq6JHfwNdhmdF6hFW@public.gmane.org>
2010-10-08 23:57       ` Valerie Aurora [this message]
2010-09-30 18:15   ` [PATCH 09/17] autofs4: add d_automount() dentry operation David Howells
2010-09-30 18:15   ` [PATCH 11/17] autofs4: removed unused code David Howells
2010-09-30 18:16   ` [PATCH 13/17] autofs4: cleanup dentry operations David Howells
2010-09-30 18:16   ` [PATCH 14/17] autofs4: cleanup autofs4_free_ino() David Howells
2010-09-30 18:16   ` [PATCH 17/17] autofs4 - bump version David Howells
2010-10-01  7:57   ` [autofs] [PATCH 00/17] Introduce automounter dentry ops Stef Bon
2010-10-01 11:49     ` Ian Kent
2010-09-30 18:15 ` [PATCH 04/17] CIFS: Use d_automount() rather than abusing follow_link() David Howells
2010-09-30 18:15 ` [PATCH 05/17] Remove the automount through follow_link() kludge code from pathwalk David Howells
     [not found]   ` <20100930181521.30939.31415.stgit-S6HVgzuS8uM4Awkfq6JHfwNdhmdF6hFW@public.gmane.org>
2010-10-08 23:41     ` Valerie Aurora
2010-10-10  1:16       ` Ian Kent
2010-09-30 18:15 ` [PATCH 06/17] Add an AT_NO_AUTOMOUNT flag to suppress terminal automount David Howells
2010-10-01  8:55   ` Ian Kent
2010-09-30 18:15 ` [PATCH 08/17] Make follow_down() handle d_manage() David Howells
     [not found]   ` <20100930181536.30939.6776.stgit-S6HVgzuS8uM4Awkfq6JHfwNdhmdF6hFW@public.gmane.org>
2010-10-09  0:28     ` Valerie Aurora
2010-09-30 18:15 ` [PATCH 10/17] autofs4: add d_manage() dentry operation David Howells
2010-09-30 18:15 ` [PATCH 12/17] autofs4: cleanup inode operations David Howells
2010-09-30 18:16 ` [PATCH 15/17] autofs4 - fix wait validation David Howells
2010-09-30 18:16 ` [PATCH 16/17] autofs4 - add v4 pseudo direct mount support David Howells
     [not found] ` <4CA59482.1040508-/E4qUF3RmbfcqV2Q6a4srg@public.gmane.org>
2010-10-01  8:31   ` [autofs] [PATCH 00/17] Introduce automounter dentry ops David Howells
     [not found]     ` <8948.1285921904-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2010-10-01 18:01       ` Stef Bon
     [not found]     ` <AANLkTinoJCY=Fy_0UPWdasJg3GE5WQP0pQA8cLnir1BL-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-10-04  9:23       ` David Howells

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=20101008235741.GF30846@shell \
    --to=vaurora-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
    --cc=autofs-CPWUtch7KCBzeIdxy0IIJw@public.gmane.org \
    --cc=dhowells-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=jmoyer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=linux-afs-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-fs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=raven-PKsaG3nR2I+sTnJN9+BGXg@public.gmane.org \
    --cc=viro-rfM+Q5joDG/XmaaqVzeoHQ@public.gmane.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