Messages in this thread | | | From | Trond Myklebust <> | Subject | Re: [PATCH] Fix NFS option parsing bit-rot. | Date | Wed, 8 Jun 2016 20:35:21 +0000 |
| |
On 6/8/16, 15:41, "Rob Landley" <rob@landley.net> wrote: >So addr= is at _best_ redundant. You MUST supply host:/ always, >the current code just wasn't using it.
That’s because you just happen to be supplying an IP address instead of a hostname. The kernel has no DNS resolving functionality. It cannot resolve hostnames into IP addresses without help either by the caller or by means of an upcall. That is why ‘addr=’ has been a mandatory parameter ever since we introduced the text based parser.
Now we could, theoretically, have the client call nfs_dns_resolve_name() on the hostname to resolve it. However that breaks when you have net namespaces and such, since the kernel’s dns_query() call is not container aware.
Trond
| |