Return-Path: Received: from mail-gy0-f174.google.com ([209.85.160.174]:62196 "EHLO mail-gy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752834Ab0EXNnI (ORCPT ); Mon, 24 May 2010 09:43:08 -0400 Received: by gyg13 with SMTP id 13so1475404gyg.19 for ; Mon, 24 May 2010 06:43:02 -0700 (PDT) In-Reply-To: References: <1274371457-10003-1-git-send-email-andros@netapp.com> Date: Mon, 24 May 2010 09:43:02 -0400 Message-ID: Subject: Re: [PATCH 1/1] SQUASHME pnfs-submit: replace layoutcommit_ctx with rpc_cred From: "William A. (Andy) Adamson" To: Tao Guo Cc: bhalevy@panasas.com, linux-nfs@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-nfs-owner@vger.kernel.org List-ID: MIME-Version: 1.0 On Mon, May 24, 2010 at 2:37 AM, Tao Guo wrote: > I see... But what if sync == 1? At that time, data must have already > been freed, so you should not use data->cred anymore. Agreed. Since LAYOUTCOMMIT is not called after close, we can get rid of taking the reference all together since a reference is taken on OPEN and dropped on CLOSE. -->Andy > > -- > tao. >