xfs
[Top] [All Lists]

Re: The segment fault with NULL point using when recovering failure

To: Mike Gao <ygao.linux@xxxxxxxxx>
Subject: Re: The segment fault with NULL point using when recovering failure
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Fri, 17 Sep 2010 11:44:12 +1000
Cc: xfs@xxxxxxxxxxx
In-reply-to: <AANLkTimR-dBLmQQ-Nh0mmjHJMfFidePKxfO6P76y48n8@xxxxxxxxxxxxxx>
References: <AANLkTimR-dBLmQQ-Nh0mmjHJMfFidePKxfO6P76y48n8@xxxxxxxxxxxxxx>
User-agent: Mutt/1.5.20 (2009-06-14)
On Wed, Sep 15, 2010 at 10:59:07AM -0500, Mike Gao wrote:
> xlog_recover_process_iunlinks(
>     xlog_t        *log)
> {
>                 /*
>                  * Reacquire the agibuffer and continue around
>                  * the loop. This should never fail as we know
>                  * the buffer was good earlier on.
>                  */
>                 error = xfs_read_agi(mp, NULL, agno, &agibp);
>                 ASSERT(error == 0);
>                 agi = XFS_BUF_TO_AGI(agibp);
> 
> }
> If no defined DEBUG, ASSERT will not work and agibp could be ZERO if there
> is a error in xfs_read_agi.
> And the comment shouldn't say it never fail because  xfs_read_agi will
> return error in function and it does when there is forced shutdown.

Have you seen a failure here?

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>