IB: directly cast the sockaddr union to aockaddr
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 7 Aug 2019 16:44:12 +0000 (18:44 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 9 Aug 2019 15:52:29 +0000 (17:52 +0200)
Like commit 641114d2af31 ("RDMA: Directly cast the sockaddr union to
sockaddr") we need to quiet gcc 9 from warning about this crazy union.
That commit did not fix all of the warnings in 4.19 and older kernels
because the logic in roce_resolve_route_from_path() was rewritten
between 4.19 and 5.2 when that change happened.

Cc: Jason Gunthorpe <jgg@mellanox.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/infiniband/core/sa_query.c

index 7b794a1..8be082e 100644 (file)
@@ -1232,7 +1232,6 @@ static int roce_resolve_route_from_path(struct sa_path_rec *rec,
 {
        struct rdma_dev_addr dev_addr = {};
        union {
-               struct sockaddr     _sockaddr;
                struct sockaddr_in  _sockaddr_in;
                struct sockaddr_in6 _sockaddr_in6;
        } sgid_addr, dgid_addr;
@@ -1249,12 +1248,12 @@ static int roce_resolve_route_from_path(struct sa_path_rec *rec,
         */
        dev_addr.net = &init_net;
 
-       rdma_gid2ip(&sgid_addr._sockaddr, &rec->sgid);
-       rdma_gid2ip(&dgid_addr._sockaddr, &rec->dgid);
+       rdma_gid2ip((struct sockaddr *)&sgid_addr, &rec->sgid);
+       rdma_gid2ip((struct sockaddr *)&dgid_addr, &rec->dgid);
 
        /* validate the route */
-       ret = rdma_resolve_ip_route(&sgid_addr._sockaddr,
-                                   &dgid_addr._sockaddr, &dev_addr);
+       ret = rdma_resolve_ip_route((struct sockaddr *)&sgid_addr,
+                                   (struct sockaddr *)&dgid_addr, &dev_addr);
        if (ret)
                return ret;