Subject: Re: commit 0b5da8d: fuse: add support for SEEK_HOLE and SEEK_DATA in
 lseek
To: Miklos Szeredi <mszeredi@redhat.com>,
        linux-fsdevel <linux-fsdevel@vger.kernel.org>,
        linux-kernel <linux-kernel@vger.kernel.org>,
        fuse-devel <fuse-devel@lists.sourceforge.net>
References: <87d1hvmbjh.fsf@thinkpad.rath.org>
From: Ravishankar N <ravishankar@redhat.com>
Message-ID: <2756c158-de24-01ac-8a52-4a9a332ff5e8@redhat.com>
Date: Thu, 17 Nov 2016 07:04:26 +0530
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101
 Thunderbird/45.4.0
MIME-Version: 1.0
In-Reply-To: <87d1hvmbjh.fsf@thinkpad.rath.org>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1098
Lines: 37

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@lists.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
>