question about xfs_alloc_fix_freelist()

["Michael Nishimoto" <miken@xxxxxxxxx>]

The following code can be found near the end of xfs_alloc_fix_freelist:

                if (targs.agbno == NULLAGBLOCK) {
                        if (flags & XFS_ALLOC_FLAG_FREEING)
                                break;
                        xfs_trans_brelse(tp, agflbp);
                        args->agbp = NULL;
                        return 0;
                }

Don't we need to release agbp too by calling xfs_trans_brelse(tp, agbp)?

   Michael



[[HTML alternate version deleted]]