From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cuda.sgi.com (cuda1.sgi.com [192.48.157.11]) by oss.sgi.com (8.14.3/8.14.3/SuSE Linux 0.8) with ESMTP id oB2Bb8WJ014392 for ; Thu, 2 Dec 2010 05:37:10 -0600 Received: from bombadil.infradead.org (localhost [127.0.0.1]) by cuda.sgi.com (Spam Firewall) with ESMTP id A0FBE1404F6A for ; Thu, 2 Dec 2010 03:38:52 -0800 (PST) Received: from bombadil.infradead.org (bombadil.infradead.org [18.85.46.34]) by cuda.sgi.com with ESMTP id qm1feOlqniwCKSQC for ; Thu, 02 Dec 2010 03:38:52 -0800 (PST) Date: Thu, 2 Dec 2010 06:38:49 -0500 From: Christoph Hellwig Subject: Re: [PATCH 1/8] xfs: Pull EFI/EFD handling out from under the AIL lock Message-ID: <20101202113849.GA21365@infradead.org> References: <1290993152-20999-1-git-send-email-david@fromorbit.com> <1290993152-20999-2-git-send-email-david@fromorbit.com> <20101130201734.GA16079@infradead.org> <20101202012841.GL16922@dastard> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20101202012841.GL16922@dastard> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: xfs-bounces@oss.sgi.com Errors-To: xfs-bounces@oss.sgi.com To: Dave Chinner Cc: Christoph Hellwig , xfs@oss.sgi.com On Thu, Dec 02, 2010 at 12:28:41PM +1100, Dave Chinner wrote: > > - there is a behaviour change about the xfs_trans_del_item call > > in xfs_efi_item_unpin - before it was protected by the > > XFS_EFI_CANCELED which was never set, and now it's not. > > XFS_EFI_CANCELED has not been set in the code base since > xfs_efi_cancel() was removed back in 2006 by commit > 065d312e15902976d256ddaf396a7950ec0350a8 ("[XFS] Remove unused > iop_abort log item operation), and even then xfs_efi_cancel() was > never called. I haven't tracked it back further than that (beyond > git history), but handling of efis in cancelled transactions has > been broken for a long time. > > Basically, when we get an IOP_UNPIN(lip, 1); call from > xfs_trans_uncommit() (i.e. remove == 1), if we don't free the log > item descriptor we leak it. IOWs, the new behaviour introduced in > this patch is actually the correct behaviour. Maybe fix this issue first in a separate patch, instead of hiding it in a bigger one. _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs