From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay2.corp.sgi.com [137.38.102.29]) by oss.sgi.com (Postfix) with ESMTP id E1D1F7F51 for ; Fri, 14 Feb 2014 13:51:16 -0600 (CST) Message-ID: <52FE73B3.1020804@sgi.com> Date: Fri, 14 Feb 2014 13:51:15 -0600 From: Mark Tinguely MIME-Version: 1.0 Subject: Re: [PATCH 1/5] metadump: sanitise write_buf/index return values References: <1390472635-17225-1-git-send-email-david@fromorbit.com> <1390472635-17225-2-git-send-email-david@fromorbit.com> <52FD1D38.1010006@sgi.com> <20140214022053.GQ13997@dastard> In-Reply-To: <20140214022053.GQ13997@dastard> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Dave Chinner Cc: xfs@oss.sgi.com On 02/13/14 20:20, Dave Chinner wrote: > On Thu, Feb 13, 2014 at 01:30:00PM -0600, Mark Tinguely wrote: >> On 01/23/14 04:23, Dave Chinner wrote: >>> From: Dave Chinner >>> >>> Write_buf/write_index use confusing boolean values for return, >>> meaning that it's hard to tell what the correct error return is >>> supposed to be. Convert them to return zero on success or a >>> negative errno otherwise so that it's clear what the error case is. >>> >>> Signed-off-by: Dave Chinner >>> --- >> >> Looks like this patch broke metadumps on some corrupted filesystems. >> This is a legacy filesystem that has zeroes overwriting the >> SB/AGF/AGI on AG 1/2/3: >> >> >> # xfs_metadump -wgo /dev/sda8 myfile.metadata >> xfs_agf_read_verify: XFS_CORRUPTION_ERROR >> xfs_metadump: cannot init perag data (117). Continuing anyway. >> Copied 64 of 64 inodes (0 of 4 AGs) xfs_agf_write_verify: XF >> S_CORRUPTION_ERROR >> write_buf: write verifer failed on bno 0x1100919/0x200 >> (no output) > > Where did it crash? Can you post the stack trace from gdb? Even > better, can you send a patch to fix the problem? ;) > > Cheers, > > Dave. No crash, just exits without performing the dump. --Mark. _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs