On Fri, Nov 25, 2011 at 10:34:54AM +0800, Tao Ma wrote: > Sorry, but I don't get your meaning here. > In bitmap_uptodate, we check both the flag of BH_uptodate and > BH_BITMAP_UPTODATE. And in your patch below, we just move the set of > bitmap_uptodate after bh_uptodate. So I don't think the above scenario > would ever happen. Could you please explain it in more detail? Yes, you're right. I didn't realize bitmap_uptodate() checked both bits. I had assumed it used the same convention as the other functions that set/git bits in bh_state. Rats, and I thought I had finally nailed the bug... - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html