From nobody Fri Apr 3 09:43:26 2026 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id AAF38C54EE9 for ; Wed, 14 Sep 2022 01:53:49 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230039AbiINBxf (ORCPT ); Tue, 13 Sep 2022 21:53:35 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43564 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229907AbiINBxa (ORCPT ); Tue, 13 Sep 2022 21:53:30 -0400 Received: from mail-pl1-x62f.google.com (mail-pl1-x62f.google.com [IPv6:2607:f8b0:4864:20::62f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F0D386CF55 for ; Tue, 13 Sep 2022 18:53:29 -0700 (PDT) Received: by mail-pl1-x62f.google.com with SMTP id x1so13649396plv.5 for ; Tue, 13 Sep 2022 18:53:29 -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=cL/tIukZIOFMstJlAg2S6//ao9+9L918f7XfDYUQAYJzrPFodXh13gd3+EpGoEW9l8 HLBvwUkBq4D84USAjbGlnMYmjfEJtDAOMMhVcMXI6VePUPKBvt1eVxEyQeq2hxSpC02y fYctH7LxFD41s3f/ukzakH0VVxgPPtvEPJxdYPhi2naqrjTy2LYzsR0ZlJCX8vN11xSC 7XQryyqafHXoM1ONRGeD0X3cbGwzzgQzzjfIwa2/xOr2csL0MoyIwC/fLK3kaGAWcDnC iFz9v/eaW5vRu8v2o4EwwwMbVmlcFHbBVboPvr4hg739smwz/Hlj+CDU7eATkwcflUwd uiyg== X-Gm-Message-State: ACrzQf1HreAx3aXJp6trz4hCMA9wE0oNUnD8Zbe+IyfzTl+yDqlPw0xn 1Ddh3ineZ2gSYq+TMyBX/YiCTg== 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> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" 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++; =20 /* Can only happen on remote CPUs */ if (f->mm && f->mm !=3D loaded_mm) @@ -895,6 +896,7 @@ STATIC_NOPV void native_flush_tlb_multi(const struct cp= umask *cpumask, * would not happen. */ count_vm_tlb_event(NR_TLB_REMOTE_FLUSH); + current->ngtlbflush++; if (info->end =3D=3D 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 =3D 0, cmaj_flt =3D 0; unsigned long min_flt =3D 0, maj_flt =3D 0; + unsigned long ngtlbflush =3D 0, nrtlbflush =3D 0; u64 cutime, cstime, utime, stime; u64 cgtime, gtime; unsigned long rsslim =3D 0; @@ -530,11 +531,15 @@ static int do_task_stat(struct seq_file *m, struct pi= d_namespace *ns, do { min_flt +=3D t->min_flt; maj_flt +=3D t->maj_flt; + ngtlbflush +=3D t->ngtlbflush; + nrtlbflush +=3D t->nrtlbflush; gtime +=3D task_gtime(t); } while_each_thread(task, t); =20 min_flt +=3D sig->min_flt; maj_flt +=3D sig->maj_flt; + ngtlbflush +=3D sig->ngtlbflush; + nrtlbflush +=3D sig->nrtlbflush; thread_group_cputime_adjusted(task, &utime, &stime); gtime +=3D sig->gtime; =20 @@ -554,6 +559,8 @@ static int do_task_stat(struct seq_file *m, struct pid_= namespace *ns, if (!whole) { min_flt =3D task->min_flt; maj_flt =3D task->maj_flt; + nrtlbflush =3D task->nrtlbflush; + ngtlbflush =3D task->ngtlbflush; task_cputime_adjusted(task, &utime, &stime); gtime =3D 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"); =20 + 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; =20 + /* 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=3Dn */ struct posix_cputimers posix_cputimers; =20 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 +=3D task_gtime(tsk); sig->min_flt +=3D tsk->min_flt; sig->maj_flt +=3D tsk->maj_flt; + sig->ngtlbflush +=3D tsk->ngtlbflush; + sig->nrtlbflush +=3D tsk->nrtlbflush; sig->nvcsw +=3D tsk->nvcsw; sig->nivcsw +=3D tsk->nivcsw; sig->inblock +=3D task_io_get_inblock(tsk); @@ -1095,6 +1097,10 @@ static int wait_task_zombie(struct wait_opts *wo, st= ruct task_struct *p) p->min_flt + sig->min_flt + sig->cmin_flt; psig->cmaj_flt +=3D p->maj_flt + sig->maj_flt + sig->cmaj_flt; + psig->ngtlbflush +=3D + p->ngtlbflush + sig->ngtlbflush; + psig->nrtlbflush +=3D + p->nrtlbflush + sig->nrtlbflush; psig->cnvcsw +=3D p->nvcsw + sig->nvcsw + sig->cnvcsw; psig->cnivcsw +=3D 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; =20 tsk->min_flt =3D tsk->maj_flt =3D 0; + tsk->ngtlbflush =3D tsk->nrtlbflush =3D 0; tsk->nvcsw =3D tsk->nivcsw =3D 0; #ifdef CONFIG_DETECT_HUNG_TASK tsk->last_switch_count =3D tsk->nvcsw + tsk->nivcsw; --=20 2.7.4