mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: David Laight <David.Laight@ACULAB.COM>
To: 'John Stultz' <jstultz@google.com>, LKML <linux-kernel@vger.kernel.org>
Cc: Pavankumar Kondeti <pkondeti@codeaurora.org>,
	John Dias <joaodias@google.com>,
	Connor O'Brien <connoro@google.com>,
	Rick Yiu <rickyiu@google.com>, John Kacur <jkacur@redhat.com>,
	Qais Yousef <qais.yousef@arm.com>,
	Chris Redpath <chris.redpath@arm.com>,
	"Abhijeet Dharmapurikar" <adharmap@quicinc.com>,
	Peter Zijlstra <peterz@infradead.org>,
	Ingo Molnar <mingo@redhat.com>,
	Juri Lelli <juri.lelli@redhat.com>,
	"Vincent Guittot" <vincent.guittot@linaro.org>,
	Dietmar Eggemann <dietmar.eggemann@arm.com>,
	Steven Rostedt <rostedt@goodmis.org>,
	"Thomas Gleixner" <tglx@linutronix.de>,
	"kernel-team@android.com" <kernel-team@android.com>,
	Satya Durga Srinivasu Prabhala <satyap@codeaurora.org>,
	"J . Avila" <elavila@google.com>
Subject: RE: [RFC PATCH v4 3/3] softirq: defer softirq processing to ksoftirqd if CPU is busy with RT
Date: Tue, 4 Oct 2022 10:45:43 +0000	[thread overview]
Message-ID: <ac37c63a5039435ab775a0e983213902@AcuMS.aculab.com> (raw)
In-Reply-To: <20221003232033.3404802-4-jstultz@google.com>

From: John Stultz
> Sent: 04 October 2022 00:21
> 
> From: Pavankumar Kondeti <pkondeti@codeaurora.org>
> 
> Defer the softirq processing to ksoftirqd if a RT task is
> running or queued on the current CPU. This complements the RT
> task placement algorithm which tries to find a CPU that is not
> currently busy with softirqs.
> 
> Currently NET_TX, NET_RX, BLOCK and IRQ_POLL softirqs are only
> deferred as they can potentially run for long time.

Deferring NET_RX to ksoftirqd stops the NET_RX code from
running until the RT process completes.

This has exactly the same problems as the softint taking
priority of the RT task - just the other way around.

Under very high traffic loads receive packets get lost.
In many cases that is actually far worse that the wakeup
of an RT process being delayed slightly.

The is no 'one size fits all' answer to the problem.

Plausibly depending on the priority of the RT task
might be useful.
But sometimes it depends on the actual reason for the
wakeup.
For instance a wakeup from an ISR or a hish-res timer
might need lower latency than one from a futex.

	David

-
Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK
Registration No: 1397386 (Wales)

  reply	other threads:[~2022-10-04 10:45 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-03 23:20 [RFC PATCH v4 0/3] Softirq -rt Optimizations John Stultz
2022-10-03 23:20 ` [RFC PATCH v4 1/3] softirq: Add generic accessor to percpu softirq_pending data John Stultz
2022-10-03 23:20 ` [RFC PATCH v4 2/3] sched: Avoid placing RT threads on cores handling long softirqs John Stultz
     [not found]   ` <20221004013611.1822-1-hdanton@sina.com>
2022-10-04  2:29     ` John Stultz
     [not found]       ` <20221005002149.1876-1-hdanton@sina.com>
2022-10-05  1:13         ` John Stultz
     [not found]         ` <20221005060155.1571-1-hdanton@sina.com>
2022-10-10 15:42           ` Qais Yousef
     [not found]             ` <20221011111846.284-1-hdanton@sina.com>
2022-10-12 14:10               ` Qais Yousef
2022-10-17 12:40   ` [PATCH RFC " Alexander Gordeev
2022-10-18  3:42     ` John Stultz
2022-10-18  3:59       ` John Stultz
2022-10-18  5:32       ` John Stultz
2022-10-19  9:11       ` Alexander Gordeev
2022-10-19 22:09         ` John Stultz
2022-10-20  0:15           ` Qais Yousef
2022-10-20 12:47           ` Alexander Gordeev
2022-10-22 18:34             ` Joel Fernandes
2022-10-23  7:45               ` Alexander Gordeev
2022-11-15  7:08                 ` John Stultz
2022-11-15 12:55                   ` Alexander Gordeev
2022-10-19 11:23       ` Qais Yousef
2022-10-22 18:28   ` [RFC PATCH " Joel Fernandes
2022-11-15 21:36     ` John Stultz
2022-10-03 23:20 ` [RFC PATCH v4 3/3] softirq: defer softirq processing to ksoftirqd if CPU is busy with RT John Stultz
2022-10-04 10:45   ` David Laight [this message]
2022-10-04 19:19     ` John Stultz
2022-10-10 16:09   ` Qais Yousef
2022-10-17 14:44     ` Qais Yousef
2022-10-18  0:04       ` John Stultz
2022-10-19 11:01         ` Qais Yousef

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=ac37c63a5039435ab775a0e983213902@AcuMS.aculab.com \
    --to=david.laight@aculab.com \
    --cc=adharmap@quicinc.com \
    --cc=chris.redpath@arm.com \
    --cc=connoro@google.com \
    --cc=dietmar.eggemann@arm.com \
    --cc=elavila@google.com \
    --cc=jkacur@redhat.com \
    --cc=joaodias@google.com \
    --cc=jstultz@google.com \
    --cc=juri.lelli@redhat.com \
    --cc=kernel-team@android.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=peterz@infradead.org \
    --cc=pkondeti@codeaurora.org \
    --cc=qais.yousef@arm.com \
    --cc=rickyiu@google.com \
    --cc=rostedt@goodmis.org \
    --cc=satyap@codeaurora.org \
    --cc=tglx@linutronix.de \
    --cc=vincent.guittot@linaro.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®