cluster-devel.redhat.com archive mirror
 help / color / mirror / Atom feed
From: david m. richter <richterd@gmail.com>
To: cluster-devel.redhat.com
Subject: [Cluster-devel] gfs uevent and sysfs changes
Date: Thu, 4 Dec 2008 13:32:31 -0500	[thread overview]
Message-ID: <1d07ca700812041032o6f82fecew3fb93545fe64ed2d@mail.gmail.com> (raw)
In-Reply-To: <20081201173137.GA25171@redhat.com>

On Mon, Dec 1, 2008 at 12:31 PM, David Teigland <teigland@redhat.com> wrote:
> Here are the compatibility aspects to the recent ideas about changes to
> the user/kernel interface between gfs (1 & 2) and gfs_controld.
>
> . gfs_controld can remove id from hostdata string in mount options

hi david,

I know I'm a peripheral consumer of the cluster suite, but I thought
I'd chime in and say that I am currently using the "id" as passed into
the kernel in the hostdata string (I believe by mount.gfs2?) in my
pNFS work.  does the above "gfs_controld can remove id from hostdata
string" comment refer to something orthogonal, or would it affect what
gets stored in the superblock's hostdata at mount time?

..hm, sorry, I don't have the code right in front of me, but is that
"id" in the hostdata string the same thing as the mountgroup id?  if
so, then my above worry about the hostdata string is moot, because if
gfs_controld still has that info I can just make a downcall.

thanks,

  d
  .

>
>  - no compat issues AFAICT
>
> . getting rid of "id" sysfs file from lock_dlm
>
>  - new gfs_controld old gfs-kernel
>    old kernel provides both "block" and "id" sysfs files
>    new daemon looks for "block" instead of "id" in sysfs
>
>  - old gfs_controld new gfs-kernel
>    old daemon looks for "id" sysfs file
>    new kernel needs to provide "id" as well as "block" sysfs files
>
>  Once everyone is using the new daemon, we can remove the "id" sysfs
>  file from the kernel.
>
> . uevent strings to replace recover_done/recover_status sysfs files
>
>  - new gfs_controld old gfs-kernel
>    old kernel has recover sysfs files, and no new uevent strings
>    new daemon needs to look for either sysfs files or uevent strings
>
>  - old gfs_controld new gfs-kernel
>    old daemon looks for recover sysfs files, not new uevent strings
>    new kernel needs to provide both sysfs files and uevent strings
>
>  Once everyone is using new kernel and new daemon, we can remove
>  the recover sysfs files from kernel, and daemon can stop looking for
>  recover sysfs files.
>
>
>



  parent reply	other threads:[~2008-12-04 18:32 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-01 17:31 [Cluster-devel] gfs uevent and sysfs changes David Teigland
2008-12-02 14:02 ` Steven Whitehouse
2008-12-04 18:32 ` david m. richter [this message]
2008-12-04 21:07   ` David Teigland
2008-12-04 21:59     ` david m. richter
2008-12-04 22:38       ` David Teigland
2008-12-05  9:51         ` Steven Whitehouse
2008-12-05 14:52           ` David Teigland
2008-12-05 15:03             ` David Teigland
2008-12-05 17:35           ` david m. richter
2008-12-05 17:31         ` david m. richter

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=1d07ca700812041032o6f82fecew3fb93545fe64ed2d@mail.gmail.com \
    --to=richterd@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).