Re: [PATCH v4] overlayfs: Provide mount options sync=off/fs to skip sync

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> >
> > Naming: I'm not at all convinced by any name having "sync" in it.  I
> > think "sync=no" is about the implementation, not the functionality,
> > and so it's confusing. The functionality is better described by
> > "volatile" or "temporary".   But I can live with sync=... if voted
> > down.
>
> I am fine with the name "volatile/temporary" for sync=off.
>
> Amir, WDYT?

If we knew we were going to stop here, I wouldn't mind "volatile".
I too like the fact that the user config means something functional.
But if we are considering to expand it later to sync=fs and maybe
sync=writeback, then it is going to be quite hard to come by with
equally meaningful names and the "sync=" modes would end up
being less confusing IMO.

So I cast my vote for "sync=off".
We could also make "volatile" an alias to "sync=off" if it makes
anyone happier.

Thanks,
Amir.



[Index of Archives]     [Linux Filesystems Devel]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux