# git rev-parse -q --verify 4df910620bebb5cfe234af16ac8f6474b60215fd^{commit}
4df910620bebb5cfe234af16ac8f6474b60215fd
already have revision, skipping fetch
# git checkout -q -f -B kisskb 4df910620bebb5cfe234af16ac8f6474b60215fd
# git clean -qxdf
# < git log -1
# commit 4df910620bebb5cfe234af16ac8f6474b60215fd
# Author: Feng Tang <feng.tang@intel.com>
# Date:   Wed Nov 25 13:22:21 2020 +0800
# 
#     mm: memcg: relayout structure mem_cgroup to avoid cache interference
#     
#     0day reported one -22.7% regression for will-it-scale page_fault2
#     case [1] on a 4 sockets 144 CPU platform, and bisected to it to be
#     caused by Waiman's optimization (commit bd0b230fe1) of saving one
#     'struct page_counter' space for 'struct mem_cgroup'.
#     
#     Initially we thought it was due to the cache alignment change introduced
#     by the patch, but further debug shows that it is due to some hot data
#     members ('vmstats_local', 'vmstats_percpu', 'vmstats') sit in 2 adjacent
#     cacheline (2N and 2N+1 cacheline), and when adjacent cache line prefetch
#     is enabled, it triggers an "extended level" of cache false sharing for
#     2 adjacent cache lines.
#     
#     So exchange the 2 member blocks, while keeping mostly the original
#     cache alignment, which can restore and even enhance the performance,
#     and save 64 bytes of space for 'struct mem_cgroup' (from 2880 to 2816,
#     with 0day's default RHEL-8.3 kernel config)
#     
#     [1]. https://lore.kernel.org/lkml/20201102091543.GM31092@shao2-debian/
#     
#     Fixes: bd0b230fe145 ("mm/memcg: unify swap and memsw page counters")
#     Reported-by: kernel test robot <rong.a.chen@intel.com>
#     Signed-off-by: Feng Tang <feng.tang@intel.com>
#     Acked-by: Waiman Long <longman@redhat.com>
#     Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
# < /opt/cross/kisskb/korg/gcc-8.1.0-nolibc/m68k-linux/bin/m68k-linux-gcc --version
# < /opt/cross/kisskb/korg/gcc-8.1.0-nolibc/m68k-linux/bin/m68k-linux-ld --version
# < git log --format=%s --max-count=1 4df910620bebb5cfe234af16ac8f6474b60215fd
# < make -s -j 24 ARCH=m68k O=/kisskb/build/linus_m5272c3_defconfig_m68k-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/m68k-linux/bin/m68k-linux-  m5272c3_defconfig
# < make -s -j 24 ARCH=m68k O=/kisskb/build/linus_m5272c3_defconfig_m68k-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/m68k-linux/bin/m68k-linux-  help
# make -s -j 24 ARCH=m68k O=/kisskb/build/linus_m5272c3_defconfig_m68k-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/m68k-linux/bin/m68k-linux-  olddefconfig
# make -s -j 24 ARCH=m68k O=/kisskb/build/linus_m5272c3_defconfig_m68k-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/m68k-linux/bin/m68k-linux-  
/kisskb/src/drivers/net/ethernet/freescale/fec_main.c: In function 'fec_restart':
/kisskb/src/drivers/net/ethernet/freescale/fec_main.c:947:6: warning: unused variable 'val' [-Wunused-variable]
  u32 val;
      ^~~
/kisskb/src/drivers/net/ethernet/freescale/fec_main.c: In function 'fec_get_mac':
/kisskb/src/drivers/net/ethernet/freescale/fec_main.c:1665:28: warning: unused variable 'pdata' [-Wunused-variable]
  struct fec_platform_data *pdata = dev_get_platdata(&fep->pdev->dev);
                            ^~~~~
Completed OK
# rm -rf /kisskb/build/linus_m5272c3_defconfig_m68k-gcc8
# Build took: 0:00:36.754977