From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 98436C6FA86 for ; Wed, 14 Sep 2022 01:53:31 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 2E0B06B0071; Tue, 13 Sep 2022 21:53:31 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 26AAA80007; Tue, 13 Sep 2022 21:53:31 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 0476A6B0074; Tue, 13 Sep 2022 21:53:30 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id E7A916B0071 for ; Tue, 13 Sep 2022 21:53:30 -0400 (EDT) Received: from smtpin23.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id C0E03120869 for ; Wed, 14 Sep 2022 01:53:30 +0000 (UTC) X-FDA: 79909018980.23.FF00B8F Received: from mail-pl1-f180.google.com (mail-pl1-f180.google.com [209.85.214.180]) by imf31.hostedemail.com (Postfix) with ESMTP id 87DDE200B7 for ; Wed, 14 Sep 2022 01:53:30 +0000 (UTC) Received: by mail-pl1-f180.google.com with SMTP id k21so3106594pls.11 for ; Tue, 13 Sep 2022 18:53:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fastly.com; s=google; h=references:in-reply-to:message-id:date:subject:cc:to:from:from:to :cc:subject:date; bh=RtBmjBNUZvS0RgMzRUDGO5THvzbS2zrx//6TWP3NPXc=; b=qAnqAw8/L7vDPOoX7d4dRwgEvkS5SqRkYNqZSxJzz/qOBiv6K6P9XOAk1jnb8w9pYv KcIVfxw1mdCmuvEYLVXJepszcF0735Uh1tIsQb59ZMTgp6/vivSpLmBi9FK7btWO089e iATQ9exm0Wy9DdTte8mp+oy3Colw83lsiXrG8= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=references:in-reply-to:message-id:date:subject:cc:to:from :x-gm-message-state:from:to:cc:subject:date; bh=RtBmjBNUZvS0RgMzRUDGO5THvzbS2zrx//6TWP3NPXc=; b=gDgO1ZxPt5bQqhCTuCtLfftOhLR3OU3vPmVjWqVcB+76ooDPx0XiSy8vdQTSj8OSCF L+eXZ15yTjlA5HSfWqMFryKFE5B1bTkFHwGOE36gTEn8JWIL+fH+g7HnOU0njT1SfnKB lzKbnxL3V1Yl+tpZZEwiGPNxPsWoJl7V8gUUB1t18ACiQvhIyjQQD3toG0hIDGvhQkuW nyKQDG2f3YEgDS2oC6lNVl4RaPpmTGT3oy9CL7s+l2hYfloYXeTNMH/QW7uLfhS+SVRD fTcdWXm32LFfHOdMZI/Il5o7BZ5KZ8btHlFHI+HCuNSSKq+3hJIhtwJdYb4F+lnE/37R rCpA== X-Gm-Message-State: ACrzQf2sSosYdEatpBm2ZTxJoV162Hrt114Ah7LTDbMR9JNLwbVoy2b9 0asAdUCWZUs0nHZ/4SDaTAzPEQ== X-Google-Smtp-Source: AMsMyM6Su9yxFun7Jkm9+A/9QsApEWdfEMuQb4mxhuBiJWsJasvWOG2MtZ2+7066YGuhgR0KaxNDCQ== X-Received: by 2002:a17:90b:384b:b0:200:3215:878b with SMTP id nl11-20020a17090b384b00b002003215878bmr2152164pjb.176.1663120409407; Tue, 13 Sep 2022 18:53:29 -0700 (PDT) Received: from localhost.localdomain (c-73-223-190-181.hsd1.ca.comcast.net. [73.223.190.181]) by smtp.gmail.com with ESMTPSA id w189-20020a627bc6000000b0053e61633057sm8524481pfc.132.2022.09.13.18.53.27 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Tue, 13 Sep 2022 18:53:29 -0700 (PDT) From: Joe Damato To: x86@kernel.org, linux-mm@kvack.org, Dave Hansen , Andy Lutomirski , Peter Zijlstra , Thomas Gleixner , Ingo Molnar , Borislav Petkov , "H. Peter Anvin" , Juri Lelli , Vincent Guittot , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Daniel Bristot de Oliveira , Valentin Schneider , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org Cc: Joe Damato Subject: [RFC 1/1] mm: Add per-task struct tlb counters Date: Tue, 13 Sep 2022 18:51:09 -0700 Message-Id: <1663120270-2673-2-git-send-email-jdamato@fastly.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1663120270-2673-1-git-send-email-jdamato@fastly.com> References: <1663120270-2673-1-git-send-email-jdamato@fastly.com> ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1663120410; a=rsa-sha256; cv=none; b=OP2q/3SLaSdjmBdlADZjArOEviHXgbgx0yjI3O1YlaWJy9y4GwJpGstr4ohl3zaaCFWLGN lk/2P4yYUcJbF1jDM3L7JI/ZjK5M6uh/9+Jm9EmvKTqY4XR8mQ6Sw1M6D9jsaphW9cHVF1 cg+FYnZYXCVM9Wj/T9LY2m4iRi1wlSw= ARC-Authentication-Results: i=1; imf31.hostedemail.com; dkim=pass header.d=fastly.com header.s=google header.b="qAnqAw8/"; dmarc=pass (policy=reject) header.from=fastly.com; spf=pass (imf31.hostedemail.com: domain of jdamato@fastly.com designates 209.85.214.180 as permitted sender) smtp.mailfrom=jdamato@fastly.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1663120410; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:in-reply-to: references:references:dkim-signature; bh=RtBmjBNUZvS0RgMzRUDGO5THvzbS2zrx//6TWP3NPXc=; b=CU690uRjhijq8AI0zk0vC1gKo7BNTU+mqZyZ4nbfvakupW0GMFJ5rMN2JnW5lrRvUeHLtM Cx11A51xON0/INQBeHM01w494C+nwRORJVxDdQRs99ximNSWTa4b5WjYOVWBzU4OvgjkCx xpB0uHDCeO8rSV7vFnKXS2dU2PfL3K4= X-Rspam-User: Authentication-Results: imf31.hostedemail.com; dkim=pass header.d=fastly.com header.s=google header.b="qAnqAw8/"; dmarc=pass (policy=reject) header.from=fastly.com; spf=pass (imf31.hostedemail.com: domain of jdamato@fastly.com designates 209.85.214.180 as permitted sender) smtp.mailfrom=jdamato@fastly.com X-Rspamd-Server: rspam11 X-Stat-Signature: 6prgow91qdm6qfo3ejqowc3yt55cahzb X-Rspamd-Queue-Id: 87DDE200B7 X-HE-Tag: 1663120410-997667 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: TLB shootdowns are tracked globally, but on a busy system it can be difficult to disambiguate the source of TLB shootdowns. Add two counter fields: - nrtlbflush: number of tlb flush events received - ngtlbflush: number of tlb flush events generated Expose those fields in /proc/[pid]/stat so that they can be analyzed alongside similar metrics (e.g. min_flt and maj_flt). Signed-off-by: Joe Damato --- arch/x86/mm/tlb.c | 2 ++ fs/proc/array.c | 9 +++++++++ include/linux/sched.h | 6 ++++++ include/linux/sched/signal.h | 1 + kernel/exit.c | 6 ++++++ kernel/fork.c | 1 + 6 files changed, 25 insertions(+) diff --git a/arch/x86/mm/tlb.c b/arch/x86/mm/tlb.c index c1e31e9..58f7c59 100644 --- a/arch/x86/mm/tlb.c +++ b/arch/x86/mm/tlb.c @@ -745,6 +745,7 @@ static void flush_tlb_func(void *info) if (!local) { inc_irq_stat(irq_tlb_count); count_vm_tlb_event(NR_TLB_REMOTE_FLUSH_RECEIVED); + current->nrtlbflush++; /* Can only happen on remote CPUs */ if (f->mm && f->mm != loaded_mm) @@ -895,6 +896,7 @@ STATIC_NOPV void native_flush_tlb_multi(const struct cpumask *cpumask, * would not happen. */ count_vm_tlb_event(NR_TLB_REMOTE_FLUSH); + current->ngtlbflush++; if (info->end == TLB_FLUSH_ALL) trace_tlb_flush(TLB_REMOTE_SEND_IPI, TLB_FLUSH_ALL); else diff --git a/fs/proc/array.c b/fs/proc/array.c index 49283b81..435afdc 100644 --- a/fs/proc/array.c +++ b/fs/proc/array.c @@ -469,6 +469,7 @@ static int do_task_stat(struct seq_file *m, struct pid_namespace *ns, unsigned long long start_time; unsigned long cmin_flt = 0, cmaj_flt = 0; unsigned long min_flt = 0, maj_flt = 0; + unsigned long ngtlbflush = 0, nrtlbflush = 0; u64 cutime, cstime, utime, stime; u64 cgtime, gtime; unsigned long rsslim = 0; @@ -530,11 +531,15 @@ static int do_task_stat(struct seq_file *m, struct pid_namespace *ns, do { min_flt += t->min_flt; maj_flt += t->maj_flt; + ngtlbflush += t->ngtlbflush; + nrtlbflush += t->nrtlbflush; gtime += task_gtime(t); } while_each_thread(task, t); min_flt += sig->min_flt; maj_flt += sig->maj_flt; + ngtlbflush += sig->ngtlbflush; + nrtlbflush += sig->nrtlbflush; thread_group_cputime_adjusted(task, &utime, &stime); gtime += sig->gtime; @@ -554,6 +559,8 @@ static int do_task_stat(struct seq_file *m, struct pid_namespace *ns, if (!whole) { min_flt = task->min_flt; maj_flt = task->maj_flt; + nrtlbflush = task->nrtlbflush; + ngtlbflush = task->ngtlbflush; task_cputime_adjusted(task, &utime, &stime); gtime = task_gtime(task); } @@ -643,6 +650,8 @@ static int do_task_stat(struct seq_file *m, struct pid_namespace *ns, else seq_puts(m, " 0"); + seq_put_decimal_ull(m, " ", ngtlbflush); + seq_put_decimal_ull(m, " ", nrtlbflush); seq_putc(m, '\n'); if (mm) mmput(mm); diff --git a/include/linux/sched.h b/include/linux/sched.h index 5cdf746..2a0d879 100644 --- a/include/linux/sched.h +++ b/include/linux/sched.h @@ -1047,6 +1047,12 @@ struct task_struct { unsigned long min_flt; unsigned long maj_flt; + /* Number of TLB flushes generated by this task */ + unsigned long ngtlbflush; + + /* Number of TLB flushes received by this task */ + unsigned long nrtlbflush; + /* Empty if CONFIG_POSIX_CPUTIMERS=n */ struct posix_cputimers posix_cputimers; diff --git a/include/linux/sched/signal.h b/include/linux/sched/signal.h index 2009926..4e0b09c 100644 --- a/include/linux/sched/signal.h +++ b/include/linux/sched/signal.h @@ -189,6 +189,7 @@ struct signal_struct { struct prev_cputime prev_cputime; unsigned long nvcsw, nivcsw, cnvcsw, cnivcsw; unsigned long min_flt, maj_flt, cmin_flt, cmaj_flt; + unsigned long ngtlbflush, nrtlbflush; unsigned long inblock, oublock, cinblock, coublock; unsigned long maxrss, cmaxrss; struct task_io_accounting ioac; diff --git a/kernel/exit.c b/kernel/exit.c index 35e0a31..5a72755 100644 --- a/kernel/exit.c +++ b/kernel/exit.c @@ -141,6 +141,8 @@ static void __exit_signal(struct task_struct *tsk) sig->gtime += task_gtime(tsk); sig->min_flt += tsk->min_flt; sig->maj_flt += tsk->maj_flt; + sig->ngtlbflush += tsk->ngtlbflush; + sig->nrtlbflush += tsk->nrtlbflush; sig->nvcsw += tsk->nvcsw; sig->nivcsw += tsk->nivcsw; sig->inblock += task_io_get_inblock(tsk); @@ -1095,6 +1097,10 @@ static int wait_task_zombie(struct wait_opts *wo, struct task_struct *p) p->min_flt + sig->min_flt + sig->cmin_flt; psig->cmaj_flt += p->maj_flt + sig->maj_flt + sig->cmaj_flt; + psig->ngtlbflush += + p->ngtlbflush + sig->ngtlbflush; + psig->nrtlbflush += + p->nrtlbflush + sig->nrtlbflush; psig->cnvcsw += p->nvcsw + sig->nvcsw + sig->cnvcsw; psig->cnivcsw += diff --git a/kernel/fork.c b/kernel/fork.c index b339918..5fa9f64 100644 --- a/kernel/fork.c +++ b/kernel/fork.c @@ -1555,6 +1555,7 @@ static int copy_mm(unsigned long clone_flags, struct task_struct *tsk) struct mm_struct *mm, *oldmm; tsk->min_flt = tsk->maj_flt = 0; + tsk->ngtlbflush = tsk->nrtlbflush = 0; tsk->nvcsw = tsk->nivcsw = 0; #ifdef CONFIG_DETECT_HUNG_TASK tsk->last_switch_count = tsk->nvcsw + tsk->nivcsw; -- 2.7.4