On Thu, 2013-06-27 at 14:43 +0800, Cong Wang wrote: [...] > -/** > - * rpc_cmp_addr - compare the address portion of two sockaddrs. > - * @sap1: first sockaddr > - * @sap2: second sockaddr > - * > - * Just compares the family and address portion. Ignores port, but > - * compares the scope if it's a link-local address. You're removing the scope comparison. Ben. > - * Returns true if the addrs are equal, false if they aren't. > - */ > static inline bool rpc_cmp_addr(const struct sockaddr *sap1, > const struct sockaddr *sap2) > { > - if (sap1->sa_family == sap2->sa_family) { > - switch (sap1->sa_family) { > - case AF_INET: > - return __rpc_cmp_addr4(sap1, sap2); > - case AF_INET6: > - return __rpc_cmp_addr6(sap1, sap2); > - } > - } > - return false; > + return inet_addr_equal((const union inet_addr *)sap1, > + (const union inet_addr *)sap2); > } [...] -- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked. -- 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