| To: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 1/1] XFS: replace *_IDELETE with *_IKEEP |
| From: | Eric Sandeen <sandeen@xxxxxxxxxxx> |
| Date: | Fri, 15 Feb 2008 10:23:43 -0600 |
| Cc: | "Josef 'Jeff' Sipek" <jeffpc@xxxxxxxxxxxxxx>, xfs@xxxxxxxxxxx |
| In-reply-to: | <20080215161902.GA32398@infradead.org> |
| References: | <47B3B6AE.4030505@sandeen.net> <1202975139-10546-1-git-send-email-jeffpc@josefsipek.net> <20080215161902.GA32398@infradead.org> |
| Sender: | xfs-bounce@xxxxxxxxxxx |
| User-agent: | Thunderbird 2.0.0.9 (Macintosh/20071031) |
Christoph Hellwig wrote: > On Thu, Feb 14, 2008 at 02:45:39AM -0500, Josef 'Jeff' Sipek wrote: >> Change the *_IDELETE flags to *_IKEEP, and flip the logic as necessary. >> >> This completely eliminates the no-no-no-idelete madness. >> >> Additionally, "ikeep" or "noikeep" is always displayed in /proc/mounts >> option string. This should help clear up any confusion about what the >> current mode is. > > Looks fine to me, and I think the changed display in /proc/<pid>/mounts > is fine aswell. > IMHO if we want to display defaults, then we should probably change it so that all defaults are displayed, and not make noikeep special in this respect. (oh, and "noquota" is already there, too) Doesn't matter much to me either way but it should be consistent across all options, I think. -Eric |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH 1/1] XFS: replace *_IDELETE with *_IKEEP, Christoph Hellwig |
|---|---|
| Next by Date: | Marking inode dirty latency > 1000 msec on XFS!, Török Edwin |
| Previous by Thread: | Re: [PATCH 1/1] XFS: replace *_IDELETE with *_IKEEP, Christoph Hellwig |
| Next by Thread: | Re: [PATCH 1/1] XFS: replace *_IDELETE with *_IKEEP, Christoph Hellwig |
| Indexes: | [Date] [Thread] [Top] [All Lists] |