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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080914135138.GC21746@infradead.org>
Date:	Sun, 14 Sep 2008 09:51:38 -0400
From:	Christoph Hellwig <hch@...radead.org>
To:	Anton Altaparmakov <aia21@....ac.uk>
Cc:	Theodore Tso <tytso@....EDU>, linux-fsdevel@...r.kernel.org,
	linux-ext4@...r.kernel.org, akpm@...uxfoundation.org,
	Mark Fasheh <mfasheh@...e.com>
Subject: Re: [PATCH 1/4] vfs: vfs-level fiemap interface

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.

--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html