Re: [PATCH v2 02/10] NFS: Move the flock open mode check into nfs_flock()

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

 



On Mon, Dec 7, 2015 at 2:24 PM, Benjamin Coddington <bcodding@xxxxxxxxxx> wrote:
> On Mon, 7 Dec 2015, Christoph Hellwig wrote:
>
>> On Mon, Dec 07, 2015 at 02:13:47PM -0500, Benjamin Coddington wrote:
>> > I can fixup the comment for clarity as I just moved this chunk over.
>> >
>> > I'm not aware that flock() has ever had this check, but posix locks requires
>> > it.  My understanding is that since NFS may simulate flock() with posix
>> > locking, the check is necessary for NFS.
>>
>> Ah, right.  Maybe that's what should be in the comment here..
>
> Yes, good idea.  That's where someone is going to look when they investigate
> why their application breaks on NFS.
>

Should I expect a resend?

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



[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux