summaryrefslogtreecommitdiff
path: root/fs/xattr.c
diff options
context:
space:
mode:
authorTejun Heo <tj@kernel.org>2020-04-09 14:55:35 -0400
committerTejun Heo <tj@kernel.org>2020-04-09 14:55:46 -0400
commitd8ef4b38cb69d907f9b0e889c44d05fc0f890977 (patch)
treea1e7aeebe14d303c8ed6254ca8cbdf99f420bc68 /fs/xattr.c
parent87ebc45d2d32936fb1c8242032eb0b9bcd058858 (diff)
Revert "cgroup: Add memory barriers to plug cgroup_rstat_updated() race window"
This reverts commit 9a9e97b2f1f2 ("cgroup: Add memory barriers to plug cgroup_rstat_updated() race window"). The commit was added in anticipation of memcg rstat conversion which needed synchronous accounting for the event counters (e.g. oom kill count). However, the conversion didn't get merged due to percpu memory overhead concern which couldn't be addressed at the time. Unfortunately, the patch's addition of smp_mb() to cgroup_rstat_updated() meant that every scheduling event now had to go through an additional full barrier and Mel Gorman noticed it as 1% regression in netperf UDP_STREAM test. There's no need to have this barrier in tree now and even if we need synchronous accounting in the future, the right thing to do is separating that out to a separate function so that hot paths which don't care about synchronous behavior don't have to pay the overhead of the full barrier. Let's revert. Signed-off-by: Tejun Heo <tj@kernel.org> Reported-by: Mel Gorman <mgorman@techsingularity.net> Link: http://lkml.kernel.org/r/20200409154413.GK3818@techsingularity.net Cc: v4.18+
Diffstat (limited to 'fs/xattr.c')
0 files changed, 0 insertions, 0 deletions