On Thu, Jan 17, 2013 at 01:11:29PM -0500, Brian Foster wrote: > The stack_switch check currently occurs in __xfs_bmapi_allocate, > which means the stack switch only occurs when xfs_bmapi_allocate() > is called in a loop. Pull the check up before the loop in > xfs_bmapi_write() such that the first iteration of the loop has > consistent behavior. > > Signed-off-by: Brian Foster <bfoster@xxxxxxxxxx> Yeah, that's a bug. As i mentioned on IRC, the backup copy of the patch I have has these two hunks in it. Why they got dropped from the primary copy is have no idea - perhaps it was an issue with popping and pushing patches on and off my stack, and the hunks got silently dropped... I note that the port to the RHEL6 codebase has this correct - and looking a the patch I ported it is the same as the backup copy in my mainline guilt patch directory. So the intention was there that the stack switch should have been moved, but it wasn't and the logic problem wasn't picked up on review, so here we are... Anyway, looks good. Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
