From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111]) by oss.sgi.com (Postfix) with ESMTP id DCA747F37 for ; Thu, 31 Jul 2014 02:51:40 -0500 (CDT) Received: from cuda.sgi.com (cuda2.sgi.com [192.48.176.25]) by relay1.corp.sgi.com (Postfix) with ESMTP id C8EBB8F8081 for ; Thu, 31 Jul 2014 00:51:40 -0700 (PDT) Received: from aserp1040.oracle.com (aserp1040.oracle.com [141.146.126.69]) by cuda.sgi.com with ESMTP id RFE2hdwMS3Q1vSqc (version=TLSv1 cipher=AES256-SHA bits=256 verify=NO) for ; Thu, 31 Jul 2014 00:51:39 -0700 (PDT) Message-ID: <53D9F67F.9060609@oracle.com> Date: Thu, 31 Jul 2014 15:55:43 +0800 From: Jeff Liu MIME-Version: 1.0 Subject: Re: [PATCH v3 00/10] xfs_ioc_bulkstat code refactoring and consolidation References: <538D92B6.5050402@oracle.com> <20140729232748.GI26465@dastard> In-Reply-To: <20140729232748.GI26465@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 Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Dave Chinner Cc: "xfs@oss.sgi.com" Hi Dave, On 07/30/2014 07:27 AM, Dave Chinner wrote: > On Tue, Jun 03, 2014 at 05:17:42PM +0800, Jeff Liu wrote: >> Hi folk, >> >> This is the revised patches for xfs_ioc_bulkstat consolidation and code >> refactoring. As per Christoph's comments, I'm not include the per AG >> inumber patch in this series given that I don't actually introduce the >> relevant inumbers interface now. Similar to that reason, I also dropped >> the per AG bulkstat patch, it would be included in parallel quota check >> series. >> >> >> v3->v2: >> - one major bug fix is at xfs_bulkstat_ag_ichunk() regarding the user buffer >> pointer operations, it should be defined as a pointer-to-pointer since it >> would be updated inside xfs_bulkstat_ag_ichunk(). >> >> - separate xfs_inumber consolidate patch into two patches, the first one >> fix the formater function return value and consolidate the codes, another >> one does the actual logic changes for better error handling. >> >> - Add a separate patch to get rid of the redundant user buffer count >> checks at xfs_bulkstat() >> >> - fixed agino calculation issue at xfs_bulkstat_grab_ichunk(). >> >> v2: http://oss.sgi.com/archives/xfs/2014-04/msg00554.html >> v1: http://oss.sgi.com/archives/xfs/2013-12/msg00901.html >> >> >> Any comments are welcome! > > Hi Jeff, I ported this to the current dev tree based on the > xfs-libxfs-restructure branch, and I keep seeing fsstress failing > with memory corruption after random bulkstat ioctls. I see regular > failures with generic/013, generic/068, xfs/167 and the other > fstress tests also randomly fail. The typical failure is glibc > detected memory heap corruption on freeing the bulkstat structure > after the ioctl: > > generic/068 42s ...*** Error in `./ltp/fsstress': double free or corruption (!prev): 0x00007f0224000b70 *** > ======= Backtrace: ========= Sorry for my late response and thanks for help porting this patch series. Now I can reproduce this issue frequently with generic/013, will try to fix it ASAP. Cheers, -Jeff _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs