Re: XFS internal error xfs_trans_cancel at line 1150 of file fs/xfs/xfs_trans.c

["Christian Røsnes" <christian.rosnes@xxxxxxxxx>]

On Thu, Mar 13, 2008 at 3:53 PM, David Chinner <dgc@xxxxxxx> wrote:

>  The patch below fixes the shutdown for me. Can you give it a go?
>
>
>  Cheers,
>
>  Dave.
>  --
>  Dave Chinner
>  Principal Engineer
>  SGI Australian Software Group
>
>
>  ---
>   fs/xfs/xfs_ialloc.c |    7 ++++++-
>   1 file changed, 6 insertions(+), 1 deletion(-)
>
>  Index: 2.6.x-xfs-new/fs/xfs/xfs_ialloc.c
>  ===================================================================
>  --- 2.6.x-xfs-new.orig/fs/xfs/xfs_ialloc.c      2008-03-13 
> 13:07:24.000000000 +1100
>  +++ 2.6.x-xfs-new/fs/xfs/xfs_ialloc.c   2008-03-14 01:40:21.926153338 +1100
>  @@ -167,7 +167,12 @@ xfs_ialloc_ag_alloc(
>
>                 args.mod = args.total = args.wasdel = args.isfl =
>                         args.userdata = args.minalignslop = 0;
>                 args.prod = 1;
>  -               args.alignment = 1;
>  +               if (xfs_sb_version_hasalign(&args.mp->m_sb) &&
>
> +                       args.mp->m_sb.sb_inoalignmt >=
>  +                       XFS_B_TO_FSBT(args.mp, 
> XFS_INODE_CLUSTER_SIZE(args.mp)))
>  +                               args.alignment = args.mp->m_sb.sb_inoalignmt;
>  +               else
>  +                       args.alignment = 1;
>
>
>                 /*
>                  * Allow space for the inode btree to split.
>                  */
>

Yes. This patch fixes the shutdown problem for me too.

Once again, thanks.

Christian