On Fri, Dec 01, 2006 at 07:14:28AM +0900, OGAWA Hirofumi wrote: > > quick look. Doesn't this break reiserfs? IIRC, the reiserfs is using > it for another reason. I was also working for this, but I lost the > thread of this, sorry. Yes I think it will break reiserfs, so I just have to have a look at converting it. Shouldn't take too long. > > I found some another users (affs, hfs, hfsplus). Those seem have same > problem, but probably those also can use this... > > What do you think? Well I guess this is your code, so it is up to you ;) I would be happy if you come up with a quick fix, I'm just trying to stamp out a few big bugs in mm. However I did prefer my way of moving all the exapand code into generic_cont_expand, out of prepare_write, and avoiding holding the target page locked while we're doing all the expand work (strictly, you might be able to get away with this, but it is fragile and ugly). AFAIKS, the only reason to use prepare_write is to avoid passing the get_block into generic_cont_expand? Thanks, Nick - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
