public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Cyrill Gorcunov <gorcunov@gmail.com>
To: mingo@redhat.com, hpa@zytor.com, linux-kernel@vger.kernel.org,
	yinghai@kernel.org, tglx@linutronix.de, mingo@elte.hu
Subject: Re: [tip:irq/numa] x86, apic: Fix dummy apic read operation together with broken MP handling
Date: Sun, 7 Jun 2009 18:33:56 +0400	[thread overview]
Message-ID: <20090607143356.GE4547@lenovo> (raw)
In-Reply-To: <tip-103428e57be323c3c5545db8ad12667099bc6005@git.kernel.org>

[tip-bot for Cyrill Gorcunov - Sun, Jun 07, 2009 at 02:24:31PM +0000]
| Commit-ID:  103428e57be323c3c5545db8ad12667099bc6005
| Gitweb:     http://git.kernel.org/tip/103428e57be323c3c5545db8ad12667099bc6005
| Author:     Cyrill Gorcunov <gorcunov@openvz.org>
| AuthorDate: Sun, 7 Jun 2009 16:48:40 +0400
| Committer:  Ingo Molnar <mingo@elte.hu>
| CommitDate: Sun, 7 Jun 2009 16:08:05 +0200
| 
| x86, apic: Fix dummy apic read operation together with broken MP handling
|

Hi Ingo, this commit will fix your case but to be fair
I'm a bit scared of disable_apic variable now spreading
all over the files. I've checked Makefile's and Kconfig
for situation when SMP turned on and apic.c is not compiled
(from hw POV on x86 it should not be possible since SMP
 requires APIC chip but dunno -- just something from
 inside of me says there could be a hidden problem).

I was also thinking about some common bitfiels for
apic features -- like one bit to figure out if
it was disabled via boot line, one bit for x2apic
mode enabled and so on. We have the cpu_has_...
helpers maybe the same for apic could be implemented?

How do you think?
 
	-- Cyrill

  reply	other threads:[~2009-06-07 14:34 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-07 12:48 [PATCH -tip] x86: apic - fix dummy apic read operation together with broken MP handling Cyrill Gorcunov
2009-06-07 14:24 ` [tip:irq/numa] x86, apic: Fix " tip-bot for Cyrill Gorcunov
2009-06-07 14:33   ` Cyrill Gorcunov [this message]
2009-06-07 19:23   ` Yinghai Lu
2009-06-07 20:39     ` Cyrill Gorcunov
2009-06-07 22:59       ` Yinghai Lu
2009-06-08 15:16         ` Cyrill Gorcunov
2009-06-08 17:47           ` Cyrill Gorcunov

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=20090607143356.GE4547@lenovo \
    --to=gorcunov@gmail.com \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=mingo@redhat.com \
    --cc=tglx@linutronix.de \
    --cc=yinghai@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