On 09/07/2010 02:12 PM, Trond Myklebust wrote:
On Tue, 2010-09-07 at 14:10 -0700, Ben Greear wrote:
On 09/07/2010 01:54 PM, Trond Myklebust wrote:
No. As I said, it needs to match the clientaddr argument, not the
srcaddr.
The problem is that you are now potentially introducing cases where the
server may have multiple combinations of clientaddr and srcaddr.
Ok, so what do you think about allowing a flag to bind() to clientaddr
instead of having the separate srcaddr option?
That might be slightly less intrusive, but I'm still unconvinced it is
something we need to support in the upstream kernels.
It seems clientaddr is saved as a string instead of a struct sockaddr_storage,
which means doing conversion in lots of places. So, using a bindclient flag
and clientaddr would be even trickier than my srcaddr= approach.
I can still try to get that working if that's the only hurdle to upstream
inclusion, but if it's all for my own use, I'm just going to use the same
api as for cifs, and ensure the caller always uses same thing for srcaddr
and clientaddr.
Thanks,
Ben
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
--
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
