From: "Siddha, Suresh B" <suresh.b.siddha@intel.com>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: "Siddha, Suresh B" <suresh.b.siddha@intel.com>,
Andi Kleen <ak@suse.de>, Pavel Machek <pavel@ucw.cz>,
Andrew Morton <akpm@osdl.org>,
LKML <linux-kernel@vger.kernel.org>,
discuss@x86-64.org
Subject: Re: [PATCH][Fix][Resend] Fix Bug #4959: Page tables corrupted during resume on x86-64 (take 3)
Date: Thu, 29 Sep 2005 16:59:06 -0700 [thread overview]
Message-ID: <20050929165905.B15943@unix-os.sc.intel.com> (raw)
In-Reply-To: <200509300104.36454.rjw@sisk.pl>; from rjw@sisk.pl on Fri, Sep 30, 2005 at 01:04:35AM +0200
On Fri, Sep 30, 2005 at 01:04:35AM +0200, Rafael J. Wysocki wrote:
> On Friday, 30 of September 2005 00:29, Siddha, Suresh B wrote:
> > Did you try just only my patch on top of 2.6.14-rc2? You can get that
> > patch from http://www.x86-64.org/lists/discuss/msg07313.html
>
> The patch that I tested is attached. I think it's the same one. I've just applied
> it on top of 2.6.14-rc2-git7, and it doesn't boot.
It works fine for me. Only thing I see though is a warning for UP configuration.
Other than that UP, SMP(with and without hotplug) kernels boot fine. I will
send the warning fix to Andrew.
> The problem (as I see it) is this:
> In x86_64_start_kernel() you copy boot_level4_pgt[] into init_level4_pgt[],
> and you make the latter your current PGD by loading cr3 with its address.
> Fine. With this PGD you call start_kernel() which calls setup_arch(), which
> calls zap_low_mappings(0) that fills init_level4_pgt[] (which at this moment
> is still your current PGD) with zeros ...
It clears only the zeroth entry. Not the whole pgd.
Please send me your .config so that I can try reproducing the issue locally
here.
thanks,
suresh
next prev parent reply other threads:[~2005-09-29 23:59 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-09-28 14:24 [PATCH][Fix][Resend] Fix Bug #4959: Page tables corrupted during resume on x86-64 (take 3) Rafael J. Wysocki
2005-09-28 19:18 ` Andi Kleen
2005-09-28 20:24 ` Rafael J. Wysocki
2005-09-28 20:33 ` [discuss] " Andi Kleen
2005-09-28 22:11 ` Rafael J. Wysocki
2005-09-28 22:35 ` Pavel Machek
2005-09-29 11:25 ` Rafael J. Wysocki
2005-09-29 0:00 ` Siddha, Suresh B
2005-09-29 10:58 ` Rafael J. Wysocki
2005-09-29 22:01 ` Rafael J. Wysocki
2005-09-29 22:29 ` Siddha, Suresh B
2005-09-29 23:04 ` Rafael J. Wysocki
2005-09-29 23:59 ` Siddha, Suresh B [this message]
2005-09-30 5:26 ` Rafael J. Wysocki
2005-09-30 6:51 ` [discuss] " Rafael J. Wysocki
2005-10-01 1:25 ` Siddha, Suresh B
2005-10-01 7:47 ` Rafael J. Wysocki
2005-10-01 10:03 ` Rafael J. Wysocki
2005-10-02 1:08 ` Siddha, Suresh B
2005-10-02 9:54 ` Rafael J. Wysocki
2005-09-29 20:02 ` Andrew Morton
2005-09-29 21:35 ` Rafael J. Wysocki
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=20050929165905.B15943@unix-os.sc.intel.com \
--to=suresh.b.siddha@intel.com \
--cc=ak@suse.de \
--cc=akpm@osdl.org \
--cc=discuss@x86-64.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pavel@ucw.cz \
--cc=rjw@sisk.pl \
/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.