On Sat, Jun 08, 2019 at 12:04:54PM +0300, Amir Goldstein wrote: > Hi Matt, > > Thank you for trying to address this, but I see problems both in Why and > How you did it. > > On Fri, Jun 7, 2019 at 11:51 PM Matt Coffin <mcoffin13@xxxxxxxxx> wrote: > > > > [Why] > > Currently, if the redirect_dir option is set as a kernel or module > > parameter, then even if metacopy is only enabled config, then both > > metacopy and redirect_dir will be enabled when one creates a mount > > point. This is not desirable because /sys/module/overlay/parameters will > > still report that redirect_dir is not enabled > > /sys/module/overlay/parameters reports that redirect_dir is not enabled > *by default* not per mount. > > > and there will be no redirect_dir=on line in the mount options in /proc/mounts. > > That is a bug. This code: > /* Automatically enable redirect otherwise. */ > config->redirect_follow = config->redirect_dir = true; > > Needs to update of config->redirect_mode. Hi Amir, It took me a while to understand what's the problem. So IIUC, issue is that kernel has enabled redirect_dir by default. But it was disabled using module parameter. But it was enabled again as a side affect because metacopy=on was passed as mount option. And /proc/self/mountinfo does not show redirect_dir=on and hence the confusion. IIRC, once you mentioned that we only show those options which needs to be specified if same mount has to be reproduced on different machine with same kernel/module options. If yes, then setting config->redirect_dir=on is not needed because passing metacopy=on will ensure that. To me problem is that sometimes I want to know what options are enabled on a particular mount. And in this case user can't figure that out by parsing /proc/self/mountinfo. And that's the reason apps create a overlay mount point, do bunch of file operations and analyze xattrs to figure out what options are enabled. It will be nice if there was an interface to query all that. And if there is one, then this should go in there. Otherwise given the current interface, it does not sound like a bug to me. Current interface does not tell you what all features are enabled. It only tells you what you need to provide as mount option to create similar mount point. Thanks Vivek
