Re: [PATCH] hpfs: add fstrim support

2015-06-28 Thread Linus Torvalds
On Sun, Jun 28, 2015 at 1:59 PM, Al Viro wrote: > > More to the point, why bother with ->ioctl() at all? Why not make > ->fitrim() a super_block method and let do_vfs_ioctl() handle all > marshalling? As in > (int *)fitrim(struct super_block *, struct fstrim_range *); > guaranteed to be

Re: [PATCH] hpfs: add fstrim support

2015-06-28 Thread Al Viro
On Sun, Jun 28, 2015 at 12:52:11PM -0700, Linus Torvalds wrote: > On Sun, Jun 28, 2015 at 6:16 AM, Mikulas Patocka > wrote: > > This patch adds support for fstrim to the HPFS filesystem. > ... > > +#ifdef CONFIG_COMPAT > > + .compat_ioctl = hpfs_compat_ioctl, > > +#endif > ... > > +#ifdef

Re: [PATCH] hpfs: add fstrim support

2015-06-28 Thread Linus Torvalds
On Sun, Jun 28, 2015 at 6:16 AM, Mikulas Patocka wrote: > This patch adds support for fstrim to the HPFS filesystem. ... > +#ifdef CONFIG_COMPAT > + .compat_ioctl = hpfs_compat_ioctl, > +#endif ... > +#ifdef CONFIG_COMPAT > + .compat_ioctl = hpfs_compat_ioctl, > +#endif ... > +#ifd

[PATCH] hpfs: add fstrim support

2015-06-28 Thread Mikulas Patocka
This patch adds support for fstrim to the HPFS filesystem. Signed-off-by: Mikulas Patocka --- fs/hpfs/alloc.c | 95 ++ fs/hpfs/dir.c |4 ++ fs/hpfs/file.c|4 ++ fs/hpfs/hpfs_fn.h |7 +++ fs/hpfs/super.c | 35 +++