xfs
[Top] [All Lists]

Re: [PATCH v5 06/10] xfs: add XFS_IOC_FREE_EOFBLOCKS ioctl

To: Brian Foster <bfoster@xxxxxxxxxx>
Subject: Re: [PATCH v5 06/10] xfs: add XFS_IOC_FREE_EOFBLOCKS ioctl
From: Ben Myers <bpm@xxxxxxx>
Date: Mon, 15 Oct 2012 17:46:26 -0500
Cc: xfs@xxxxxxxxxxx
In-reply-to: <507749A2.4020206@xxxxxxxxxx>
References: <1349446636-8611-1-git-send-email-bfoster@xxxxxxxxxx> <1349446636-8611-7-git-send-email-bfoster@xxxxxxxxxx> <20121011141335.GY13214@xxxxxxx> <507749A2.4020206@xxxxxxxxxx>
User-agent: Mutt/1.5.20 (2009-06-14)
Hey Brian,

On Thu, Oct 11, 2012 at 06:35:14PM -0400, Brian Foster wrote:
> On 10/11/2012 10:13 AM, Ben Myers wrote:
> > Hey Brian,
> > 
> > On Fri, Oct 05, 2012 at 10:17:12AM -0400, Brian Foster wrote:
> >> The XFS_IOC_FREE_EOFBLOCKS ioctl allows users to invoke an EOFBLOCKS
> >> scan. The xfs_eofblocks structure is defined to support the command
> >> parameters (scan mode).
> > 
> > It would help to have an xfstest to exercise this ioctl to pull in with this
> > series.  Do you have any code that could be wrangled into a test case?
> > 
> 
> Yes, makes sense. I have some very basic test code I could put somewhere
> to invoke the ioctl(). One of the questions I've been meaning to ask is
> whether it would be relevant for that code to live in a common tool,
> such as adding a new command to xfs_io. Then perhaps create an xfstests
> test using that. Thoughts?

IMO you are right on the mark.  xfs_io is a great place for this.

> FYI, I have a few other things on my plate at the moment so
> unfortunately it will be a bit before I can get back to XFS work... But
> I'm fine with the set pending until I can come up with some test
> coverage if that is preferable, of course.

I do think it is preferable to have a test case go in with the code where
possible.  Since you don't mind waiting a bit, that seems to be the way to go.
The other option could be to look for a volunteer to work on the test.  ;)

Regards,
        Ben

