net/packet: convert po->tp_tx_has_off to an atomic flag
This is to use existing space in po->flags, and reclaim the storage used by the non atomic bit fields. Signed-off-by: Eric Dumazet <edumazet@google.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Родитель
1051ce4ab6
Коммит
7438344660
|
@ -2672,7 +2672,7 @@ static int tpacket_parse_header(struct packet_sock *po, void *frame,
|
||||||
return -EMSGSIZE;
|
return -EMSGSIZE;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (unlikely(po->tp_tx_has_off)) {
|
if (unlikely(packet_sock_flag(po, PACKET_SOCK_TX_HAS_OFF))) {
|
||||||
int off_min, off_max;
|
int off_min, off_max;
|
||||||
|
|
||||||
off_min = po->tp_hdrlen - sizeof(struct sockaddr_ll);
|
off_min = po->tp_hdrlen - sizeof(struct sockaddr_ll);
|
||||||
|
@ -3993,7 +3993,7 @@ packet_setsockopt(struct socket *sock, int level, int optname, sockptr_t optval,
|
||||||
|
|
||||||
lock_sock(sk);
|
lock_sock(sk);
|
||||||
if (!po->rx_ring.pg_vec && !po->tx_ring.pg_vec)
|
if (!po->rx_ring.pg_vec && !po->tx_ring.pg_vec)
|
||||||
po->tp_tx_has_off = !!val;
|
packet_sock_flag_set(po, PACKET_SOCK_TX_HAS_OFF, val);
|
||||||
|
|
||||||
release_sock(sk);
|
release_sock(sk);
|
||||||
return 0;
|
return 0;
|
||||||
|
@ -4120,7 +4120,7 @@ static int packet_getsockopt(struct socket *sock, int level, int optname,
|
||||||
lv = sizeof(rstats);
|
lv = sizeof(rstats);
|
||||||
break;
|
break;
|
||||||
case PACKET_TX_HAS_OFF:
|
case PACKET_TX_HAS_OFF:
|
||||||
val = po->tp_tx_has_off;
|
val = packet_sock_flag(po, PACKET_SOCK_TX_HAS_OFF);
|
||||||
break;
|
break;
|
||||||
case PACKET_QDISC_BYPASS:
|
case PACKET_QDISC_BYPASS:
|
||||||
val = packet_use_direct_xmit(po);
|
val = packet_use_direct_xmit(po);
|
||||||
|
|
|
@ -119,8 +119,7 @@ struct packet_sock {
|
||||||
unsigned long flags;
|
unsigned long flags;
|
||||||
unsigned int running; /* bind_lock must be held */
|
unsigned int running; /* bind_lock must be held */
|
||||||
unsigned int has_vnet_hdr:1, /* writer must hold sock lock */
|
unsigned int has_vnet_hdr:1, /* writer must hold sock lock */
|
||||||
tp_loss:1,
|
tp_loss:1;
|
||||||
tp_tx_has_off:1;
|
|
||||||
int pressure;
|
int pressure;
|
||||||
int ifindex; /* bound device */
|
int ifindex; /* bound device */
|
||||||
__be16 num;
|
__be16 num;
|
||||||
|
@ -146,6 +145,7 @@ static inline struct packet_sock *pkt_sk(struct sock *sk)
|
||||||
enum packet_sock_flags {
|
enum packet_sock_flags {
|
||||||
PACKET_SOCK_ORIGDEV,
|
PACKET_SOCK_ORIGDEV,
|
||||||
PACKET_SOCK_AUXDATA,
|
PACKET_SOCK_AUXDATA,
|
||||||
|
PACKET_SOCK_TX_HAS_OFF,
|
||||||
};
|
};
|
||||||
|
|
||||||
static inline void packet_sock_flag_set(struct packet_sock *po,
|
static inline void packet_sock_flag_set(struct packet_sock *po,
|
||||||
|
|
Загрузка…
Ссылка в новой задаче