mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Nick Piggin <nickpiggin@yahoo.com.au>
To: Andi Kleen <ak@suse.de>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	ashok.raj@intel.com, linux-kernel@vger.kernel.org
Subject: Re: [RFC][PATCH] avoid cpu hot remove of cpus which have special RT tasks.
Date: Sat, 17 Jun 2006 13:46:30 +1000	[thread overview]
Message-ID: <44937B16.3050204@yahoo.com.au> (raw)
In-Reply-To: <200606161236.50302.ak@suse.de>

Andi Kleen wrote:
> On Friday 16 June 2006 12:26, KAMEZAWA Hiroyuki wrote:
> 
>>On 16 Jun 2006 11:14:57 +0200
>>Andi Kleen <ak@suse.de> wrote:
>>
>>
>>>KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com> writes:
>>>
>>>>This is a bit pessimistic. But forecd migration of RT task which is bounded
>>>>to the special cpu will cause unpredictable trouble, I think.
>>>
>>>More trouble than running it on a CPU that is about to fail?
>>>Doubtful.
>>>
>>
>>With my patch, RT tasks will continute to run.
> 
> 
> That's the problem - if the CPU is failing and you have to remove
> it the task will likely corrupt its data or fail in other ways
> if it doesn't allow it.
> 
> Better to let RT tasks run a little slower on another CPU.
> 
>  
> 
>>Assume there are some multi-threaded tasks with SCHED_FIFO.
>>If they uses some kind of synchronization in user land and task is migrated to
>>other cpus, it will cause dead-lock.
> 
> 
> If its CPU fails much worse things than that will happen.
> 
> One way might be to break affinity of all processes in the system on hot unplug
> - then your deadlock would be avoided - but it might be a bit radical.

Agreed. The kernel is just doing some basic fallback behaviour. If you
actually have a critical RT system, you probably need to have much more
sophisticated handling of CPU unplug anyway. So it doesn't make much
sense to complicate the kernel for this.

-- 
SUSE Labs, Novell Inc.
Send instant messages to your online friends http://au.messenger.yahoo.com 

  parent reply	other threads:[~2006-06-17  3:46 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-16  7:23 KAMEZAWA Hiroyuki
2006-06-16  9:14 ` Andi Kleen
2006-06-16 10:26   ` KAMEZAWA Hiroyuki
2006-06-16 10:36     ` Andi Kleen
2006-06-16 10:58       ` KAMEZAWA Hiroyuki
2006-06-16 11:20         ` Andi Kleen
2006-06-16 11:25           ` KAMEZAWA Hiroyuki
2006-06-17  3:46       ` Nick Piggin [this message]
2006-06-17  5:12         ` KAMEZAWA Hiroyuki
2006-06-17  7:29           ` Nick Piggin
2006-06-17  7:53             ` KAMEZAWA Hiroyuki
2006-06-17  8:48               ` Nick Piggin
2006-06-17  8:58                 ` KAMEZAWA Hiroyuki
2006-06-16 16:09 ` Christoph Lameter
2006-06-16 16:46   ` KAMEZAWA Hiroyuki
2006-06-16 17:29     ` Ashok Raj
2006-06-16 23:47       ` KAMEZAWA Hiroyuki
2006-06-18 16:46 ` Pavel Machek
2006-06-19  1:12   ` KAMEZAWA Hiroyuki

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=44937B16.3050204@yahoo.com.au \
    --to=nickpiggin@yahoo.com.au \
    --cc=ak@suse.de \
    --cc=ashok.raj@intel.com \
    --cc=kamezawa.hiroyu@jp.fujitsu.com \
    --cc=linux-kernel@vger.kernel.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®