From: Nikolaus Rath <Nikolaus@rath.org>
To: Maxim Patlasov <mpatlasov@virtuozzo.com>
Cc: mszeredi@redhat.com, fuse-devel@lists.sourceforge.net,
linux-fsdevel <linux-fsdevel@vger.kernel.org>
Subject: Re: [fuse-devel] [fuse] getattr() results ignored when writeback cache is active
Date: Fri, 22 Sep 2017 10:34:15 +0100 [thread overview]
Message-ID: <87shffyt48.fsf@vostro.rath.org> (raw)
In-Reply-To: <b270fa6e-d4ff-514d-0a31-b78f1bd9c2d9@virtuozzo.com> (Maxim Patlasov's message of "Thu, 21 Sep 2017 12:44:58 -0700")
On Sep 21 2017, Maxim Patlasov <mpatlasov@virtuozzo.com> wrote:
> Oh, I understand now. Your example actually demonstrates that
> FUSE_WRITEBACK_CACHE must not be used if it's possible to modify
> underlying data (/tmp/issue_93/file_1) externally w.r.t fuse
> filesystem. That's correct. But initial statement:
>
>> this mode of operation is not suitable for any network filesystem
>> even if no write operations are actually carried out
>
> is not, because it's not impossible to protect underlying data against
> such an external change somehow. A network filesystem is not obliged
> to keep transparent correspondence like /tmp/issue_93_mnt/file_1 <-->
> /tmp/issue_93/file_1. It can keep all user data (and metadata) in
> internal structures making any external modifications
> impossible. Then, implementing exclusive write semantics would make
> using FUSE_WRITEBACK_CACHE safe. Agree?
I agree with what you're saying, but I don't agree that this should go
into the documentation for which the sentence in dispute was intended
:-). I think in 93% of all cases, the simplified statement will be more
helpful to a potential FUSE user than the correct one.
If this bothers you a lot, can we compromise and just be a little more
vague overall? I.e.,
| This mode of operation is therefore generally not suitable for
| filesystems where modifications may not come through the (local) FUSE
| layer (i.e, most network filesystems).
Best,
-Nikolaus
--
GPG Fingerprint: ED31 791B 2C5C 1613 AF38 8B8A D113 FCAC 3C4E 599F
»Time flies like an arrow, fruit flies like a Banana.«
next prev parent reply other threads:[~2017-09-25 20:01 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-20 11:50 [fuse] getattr() results ignored when writeback cache is active Nikolaus Rath
2017-09-20 15:23 ` Miklos Szeredi
2017-09-20 15:31 ` [fuse-devel] " Nikolaus Rath
2017-09-20 15:59 ` Miklos Szeredi
2017-09-20 16:37 ` Nikolaus Rath
2017-09-21 15:04 ` Miklos Szeredi
2017-09-21 17:59 ` Nikolaus Rath
2017-09-25 9:12 ` Miklos Szeredi
2017-09-25 9:25 ` Nikolaus Rath
2017-09-21 10:12 ` Nikolaus Rath
2017-09-21 17:28 ` Maxim Patlasov
2017-09-21 17:53 ` Nikolaus Rath
2017-09-21 18:24 ` Maxim Patlasov
2017-09-21 18:31 ` Nikolaus Rath
2017-09-21 18:45 ` Maxim Patlasov
2017-09-21 19:21 ` Nikolaus Rath
2017-09-21 19:44 ` Maxim Patlasov
2017-09-22 9:34 ` Nikolaus Rath [this message]
2017-09-26 6:55 ` Maxim Patlasov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87shffyt48.fsf@vostro.rath.org \
--to=nikolaus@rath.org \
--cc=fuse-devel@lists.sourceforge.net \
--cc=linux-fsdevel@vger.kernel.org \
--cc=mpatlasov@virtuozzo.com \
--cc=mszeredi@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.