linux-kernel - Re: commit 0b5da8d: fuse: add support for SEEK_HOLE and SEEK

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] [day] [month] [year] [list]

Date:   Thu, 17 Nov 2016 07:04:26 +0530
From:   Ravishankar N <ravishankar@...hat.com>
To:     Miklos Szeredi <mszeredi@...hat.com>,
        linux-fsdevel <linux-fsdevel@...r.kernel.org>,
        linux-kernel <linux-kernel@...r.kernel.org>,
        fuse-devel <fuse-devel@...ts.sourceforge.net>
Subject: Re: commit 0b5da8d: fuse: add support for SEEK_HOLE and SEEK_DATA in
 lseek

Hi Nikolaus,

On 11/17/2016 01:39 AM, Nikolaus Rath wrote:
> Hi Ravishankar,
>
> In commit 0b5da8d you added support for a new FUSE_LSEEK
> operation. However, as far as I can tell the corresponding userspace
> code never landed in libfuse.
>
> Do you have a corresponding patch for libfuse?
No, unfortunately. I only used gluster to test what I wrote.
>
> Looking at the commit message, I assume this functionality was added
> specifically for glusterfs which presumably does not use libfuse.

You're right, gluster has its own code [1].
> But
> even in that case it would be nice if you could Cc
> fuse-devel@...ts.sourceforge.net
The discussion[2] actually did start on fuse-devel.
> so that someone else can add support
That would be great.
> for this. This is important because AFAIR the libfuse documentation is
> the only thing that comes close to an official documentation of the FUSE
> API.
>

Best regards,
Ravi


[1] 
https://github.com/gluster/glusterfs/blob/master/xlators/mount/fuse/src/fuse-bridge.c
[2] https://sourceforge.net/p/fuse/mailman/message/34227449/
> Best,
> -Nikolaus
>