forked from mirrors/linux
		
	net: Update RFS target at poll for tcp/udp
The current state of affairs is that read()/write() will setup RFS (Receive Flow Steering) for internet protocol sockets while poll()/epoll() does not. When poll() gets called with a TCP or UDP socket, we should update the flow target. This permits to RFS (if enabled) to select the appropriate CPU for following incoming packets. Note: Only connected UDP sockets can benefit from RFS. Signed-off-by: David Majnemer <majnemer@google.com> Signed-off-by: Eric Dumazet <edumazet@google.com> Cc: Paul Turner <pjt@google.com> Cc: Tom Herbert <therbert@google.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
		
							parent
							
								
									aef6de511a
								
							
						
					
					
						commit
						c3f1dbaf6e
					
				
					 2 changed files with 4 additions and 0 deletions
				
			
		| 
						 | 
					@ -436,6 +436,8 @@ unsigned int tcp_poll(struct file *file, struct socket *sock, poll_table *wait)
 | 
				
			||||||
	struct sock *sk = sock->sk;
 | 
						struct sock *sk = sock->sk;
 | 
				
			||||||
	const struct tcp_sock *tp = tcp_sk(sk);
 | 
						const struct tcp_sock *tp = tcp_sk(sk);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						sock_rps_record_flow(sk);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	sock_poll_wait(file, sk_sleep(sk), wait);
 | 
						sock_poll_wait(file, sk_sleep(sk), wait);
 | 
				
			||||||
	if (sk->sk_state == TCP_LISTEN)
 | 
						if (sk->sk_state == TCP_LISTEN)
 | 
				
			||||||
		return inet_csk_listen_poll(sk);
 | 
							return inet_csk_listen_poll(sk);
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -1967,6 +1967,8 @@ unsigned int udp_poll(struct file *file, struct socket *sock, poll_table *wait)
 | 
				
			||||||
	unsigned int mask = datagram_poll(file, sock, wait);
 | 
						unsigned int mask = datagram_poll(file, sock, wait);
 | 
				
			||||||
	struct sock *sk = sock->sk;
 | 
						struct sock *sk = sock->sk;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						sock_rps_record_flow(sk);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	/* Check for false positives due to checksum errors */
 | 
						/* Check for false positives due to checksum errors */
 | 
				
			||||||
	if ((mask & POLLRDNORM) && !(file->f_flags & O_NONBLOCK) &&
 | 
						if ((mask & POLLRDNORM) && !(file->f_flags & O_NONBLOCK) &&
 | 
				
			||||||
	    !(sk->sk_shutdown & RCV_SHUTDOWN) && !first_packet_length(sk))
 | 
						    !(sk->sk_shutdown & RCV_SHUTDOWN) && !first_packet_length(sk))
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue