Linux Overlay Filesystem development
 help / color / mirror / Atom feed
From: Gao Xiang <hsiangkao@linux.alibaba.com>
To: Yafang Shao <laoar.shao@gmail.com>
Cc: miklos@szeredi.hu, amir73il@gmail.com,
	linux-unionfs@vger.kernel.org, fuweid89@gmail.com
Subject: Re: [PATCH] ovl: Allow changing default fsync_mode
Date: Tue, 23 Jun 2026 19:59:09 +0800	[thread overview]
Message-ID: <13bb8cba-c82e-4a41-aff1-0a6418873bf1@linux.alibaba.com> (raw)
In-Reply-To: <CALOAHbCBo2eB0xVoSv8Vi+r3DwbwTticismFSL3EXyYuwFkEzA@mail.gmail.com>



On 2026/6/23 19:38, Yafang Shao wrote:
> On Tue, Jun 23, 2026 at 6:25 PM Gao Xiang <hsiangkao@linux.alibaba.com> wrote:
>>
>>
>>
>> On 2026/6/23 18:18, Yafang Shao wrote:
>>> On Tue, Jun 23, 2026 at 6:12 PM Gao Xiang <hsiangkao@linux.alibaba.com> wrote:
>>>>
>>
>> ...
>>
>>>>
>>>> Again, I don't want such customized messy breaks userspace
>>>> again; with that patch, container runtime needs to consider
>>>> if `volatile` is the default which just breaks the existing
>>>> containerd versions.
>>>
>>> I'll leave this debate to the overlayfs maintainers ;)
>>
>> On my own perspective and be responsible for common users
>> (and as a containerd maintainer [1]),
> 
> No wonder containerd is getting harder and harder to use ;)

What do you mean, can you explain exactly?

You're just adding a new way to break the existing
applications, no? You just breaks previous shipped
containerd.

Add a way to change the default behavior is fine, but
the new default behavior should be worked with the
same functionality and compatible, but switching to
`volatile` feature is non-compatible and what is why
containerd dropped volatile.

I've explained the technical reasons, can you also
show your technical argument why you cannot patch
docker with a very little change (if you can
livepatch the kernel) and just restart the docker
daemon, is that hard?  -- Also, is that requirement
common?

How pre-existing container runtime versions know
their default option is not "auto", how do they add
mount option "fsync=auto"? and how do they know there
is a new sysfs knob to work around incompatible
behavior?  there are so many container runtime
"docker", "containerd", "crio", how to make sure
all these container runtime aware of the new
default may not be "auto"?

Thanks,
Gao Xiang

  reply	other threads:[~2026-06-23 11:59 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-23  8:43 [PATCH] ovl: Allow changing default fsync_mode Yafang Shao
2026-06-23  9:00 ` Gao Xiang
2026-06-23  9:15   ` Yafang Shao
2026-06-23  9:25     ` Gao Xiang
2026-06-23  9:34       ` Yafang Shao
2026-06-23  9:49         ` Gao Xiang
2026-06-23  9:59           ` Yafang Shao
2026-06-23 10:06             ` Gao Xiang
2026-06-23 10:12             ` Gao Xiang
2026-06-23 10:18               ` Yafang Shao
2026-06-23 10:25                 ` Gao Xiang
2026-06-23 11:38                   ` Yafang Shao
2026-06-23 11:59                     ` Gao Xiang [this message]
2026-06-23 12:47                       ` Yafang Shao
2026-06-23 13:11                         ` Gao Xiang
2026-06-23 13:19                           ` Yafang Shao
2026-06-23 13:35                             ` Gao Xiang
2026-06-23 13:39                               ` Yafang Shao
2026-06-23 13:42                               ` Christoph Hellwig
2026-06-23 13:46                                 ` Yafang Shao
2026-06-23 13:47                                 ` Gao Xiang
2026-06-23 14:52                                   ` Amir Goldstein
2026-06-24  2:09                                     ` Yafang Shao
2026-06-24  2:28                                       ` Gao Xiang
2026-06-24  2:30                                         ` Yafang Shao
2026-06-24  3:17                                           ` Gao Xiang
2026-06-24  3:28                                             ` Yafang Shao
2026-06-24  3:36                                               ` Gao Xiang
2026-06-24  3:41                                                 ` Yafang Shao
2026-06-23 13:03             ` Gao Xiang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=13bb8cba-c82e-4a41-aff1-0a6418873bf1@linux.alibaba.com \
    --to=hsiangkao@linux.alibaba.com \
    --cc=amir73il@gmail.com \
    --cc=fuweid89@gmail.com \
    --cc=laoar.shao@gmail.com \
    --cc=linux-unionfs@vger.kernel.org \
    --cc=miklos@szeredi.hu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox