All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Zijlstra <peterz@infradead.org>
To: Marcelo Tosatti <mtosatti@redhat.com>
Cc: Frederic Weisbecker <frederic@kernel.org>,
	Yair Podemsky <ypodemsk@redhat.com>,
	linux@armlinux.org.uk, mpe@ellerman.id.au, npiggin@gmail.com,
	christophe.leroy@csgroup.eu, hca@linux.ibm.com,
	gor@linux.ibm.com, agordeev@linux.ibm.com,
	borntraeger@linux.ibm.com, svens@linux.ibm.com,
	davem@davemloft.net, tglx@linutronix.de, mingo@redhat.com,
	bp@alien8.de, dave.hansen@linux.intel.com, x86@kernel.org,
	hpa@zytor.com, will@kernel.org, aneesh.kumar@linux.ibm.com,
	akpm@linux-foundation.org, arnd@arndb.de, keescook@chromium.org,
	paulmck@kernel.org, jpoimboe@kernel.org, samitolvanen@google.com,
	ardb@kernel.org, juerg.haefliger@canonical.com,
	rmk+kernel@armlinux.org.uk, geert+renesas@glider.be,
	tony@atomide.com, linus.walleij@linaro.org,
	sebastian.reichel@collabora.com, nick.hawkins@hpe.com,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linuxppc-dev@lists.ozlabs.org, linux-s390@vger.kernel.org,
	sparclinux@vger.kernel.org, linux-arch@vger.kernel.org,
	linux-mm@kvack.org, vschneid@redhat.com, dhildenb@redhat.com,
	alougovs@redhat.com
Subject: Re: [PATCH 3/3] mm/mmu_gather: send tlb_remove_table_smp_sync IPI only to CPUs in kernel mode
Date: Thu, 6 Apr 2023 15:32:06 +0200	[thread overview]
Message-ID: <20230406133206.GN386572@hirez.programming.kicks-ass.net> (raw)
In-Reply-To: <ZC6/0hRXztNwqXg0@tpad>

On Thu, Apr 06, 2023 at 09:49:22AM -0300, Marcelo Tosatti wrote:

> > > 2) Depends on the application and the definition of "occasional".
> > > 
> > > For certain types of applications (for example PLC software or
> > > RAN processing), upon occurrence of an event, it is necessary to
> > > complete a certain task in a maximum amount of time (deadline).
> > 
> > If the application is properly NOHZ_FULL and never does a kernel entry,
> > it will never get that IPI. If it is a pile of shit and does kernel
> > entries while it pretends to be NOHZ_FULL it gets to keep the pieces and
> > no amount of crying will get me to care.
> 
> I suppose its common practice to use certain system calls in latency
> sensitive applications, for example nanosleep. Some examples:
> 
> 1) cyclictest		(nanosleep)

cyclictest is not a NOHZ_FULL application, if you tihnk it is, you're
deluded.

> 2) PLC programs		(nanosleep)

What's a PLC? Programmable Logic Circuit?

> A system call does not necessarily have to take locks, does it ?

This all is unrelated to locks

> Or even if application does system calls, but runs under a VM,
> then you are requiring it to never VM-exit.

That seems to be a goal for performance anyway.

> This reduces the flexibility of developing such applications.

Yeah, that's the cards you're dealt, deal with it.

WARNING: multiple messages have this Message-ID (diff)
From: Peter Zijlstra <peterz@infradead.org>
To: Marcelo Tosatti <mtosatti@redhat.com>
Cc: Frederic Weisbecker <frederic@kernel.org>,
	Yair Podemsky <ypodemsk@redhat.com>,
	linux@armlinux.org.uk, mpe@ellerman.id.au, npiggin@gmail.com,
	christophe.leroy@csgroup.eu, hca@linux.ibm.com,
	gor@linux.ibm.com, agordeev@linux.ibm.com,
	borntraeger@linux.ibm.com, svens@linux.ibm.com,
	davem@davemloft.net, tglx@linutronix.de, mingo@redhat.com,
	bp@alien8.de, dave.hansen@linux.intel.com, x86@kernel.org,
	hpa@zytor.com, will@kernel.org, aneesh.kumar@linux.ibm.com,
	akpm@linux-foundation.org, arnd@arndb.de, keescook@chromium.org,
	paulmck@kernel.org, jpoimboe@kernel.org, samitolvanen@google.com,
	ardb@kernel.org, juerg.haefliger@canonical.com,
	rmk+kernel@armlinux.org.uk, geert+renesas@glider.be,
	tony@atomide.com, linus.walleij@linaro.org,
	sebastian.reichel@collabora.com, nick.hawkins@hpe.com,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linuxppc-dev@lists.ozlabs.org, linux-s390@vger.kernel.org,
	sparclinux@vger.kernel.org, linux-arch@vger.kernel.org,
	linux-mm@kvack.org, vschneid@redhat.com, dhildenb@redhat.com,
	alougovs@redhat.com
