lkml.org 
[lkml]   [2010]   [Feb]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [fuse-devel] [PATCH] FUSE/CUSE: implement direct mmap support
From
Date
On Thu, 11 Feb 2010, Miklos Szeredi wrote:
> On one hand it's simple, on the other it has pretty weird limitations,
> considering that server side mmap shouldn't be mandatory. But of
> course if server side mmap isn't used, then the SHMLBA limitation is
> not necessary anymore so the implementation could choose an arbitrary
> "offset".
>
> My biggest gripe with the kernel API is that we shouldn't be calling
> that thing in fuse_mmap_out an offset at all, because it's not and
> it's confusing (like making you set vma->vm_pgoff to that value, which
> is bogus). Adding a separate "page_id" or whatever would make me
> happy.

And even a page_id is superfluous, since all we want is differentiate
between possibly separate maps for the same file and don't care about
individual pages.

And even differentiating between maps on the same file is only ever
possible for char devs, normal files will only ever have a single map.

So instead of "page_id" it could just be a "map_id", which will be
zero for normal maps, and whatever for /dev/fuse maps.

> And if the server wants to mmap /dev/fuse then it can do that and send
> the result in "dev_offset", to make it clear that it's a different
> offset from the one the client used on the mmap. And it can even use
> that value as page_id, if it wants to or it can use a different
> page_id.

And there's a weakness in the current server side mmap interface, for
example:

1) user mmaps region 3-5 (in page index)
server maps this region at 3-5

2) user mmaps region 1-7
server can't tell kernel that it wants to reuse region 3-5 but
wants to create two other regions

What happens in that case?

On the other hand if the map_id is separate from the dev_offset, then
the server can map the second region as one map and the kernel can
connect it up with the right pages based on the map_id returned in the
MMAP reply.

Thanks,
Miklos


\
 
 \ /
  Last update: 2010-02-11 10:55    [W:0.101 / U:0.236 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site