Dave,

On Thu, Feb 28, 2013 at 10:31:55AM +1100, Dave Chinner wrote:

On Thu, Feb 28, 2013 at 10:20:45AM +1100, Dave Chinner wrote:

quoted

On Wed, Feb 27, 2013 at 04:37:50PM -0600, Ben Myers wrote:

quoted

Hi Dave,

....

quoted

quoted

quoted

+	case XFS_BLF_AGFL_BUF:
+		if (*(__be32 *)bp->b_addr != cpu_to_be32(XFS_AGFL_MAGIC)) {
+			xfs_warn(mp, "Bad AGFL block magic!");
+			ASSERT(0);
+		}
+		bp->b_ops = &xfs_agfl_buf_ops;
+		break;

Your changes for v2 in this section look good.

Actually, the above hunk is broken. The magic number should only be
checked for CRC enabled filesystems. My current code has this check,
though I thought I fixed that long before I reposted this series...

I just realised you're commenting on the original version of the
patch series, no the new version I posted a couple of days ago. It
is fixed in that version of the patch, so I'm not going totally
crazy (yet!)....

Yep.  The new rev looks good.

Can you switch over to the newer version of the patch set?

Will do.

-Ben

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs