Re: [PATCH 1/4] vfs: vfs-level fiemap interface

Christoph Hellwig
2008-09-14T13:51:40Z

On Sat, Sep 13, 2008 at 10:59:40PM +0100, Anton Altaparmakov wrote:
>> In fact, the previous version of the patch had a flag for compression,
>> but Cristoph Hellwig objected because he claimed there were no users
>> for it.
>
> He is plain wrong then! NTFS supports compressed files and always has
> done! We only support them read-only at present but that does not
> matter for the purposes of FIEMAP. And in any case one day we will have
> write support for compressed files, too...

Note that I didn't object it because there are not user, this is just
Ted making up BS because he's frustrated. I said don't add it until we
actually have users so we know their requirements. The nice part about
this interface is that there is no problem adding more flags once we
need them, if it's just flags, and if we need mor information than just
the flags we're better of discussing these bits beforehand.

> We already have two file systems in the mainline kernel tree that
> support transparent compression/decompression: ntfs and zisofs so there
> is no "future" anywhere in sight... (-;

or cramfs. But no one involved with this patch srries bothered actually
looking into the requirements.