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-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to