Hi Junxiao, On 2017/12/27 15:35, Junxiao Bi wrote: > Hi Changwei, > > On 12/26/2017 05:20 PM, Changwei Ge wrote: >> Hi Alex >> >> On 2017/12/26 16:20, alex chen wrote: >>> Hi Changwei, >>> >>> On 2017/12/26 15:03, Changwei Ge wrote: >>>> The intention of this patch is to provide an option to ocfs2 users whether >>>> to allocate disk space while doing dio write. >>>> >>>> Whether to make ocfs2 fall back to buffer io is up to ocfs2 users through >>>> toggling append-dio feature. It rather makes ocfs2 configurable and >>>> flexible. >>>> >>> It is too strange to make ocfs2 fall back to buffer io by toggling >>> append-dio feature. >> >> It might be. >> But as my changelog said, I think, append-dio is key to whether to allocate >> space with dio writing. So filling hole and appending file should have the >> same reflection. >> >> Besides, in the early days, ocfs2 truly falls back to buffer io when >> append-dio is disabled and file hole is encountered. > I think we have discussed a lot in your v1 version. This wasn't fit for > mainline, you can keep one off-mainline patch.
Fine. I agree now. I give up trying to push this patch into mainline. BTW, could you please help review my another patch (ocfs2: try to reuse extent block in dealloc without meta_alloc) which is used to fix a dio crash issue? So without this patch, dio still won't hit such an issue. Your comments are very important to me. Thanks, Changwei > > Thanks, > Junxiao. > _______________________________________________ Ocfs2-devel mailing list Ocfs2-devel@oss.oracle.com https://oss.oracle.com/mailman/listinfo/ocfs2-devel