Subject: Re: [PATCH 3/3] mm/mmu_gather: send tlb_remove_table_smp_sync IPI only to CPUs in kernel mode
Date: Thu, 6 Apr 2023 15:32:06 +0200	[thread overview]
Message-ID: <20230406133206.GN386572@hirez.programming.kicks-ass.net> (raw)
In-Reply-To: <ZC6/0hRXztNwqXg0@tpad>

On Thu, Apr 06, 2023 at 09:49:22AM -0300, Marcelo Tosatti wrote:

> > > 2) Depends on the application and the definition of "occasional".
> > > 
> > > For certain types of applications (for example PLC software or
> > > RAN processing), upon occurrence of an event, it is necessary to
> > > complete a certain task in a maximum amount of time (deadline).
> > 
> > If the application is properly NOHZ_FULL and never does a kernel entry,
> > it will never get that IPI. If it is a pile of shit and does kernel
> > entries while it pretends to be NOHZ_FULL it gets to keep the pieces and
> > no amount of crying will get me to care.
> 
> I suppose its common practice to use certain system calls in latency
> sensitive applications, for example nanosleep. Some examples:
> 
> 1) cyclictest		(nanosleep)

cyclictest is not a NOHZ_FULL application, if you tihnk it is, you're
deluded.

> 2) PLC programs		(nanosleep)

What's a PLC? Programmable Logic Circuit?

> A system call does not necessarily have to take locks, does it ?

This all is unrelated to locks

> Or even if application does system calls, but runs under a VM,
> then you are requiring it to never VM-exit.

That seems to be a goal for performance anyway.

> This reduces the flexibility of developing such applications.

Yeah, that's the cards you're dealt, deal with it.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Peter Zijlstra <peterz@infradead.org>
To: Marcelo Tosatti <mtosatti@redhat.com>
Cc: geert+renesas@glider.be, tony@atomide.com,
	linus.walleij@linaro.org, dave.hansen@linux.intel.com,
	Yair Podemsky <ypodemsk@redhat.com>,
	sebastian.reichel@collabora.com, linux-mm@kvack.org,
	hpa@zytor.com, sparclinux@vger.kernel.org,
	agordeev@linux.ibm.com, will@kernel.org, ardb@kernel.org,
	linux-arch@vger.kernel.org, linux-s390@vger.kernel.org,
	vschneid@redhat.com, arnd@arndb.de, paulmck@kernel.org,
	aneesh.kumar@linux.ibm.com, x86@kernel.org,
	linux@armlinux.org.uk, mingo@redhat.com, samitolvanen@google.com,
	borntraeger@linux.ibm.com, hca@linux.ibm.com,
	keescook@chromium.org, gor@linux.ibm.com,
	Frederic Weisbecker <frederic@kernel.org>,
	npiggin@gmail.com, rmk+kernel@armlinux.org.uk, bp@alien8.de,
	nick.hawkins@hpe.com, tglx@linutronix.de, jpoimboe@kernel.org,
	linux-arm-kernel@lists.infradead.org, alougovs@redhat.com,
	linux-kernel@vger.kernel.org, juerg.haefliger@canonical.com,
	svens@linux.ibm.com, dhildenb@redhat.com,
	akpm@linux-foundation.org, linuxppc-dev@lists.ozlabs.org,
	davem@da vemloft.net
Subject: Re: [PATCH 3/3] mm/mmu_gather: send tlb_remove_table_smp_sync IPI only to CPUs in kernel mode
Date: Thu, 6 Apr 2023 15:32:06 +0200	[thread overview]
Message-ID: <20230406133206.GN386572@hirez.programming.kicks-ass.net> (raw)
In-Reply-To: <ZC6/0hRXztNwqXg0@tpad>

On Thu, Apr 06, 2023 at 09:49:22AM -0300, Marcelo Tosatti wrote:

> > > 2) Depends on the application and the definition of "occasional".
> > > 
> > > For certain types of applications (for example PLC software or
> > > RAN processing), upon occurrence of an event, it is necessary to
> > > complete a certain task in a maximum amount of time (deadline).
> > 
> > If the application is properly NOHZ_FULL and never does a kernel entry,
> > it will never get that IPI. If it is a pile of shit and does kernel
> > entries while it pretends to be NOHZ_FULL it gets to keep the pieces and
> > no amount of crying will get me to care.
> 
> I suppose its common practice to use certain system calls in latency
> sensitive applications, for example nanosleep. Some examples:
> 
> 1) cyclictest		(nanosleep)

cyclictest is not a NOHZ_FULL application, if you tihnk it is, you're
deluded.

> 2) PLC programs		(nanosleep)

What's a PLC? Programmable Logic Circuit?

> A system call does not necessarily have to take locks, does it ?

This all is unrelated to locks

> Or even if application does system calls, but runs under a VM,
> then you are requiring it to never VM-exit.

That seems to be a goal for performance anyway.

> This reduces the flexibility of developing such applications.

Yeah, that's the cards you're dealt, deal with it.

  reply	other threads:[~2023-04-06 13:33 UTC|newest]

Thread overview: 107+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-04 13:42 [PATCH 0/3] send tlb_remove_table_smp_sync IPI only to necessary CPUs Yair Podemsky
2023-04-04 13:42 ` Yair Podemsky
2023-04-04 13:42 ` Yair Podemsky
2023-04-04 13:42 ` [PATCH 1/3] arch: Introduce ARCH_HAS_CPUMASK_BITS Yair Podemsky
2023-04-04 13:42   ` Yair Podemsky
2023-04-04 13:42   ` Yair Podemsky
2023-04-04 13:47   ` David Hildenbrand
2023-04-04 13:47     ` David Hildenbrand
2023-04-04 13:42 ` [PATCH 2/3] mm/mmu_gather: send tlb_remove_table_smp_sync IPI only to MM CPUs Yair Podemsky
2023-04-04 13:42   ` Yair Podemsky
2023-04-04 13:42   ` Yair Podemsky
2023-04-04 14:57   ` Peter Zijlstra
2023-04-04 14:57     ` Peter Zijlstra
2023-04-04 14:57     ` Peter Zijlstra
2023-04-04 13:42 ` [PATCH 3/3] mm/mmu_gather: send tlb_remove_table_smp_sync IPI only to CPUs in kernel mode Yair Podemsky
2023-04-04 13:42   ` Yair Podemsky
2023-04-04 13:42   ` Yair Podemsky
2023-04-04 14:03   ` David Hildenbrand
2023-04-04 14:03     ` David Hildenbrand
2023-04-04 15:12   ` Peter Zijlstra
2023-04-04 15:12     ` Peter Zijlstra
2023-04-04 15:12     ` Peter Zijlstra
2023-04-04 16:00     ` Peter Zijlstra
2023-04-04 16:00       ` Peter Zijlstra
2023-04-04 16:00       ` Peter Zijlstra
2023-04-05  0:53       ` Hillf Danton
2023-04-05 10:43   ` Frederic Weisbecker
2023-04-05 10:43     ` Frederic Weisbecker
2023-04-05 10:43     ` Frederic Weisbecker
2023-04-05 11:10     ` Frederic Weisbecker
2023-04-05 11:10       ` Frederic Weisbecker
2023-04-05 11:10       ` Frederic Weisbecker
2023-04-05 11:41       ` Peter Zijlstra
2023-04-05 11:41         ` Peter Zijlstra
2023-04-05 11:41         ` Peter Zijlstra
2023-04-05 12:00         ` David Hildenbrand
2023-04-05 12:00           ` David Hildenbrand
2023-04-05 12:00           ` David Hildenbrand
2023-04-05 12:05         ` Frederic Weisbecker
2023-04-05 12:05           ` Frederic Weisbecker
2023-04-05 12:05           ` Frederic Weisbecker
2023-04-05 12:31           ` Frederic Weisbecker
2023-04-05 12:31             ` Frederic Weisbecker
2023-04-05 12:31             ` Frederic Weisbecker
2023-04-05 12:45           ` Valentin Schneider
2023-04-05 12:45             ` Valentin Schneider
2023-04-05 12:45             ` Valentin Schneider
2023-04-06 13:38             ` Peter Zijlstra
2023-04-06 13:38               ` Peter Zijlstra
2023-04-06 13:38               ` Peter Zijlstra
2023-04-06 14:11               ` Valentin Schneider
2023-04-06 14:11                 ` Valentin Schneider
2023-04-06 14:11                 ` Valentin Schneider
2023-04-06 14:39                 ` Peter Zijlstra
2023-04-06 14:39                   ` Peter Zijlstra
2023-04-06 14:39                   ` Peter Zijlstra
2023-04-05 19:45       ` Marcelo Tosatti
2023-04-05 19:45         ` Marcelo Tosatti
2023-04-05 19:45         ` Marcelo Tosatti
2023-04-05 19:52         ` Peter Zijlstra
2023-04-05 19:52           ` Peter Zijlstra
2023-04-05 19:52           ` Peter Zijlstra
2023-04-06 12:38           ` Marcelo Tosatti
2023-04-06 12:38             ` Marcelo Tosatti
2023-04-06 12:38             ` Marcelo Tosatti
2023-04-06 13:29             ` Peter Zijlstra
2023-04-06 13:29               ` Peter Zijlstra
2023-04-06 13:29               ` Peter Zijlstra
2023-04-06 14:04               ` Peter Zijlstra
2023-04-06 14:04                 ` Peter Zijlstra
2023-04-06 14:04                 ` Peter Zijlstra
2023-04-06 14:42                 ` David Hildenbrand
2023-04-06 14:42                   ` David Hildenbrand
2023-04-06 14:42                   ` David Hildenbrand
2023-04-06 15:06                   ` Peter Zijlstra
2023-04-06 15:06                     ` Peter Zijlstra
2023-04-06 15:06                     ` Peter Zijlstra
2023-04-06 15:02                 ` Peter Zijlstra
2023-04-06 15:02                   ` Peter Zijlstra
2023-04-06 15:02                   ` Peter Zijlstra
2023-04-06 15:51                   ` David Hildenbrand
2023-04-06 15:51                     ` David Hildenbrand
2023-04-06 15:51                     ` David Hildenbrand
2023-04-06 18:27                     ` Peter Zijlstra
2023-04-06 18:27                       ` Peter Zijlstra
2023-04-06 18:27                       ` Peter Zijlstra
2023-04-19 11:30                       ` David Hildenbrand
2023-04-19 11:30                         ` David Hildenbrand
2023-04-19 11:30                         ` David Hildenbrand
2023-04-19 11:39                         ` Marcelo Tosatti
2023-04-19 11:39                           ` Marcelo Tosatti
2023-04-19 11:39                           ` Marcelo Tosatti
2023-04-05 19:43     ` Marcelo Tosatti
2023-04-05 19:43       ` Marcelo Tosatti
2023-04-05 19:43       ` Marcelo Tosatti
2023-04-05 19:54       ` Peter Zijlstra
2023-04-05 19:54         ` Peter Zijlstra
2023-04-05 19:54         ` Peter Zijlstra
2023-04-06 12:49         ` Marcelo Tosatti
2023-04-06 12:49           ` Marcelo Tosatti
2023-04-06 12:49           ` Marcelo Tosatti
2023-04-06 13:32           ` Peter Zijlstra [this message]
2023-04-06 13:32             ` Peter Zijlstra
2023-04-06 13:32             ` Peter Zijlstra
2023-04-19 11:01             ` Marcelo Tosatti
2023-04-19 11:01               ` Marcelo Tosatti
2023-04-19 11:01               ` Marcelo Tosatti

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=20230406133206.GN386572@hirez.programming.kicks-ass.net \
    --to=peterz@infradead.org \
    --cc=agordeev@linux.ibm.com \
    --cc=akpm@linux-foundation.org \
    --cc=alougovs@redhat.com \
    --cc=aneesh.kumar@linux.ibm.com \
    --cc=ardb@kernel.org \
    --cc=arnd@arndb.de \
    --cc=borntraeger@linux.ibm.com \
    --cc=bp@alien8.de \
    --cc=christophe.leroy@csgroup.eu \
    --cc=dave.hansen@linux.intel.com \
    --cc=davem@davemloft.net \
    --cc=dhildenb@redhat.com \
    --cc=frederic@kernel.org \
    --cc=geert+renesas@glider.be \
    --cc=gor@linux.ibm.com \
    --cc=hca@linux.ibm.com \
    --cc=hpa@zytor.com \
    --cc=jpoimboe@kernel.org \
    --cc=juerg.haefliger@canonical.com \
    --cc=keescook@chromium.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=mingo@redhat.com \
    --cc=mpe@ellerman.id.au \
    --cc=mtosatti@redhat.com \
    --cc=nick.hawkins@hpe.com \
    --cc=npiggin@gmail.com \
    --cc=paulmck@kernel.org \
    --cc=rmk+kernel@armlinux.org.uk \
    --cc=samitolvanen@google.com \
    --cc=sebastian.reichel@collabora.com \
    --cc=sparclinux@vger.kernel.org \
    --cc=svens@linux.ibm.com \
    --cc=tglx@linutronix.de \
    --cc=tony@atomide.com \
    --cc=vschneid@redhat.com \
    --cc=will@kernel.org \
    --cc=x86@kernel.org \
    --cc=ypodemsk@redhat.com \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.