Re: Proposal to improve filesystem/block snapshot interaction

2007-10-30 Thread Greg Banks
that in the filesystem, it certainly solve the problem. In which case I'll explicitly allow for the hint's extent to overlap extents previous extents thus hinted, and define the semantics for overlaps. I think I'll rename the hint to BIO_HINT_RELEASED, I think that will make the s

Re: Proposal to improve filesystem/block snapshot interaction

2007-10-30 Thread Greg Banks
On Tue, Oct 30, 2007 at 06:35:08PM +0900, Dongjun Shin wrote: > On 10/30/07, Greg Banks <[EMAIL PROTECTED]> wrote: > > > > BIO_HINT_RELEASE > > The bio's block extent is no longer in use by the filesystem > > and will not be read in the future. Any s

Re: Proposal to improve filesystem/block snapshot interaction

2007-10-29 Thread Greg Banks
get bad > old data, where as if you try to access the subordinate device of a > snapshot, you get an IO error - which is probably safer. I believe (Dave or Roger will correct me here) that XFS needs a log when you mount, and you get to either provide an external one or use the internal one.

Re: Proposal to improve filesystem/block snapshot interaction

2007-10-29 Thread Greg Banks
On Tue, Oct 30, 2007 at 12:51:47AM +0100, Arnd Bergmann wrote: > On Monday 29 October 2007, Christoph Hellwig wrote: > > - Forwarded message from Greg Banks <[EMAIL PROTECTED]> - > > > > Date: Thu, 27 Sep 2007 16:31:13 +1000 > > From: Greg Banks <[EMA

Proposal to improve filesystem/block snapshot interaction

2007-10-29 Thread Greg Banks
cks in xfs_bunmapi. The snapshot implementation can then free unnecessary copies of those blocks. Problem c) is solved by the filesystem giving to the block device a BIO_HINT_DONTCOW hint describing the block extent of the internal log, at filesystem mount time. The snapshot implementation ma

Re: [NFS] [PATH 10/19] xfs: new export ops

2007-09-14 Thread Greg Banks
On Fri, Sep 14, 2007 at 06:03:49PM +0200, Christoph Hellwig wrote: > On Sat, Sep 15, 2007 at 01:22:16AM +1000, Greg Banks wrote: > > Not really a comment on your patches, but I got the original logic > > wrong here. The VFS_32BITINODES flag only affects newly allocated >

Re: [NFS] [PATCH 2/7] NFS: if ATTR_KILL_S*ID bits are set, then skip mode change

2007-09-14 Thread Greg Banks
On Fri, Sep 14, 2007 at 10:58:38AM -0400, Jeff Layton wrote: > On Sat, 15 Sep 2007 00:40:33 +1000 > Greg Banks <[EMAIL PROTECTED]> wrote: > > > > Ok, you convinced me. > > Right. When I was first looking at this, I considered some similar > approaches, but hit

Re: [NFS] [PATH 18/19] exportfs: update documentation

2007-09-14 Thread Greg Banks
dentry for it (possibly with d_alloc_anon). > +May simplify fail if the filehandle fragment is too small. Greg. -- Greg Banks, R&D Software Engineer, SGI Australian Software Group. Apparently, I'm Bedevere. Which MPHG character are you? I don't speak for SGI. - To u

Re: [NFS] [PATH 10/19] xfs: new export ops

2007-09-14 Thread Greg Banks
f that flag; we recently saw this happen by accident on a customer site. So the right thing to do is probably to check the inode number against (u32)~0. Unfortunately, given the current encoding scheme, you have to check both the inode and the parent inode, which complicates the logic. Greg. -- Gre

Re: [NFS] [PATH 04/19] ext2: new export ops

2007-09-14 Thread Greg Banks
little helper functions in each filesystem, because you could point export_ops.fh_to_dentry directly at generic_fh_to_dentry. Greg. -- Greg Banks, R&D Software Engineer, SGI Australian Software Group. Apparently, I'm Bedevere. Which MPHG character are you? I don't speak for SGI. - T

Re: [NFS] [PATCH 2/7] NFS: if ATTR_KILL_S*ID bits are set, then skip mode change

2007-09-14 Thread Greg Banks
On Fri, Sep 14, 2007 at 09:38:46AM -0400, Jeff Layton wrote: > On Fri, 14 Sep 2007 23:09:24 +1000 > Greg Banks <[EMAIL PROTECTED]> wrote: > > > On Fri, Sep 14, 2007 at 07:02:58AM -0400, Jeff Layton wrote: > > > On Fri, 14 Sep 2007 20:25:45 +1000 > > &g

Re: [NFS] [PATCH 2/7] NFS: if ATTR_KILL_S*ID bits are set, then skip mode change

2007-09-14 Thread Greg Banks
On Fri, Sep 14, 2007 at 07:02:58AM -0400, Jeff Layton wrote: > On Fri, 14 Sep 2007 20:25:45 +1000 > Greg Banks <[EMAIL PROTECTED]> wrote: > > > I'm curious about the reasons behind this change. You mention > > credential issues; how exactly is it that you have th

Re: [NFS] [PATCH 2/7] NFS: if ATTR_KILL_S*ID bits are set, then skip mode change

2007-09-14 Thread Greg Banks
WRITE rpc but not a SETATTR rpc? Greg. -- Greg Banks, R&D Software Engineer, SGI Australian Software Group. Apparently, I'm Bedevere. Which MPHG character are you? I don't speak for SGI. - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body

Re: O_NONBLOCK for regular files

2005-04-13 Thread Greg Banks
On Wed, 2005-04-13 at 16:31, Trond Myklebust wrote: > on den 13.04.2005 Klokka 16:03 (+1000) skreiv Greg Banks: > > > Any other issues? > > > > Greg. > > Well, I'm sa bit curious as to what "iap->ia_valid |= ATTR_NO_BLOCK" > will do. AFAICS, t

Re: O_NONBLOCK for regular files

2005-04-12 Thread Greg Banks
On Wed, 2005-04-13 at 15:56, Trond Myklebust wrote: > on den 13.04.2005 Klokka 10:45 (+1000) skreiv Greg Banks: > > > There's now a patch to actually do it, see > > > > http://marc.theaimsgroup.com/?l=linux-nfs&m=111087383132762&w=2 > > > > D

Re: O_NONBLOCK for regular files

2005-04-12 Thread Greg Banks
G'day, On Sat, 2004-03-20 at 20:24, Greg Banks wrote: > Jamie Lokier wrote: > > Christoph Hellwig wrote: > > > It's used to [...] return EAGAIN to the nfs server so it doesn't block all > > > nfsd threads on possible long-enduring HSM operations. &g

Re: [NFS] [PATCH] SGI 926917: make knfsd interact cleanly with HSMs

2005-03-30 Thread Greg Banks
On Thu, 2005-03-31 at 11:58, Neil Brown wrote: > On Thursday March 31, [EMAIL PROTECTED] wrote: > > On Tue, 2005-03-15 at 18:49, Greg Banks wrote: > > > This patch seeks to remedy the interaction between knfsd and HSMs by > > > providing mechanisms to allow knfsd to te