net: sched: shrink struct Qdisc
authorPaolo Abeni <pabeni@redhat.com>
Fri, 25 May 2018 14:28:44 +0000 (16:28 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 29 May 2018 03:13:39 +0000 (23:13 -0400)
commite9be0e993d95adbe5efe0e0f03b2a3e71f5bb2b6
tree59c29c303e00e10e68efc5c048f1bd1b239ec12e
parent102cd909635612c0be784a519651954a7924c786
net: sched: shrink struct Qdisc

The struct Qdisc has a lot of holes, especially after commit
a53851e2c321 ("net: sched: explicit locking in gso_cpu fallback"),
which as a side effect, moved the fields just after 'busylock'
on a new cacheline.

Since both 'padded' and 'refcnt' are not updated frequently, and
there is a hole before 'gso_skb', we can move such fields there,
saving a cacheline without any performance side effect.

Before this commit:

pahole -C Qdisc net/sche/sch_generic.o
# ...
        /* size: 384, cachelines: 6, members: 25 */
        /* sum members: 236, holes: 3, sum holes: 92 */
        /* padding: 56 */

After this commit:
pahole -C Qdisc net/sche/sch_generic.o
# ...
/* size: 320, cachelines: 5, members: 25 */
/* sum members: 236, holes: 2, sum holes: 28 */
/* padding: 56 */

Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Acked-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/sch_generic.h