public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Myron Stowe <myron.stowe@hp.com>
To: Yinghai Lu <yinghai@kernel.org>
Cc: Nikanth Karthikesan <knikanth@suse.de>,
	Eric Dumazet <eric.dumazet@gmail.com>,
	Bjorn Helgaas <bjorn.helgaas@hp.com>, Ingo Molnar <mingo@elte.hu>,
	Peter Zijlstra <peterz@infradead.org>,
	Venkatesh Pallipadi <venki@google.com>,
	Nikhil Rao <ncrao@google.com>,
	Takuya Yoshikawa <yoshikawa.takuya@oss.ntt.co.jp>,
	linux-kernel@vger.kernel.org, rjenties@google.com
Subject: Re: divide error in select_task_rq_fair()
Date: Thu, 18 Nov 2010 16:32:41 -0700	[thread overview]
Message-ID: <1290123161.2447.27.camel@zim> (raw)
In-Reply-To: <AANLkTikUnLc8L_0+CQOwPgr-oifws=sqnFPxWz46MTCi@mail.gmail.com>

On Sun, 2010-11-14 at 11:11 -0800, Yinghai Lu wrote:
> On Sun, Nov 14, 2010 at 9:36 AM, Myron Stowe <myron.stowe@hp.com> wrote:
> >
> > I got the same divide error with this latest patch (see attachment).  If
> > I revert commit 50f2d7f682f9, the platform boots successfully.
> 
> please check patch in
> http://lkml.org/lkml/2010/11/13/181

I was able to test this patch and with it applied the system did boot
successfully.

While I think you are on the correct path with respect to this issue I
could not make any sense out of the patch heading and description.
Worse - I'm thinking that it is even mis-leading as currently written
(especially the patch heading).

Thanks,

Myron
> 
> BTW, you also need to ask your BIOS guys to fix the SRAT table.
> If you only have 128 cpu entries in MADT, SRAT table should have 128
> cpu entries instead of 256 cpu entries
> otherwise, RHEL 5.5 could have problem. it will throw away last cpu
> entry in SRAT,
>  (NR_CPUS is 255..., and last entry still could point the right cpu in MADT)
> Also BIOS should keep cpu entries in SRAT have same order to that in MADT.
> 
> Thanks
> 
> Yinghai
> 


-- 
Myron Stowe                             Linux Kernel Developer
Fort Collins, CO                        Office of Corporate Strategy and Technology


  reply	other threads:[~2010-11-18 23:32 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-04  4:12 divide error in select_task_rq_fair() Bjorn Helgaas
2010-11-04  5:19 ` Eric Dumazet
2010-11-04 14:28   ` Bjorn Helgaas
2010-11-04 14:37     ` Eric Dumazet
2010-11-05  2:00       ` Bjorn Helgaas
2010-11-05  6:17         ` Eric Dumazet
2010-11-11 18:28           ` Myron Stowe
2010-11-12  6:22             ` Nikanth Karthikesan
2010-11-12 14:06               ` Myron Stowe
2010-11-14 17:36               ` Myron Stowe
2010-11-14 19:11                 ` Yinghai Lu
2010-11-18 23:32                   ` Myron Stowe [this message]
2010-11-22  5:25                     ` [PATCH] x86, acpi: Parse all SRAT cpu entries even have cpu num limitation Yinghai Lu
2010-12-15 22:09                     ` divide error in select_task_rq_fair() Venkatesh Pallipadi
2010-11-14  1:15             ` Yinghai Lu

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=1290123161.2447.27.camel@zim \
    --to=myron.stowe@hp.com \
    --cc=bjorn.helgaas@hp.com \
    --cc=eric.dumazet@gmail.com \
    --cc=knikanth@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=ncrao@google.com \
    --cc=peterz@infradead.org \
    --cc=rjenties@google.com \
    --cc=venki@google.com \
    --cc=yinghai@kernel.org \
    --cc=yoshikawa.takuya@oss.ntt.co.jp \
    /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