All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yinghai Lu <yinghai@kernel.org>
To: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Cc: mingo@elte.hu, rdreier@cisco.com, linux-kernel@vger.kernel.org,
	Christoph Lameter <cl@linux-foundation.org>
Subject: Re: bisected crash due to "x86: Move swiotlb initialization before dma32_free_bootmem"
Date: Tue, 15 Dec 2009 03:25:41 -0800	[thread overview]
Message-ID: <4B277235.4010308@kernel.org> (raw)
In-Reply-To: <20091215201039V.fujita.tomonori@lab.ntt.co.jp>

FUJITA Tomonori wrote:
> On Tue, 15 Dec 2009 11:56:50 +0100
> Ingo Molnar <mingo@elte.hu> wrote:
> 
>> * FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> wrote:
>>
>>> There are a few people who hit this. How many people use a box with over 
>>> 256GB memory?
>>>
>>> And you can work around this with "dma32_size" kernel boot option.
>> Well, since the kernel has not crashed before this change there's really just 
>> two options as per upstream kernel regression policy: either we fix it or we 
>> revert it.
> 
> As I wrote, here is a patch that can be applied to cleanly to the git
> head:
> 
> http://www.kernel.org/pub/linux/kernel/people/tomo/misc/0001-x86-two-stage-swiotlb-initialization.patch
> 
> It fixes the problem. Yinghai, can you test it? It should work but
> it's good to confirm it.

i tested already, it works.

> 
> I simply wanted to say that it's not a bug that breaks lots of boxes
> or leads to something serious like data corruption (no need to say
> something like "revert it now!"). It's also worth investigating why it
> breaks, I think.

will look at it later

YH


  reply	other threads:[~2009-12-15 11:27 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-15  7:47 bisected crash due to "x86: Move swiotlb initialization before dma32_free_bootmem" Roland Dreier
2009-12-15  8:55 ` FUJITA Tomonori
2009-12-15  9:04   ` Yinghai Lu
2009-12-15  9:23     ` FUJITA Tomonori
2009-12-15  9:11   ` Yinghai Lu
2009-12-15  9:23     ` FUJITA Tomonori
2009-12-15 10:56       ` Ingo Molnar
2009-12-15 11:10         ` FUJITA Tomonori
2009-12-15 11:25           ` Yinghai Lu [this message]
2009-12-15 11:43             ` Ingo Molnar
2009-12-15 11:50             ` FUJITA Tomonori
2009-12-15 13:04             ` 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=4B277235.4010308@kernel.org \
    --to=yinghai@kernel.org \
    --cc=cl@linux-foundation.org \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=rdreier@cisco.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.