Theodore Ts'o <tytso@xxxxxxx> writes: > On Wed, Feb 24, 2016 at 03:17:40PM +0400, Dmitry Monakhov wrote: >> > So in principle this patch is fine but I hate how handling of extent status >> > tree is proliferating to fs/ext4/extent.c to handle some cases especially >> > through special function flags. So looking into the code, probably it would >> > make sense to move all updates of the extent status tree handling down into >> > ext4_ext_map_blocks() and ext4_ind_map_blocks(). Thoughts? >> >> Agree. This was my initial diea but patch was too big for stable >> releases. In fact this is not critical fix so we can simply make is >> in good way and simply ignore @stable. Will send updated version shortly. > > Hi Dmitry, > > Have you had a chance to come up with an updated patch? Not yet. But I plan to do it this week. > > Thanks, > > - 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 -- 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
