Re: [nfsv4] New version of sparse draft(draft-hildebrand-nfsv4-read-sparse-01.txt)

"J. Bruce Fields" <bfields@fieldses.org> Sat, 02 October 2010 20:35 UTC

Return-Path: <bfields@fieldses.org>
X-Original-To: nfsv4@core3.amsl.com
Delivered-To: nfsv4@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 5CB6C3A6DDE for <nfsv4@core3.amsl.com>; Sat, 2 Oct 2010 13:35:20 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.594
X-Spam-Level:
X-Spam-Status: No, score=-2.594 tagged_above=-999 required=5 tests=[AWL=0.005, BAYES_00=-2.599]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id j7PrIG542XFZ for <nfsv4@core3.amsl.com>; Sat, 2 Oct 2010 13:35:19 -0700 (PDT)
Received: from fieldses.org (fieldses.org [174.143.236.118]) by core3.amsl.com (Postfix) with ESMTP id 306FB3A6DB3 for <nfsv4@ietf.org>; Sat, 2 Oct 2010 13:35:19 -0700 (PDT)
Received: from bfields by fieldses.org with local (Exim 4.71) (envelope-from <bfields@fieldses.org>) id 1P28oB-0006M7-2P; Sat, 02 Oct 2010 16:35:51 -0400
Date: Sat, 02 Oct 2010 16:35:51 -0400
To: "Erasani, Pranoop" <Pranoop.Erasani@netapp.com>
Message-ID: <20101002203551.GC18079@fieldses.org>
References: <4CA3CE95.10407@gmail.com> <E043D9D8EE3B5743B8B174A814FD584F0A64D38F@TK5EX14MBXC124.redmond.corp.microsoft.com> <4CA63E48.5070903@gmail.com> <E043D9D8EE3B5743B8B174A814FD584F0A69A182@TK5EX14MBXC124.redmond.corp.microsoft.com> <4CA6482F.2000609@gmail.com> <4CA65309.4060600@panasas.com> <4CA65D71.50000@gmail.com> <4CA66570.3010207@panasas.com> <4CA6678B.3090902@gmail.com> <43EEF8704A569749804F545E3306FCE30910E822@SACMVEXC3-PRD.hq.netapp.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <43EEF8704A569749804F545E3306FCE30910E822@SACMVEXC3-PRD.hq.netapp.com>
User-Agent: Mutt/1.5.20 (2009-06-14)
From: "J. Bruce Fields" <bfields@fieldses.org>
Cc: Benny Halevy <bhalevy@panasas.com>, nfsv4@ietf.org
Subject: Re: [nfsv4] New version of sparse draft(draft-hildebrand-nfsv4-read-sparse-01.txt)
X-BeenThere: nfsv4@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: NFSv4 Working Group <nfsv4.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/nfsv4>, <mailto:nfsv4-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/nfsv4>
List-Post: <mailto:nfsv4@ietf.org>
List-Help: <mailto:nfsv4-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/nfsv4>, <mailto:nfsv4-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 02 Oct 2010 20:35:20 -0000

Their current proposal is mainly just an optimization to allow returning
long strings of zeroes to clients in read requests.  (Servers may be
using file allocation information to identify long strings of zeroes
efficiently, but that's an implementation detail.)

This isn't about allocation, or metadata--it's just a minor incremental
improvement to READ.

The proposal decreases the size of read responses, and may decrease the
number of read requests as well in the case of a sequential reader.

As such, agreed: it doesn't allow a lot of optimizations that an
allocation map would.

It's also simpler than an allocation-map operation:

	- It's just a read, so we already know the semantics.
	- It never performs worse than ordinary read.

You may well be right that it simply isn't worth the trouble, whereas a
GET_HOLE_MAP operation would be.

But I think your proposal is sufficiently different that it should be a
separate proposal.  Beats me whether it would be a competitor to this
one, or complementary to it.

A few questions about a map:

	- What is its lifetime?  Will it be a recallable object like a
	  layout, or does the client invalidate it normally whenever it
	  would invalidate its data cache?
	- Does requesting the block map break write delegations, or (on
	  servers that support atime) update the atime?
	- How does a request for a map they interact with mandatory
	  locks?

--b.