Revert "udp: move udp->gro_enabled to udp->udp_flags"
This reverts commit 753886c0b9
which is
commit e1dc0615c6b08ef36414f08c011965b8fb56198b upstream.
It breaks the Android kernel abi and can be brought back in the future
in an abi-safe way if it is really needed.
Bug: 161946584
Change-Id: I7d75a59f256d9e2a5b7e2adb8afc0e649fe8dba6
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
This commit is contained in:
parent
583a8f8204
commit
7b70acfd8e
@ -34,7 +34,6 @@ enum {
|
||||
UDP_FLAGS_CORK, /* Cork is required */
|
||||
UDP_FLAGS_NO_CHECK6_TX, /* Send zero UDP6 checksums on TX? */
|
||||
UDP_FLAGS_NO_CHECK6_RX, /* Allow zero UDP6 checksums on RX? */
|
||||
UDP_FLAGS_GRO_ENABLED, /* Request GRO aggregation */
|
||||
};
|
||||
|
||||
struct udp_sock {
|
||||
@ -53,6 +52,7 @@ struct udp_sock {
|
||||
* different encapsulation layer set
|
||||
* this
|
||||
*/
|
||||
gro_enabled:1, /* Request GRO aggregation */
|
||||
accept_udp_l4:1,
|
||||
accept_udp_fraglist:1;
|
||||
/* indicator bits used by pcflag: */
|
||||
|
@ -1901,7 +1901,7 @@ int udp_recvmsg(struct sock *sk, struct msghdr *msg, size_t len, int flags,
|
||||
(struct sockaddr *)sin);
|
||||
}
|
||||
|
||||
if (udp_test_bit(GRO_ENABLED, sk))
|
||||
if (udp_sk(sk)->gro_enabled)
|
||||
udp_cmsg_recv(msg, sk, skb);
|
||||
|
||||
if (inet->cmsg_flags)
|
||||
@ -2730,7 +2730,7 @@ int udp_lib_setsockopt(struct sock *sk, int level, int optname,
|
||||
/* when enabling GRO, accept the related GSO packet type */
|
||||
if (valbool)
|
||||
udp_tunnel_encap_enable(sk->sk_socket);
|
||||
udp_assign_bit(GRO_ENABLED, sk, valbool);
|
||||
up->gro_enabled = valbool;
|
||||
up->accept_udp_l4 = valbool;
|
||||
release_sock(sk);
|
||||
break;
|
||||
@ -2820,7 +2820,7 @@ int udp_lib_getsockopt(struct sock *sk, int level, int optname,
|
||||
break;
|
||||
|
||||
case UDP_GRO:
|
||||
val = udp_test_bit(GRO_ENABLED, sk);
|
||||
val = up->gro_enabled;
|
||||
break;
|
||||
|
||||
/* The following two cannot be changed on UDP sockets, the return is
|
||||
|
@ -549,10 +549,10 @@ struct sk_buff *udp_gro_receive(struct list_head *head, struct sk_buff *skb,
|
||||
NAPI_GRO_CB(skb)->is_flist = 0;
|
||||
if (!sk || !udp_sk(sk)->gro_receive) {
|
||||
if (skb->dev->features & NETIF_F_GRO_FRAGLIST)
|
||||
NAPI_GRO_CB(skb)->is_flist = sk ? !udp_test_bit(GRO_ENABLED, sk) : 1;
|
||||
NAPI_GRO_CB(skb)->is_flist = sk ? !udp_sk(sk)->gro_enabled : 1;
|
||||
|
||||
if ((!sk && (skb->dev->features & NETIF_F_GRO_UDP_FWD)) ||
|
||||
(sk && udp_test_bit(GRO_ENABLED, sk)) || NAPI_GRO_CB(skb)->is_flist)
|
||||
(sk && udp_sk(sk)->gro_enabled) || NAPI_GRO_CB(skb)->is_flist)
|
||||
return call_gro_receive(udp_gro_receive_segment, head, skb);
|
||||
|
||||
/* no GRO, be sure flush the current packet */
|
||||
|
@ -440,7 +440,7 @@ int udpv6_recvmsg(struct sock *sk, struct msghdr *msg, size_t len,
|
||||
(struct sockaddr *)sin6);
|
||||
}
|
||||
|
||||
if (udp_test_bit(GRO_ENABLED, sk))
|
||||
if (udp_sk(sk)->gro_enabled)
|
||||
udp_cmsg_recv(msg, sk, skb);
|
||||
|
||||
if (np->rxopt.all)
|
||||
|
Loading…
Reference in New Issue
Block a user