Re: [PATCH 0/3] client-side lockd doesn't start UDP listener

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

 



On Mon, Oct 13, 2008 at 11:42:47AM -0400, Chuck Lever wrote:
> Hi Neil-
>
> On Oct 12, 2008, at Oct 12, 2008, 7:15 PM, Neil Brown wrote:
>> On Saturday October 4, bfields@xxxxxxxxxxxx wrote:
>>> On Fri, Oct 03, 2008 at 05:15:14PM -0400, Chuck Lever wrote:
>>>> Hi Bruce, Neil-
>>>>
>>>> Here's my initial proposal to address the NFSv2/v3 lock recovery  
>>>> issue
>>>> that results from having no UDP lockd listener.
>>>>
>>>> Comments?  Did I miss anything?
>>>
>>> Looks fine; I can't see any problem.  So I've applied to for-2.6.28.
>>> (An ack from Neil would be reassuring, though, if he gets a chance.)
>>
>> Sorry for my tardiness.  September was a very hectic month for me.
>>
>> One consequence of this change is that lockd always listens on TCP
>> even if NFSD and NFS are only using UDP.
>> Do we care?  I suspect not.
>
> The server side usually has to start both anyway, so I thought making  
> both sides work the same way was slightly nicer than keeping the "proto" 
> argument to lockd_up(), and the run-time cost on the client is fairly 
> minimal.  Plus, the overall trend is away from NFS over UDP, and towards 
> NFS over TCP.  UDP is legacy, and TCP is the common case, going forward, 
> so it's likely the TCP listener will nearly always be running anyway.
>
> However, do we care about the -T and -U options on rpc.nfsd affecting  
> how server-side lockd works?  Maybe that is a valid reason to keep the  
> "proto" argument to lockd_up().

We might at least want to fix the man page.

I suppose worst case scenarios would be:

	- a bug is found that affects lockd/tcp and not lockd/udp, and
	  people that used -T are affected when they needn't have been,
	  or
	- someone uses -T and only bothers to firewall the udp port?

Is there any evidence that anyone uses -U or -T?

--b.
--
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