On Tue, Mar 15, 2016 at 12:42:44PM -0700, Darrick J. Wong wrote:
quoted hunk ↗ jump to hunk
#include <linux/cleancache.h>
#include <linux/dax.h>
#include <asm/uaccess.h>
+#include <linux/falloc.h>
Maybe keep this before asm/uaccess.h
+long blkdev_fallocate(struct file *file, int mode, loff_t start, loff_t len)
should be marked static.
quoted hunk ↗ jump to hunk
+ /* We haven't a primitive for "ensure space exists" right now. */
+ if (!(mode & ~FALLOC_FL_KEEP_SIZE))
+ return -EOPNOTSUPP;
I don't really understand the comment. But I think you'd be much
better off with having blkdev_fallocate as just a tiny wrapper that has
a switch for the supported modes, e.g.
switch (mode) {
case FALLOC_FL_PUNCH_HOLE | FALLOC_FL_KEEP_SIZE:
return blkdev_punch_hole();
case FALLOC_FL_ZERO_RANGE | FALLOC_FL_KEEP_SIZE::
return blkdev_zero_range();
default:
return -EOPNOTSUPP;
}
+EXPORT_SYMBOL_GPL(blkdev_fallocate);
and no need to export it either..