> Brian
> 
> > Regards,
> >     Ben
> > 
> >>
> >> Signed-off-by: Brian Foster <bfoster@xxxxxxxxxx>
> >> ---
> >>  fs/xfs/xfs_fs.h     |   15 +++++++++++++++
> >>  fs/xfs/xfs_icache.c |    5 +++--
> >>  fs/xfs/xfs_icache.h |    2 +-
> >>  fs/xfs/xfs_ioctl.c  |   16 ++++++++++++++++
> >>  4 files changed, 35 insertions(+), 3 deletions(-)
> >>
> >> diff --git a/fs/xfs/xfs_fs.h b/fs/xfs/xfs_fs.h
> >> index c13fed8..5f48b3e 100644
> >> --- a/fs/xfs/xfs_fs.h
> >> +++ b/fs/xfs/xfs_fs.h
> >> @@ -339,6 +339,20 @@ typedef struct xfs_error_injection {
> >>  
> >>  
> >>  /*
> >> + * Speculative preallocation trimming.
> >> + */
> >> +#define XFS_EOFBLOCKS_VERSION             1
> >> +struct xfs_eofblocks {
> >> +  __u32           eof_version;
> >> +  __u32           eof_flags;
> >> +  unsigned char   pad[12];
> >> +};
> >> +
> >> +/* eof_flags values */
> >> +#define XFS_EOF_FLAGS_SYNC                0x01    /* sync/wait mode scan 
> >> */
> >> +
> >> +
> >> +/*
> >>   * The user-level Handle Request interface structure.
> >>   */
> >>  typedef struct xfs_fsop_handlereq {
> >> @@ -456,6 +470,7 @@ typedef struct xfs_handle {
> >>  /*        XFS_IOC_GETBIOSIZE ---- deprecated 47      */
> >>  #define XFS_IOC_GETBMAPX  _IOWR('X', 56, struct getbmap)
> >>  #define XFS_IOC_ZERO_RANGE        _IOW ('X', 57, struct xfs_flock64)
> >> +#define XFS_IOC_FREE_EOFBLOCKS    _IOR ('X', 58, struct xfs_eofblocks)
> >>  
> >>  /*
> >>   * ioctl commands that replace IRIX syssgi()'s
> >> diff --git a/fs/xfs/xfs_icache.c b/fs/xfs/xfs_icache.c
> >> index b02a3df..35efdda 100644
> >> --- a/fs/xfs/xfs_icache.c
> >> +++ b/fs/xfs/xfs_icache.c
> >> @@ -1206,11 +1206,12 @@ xfs_inode_free_eofblocks(
> >>  int
> >>  xfs_icache_free_eofblocks(
> >>    struct xfs_mount        *mp,
> >> -  int                     flags)
> >> +  int                     flags,
> >> +  struct xfs_eofblocks    *eofb)
> >>  {
> >>    ASSERT((flags & ~(SYNC_TRYLOCK|SYNC_WAIT)) == 0);
> >>    return xfs_inode_ag_iterator_tag(mp, xfs_inode_free_eofblocks, flags,
> >> -                                   NULL, XFS_ICI_EOFBLOCKS_TAG);
> >> +                                   eofb, XFS_ICI_EOFBLOCKS_TAG);
> >>  }
> >>  
> >>  void
> >> diff --git a/fs/xfs/xfs_icache.h b/fs/xfs/xfs_icache.h
> >> index abca9fb..b46716c 100644
> >> --- a/fs/xfs/xfs_icache.h
> >> +++ b/fs/xfs/xfs_icache.h
> >> @@ -37,7 +37,7 @@ void xfs_inode_set_reclaim_tag(struct xfs_inode *ip);
> >>  
> >>  void xfs_inode_set_eofblocks_tag(struct xfs_inode *ip);
> >>  void xfs_inode_clear_eofblocks_tag(struct xfs_inode *ip);
> >> -int xfs_icache_free_eofblocks(struct xfs_mount *, int);
> >> +int xfs_icache_free_eofblocks(struct xfs_mount *, int, struct 
> >> xfs_eofblocks *);
> >>  
> >>  int xfs_sync_inode_grab(struct xfs_inode *ip);
> >>  int xfs_inode_ag_iterator(struct xfs_mount *mp,
> >> diff --git a/fs/xfs/xfs_ioctl.c b/fs/xfs/xfs_ioctl.c
> >> index 0e0232c..ad4352f 100644
> >> --- a/fs/xfs/xfs_ioctl.c
> >> +++ b/fs/xfs/xfs_ioctl.c
> >> @@ -42,6 +42,7 @@
> >>  #include "xfs_inode_item.h"
> >>  #include "xfs_export.h"
> >>  #include "xfs_trace.h"
> >> +#include "xfs_icache.h"
> >>  
> >>  #include <linux/capability.h>
> >>  #include <linux/dcache.h>
> >> @@ -1602,6 +1603,21 @@ xfs_file_ioctl(
> >>            error = xfs_errortag_clearall(mp, 1);
> >>            return -error;
> >>  
> >> +  case XFS_IOC_FREE_EOFBLOCKS: {
> >> +          struct xfs_eofblocks eofb;
> >> +          int flags;
> >> +
> >> +          if (copy_from_user(&eofb, arg, sizeof(eofb)))
> >> +                  return -XFS_ERROR(EFAULT);
> >> +
> >> +          if (eofb.eof_version != XFS_EOFBLOCKS_VERSION)
> >> +                  return -XFS_ERROR(EINVAL);
> >> +
> >> +          flags = (eofb.eof_flags & XFS_EOF_FLAGS_SYNC) ? SYNC_WAIT : 
> >> SYNC_TRYLOCK;
> >> +          error = xfs_icache_free_eofblocks(mp, flags, &eofb);
> >> +          return -error;
> >> +  }
> >> +
> >>    default:
> >>            return -ENOTTY;
> >>    }
> >> -- 
> >> 1.7.7.6
> >>
> >> _______________________________________________
> >> xfs mailing list
> >> xfs@xxxxxxxxxxx
> >> http://oss.sgi.com/mailman/listinfo/xfs
> 

<Prev in Thread] Current Thread [Next in Thread>