]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
MF stable/12 r366190:
authorAlan Somers <asomers@FreeBSD.org>
Mon, 28 Sep 2020 00:23:59 +0000 (00:23 +0000)
committerAlan Somers <asomers@FreeBSD.org>
Mon, 28 Sep 2020 00:23:59 +0000 (00:23 +0000)
commitdf309babcf29db1895ec5fe6bc7f5289231a3b31
treee27f063fbf8ba99158fa139a46f358c4c84fa76e
parent7d38c4af1773bf0278414e4cf60d540e0a280eb6
MF stable/12 r366190:

fusefs: fix mmap'd writes in direct_io mode

If a FUSE server returns FOPEN_DIRECT_IO in response to FUSE_OPEN, that
instructs the kernel to bypass the page cache for that file. This feature
is also known by libfuse's name: "direct_io".

However, when accessing a file via mmap, there is no possible way to bypass
the cache completely. This change fixes a deadlock that would happen when
an mmap'd write tried to invalidate a portion of the cache, wrongly assuming
that a write couldn't possibly come from cache if direct_io were set.

Arguably, we could instead disable mmap for files with FOPEN_DIRECT_IO set.
But allowing it is less likely to cause user complaints, and is more in
keeping with the spirit of open(2), where O_DIRECT instructs the kernel to
"reduce", not "eliminate" cache effects.

PR: 247276
Approved by: re (gjb)
Reported by: trapexit@spawn.link
Reviewed by: cem
Differential Revision: https://reviews.freebsd.org/D26485
sys/fs/fuse/fuse_io.c
tests/sys/fs/fusefs/write.cc