public inbox for linux-nfs@vger.kernel.org
 help / color / mirror / Atom feed
From: Boaz Harrosh <bharrosh@panasas.com>
To: NFS list <linux-nfs@vger.kernel.org>,
	Benny Halevy <bhalevy@panasas.com>,
	Andy Adamson <andros@netapp.com>,
	Fred Isaman <iisaman@netapp.com>
Subject: [RFC 0/3] Ugliness in struct pnfs_layout_segment and possible uselessness of the device cache
Date: Sun, 12 Sep 2010 19:01:13 +0200	[thread overview]
Message-ID: <4C8D0759.6090201@panasas.com> (raw)

Struct pnfs_layout_segment is the generic part of the pnfs-getlayout (segment) operation
and should not hold any layout specific part.

So what is the 	struct nfs4_deviceid *deviceid; member doing there. This is a files
only useful member that has no meaning in objects and blocks. (The later have multiple
devices per layout)

I've hacked up a possible fix. Please review. These will need more work done:
- Actual testing. They are compiled only.
- Adjusted to multiple pending changes from the changes to the layout segment allocation
  to all these latest Fred changes.
- I wish the relevant parts get into the first batch of submission in the final form
  presented here, instead of changing later. Note the cleanup it gives to the files
  layout driver. Which can even be driven deeper. So they'll need to be integrated
  down deeper. (And code rebased above it)
- So I've actually attempted using the generic cache in the objects driver. And it
  should be usable once all the code changes settle down, and I fix that FIXME: which
  is there now (See [RFC 3/3])

Andy? (Fred)? would you want to take the generic and file-layout parts out of my hands
and see them through. For one they need testing which I don't have readily available.
That is if you like them and they make any sense.

[RFC 1/3] SQUASHME: Generalize the device cache so it can be used by all layouts
[RFC 2/3] SQUASHME: fileslayout: Adjust to device_cache API changes
[RFC 3/3] SQUASHME: pnfs-obj: Use the generic device cache

Thanks
Boaz

             reply	other threads:[~2010-09-12 17:01 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-12 17:01 Boaz Harrosh [this message]
2010-09-12 17:03 ` [RFC 1/3] SQUASHME: Generalize the device cache so it can be used by all layouts Boaz Harrosh
2010-09-12 19:42   ` Trond Myklebust
2010-09-13  9:50     ` Boaz Harrosh
2010-09-13 12:16       ` Fred Isaman
2010-09-13 12:45         ` Boaz Harrosh
2010-09-12 17:05 ` [RFC 2/3] SQUASHME: fileslayout: Adjust to device_cache API changes Boaz Harrosh
2010-09-12 17:06 ` [RFC 3/3] SQUASHME: pnfs-obj: Use the generic device cache Boaz Harrosh
2010-09-12 17:31   ` Boaz Harrosh

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=4C8D0759.6090201@panasas.com \
    --to=bharrosh@panasas.com \
    --cc=andros@netapp.com \
    --cc=bhalevy@panasas.com \
    --cc=iisaman@netapp.com \
    --cc=linux-nfs@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox