mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Bernd Schubert <bernd.schubert@itwm.fraunhofer.de>
To: Borislav Petkov <bp@alien8.de>, Yinghai Lu <yinghai@kernel.org>,
	Ingo Molnar <mingo@kernel.org>,
	linux-kernel@vger.kernel.org, Weidong Han <weidong.han@intel.com>,
	David Woodhouse <David.Woodhouse@intel.com>,
	sbsiddha@gmail.com
Subject: Re: [regression] 3.7 ends in APIC panic
Date: Mon, 17 Dec 2012 14:11:06 +0100	[thread overview]
Message-ID: <50CF19EA.3070203@itwm.fraunhofer.de> (raw)
In-Reply-To: <20121217100014.GA31866@liondog.tnic>

On 12/17/2012 11:00 AM, Borislav Petkov wrote:
> + Suresh.
>
> On Mon, Dec 17, 2012 at 10:34:46AM +0100, Bernd Schubert wrote:
>> On 12/16/2012 09:39 PM, Borislav Petkov wrote:
>>> On Sun, Dec 16, 2012 at 08:46:06PM +0100, Bernd Schubert wrote:
>>>> Hmm, I read it the other way around - x2apic depends on interrupt
>>>> remapping, but interrupt remapping can be used without x2apic.
>>>
>>> Ok, you're right. X2APIC should depend on IRQ_REMAP:
>>> https://lwn.net/Articles/289881/
>>>
>>>> The help text of CONFIG_IRQ_REMAP also says "x2APIC enhancements or to
>>>> support platforms with CPU's having > 8 bit APIC ID, say Y." I guess
>>>> may CPU has the latter?
>>>
>>> I think it is what Yinghai said - you obviously need x2apic kernel
>>> support if you have IRQ_REMAP on.
>>>
>>>> Can the kernel panic a bit improved to help user to understand what
>>>> needs to be enabled?
>>>
>>> Well, your kernel enables IRQ_REMAP properly:
>>>
>>> [    0.031115] Enabled IRQ remapping in x2apic mode
>>>
>>> I guess at that stage we could probably check for x2apic support and
>>> scream loudly if it is not present... IMHO.
>>>
>> Hmm, I think that would the wrong place,
>
> It has to be the right place because this "Enabled IRQ..." printk above
> is from the IRQ remapping code which detects an x2apic mode in your
> case.
>
>> as the initial 3.7.0 configuration didn't have IRQ_REMAP enabled.
>
> Huh, so why do I see the above message in your dmesg output in
> http://marc.info/?l=linux-kernel&m=135540103415652 then?

Oh huh, that is the dmesg from 3.4.7, which booted fine. I just sent it 
hoping it would help to see where the issue comes from.
When I run "make oldconfig" for 3.7.0 I accidentally unset 
CONFIG_IRQ_REMAP, wich also unset CONFIG_X86_X2APIC :(

>
> Ok, let's sort things out here. Your .config has
>
> # CONFIG_IRQ_REMAP is not set
>
> but in the original dmesg you sent, the printk above comes from
> intel_irq_remapping.c which gets enabled by CONFIG_IRQ_REMAP.
>
> So, can you try enabling only CONFIG_IRQ_REMAP and leave
> CONFIG_X86_X2APIC off to confirm the original observation?

No need, as I said above, the printk comes from a different kernel with 
a different config. With either CONFIG_IRQ_REMAP=false or 
CONFIG_X86_X2APIC=false the booting ends in a kernel panic.

>
> Also, I'd guess your machine can boot with both options off?

No, only partly with "noapic", but that makes ahci to fail to detect 
disks later on.

>
>> And that was the reason why x2apic got disabled during the "make
>> oldconfig" process...
>>
>> Is this message an indication for missing x2apic?
>>
>> "smpboot: weird, boot (#255) not listed by the BIOS"
>
> It's an indication that something is fishy with the apic IDs.
>
> Thanks.
>


Cheers,
Bernd

  reply	other threads:[~2012-12-17 13:11 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-14 14:58 Bernd Schubert
2012-12-16 11:35 ` Ingo Molnar
2012-12-16 15:33   ` Bernd Schubert
2012-12-16 17:43 ` Yinghai Lu
2012-12-16 17:45   ` Yinghai Lu
2012-12-16 18:01     ` Bernd Schubert
2012-12-16 18:07       ` Yinghai Lu
2012-12-16 18:28         ` Bernd Schubert
2012-12-16 19:13           ` Borislav Petkov
2012-12-16 19:46             ` Bernd Schubert
2012-12-16 20:39               ` Borislav Petkov
2012-12-17  9:34                 ` Bernd Schubert
2012-12-17 10:00                   ` Borislav Petkov
2012-12-17 13:11                     ` Bernd Schubert [this message]
2012-12-17  9:27         ` Bernd Schubert
2012-12-17 14:02           ` Ingo Molnar
2012-12-17 16:53             ` Yinghai Lu
2012-12-17 16:56               ` H. Peter Anvin
2012-12-17 17:03                 ` Yinghai Lu
  -- strict thread matches above, loose matches on Subject: below --
2012-12-13 12:16 Bernd Schubert

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=50CF19EA.3070203@itwm.fraunhofer.de \
    --to=bernd.schubert@itwm.fraunhofer.de \
    --cc=David.Woodhouse@intel.com \
    --cc=bp@alien8.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@kernel.org \
    --cc=sbsiddha@gmail.com \
    --cc=weidong.han@intel.com \
    --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

Powered by JetHome