mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@zytor.com>
To: "Rafał Miłecki" <zajec5@gmail.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Ingo Molnar <mingo@redhat.com>,
	x86@kernel.org
Subject: Re: Bisected 3.9 regression: Corrupted low memory (x86, mm: Move reserving low memory later in initialization)
Date: Wed, 21 May 2014 11:03:59 -0700	[thread overview]
Message-ID: <c969f4d3-8b82-456d-b126-77a3b66a88bd@email.android.com> (raw)
In-Reply-To: <CACna6ryC8jjpJwzuHMbwuskrCgr7STMN8Bs3a7HN5h-rBZ68Rg@mail.gmail.com>

Yes, I'm sure.  The old memory reservation code kept the checker from working.

On May 21, 2014 10:58:39 AM PDT, "Rafał Miłecki" <zajec5@gmail.com> wrote:
>On 21 May 2014 19:34, H. Peter Anvin <hpa@zytor.com> wrote:
>> On 05/21/2014 06:31 AM, Rafał Miłecki wrote:
>>> On 16 May 2014 07:29, Rafał Miłecki <zajec5@gmail.com> wrote:
>>>> Sorry for the late report, this bug appears on my old notebook I
>don't
>>>> commonly use. Anyway, I've noticed following problem when using
>>>> 3.15-rc1:
>>>>
>>>> Corrupted low memory at ffff88000000be98 (be98 phys) =
>b02a000400000000
>>>
>>> Ping?
>>>
>>
>> It's not a regression, but rather a progression: the checker now does
>> what it claims to do, which is to check for low memory corruption.
>>
>> However, the checker is also rather useless as anything other than a
>> diagnostic tool.  It tells you that your BIOS is corrupting memory,
>and
>> the solution to that is to reserve the memory, which we already do.
>>
>> All the checker tells us is that on your system, yes, we really do
>need
>> to reserve the memory.
>
>What about making checker a bit more user friendly?
>
>First of all, I got many repeated messages like:
>Corrupted low memory at (...)
>In the first e-mail you can see ~10 of them and I really got more.
>
>Secondly, if this is just a confirmation that low memory reservation
>was indeed needed, what about making it debugging only? Right now it
>sounds like an error for end-users. What about making it something
>like
>Low memory reservation prevented corruption at (...)
>
>The last question, just for sure... are you convinced what I'm seeing
>is not any kind of error? After all, it started appearing after commit
>that moves memory reservation. It's not a commit that enabled the
>checker or something like that.
>What it seems to be for someone not experienced is a bit opposite. It
>looks that after commit in question, kernel reserves low memory too
>late and BIOS already corrupts some data placed in it.
>However I'm not on expert, so I'll just listen to your opinion. I
>simply wanted to ask, just to be sure everything is alright.

-- 
Sent from my mobile phone.  Please pardon brevity and lack of formatting.

      reply	other threads:[~2014-05-21 18:04 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-16  5:29 Rafał Miłecki
2014-05-21 13:31 ` Rafał Miłecki
2014-05-21 17:34   ` H. Peter Anvin
2014-05-21 17:58     ` Rafał Miłecki
2014-05-21 18:03       ` H. Peter Anvin [this message]

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=c969f4d3-8b82-456d-b126-77a3b66a88bd@email.android.com \
    --to=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=tglx@linutronix.de \
    --cc=x86@kernel.org \
    --cc=zajec5@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®