From: Christoph Lameter <email@example.com> To: Thomas Gleixner <firstname.lastname@example.org> Cc: Marcelo Tosatti <email@example.com>, Matthew Wilcox <firstname.lastname@example.org>, email@example.com, Andrew Morton <firstname.lastname@example.org>, Alex Belits <email@example.com>, Phil Auld <firstname.lastname@example.org>, Frederic Weisbecker <email@example.com>, Peter Zijlstra <firstname.lastname@example.org> Subject: Re: [PATCH] mm: introduce sysctl file to flush per-cpu vmstat statistics Date: Wed, 2 Dec 2020 17:43:48 +0000 (UTC) [thread overview] Message-ID: <alpine.DEB.email@example.com> (raw) In-Reply-To: <firstname.lastname@example.org> On Wed, 2 Dec 2020, Thomas Gleixner wrote: > prctl() is the right thing to do. Ok great consensus on that one. > >> In addition to quiescing pending activities on the CPU, it would > >> also be useful to assign a per-task attribute (which is then assigned > >> to a per-CPU attribute), indicating whether that CPU is running > >> an isolated task or not. > > > > Sounds good but what would this do? Give a warning like the isolation > > patchset? > > This all needs a lot more thought about the overall picture. We already > have too many knobs and ad hoc hooks which fiddle with isolation. > > The current CPU isolation is a best effort approach and I agree that for > more strict isolation modes we need to be able to enforce that and hunt > down offenders and think about them one by one. There are two apprahces actually to make the OS quiet. One is the best effort approach which is more like the current NOHZ one with additional actions to flush things. The other is the strict approach were one wants a guarantee that the OS does not do anything at all. > > >> To be called before real time loop, one would have: > > Can we please agree in the first place, that "real time" is absolutely > the wrong term here? Sorry I once swore that I would not use real time since that term is so damaged by multiple meanings and marketing endeavor. "to be called before a loop without any disturbance by the operating system" > It's about running undisturbed CPU bound computations whatever nature > they are. It does not matter whether that loop does busy polling ala > DPDK, whether it runs a huge math computation on a data set or > whatever people come up with. Right. > > Often code can tolerate a few interruptions (in some code branches > > regular syscalls may be needed) but one wants the thread to be > > as quiet as possible. > > So you say some code can tolerate a few interrupts, then comes Alex and > says 'no disturbance' at all. Yes that is the current NOHZ approach. You switch it on and after the OS detects are polling loop it will quiet things down. Instead of detecting it we are actively telling the OS to quiet down now. > We really need to stop to look at particular workloads and defining > adhoc solutions tailored to their particular itch if we don't want to > end up with an uncoordinated and unmaintainable zoo of interfaces, hooks > and knobs. True....Multiple subsystem needs to be hooked into this to quiet things down. > That allows Marcelo to start tackling the vmstat side and Alex can > utilize that and build the other parts into it piece by piece. Yup. I agree.
next prev parent reply other threads:[~2020-12-02 17:43 UTC|newest] Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-11-17 16:28 Marcelo Tosatti 2020-11-17 18:03 ` Matthew Wilcox 2020-11-17 19:06 ` Christopher Lameter 2020-11-17 19:09 ` Matthew Wilcox 2020-11-20 18:04 ` Christopher Lameter 2020-11-17 20:23 ` Marcelo Tosatti 2020-11-20 18:02 ` Marcelo Tosatti 2020-11-20 18:20 ` Christopher Lameter 2020-11-23 18:02 ` Marcelo Tosatti 2020-11-24 17:12 ` Vlastimil Babka 2020-11-24 19:52 ` Marcelo Tosatti 2020-11-27 15:48 ` Marcelo Tosatti 2020-11-28 3:49 ` [EXT] " Alex Belits 2020-11-30 18:18 ` Marcelo Tosatti 2020-11-30 18:29 ` Marcelo Tosatti 2020-12-03 22:47 ` Alex Belits 2020-12-03 22:21 ` Alex Belits 2020-11-30 9:31 ` Christoph Lameter 2020-12-02 12:43 ` Marcelo Tosatti 2020-12-02 15:57 ` Thomas Gleixner 2020-12-02 17:43 ` Christoph Lameter [this message] 2020-12-03 3:17 ` Thomas Gleixner 2020-12-07 8:08 ` Christoph Lameter 2020-12-07 16:09 ` Thomas Gleixner 2020-12-07 19:01 ` Thomas Gleixner 2020-12-02 18:38 ` Marcelo Tosatti 2020-12-04 0:20 ` Frederic Weisbecker 2020-12-04 13:31 ` Marcelo Tosatti 2020-12-04 1:43 ` [EXT] " Alex Belits 2021-01-13 12:15 ` [RFC] tentative prctl task isolation interface Marcelo Tosatti 2021-01-14 9:22 ` Christoph Lameter 2021-01-14 19:34 ` Marcelo Tosatti 2021-01-15 13:24 ` Christoph Lameter 2021-01-15 18:35 ` Alex Belits 2021-01-21 15:51 ` Marcelo Tosatti 2021-01-21 16:20 ` Marcelo Tosatti 2021-01-22 13:05 ` Marcelo Tosatti 2021-02-01 10:48 ` Christoph Lameter 2021-02-01 12:47 ` Alex Belits 2021-02-01 18:20 ` Marcelo Tosatti 2021-01-18 15:18 ` Marcelo Tosatti 2020-11-24 5:02 ` [mm] e655d17ffa: BUG:using_smp_processor_id()in_preemptible kernel test robot
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=alpine.DEB.email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='Re: [PATCH] mm: introduce sysctl file to flush per-cpu vmstat statistics' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions on how to clone and mirror all data and code used for this inbox