[PATCH] xfs_io: allow resvsp command for foreign filesystems

[PATCH] xfs_io: allow resvsp command for foreign filesystems

[Christoph Hellwig]

Recent kernels implement the RESVP ioctl in common code, so using
command for non-XFS filesystems is fine.

Signed-off-by: Christoph Hellwig <hch@lst.de>

Index: xfsprogs-dev/io/prealloc.c
===================================================================
--- xfsprogs-dev.orig/io/prealloc.c	2010-07-21 19:11:34.943494680 +0200
+++ xfsprogs-dev/io/prealloc.c	2010-07-21 19:11:57.093494680 +0200
@@ -182,7 +182,7 @@ prealloc_init(void)
 	resvsp_cmd.cfunc = resvsp_f;
 	resvsp_cmd.argmin = 2;
 	resvsp_cmd.argmax = 2;
-	resvsp_cmd.flags = CMD_NOMAP_OK;
+	resvsp_cmd.flags = CMD_NOMAP_OK | CMD_FOREIGN_OK;
 	resvsp_cmd.args = _("off len");
 	resvsp_cmd.oneline =
 		_("reserves space associated with part of a file");

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

Re: [PATCH] xfs_io: allow resvsp command for foreign filesystems

[Nathan Scott]

----- "Christoph Hellwig" <hch@infradead.org> wrote:

> Recent kernels implement the RESVP ioctl in common code, so using
> command for non-XFS filesystems is fine.
> 
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> 

Looks good.

Reviewed-by: Nathan Scott <nathans@debian.org>


cheers.

-- 
Nathan

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs