All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ben Woodard <woodard@redhat.com>
To: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: Neil Horman <nhorman@redhat.com>,
	Neil Horman <nhorman@tuxdriver.com>,
	kexec@lists.infradead.org, Andi Kleen <ak@suse.de>,
	linux-kernel@vger.kernel.org, hbabu@us.ibm.com,
	vgoyal@in.ibm.com
Subject: Re: [PATCH] kexec: force x86_64 arches to boot kdump kernels on boot cpu
Date: Tue, 27 Nov 2007 12:50:52 -0800	[thread overview]
Message-ID: <474C832C.1060903@redhat.com> (raw)
In-Reply-To: <m14pf7k8ns.fsf@ebiederm.dsl.xmission.com>

Eric W. Biederman wrote:
> Neil Horman <nhorman@tuxdriver.com> writes:
> 
>> So, it sounds to me then, like unless I'm willing to really re-write the APIC
>> setup code (which I don't feel qualified to do quite yet), that the immediate
>> solution would be to not rely on interrupts in legacy mode, which was according
>> to my understanding, what the use of the irqpoll command line option was
>> supposed to enable.  Any thoughts as to why that might not be working in this
>> case, or suggested tests to determine a cause there?
> 
> Hmm.  It looks like irqpoll expects to have at least one irq working.
> I wonder if we can fix that.
> 
> Looking at it from the other direction what does this line
> from check_timer() look like when you boot a normal kernel?
> 
> 	apic_printk(APIC_VERBOSE,KERN_INFO "..TIMER: vector=0x%02X apic1=%d pin1=%d apic2=%d pin2=%d\n",
> 		cfg->vector, apic1, pin1, apic2, pin2);
> 

Here is what I get on a normal boot of the problem motherboard:

..TIMER: vector=0x31 apic1=0 pin1=2 apic2=-1 pin2=-1


> I'm curious what values we see at boot for the legacy mode the first
> time it is setup, and possibly what chipset you are on.
> I know we have had a few times where we have failed to reprogram
> the ioapic properly at shutdown.  So it can't hurt to look at
> that possibility one last time.
> 
> 
> For whatever it is worth my original attempt at using only the
> apic mode was commit: f2b36db692b7ff6972320ad9839ae656a3b0ee3e
> Looks like I didn't have an x86_64 version.
> 
> It looks like about half the cleanups I needed was to decouple
> smp cpu startup and apic initialization.  I think the hotplug cpu
> case has done a more thorough version of that now.
> 
> There was a bit of reshuffling needed to get the everything initialized
> in the right order when we started apic mode sooner.
> 
> Anyway I might just take another look at this if I can find enough
> moments to string together.
> 
> Eric
> 
> _______________________________________________
> kexec mailing list
> kexec@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/kexec


-- 
-ben
-=-

_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec

WARNING: multiple messages have this Message-ID (diff)
From: Ben Woodard <woodard@redhat.com>
To: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: Neil Horman <nhorman@tuxdriver.com>,
	Neil Horman <nhorman@redhat.com>,
	kexec@lists.infradead.org, Andi Kleen <ak@suse.de>,
	linux-kernel@vger.kernel.org, hbabu@us.ibm.com,
	vgoyal@in.ibm.com
Subject: Re: [PATCH] kexec: force x86_64 arches to boot kdump kernels on boot cpu
Date: Tue, 27 Nov 2007 12:50:52 -0800	[thread overview]
Message-ID: <474C832C.1060903@redhat.com> (raw)
In-Reply-To: <m14pf7k8ns.fsf@ebiederm.dsl.xmission.com>

Eric W. Biederman wrote:
> Neil Horman <nhorman@tuxdriver.com> writes:
> 
>> So, it sounds to me then, like unless I'm willing to really re-write the APIC
>> setup code (which I don't feel qualified to do quite yet), that the immediate
>> solution would be to not rely on interrupts in legacy mode, which was according
>> to my understanding, what the use of the irqpoll command line option was
>> supposed to enable.  Any thoughts as to why that might not be working in this
>> case, or suggested tests to determine a cause there?
> 
> Hmm.  It looks like irqpoll expects to have at least one irq working.
> I wonder if we can fix that.
> 
> Looking at it from the other direction what does this line
> from check_timer() look like when you boot a normal kernel?
> 
> 	apic_printk(APIC_VERBOSE,KERN_INFO "..TIMER: vector=0x%02X apic1=%d pin1=%d apic2=%d pin2=%d\n",
> 		cfg->vector, apic1, pin1, apic2, pin2);
> 

Here is what I get on a normal boot of the problem motherboard:

..TIMER: vector=0x31 apic1=0 pin1=2 apic2=-1 pin2=-1


> I'm curious what values we see at boot for the legacy mode the first
> time it is setup, and possibly what chipset you are on.
> I know we have had a few times where we have failed to reprogram
> the ioapic properly at shutdown.  So it can't hurt to look at
> that possibility one last time.
> 
> 
> For whatever it is worth my original attempt at using only the
> apic mode was commit: f2b36db692b7ff6972320ad9839ae656a3b0ee3e
> Looks like I didn't have an x86_64 version.
> 
> It looks like about half the cleanups I needed was to decouple
> smp cpu startup and apic initialization.  I think the hotplug cpu
> case has done a more thorough version of that now.
> 
> There was a bit of reshuffling needed to get the everything initialized
> in the right order when we started apic mode sooner.
> 
> Anyway I might just take another look at this if I can find enough
> moments to string together.
> 
> Eric
> 
> _______________________________________________
> kexec mailing list
> kexec@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/kexec


-- 
-ben
-=-

  parent reply	other threads:[~2007-11-27 20:49 UTC|newest]

Thread overview: 202+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-27  1:47 [PATCH] kexec: force x86_64 arches to boot kdump kernels on boot cpu Neil Horman
2007-11-27  1:47 ` Neil Horman
2007-11-27  4:12 ` Eric W. Biederman
2007-11-27  4:12   ` Eric W. Biederman
2007-11-27 13:13   ` Neil Horman
2007-11-27 13:13     ` Neil Horman
2007-11-27 13:28     ` Eric W. Biederman
2007-11-27 13:28       ` Eric W. Biederman
2007-11-27 13:45       ` Andi Kleen
2007-11-27 13:45         ` Andi Kleen
2007-11-27 14:28         ` Neil Horman
2007-11-27 14:28           ` Neil Horman
2007-11-27 14:43           ` Andi Kleen
2007-11-27 14:43             ` Andi Kleen
2007-11-27 14:48             ` Neil Horman
2007-11-27 14:48               ` Neil Horman
2007-11-27 15:24               ` Andi Kleen
2007-11-27 15:24                 ` Andi Kleen
2007-11-27 15:30               ` Eric W. Biederman
2007-11-27 15:30                 ` Eric W. Biederman
2007-11-27 16:45                 ` Neil Horman
2007-11-27 16:45                   ` Neil Horman
2007-11-27 20:50                 ` Ben Woodard [this message]
2007-11-27 20:50                   ` Ben Woodard
2007-11-27 21:05                   ` Neil Horman
2007-11-27 21:05                     ` Neil Horman
2007-11-27 22:38                     ` Eric W. Biederman
2007-11-27 22:38                       ` Eric W. Biederman
2007-11-27 23:15                       ` Ben Woodard
2007-11-27 23:15                         ` Ben Woodard
2007-11-28  0:15                         ` Eric W. Biederman
2007-11-28  0:15                           ` Eric W. Biederman
2007-11-27 23:40                       ` Neil Horman
2007-11-27 23:40                         ` Neil Horman
2007-11-28  0:43                         ` Eric W. Biederman
2007-11-28  0:43                           ` Eric W. Biederman
2007-11-28 15:54                     ` Neil Horman
2007-11-28 15:54                       ` Neil Horman
2007-11-27 14:56         ` Eric W. Biederman
2007-11-27 14:56           ` Eric W. Biederman
2007-11-27 15:34           ` Neil Horman
2007-11-27 15:34             ` Neil Horman
2007-11-27 18:41           ` Ben Woodard
2007-11-27 18:41             ` Ben Woodard
2007-11-27 19:42             ` Neil Horman
2007-11-27 19:42               ` Neil Horman
2007-11-27 20:00               ` Vivek Goyal
2007-11-27 20:00                 ` Vivek Goyal
2007-11-27 20:52                 ` Neil Horman
2007-11-27 20:52                   ` Neil Horman
2007-11-27 22:24                 ` Andi Kleen
2007-11-27 22:24                   ` Andi Kleen
2007-11-27 23:24                   ` Ben Woodard
2007-11-27 23:24                     ` Ben Woodard
2007-11-27 23:56                     ` Andi Kleen
2007-11-27 23:56                       ` Andi Kleen
2007-11-28 15:36                     ` Vivek Goyal
2007-11-28 15:36                       ` Vivek Goyal
2007-11-28 16:02                       ` Neil Horman
2007-11-28 16:02                         ` Neil Horman
2007-11-28 17:36                         ` Eric W. Biederman
2007-11-28 17:36                           ` Eric W. Biederman
2007-11-28 18:16                           ` Neil Horman
2007-11-28 18:16                             ` Neil Horman
2007-11-28 19:05                         ` Vivek Goyal
2007-11-28 19:05                           ` Vivek Goyal
2007-11-28 19:42                           ` Eric W. Biederman
2007-11-28 19:42                             ` Eric W. Biederman
2007-11-28 21:09                             ` Neil Horman
2007-11-28 21:09                               ` Neil Horman
2007-11-28 23:27                               ` Eric W. Biederman
2007-11-28 23:27                                 ` Eric W. Biederman
2007-11-30  2:16                             ` Ben Woodard
2007-11-30  2:16                               ` Ben Woodard
2007-11-30  2:54                               ` Eric W. Biederman
2007-11-30  2:54                                 ` Eric W. Biederman
2007-11-30  8:59                                 ` Yinghai Lu
2007-11-30  8:59                                   ` Yinghai Lu
2007-11-30 14:35                                   ` Vivek Goyal
2007-11-30 14:35                                     ` Vivek Goyal
2007-11-30 14:32                                 ` Neil Horman
2007-11-30 14:32                                   ` Neil Horman
2007-11-30  2:12                           ` Ben Woodard
2007-11-30  2:12                             ` Ben Woodard
2007-11-30 14:42                             ` Vivek Goyal
2007-11-30 14:42                               ` Vivek Goyal
2007-11-30 14:51                               ` Neil Horman
2007-11-30 14:51                                 ` Neil Horman
2007-12-06 21:39                                 ` Neil Horman
2007-12-06 21:39                                   ` Neil Horman
2007-12-06 22:11                                   ` Vivek Goyal
2007-12-06 22:11                                     ` Vivek Goyal
2007-12-07  0:10                                     ` Neil Horman
2007-12-07  0:10                                       ` Neil Horman
2007-12-07 14:39                                       ` Vivek Goyal
2007-12-07 14:39                                         ` Vivek Goyal
2007-12-07 14:53                                         ` Neil Horman
2007-12-07 14:53                                           ` Neil Horman
2007-12-07 15:16                                           ` Vivek Goyal
2007-12-07 15:16                                             ` Vivek Goyal
2007-12-07 15:53                                             ` Neil Horman
2007-12-07 15:53                                               ` Neil Horman
2007-12-07 18:46                                               ` Eric W. Biederman
2007-12-07 18:46                                                 ` Eric W. Biederman
2007-12-07  0:33                                     ` Eric W. Biederman
2007-12-07  0:33                                       ` Eric W. Biederman
2007-12-07  2:04                                       ` Neil Horman
2007-12-07  2:04                                         ` Neil Horman
2007-12-07  8:50                                       ` Yinghai Lu
2007-12-07  8:50                                         ` Yinghai Lu
2007-12-07  9:22                                         ` Yinghai Lu
2007-12-07  9:22                                           ` Yinghai Lu
2007-12-07 14:21                                           ` Neil Horman
2007-12-07 14:21                                             ` Neil Horman
2007-12-07 17:58                                             ` Neil Horman
2007-12-07 17:58                                               ` Neil Horman
2007-12-07 19:19                                               ` yhlu
2007-12-07 19:19                                                 ` yhlu
2007-12-07 20:13                                                 ` Neil Horman
2007-12-07 20:13                                                   ` Neil Horman
2007-12-10 15:39                                               ` Neil Horman
2007-12-10 15:39                                                 ` Neil Horman
2007-12-10 16:20                                                 ` Vivek Goyal
2007-12-10 16:20                                                   ` Vivek Goyal
2007-12-11  1:17                                                 ` Eric W. Biederman
2007-12-11  1:17                                                   ` Eric W. Biederman
2007-12-11  1:08                                               ` Eric W. Biederman
2007-12-11  1:08                                                 ` Eric W. Biederman
2007-12-11  3:43                                                 ` Neil Horman
2007-12-11  3:43                                                   ` Neil Horman
2007-12-11  4:48                                                   ` Eric W. Biederman
2007-12-11  4:48                                                     ` Eric W. Biederman
2007-12-11  6:31                                                     ` Yinghai Lu
2007-12-11  6:31                                                       ` Yinghai Lu
2007-12-11 14:39                                                     ` Neil Horman
2007-12-11 14:39                                                       ` Neil Horman
2007-12-11 15:29                                                       ` Eric W. Biederman
2007-12-11 15:29                                                         ` Eric W. Biederman
2007-12-11 18:00                                                         ` Yinghai Lu
2007-12-11 18:00                                                           ` Yinghai Lu
2007-12-11 18:29                                                           ` Neil Horman
2007-12-11 18:29                                                             ` Neil Horman
2007-12-11 18:45                                                             ` Yinghai Lu
2007-12-11 18:45                                                               ` Yinghai Lu
2007-12-11 18:22                                                         ` Neil Horman
2007-12-11 18:22                                                           ` Neil Horman
2007-12-11 18:46                                                           ` Eric W. Biederman
2007-12-11 18:46                                                             ` Eric W. Biederman
2007-12-11 19:24                                                             ` Neil Horman
2007-12-11 19:24                                                               ` Neil Horman
2007-12-11 19:51                                                               ` Yinghai Lu
2007-12-11 19:51                                                                 ` Yinghai Lu
2007-12-11 20:59                                                                 ` Neil Horman
2007-12-11 20:59                                                                   ` Neil Horman
2007-12-12  0:16                                                                   ` Ben Woodard
2007-12-12  0:16                                                                     ` Ben Woodard
2007-12-12  0:52                                                                     ` Neil Horman
2007-12-12  0:52                                                                       ` Neil Horman
2007-12-12  1:07                                                                       ` Yinghai Lu
2007-12-12  1:07                                                                         ` Yinghai Lu
2007-12-12  8:43                                                                   ` [PATCH] k8: Enable legacy irqs with extended cpu ids Eric W. Biederman
2007-12-12  8:43                                                                     ` Eric W. Biederman
2007-12-12 14:21                                                                   ` [PATCH] kexec: force x86_64 arches to boot kdump kernels on boot cpu Andi Kleen
2007-12-12 14:21                                                                     ` Andi Kleen
2007-12-12 15:55                                                                     ` Neil Horman
2007-12-12 15:55                                                                       ` Neil Horman
2007-12-12 16:07                                                                       ` Andi Kleen
2007-12-12 16:07                                                                         ` Andi Kleen
2007-12-12 19:43                                                                         ` Eric W. Biederman
2007-12-12 19:43                                                                           ` Eric W. Biederman
2007-12-12 20:22                                                                           ` Neil Horman
2007-12-12 20:22                                                                             ` Neil Horman
2007-12-12 21:32                                                                             ` Eric W. Biederman
2007-12-12 21:32                                                                               ` Eric W. Biederman
2007-12-13 14:39                                                                               ` Neil Horman
2007-12-13 14:39                                                                                 ` Neil Horman
2007-12-13 15:16                                                                                 ` Andi Kleen
2007-12-13 15:16                                                                                   ` Andi Kleen
2007-12-13 15:32                                                                                   ` Neil Horman
2007-12-13 15:32                                                                                     ` Neil Horman
2007-12-17 11:38                                                                                     ` Neil Horman
2007-12-17 11:38                                                                                       ` Neil Horman
2007-12-18  0:13                                                                                   ` Eric W. Biederman
2007-12-18  0:13                                                                                     ` Eric W. Biederman
2007-12-17 15:16                                                                                 ` Ingo Molnar
2007-12-17 15:16                                                                                   ` Ingo Molnar
2007-12-17 15:47                                                                                   ` Neil Horman
2007-12-17 15:47                                                                                     ` Neil Horman
2007-12-07 18:36                                             ` Eric W. Biederman
2007-12-07 18:36                                               ` Eric W. Biederman
2007-12-07 18:48                                               ` Neil Horman
2007-12-07 18:48                                                 ` Neil Horman
2007-12-10 19:39                                                 ` Ben Woodard
2007-12-10 20:22                                                   ` Neil Horman
2007-11-27 13:53       ` Neil Horman
2007-11-27 13:53         ` Neil Horman
2007-11-27 10:55 ` Andi Kleen
2007-11-27 10:55   ` Andi Kleen
2007-11-27 11:19   ` Eric W. Biederman
2007-11-27 11:19     ` Eric W. Biederman
2007-11-27 13:28   ` Neil Horman
2007-11-27 13:28     ` Neil Horman

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=474C832C.1060903@redhat.com \
    --to=woodard@redhat.com \
    --cc=ak@suse.de \
    --cc=ebiederm@xmission.com \
    --cc=hbabu@us.ibm.com \
    --cc=kexec@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nhorman@redhat.com \
    --cc=nhorman@tuxdriver.com \
    --cc=vgoyal@in.ibm.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.