On Mon, Sep 5, 2022 at 12:34 PM Toke Høiland-Jørgensen <toke@xxxxxxxxxx> wrote:
>
> Move the received_rps counter value next to the other RPS-related members
> in softnet_data. This closes two four-byte holes in the structure, making
> room for another pointer in the first two cache lines without bumping the
> xmit struct to its own line.
>
> Reviewed-by: Stanislav Fomichev <sdf@xxxxxxxxxx>
> Signed-off-by: Toke Høiland-Jørgensen <toke@xxxxxxxxxx>
Acked-by: Song Liu <song@xxxxxxxxxx>
> ---
> include/linux/netdevice.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
> index 1a3cb93c3dcc..fe9aeca2fce9 100644
> --- a/include/linux/netdevice.h
> +++ b/include/linux/netdevice.h
> @@ -3100,7 +3100,6 @@ struct softnet_data {
> /* stats */
> unsigned int processed;
> unsigned int time_squeeze;
> - unsigned int received_rps;
> #ifdef CONFIG_RPS
> struct softnet_data *rps_ipi_list;
> #endif
> @@ -3133,6 +3132,7 @@ struct softnet_data {
> unsigned int cpu;
> unsigned int input_queue_tail;
> #endif
> + unsigned int received_rps;
> unsigned int dropped;
> struct sk_buff_head input_pkt_queue;
> struct napi_struct backlog;
> --
> 2.37.2
>
