[Devel] [PATCH RHEL7 COMMIT] /proc/vz/latency: distinguish atomic allocations in irq from in task atomics.
Konstantin Khorenko
khorenko at virtuozzo.com
Fri Aug 31 13:29:59 MSK 2018
The commit is pushed to "branch-rh7-3.10.0-862.11.6.vz7.71.x-ovz" and will appear at https://src.openvz.org/scm/ovz/vzkernel.git
after rh7-3.10.0-862.11.6.vz7.71.6
------>
commit 62fb4398386fa408186cae3998d1813e833b0f63
Author: Andrey Ryabinin <aryabinin at virtuozzo.com>
Date: Fri Aug 31 13:29:58 2018 +0300
/proc/vz/latency: distinguish atomic allocations in irq from in task atomics.
Add to /proc/vz/latency 'alocirq' allocation type which shows allocation latencies
done in irq contexts. 'alocatomic' now shows atomic allocations in task contexts.
Also add 'Per-CPU alloc irq' which shows per-cpu 'alocirq' numbers.
Example of new output:
Version: 2.6
Latencies:
Type Lat Total_lat Calls
scheduling: 0 0 0
alocatomic: 0 0 27615
aloclow: 2000000 738000000 3166625
alochigh: 0 0 0
aloclowmp: 0 4000000 15677
alochighmp: 0 0 0
alocirq: 1000000 81000000 292975
swap_in: 10710 1553913 25
page_in: 50850 1141057002 2092877
Averages:
Type Avg1 Avg5 Avg15
scheduling: 0 0 0
alocatomic: 0 0 0
aloclow: 558150 313968 165801
alochigh: 0 0 0
aloclowmp: 5535 9701 4629
alochighmp: 0 0 0
alocirq: 309600 163409 66832
swap_in: 29962 18540 7244
page_in: 177194 60133 29625
Per-CPU alloc irq:
Type Lat Total_lat Calls
cpu0 0 0 3
cpu1 0 0 0
cpu2 0 0 0
cpu3 1000000 87000000 316298
https://jira.sw.ru/browse/PSBM-87797
Signed-off-by: Andrey Ryabinin <aryabinin at virtuozzo.com>
Cc: Pavel Borzenkov <Pavel.Borzenkov at acronis.com>
Reviewed-by: Denis V. Lunev <den at openvz.org>
---
include/linux/kstat.h | 1 +
kernel/ve/vzstat.c | 21 +++++++++++++++++++--
mm/page_alloc.c | 10 +++++++---
3 files changed, 27 insertions(+), 5 deletions(-)
diff --git a/include/linux/kstat.h b/include/linux/kstat.h
index 4fff25bcb93f..b268752f2e15 100644
--- a/include/linux/kstat.h
+++ b/include/linux/kstat.h
@@ -7,6 +7,7 @@ enum {
KSTAT_ALLOCSTAT_HIGH,
KSTAT_ALLOCSTAT_LOW_MP,
KSTAT_ALLOCSTAT_HIGH_MP,
+ KSTAT_ALLOCSTAT_IRQ,
KSTAT_ALLOCSTAT_NR,
};
diff --git a/kernel/ve/vzstat.c b/kernel/ve/vzstat.c
index 69cbb38210c0..c8cb0b525aae 100644
--- a/kernel/ve/vzstat.c
+++ b/kernel/ve/vzstat.c
@@ -7,6 +7,7 @@
#include <linux/sched.h>
#include <linux/module.h>
+#include <linux/cpu.h>
#include <linux/mm.h>
#include <linux/ve.h>
#include <linux/ve_proto.h>
@@ -32,7 +33,8 @@ static const char *alloc_descr[KSTAT_ALLOCSTAT_NR] = {
"aloclow:",
"alochigh:",
"aloclowmp:",
- "alochighmp:"
+ "alochighmp:",
+ "alocirq:"
};
/*
@@ -175,11 +177,12 @@ static void avglat_seq_show(struct seq_file *m,
static int latency_seq_show(struct seq_file *m, void *v)
{
int i;
+ int cpu;
if (!v)
return 0;
- seq_puts(m, "Version: 2.5\n");
+ seq_puts(m, "Version: 2.6\n");
seq_puts(m, "\nLatencies:\n");
seq_printf(m, "%-11s %20s %20s %20s\n",
@@ -201,6 +204,20 @@ static int latency_seq_show(struct seq_file *m, void *v)
avglat_seq_show(m, "swap_in:", kstat_glob.swap_in.avg);
avglat_seq_show(m, "page_in:", kstat_glob.page_in.avg);
+ seq_puts(m, "\nPer-CPU alloc irq:\n");
+ seq_printf(m, "%-11s %20s %20s %20s\n",
+ "Type", "Lat", "Total_lat", "Calls");
+
+ get_online_cpus();
+ for_each_online_cpu(cpu) {
+ struct kstat_lat_pcpu_snap_struct *snap;
+
+ snap = per_cpu_ptr(kstat_glob.alloc_lat[KSTAT_ALLOCSTAT_IRQ].cur, cpu);
+ seq_printf(m, "cpu%-8d %20Lu %20Lu %20lu\n", cpu,
+ snap->maxlat, snap->totlat, snap->count);
+ }
+ put_online_cpus();
+
return 0;
}
diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index 179bb0c351b2..9d8c9e4eb970 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -3222,13 +3222,17 @@ static void __alloc_collect_stats(gfp_t gfp_mask, unsigned int order,
int ind, cpu;
time = jiffies_to_usecs(jiffies - time) * 1000;
- if (!(gfp_mask & __GFP_WAIT))
- ind = KSTAT_ALLOCSTAT_ATOMIC;
- else
+ if (!(gfp_mask & __GFP_WAIT)) {
+ if (in_task())
+ ind = KSTAT_ALLOCSTAT_ATOMIC;
+ else
+ ind = KSTAT_ALLOCSTAT_IRQ;
+ } else {
if (order > 0)
ind = KSTAT_ALLOCSTAT_LOW_MP;
else
ind = KSTAT_ALLOCSTAT_LOW;
+ }
local_irq_save(flags);
cpu = smp_processor_id();
More information about the Devel
mailing list