On Mon, Dec 02, 2019 at 07:10:52PM +0000, Andrew Doran wrote:
> In light of the recent discussion, and having asked Jaromir his thoughts on
> the subject, we both think it's time to enable this by default, so it gets
> wider testing. Is there a good reason not to?
The current ubc_direct code still has the problem that I pointed out originally,
which is that it deadlocks if you read() or write() a page of a file into
a mapping of itself. We should not enable this by default until that problem
Right, I completely forgot about this.
I have a small program which triggers the deadlock quite reliably. Never got around to actually add it into test suite because it caused problems also on other system I run it on.
Andrew, would you by chance be interested to look at this?