From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from natasha.panasas.com ([67.152.220.90]:50853 "EHLO natasha.panasas.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754851Ab1G0UeL (ORCPT ); Wed, 27 Jul 2011 16:34:11 -0400 Message-ID: <4E30762E.4000508@panasas.com> Date: Wed, 27 Jul 2011 13:33:50 -0700 From: Boaz Harrosh To: Trond Myklebust CC: Jim Rees , , peter honeyman Subject: Re: [PATCH v3 03/25] NFS41: Let layoutcommit handle multiple segments References: <1311792048-12551-1-git-send-email-rees@umich.edu> <1311792048-12551-4-git-send-email-rees@umich.edu> <4E306F84.6030808@panasas.com> <1311798159.25645.32.camel@lade.trondhjem.org> In-Reply-To: <1311798159.25645.32.camel@lade.trondhjem.org> Content-Type: text/plain; charset="UTF-8" Sender: linux-nfs-owner@vger.kernel.org List-ID: MIME-Version: 1.0 On 07/27/2011 01:22 PM, Trond Myklebust wrote: > On Wed, 2011-07-27 at 13:05 -0700, Boaz Harrosh wrote: >> On 07/27/2011 11:40 AM, Jim Rees wrote: >>> From: Peng Tao >>> >>> Some layout drivers like block will have multiple segments. Generic code >>> should be able to handle it. Layoutcommit takes a list of segments and last >>> write offset is saved at inode level. >>> >>> Signed-off-by: Peng Tao >> >> Trond Hi >> >> Could you please add this patch to the current Merge window? And also include >> >> CC: Stable > > Nope. > > Firstly it exacerbates an existing bug in the current code (see my > comments). > > Secondly, it does several things at once (adding multiple segment > handling, fixing last-byte handling in layoutcommit and credential > handling). Which of these features are actually needed for stable? > OK, I agree I thought of that too. My bakethon post had a narrower scope to just the "adding multiple segment handling" & "last-byte handling" (I think let me check) But I thought for simplicity since this patchset was reviewed then, I've tested with this patch and it works good as well. If you are going to break things up, I can test with the minimal patches and report what is need in Stable. (Is anyone working on it?) Thanks Boaz