> On Mon, Nov 27, 2017 at 5:36 PM, cgxu <cgxu@xxxxxxxxxxxx> wrote: > > Hi Miklos > > > > Any suggestion for the patch? > > > > > > Best Regards, > > Chengguang > > > > > > > >> 在 2017年11月13日，下午11:59，Chengguang Xu <cgxu@xxxxxxxxxxxx> 写道： > >> > >> When upperdir has project quota and has different project id with workdir, > >> write may fail with error message "Invalid cross-device link" although mounted on r/w mode. > >> This patch checks project quota information of upperdir/workdir during ovl_fill_super(), > >> and if different mount on r/o mode. It doesn’t check detail inherit flag because > >> the implementations are different in specific filesystems. > >> > > Since overlayfs removes and re-creates 'work' directory inside workdir, > why not set projid of 'work' directory according to upper projid instead of > failing the mount? > If projid cannot be set, 'work' dir creation will fail and overlayfs will > fall back to ro mount anyway. It seems not easy to handle ioctl inside kernel module, I suggest just simply print a warning message something like “upperdir has different project id with workdir, it may cause write error”. So sysadmin can get hint from syslog when they experiencing unexpected write error. Thanks, Chengguang. -- To unsubscribe from this list: send the line "unsubscribe linux-unionfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
