Re: [PATCH 06/39] vfs: add f_op->pre_mmap()

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

 



On Wed, Jun 27, 2018 at 3:07 PM, Amir Goldstein <amir73il@xxxxxxxxx> wrote:
> On Wed, Jun 6, 2018 at 10:47 PM, Miklos Szeredi <miklos@xxxxxxxxxx> wrote:
>> On Wed, Jun 6, 2018 at 8:52 PM, Amir Goldstein <amir73il@xxxxxxxxx> wrote:
>>> On Tue, Jun 5, 2018 at 2:36 PM, Miklos Szeredi <miklos@xxxxxxxxxx> wrote:
>>>> On Mon, Jun 4, 2018 at 10:48 AM, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
>>>>> On Tue, May 29, 2018 at 04:43:06PM +0200, Miklos Szeredi wrote:
>>>>>> This is needed by overlayfs to be able to copy up a file from a read-only
>>>>>> lower layer to a writable layer when being mapped shared.  When copying up,
>>>>>> overlayfs takes VFS locks that would violate locking order when nested
>>>>>> inside mmap_sem.
>>>>>>
>>>>>> Add a new f_op->pre_mmap method, which is called before taking mmap_sem.
>>>>>
>>>>> NAK.  We really should not add multiple methods for mmap, and everytime
>>>>> this came up we found a better way to solve the problem instead.  Most
>>>>> recent example was the socket zero copy receive code.
>>>>
>>>> Okay, I'll drop this.
>>>
>>> Need to remove copy_up_shared from overlayfs.txt.
>>
>> Right.  If the pull goes trough in this cycle, I'll send a followup
>> patch to fix that.
>>
>
> Reminder: stale doc is still in current overlayfs-next.
>

Still there.

Thanks,
Amir.
--
To unsubscribe from this list: send the line "unsubscribe linux-unionfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[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