[PATCH v4 00/14] add compression ioctl support

Jeremy Allison jra at samba.org
Thu Nov 21 18:01:01 MST 2013


On Thu, Nov 21, 2013 at 04:57:59PM -0800, Jeremy Allison wrote:
> On Mon, Nov 18, 2013 at 02:54:25PM +0100, David Disseldorp wrote:
> > This patch series adds support for the FSCTL_GET_COMPRESSION and
> > FSCTL_SET_COMPRESSION ioctls, as well as the reporting of the current
> > compression state via the FILE_ATTRIBUTE_COMPRESSED flag.
> > 
> > Hooks are added to the Btrfs VFS module, which translates such requests
> > into corresponding FS_IOC_GETFLAGS and FS_IOC_SETFLAGS ioctls.
> > 
> > Changes since v3 (thanks for the feedback Jeremy):
> > - fixed and split copy-chunk dest unlock change into separate commit
> > 
> > Changes since v2:
> > - Check for valid fsp file descriptors
> > - Rebase atop filesystem specific selftest changes
> > - Change compression fsctl permission checks to match Windows behaviour
> >   + Add corresponding smbtorture test
> > 
> > Changes since v1:
> > - Only use smb_fname and fsp args with GET_COMPRESSION. The smb_fname
> >   argument is only needed for the dosmode() code-path, fsp is used
> >   everywhere else.
> > - Add an extra SetInfo(FILE_ATTRIBUTE_COMPRESSED) test.
> 
> This looks *really* good David, thanks !
> 
> Just running it through make test now, and then
> I'll push to autobuild (this may be tomorrow).
> 
> If you don't see it in the tree by the time you
> get up, feel free to push the patchset with my
> "Reviewed-by" tag attached.

Oh hang on a minute, I think one of the headers
now pulled into the tree conflicts with the
WRITE_FLUSH #define we have....

Give me a little more time...


More information about the samba-technical mailing list