Re: lseek

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Ian,

lseek calls are handled internally by the kernel and they never reach the user land for fuse calls. lseek only updates the current file offset that is stored inside the kernel file's structure. This value is what is passed to read/write fuse calls as an absolute offset.

There isn't any problem in this behavior as long as you hide all size manipulations from fuse. If you write a translator that compresses a file, you should do so in a transparent manner. This means, basically, that:

1. Whenever you are asked to return the file size, you must return the size of the uncompressed file 2. Whenever you receive an offset, you must translate that offset to the corresponding offset in the compressed file and work with that 3. Whenever you are asked to read or write data, you must return the number of uncompressed bytes read or written (even if you have compressed the chunk of data to a smaller size and you have physically written less bytes). 4. All read requests must return uncompressed data (this seems obvious though)

This guarantees that your manipulations are not seen in any way by any upper translator or even fuse, thus everything should work smoothly.

If you respect these rules, lseek (and your translator) will work as expected.

In particular, when a user calls lseek with SEEK_END, the kernel takes the size of the file from the internal kernel inode's structure. This size is obtained through a previous call to lookup or updated using the result of write operations. If you respect points 1 and 3, this value will be correct.

In gluster there are a lot of fops that return a iatt structure. You must guarantee that all these functions return the correct size of the file in the field ia_size to be sure that everything works as expected.

Xavi

On 05/14/2012 11:51 AM, Ian Latter wrote:
Hello Xavi,


   Ok - thanks.  I was hoping that this was how read
and write were working (i.e. with absolute offsets
and not just getting relative offsets from the current
seek point), however what of the raw seek
command?

      len = lseek(fd, 0, SEEK_END);

      Upon  successful completion, lseek() returns
      the resulting offset location as measured in
      bytes from the beginning of the  file.

   Any idea on where the return value comes from?
I will need to fake up a file size for this command ..



----- Original Message -----
From: "Xavier Hernandez"<xhernandez@xxxxxxxxxx>
To:<gluster-devel@xxxxxxxxxx>
Subject:  Re: lseek
Date: Mon, 14 May 2012 09:48:17 +0200

Hello Ian,

there is no such thing as an explicit seek in glusterfs.
Each readv,
writev, (f)truncate and rchecksum have an offset parameter
that tells
you the position where the operation must be performed.

If you make something that changes the size of the file
you must make it
in a way that it is transparent to upper translators. This
means that
all offsets you will receive are "real" (in your case,
offsets in the
uncompressed version of the file). You should calculate in
some way the
equivalent offset in the compressed version of the file
and send it to
the correspoding fop of the lower translators.

In the same way, you must return in all iatt structures
the real size of
the file (not the compressed size).

I'm not sure what is the intended use of NONSEEKABLE, but
I think it is
for special file types, like devices or similar that are
sequential in
nature. Anyway, this is a fuse flag that you can't return
from a regular
translator open fop.

Xavi

On 05/14/2012 03:22 AM, Ian Latter wrote:
Hello,


    I'm looking for a seek (lseek) implementation in
one of the modules and I can't see one.

    Do I need to care about seeking if my module
changes the file size (i.e. compresses) in Gluster?
I would have thought that I did except that I believe
that what I'm reading is that Gluster returns a
NONSEEKABLE flag on file open (fuse_kernel.h at
line 149).  Does this mitigate the need to correct
the user seeks?


Cheers,



--
Ian Latter
Late night coder ..
http://midnightcode.org/

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxx
https://lists.nongnu.org/mailman/listinfo/gluster-devel
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxx
https://lists.nongnu.org/mailman/listinfo/gluster-devel


--
Ian Latter
Late night coder ..
http://midnightcode.org/

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxx
https://lists.nongnu.org/mailman/listinfo/gluster-devel




[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux