stable.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Steven Rostedt <rostedt@goodmis.org>
To: <gregkh@linuxfoundation.org>
Cc: bristot@redhat.com, efault@gmx.de, jkacur@redhat.com,
	mingo@kernel.org, peterz@infradead.org, swood@redhat.com,
	tglx@linutronix.de, torvalds@linux-foundation.org,
	williams@redhat.com, <stable@vger.kernel.org>
Subject: Re: FAILED: patch "[PATCH] sched/rt: Simplify the IPI based RT balancing logic" failed to apply to 3.18-stable tree
Date: Mon, 27 Nov 2017 14:15:14 -0500	[thread overview]
Message-ID: <20171127141514.71641e37@gandalf.local.home> (raw)
In-Reply-To: <151179509410656@kroah.com>

On Mon, 27 Nov 2017 16:04:54 +0100
<gregkh@linuxfoundation.org> wrote:

> The patch below does not apply to the 3.18-stable tree.
> If someone wants it applied there, or to any other stable or longterm
> tree, then please email the backport, including the original git commit
> id to <stable@vger.kernel.org>.
> 
> thanks,
> 
> greg k-h
> 
> ------------------ original commit in Linus's tree ------------------
> 
> >From 4bdced5c9a2922521e325896a7bbbf0132c94e56 Mon Sep 17 00:00:00 2001  
> From: "Steven Rostedt (Red Hat)" <rostedt@goodmis.org>
> Date: Fri, 6 Oct 2017 14:05:04 -0400
> Subject: [PATCH] sched/rt: Simplify the IPI based RT balancing logic
> 
> When a CPU lowers its priority (schedules out a high priority task for a
> lower priority one), a check is made to see if any other CPU has overloaded
> RT tasks (more than one). It checks the rto_mask to determine this and if so
> it will request to pull one of those tasks to itself if the non running RT
> task is of higher priority than the new priority of the next task to run on
> the current CPU.
> 
> When we deal with large number of CPUs, the original pull logic suffered
> from large lock contention on a single CPU run queue, which caused a huge
> latency across all CPUs. This was caused by only having one CPU having
> overloaded RT tasks and a bunch of other CPUs lowering their priority. To
> solve this issue, commit:
> 
>   b6366f048e0c ("sched/rt: Use IPI to trigger RT task push migration instead of pulling")
> 

The bug that is being fixed was added by b6366f048e0c, which was added
in 4.1, and doesn't appear to have been backported to 3.18, so this
shouldn't be needed for 3.18.

-- Steve

      reply	other threads:[~2017-11-27 19:15 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-27 15:04 FAILED: patch "[PATCH] sched/rt: Simplify the IPI based RT balancing logic" failed to apply to 3.18-stable tree gregkh
2017-11-27 19:15 ` Steven Rostedt [this message]

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=20171127141514.71641e37@gandalf.local.home \
    --to=rostedt@goodmis.org \
    --cc=bristot@redhat.com \
    --cc=efault@gmx.de \
    --cc=gregkh@linuxfoundation.org \
    --cc=jkacur@redhat.com \
    --cc=mingo@kernel.org \
    --cc=peterz@infradead.org \
    --cc=stable@vger.kernel.org \
    --cc=swood@redhat.com \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.org \
    --cc=williams@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).