On Thu, Mar 28, 2024 at 09:06:43AM +0100, Christian Brauner wrote: > > Why do we need to set any of these for directory operations now that > > we have a clear choice? i.e. we can't mmap directories, and the rest > > of these flags are for read() and write() operations which we also > > can't do on directories... > > Yeah, I know but since your current implementation raises them for both > I just did it 1:1: Yes, sticking to the 1:1 for this patch is probably a good idea. But we should also fix this in a trivial follow on patch. I can write it and add it to your series.