All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Matt W. Benjamin" <matt@linuxbox.com>
To: ceph-devel <ceph-devel@vger.kernel.org>
Cc: Sage Weil <sage@inktank.com>, Gregory Farnum <greg@inktank.com>,
	David Zafman <david.zafman@inktank.com>
Subject: Re: ceph caps (Ganesha + Ceph pnfs)
Date: Sat, 5 Jan 2013 11:23:17 -0500 (EST)	[thread overview]
Message-ID: <674860131.10.1357402997087.JavaMail.root@thunderbeast.private.linuxbox.com> (raw)
In-Reply-To: <507490260.8.1357402950428.JavaMail.root@thunderbeast.private.linuxbox.com>

correction:

I mixed stuff together in the previous summary.

If we're committing a final state of an inode (pNFS LAYOUTCOMMIT), it's done from the MDS, which does hold caps.  The prototype DS doesn't currently handle the sync case using the bypass i/o path.  But the prototype up to now has only used that path for unstable DS writes.  If the DS uses the bypass data path, it holds no caps (currently; we might want to change that, perhaps optionally, to get fencing).  To get the spec behavior for stable DS writes, the DS instead does an ordinary ll_write--this deals with metadata correctly but it's not what we want.    To compose the bypass data path with stability, the DS gains the obligation to update inode state itself.  Looking at whether the DS could do this using Ceph protocol, is what led me to start the conversation.

Thanks,

Matt

----- "Matt W. Benjamin" <matt@linuxbox.com> wrote:

> b) on a given DS, we currently get CEPH_CAP_FILE_WR|CEPH_CAP_FILE_RD
> caps when asked to perform i/o on behalf of a valid layout--but we
> need to update metadata (size, mtime) and my question in IRC was cross
> checking these capabilities as correct to send an update message


-- 
Matt Benjamin
The Linux Box
206 South Fifth Ave. Suite 150
Ann Arbor, MI  48104

http://linuxbox.com

tel. 734-761-4689
fax. 734-769-8938
cel. 734-216-5309

       reply	other threads:[~2013-01-05 16:23 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <507490260.8.1357402950428.JavaMail.root@thunderbeast.private.linuxbox.com>
2013-01-05 16:23 ` Matt W. Benjamin [this message]
     [not found] <1538446321.14.1357663643915.JavaMail.root@thunderbeast.private.linuxbox.com>
2013-01-08 17:11 ` ceph caps (Ganesha + Ceph pnfs) Matt W. Benjamin
2013-01-10  1:47   ` Sage Weil
     [not found] <681824234.175.1357346630910.JavaMail.root@thunderbeast.private.linuxbox.com>
2013-01-05  0:51 ` Matt W. Benjamin
2013-01-05 16:36   ` Sage Weil
2013-01-05 17:29     ` Matt W. Benjamin
2013-01-08  0:23       ` Sage Weil

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=674860131.10.1357402997087.JavaMail.root@thunderbeast.private.linuxbox.com \
    --to=matt@linuxbox.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=david.zafman@inktank.com \
    --cc=greg@inktank.com \
    --cc=sage@inktank.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.