s390/qeth: trust non-IP cast type in qeth_l3_fill_header()
authorJulian Wiedmann <jwi@linux.ibm.com>
Thu, 25 Apr 2019 16:26:01 +0000 (18:26 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 26 Apr 2019 15:14:06 +0000 (11:14 -0400)
When building the L3 HW header for non-IP packets, trust the cast type
that was passed as parameter. qeth_l3_get_cast_type() has most likely
also used h_dest to determine the cast type, so we get the same
result, and can remove that duplicated code.
In the unlikely case that we would get a _different_ cast type, then
that's based off a route lookup and should be considered authoritative.

Signed-off-by: Julian Wiedmann <jwi@linux.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/s390/net/qeth_l3_main.c

index b5d76ebb488a3e8a4fd2d7f6af10ae02cdb3b4a3..0271833da6a2a42a0c78320318ed3f24f9b0b85f 100644 (file)
@@ -1976,19 +1976,14 @@ static void qeth_l3_fill_header(struct qeth_qdio_out_q *queue,
                hdr->hdr.l3.vlan_id = ntohs(veth->h_vlan_TCI);
        }
 
+       l3_hdr->flags = qeth_l3_cast_type_to_flag(cast_type);
+
        /* OSA only: */
        if (!ipv) {
-               hdr->hdr.l3.flags = QETH_HDR_PASSTHRU;
-               if (ether_addr_equal_64bits(eth_hdr(skb)->h_dest,
-                                           skb->dev->broadcast))
-                       hdr->hdr.l3.flags |= QETH_CAST_BROADCAST;
-               else
-                       hdr->hdr.l3.flags |= (cast_type == RTN_MULTICAST) ?
-                               QETH_CAST_MULTICAST : QETH_CAST_UNICAST;
+               l3_hdr->flags |= QETH_HDR_PASSTHRU;
                return;
        }
 
-       hdr->hdr.l3.flags = qeth_l3_cast_type_to_flag(cast_type);
        rcu_read_lock();
        if (ipv == 4) {
                struct rtable *rt = skb_rtable(skb);