All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@infradead.org>
To: Alex Lyakas <alex@zadarastorage.com>
Cc: Christoph Hellwig <hch@infradead.org>, linux-xfs@vger.kernel.org
Subject: Re: xfs_alloc_ag_vextent_near() takes minutes to complete
Date: Sun, 7 May 2017 02:12:09 -0700	[thread overview]
Message-ID: <20170507091209.GA6102@infradead.org> (raw)
In-Reply-To: <CAOcd+r3VYuLaCq6g3YH_ue_7AhPfjfVUHJVmn1S=vFOtEMuZag@mail.gmail.com>

On Sun, May 07, 2017 at 11:00:40AM +0300, Alex Lyakas wrote:
> We are using the sync mount option, because the XFS instances that we
> have are exposed via nfsd. Without the "sync" mount option, data from
> NFS write command would end up sitting in page cache, and the nfs
> client would not know when it has been finally persisted on disk.
> Is the any other way that you can recommend to provide data integrity
> guarantee for nfs clients? Even if it requires some development, like
> placing the data in a short-term persistent space, this is something
> we will look at.

You don't need any mount option for NFS exports.  Only NFSv2 requires
symchronous writes, while NFS3+ have the concept of unstable writes
that the client needs to explicitly committ using the COMMIT on the
write operation.  Both the NFSv2 synchronous and the NFSv3+ unstable
semantics are managed by NFSD.

(I'll speak here as a XFS and NFSD developer, and someone involved in
NFS protocol development, just in case you're doubting)

> 
> Thanks,
> Alex.
---end quoted text---

      reply	other threads:[~2017-05-08  1:11 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-01 12:37 xfs_alloc_ag_vextent_near() takes minutes to complete Alex Lyakas
2017-05-01 15:26 ` Brian Foster
2017-05-02  7:35 ` Christoph Hellwig
2017-05-04  8:07   ` Alex Lyakas
2017-05-04 11:13     ` Alex Lyakas
2017-05-04 12:29       ` Brian Foster
2017-05-04 12:25     ` Brian Foster
2017-05-04 13:53       ` Alex Lyakas
2017-05-05  3:29     ` Dave Chinner
2017-05-07  7:52       ` Alex Lyakas
2017-05-07  8:00   ` Alex Lyakas
2017-05-07  9:12     ` Christoph Hellwig [this message]

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=20170507091209.GA6102@infradead.org \
    --to=hch@infradead.org \
    --cc=alex@zadarastorage.com \
    --cc=linux-xfs@vger.kernel.org \
    /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.