Linus Torvalds writes: (Summary) That commit does look like the obviously right thing to do, and
our old behavior was obviously wrong.
our old behavior was obviously wrong.
At the same time, we really really shouldn't break user flow, even if that flow was due to an obvious bug.
that flow was due to an obvious bug.
Dang.
Dang.
That sounds like possibly the sanest fixlet for this - perhaps together with a warning message saying "ignoring DM_READONLY_FLAG for COW device" so that people see that they are doing insane things.
our old behavior was obviously wrong.
At the same time, we really really shouldn't break user flow, even if that flow was due to an obvious bug.
that flow was due to an obvious bug.
Dang.
Dang.
[...]
is probably never sane...That sounds like possibly the sanest fixlet for this - perhaps together with a warning message saying "ignoring DM_READONLY_FLAG for COW device" so that people see that they are doing insane things.