From: Ingo Molnar <mingo@elte.hu>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: linux-kernel@vger.kernel.org,
Andrew Morton <akpm@linux-foundation.org>,
Thomas Gleixner <tglx@linutronix.de>,
"H. Peter Anvin" <hpa@zytor.com>
Subject: Re: [git pull] x86 updates for v2.6.28, phase #2 - PAT updates
Date: Fri, 10 Oct 2008 19:04:26 +0200 [thread overview]
Message-ID: <20081010170426.GA5947@elte.hu> (raw)
In-Reply-To: <20081010164537.GA23664@elte.hu>
* Ingo Molnar <mingo@elte.hu> wrote:
> v2 worked well in testing:
>
> http://www.uwsg.iu.edu/hypermail/linux/kernel/0809.1/2375.html
>
> So it's basically a v1 -> v2 sequence, with artificial reverts
> surviving unintentionally. That's why there were no revert messages
> either: i never intended them to become public. I even rember having
> taken a good look at "git diff 6b5b551..6e3e492", which the v1->v2
> delta was.
here's the linearized history of the commits (older commits go last):
6e3e492: x86: track memtype for RAM in page struct
09543b6d: x86, cpa: srlz cpa(), global flush tlb after splitting big page and before doing cpa
eaaa6c4: x86, cpa: remove cpa pool code
ef91bf0: x86, cpa: no need to check alias for __set_pages_p/__set_pages_np
8e1701f: x86, cpa: dont use large pages for kernel identity mapping with DEBUG_PAGEALLOC
065ae73: x86, cpa: make the kernel physical mapping initialization a two pass sequence
eb60b73: x86, cpa: remove USER permission from the very early identity mapping attribute
fabf99f: x86, cpa: rename PTE attribute macros for kernel direct mapping in early boot
abfae7f6: Revert "x86, cpa: rename PTE attribute macros for kernel direct mapping in early boot"
f279399: Revert "x86, cpa: remove USER permission from the very early identity mapping attribute"
153eb23: Revert "x86, cpa: make the kernel physical mapping initialization a two pass sequence"
f5c3489: Revert "x86, cpa: dont use large pages for kernel identity mapping with DEBUG_PAGEALLOC"
c613e65: Revert "x86, cpa: fix taking the pgd_lock with interrupts off"
e64b1f1: Revert "x86, cpa: remove cpa pool code"
94448e3: Revert "x86, cpa: global flush tlb after splitting large page and before doing cpa"
aca9b5f: Revert "x86: track memtype for RAM in page struct"
6375b67: Revert "x86: handle error returns in set_memory_*()"
6b5b551: x86: handle error returns in set_memory_*()
5f25f5b: x86: track memtype for RAM in page struct
3196625: x86, cpa: global flush tlb after splitting large page and before doing cpa
79abc89: x86, cpa: remove cpa pool code
e96d59b: x86, cpa: fix taking the pgd_lock with interrupts off
888fdd9: x86, cpa: dont use large pages for kernel identity mapping with DEBUG_PAGEALLOC
e579af6: x86, cpa: make the kernel physical mapping initialization a two pass sequence
c86eefc: x86, cpa: remove USER permission from the very early identity mapping attribute
e8729a5: x86, cpa: rename PTE attribute macros for kernel direct mapping in early boot
f81b691: Merge commit 'v2.6.27-rc6' into x86/pat
e8729a5..6b5b551 was v1, fabf99f..6e3e492 is v2. The reverts in
6375b67..abfae7f6 are a mirror image of v1 and were supposed to be
squashed together into a delta commit.
the time gap between v1 and v2 was rather large, 10 days:
CommitDate: Sun Sep 14 17:50:41 2008 +0200
CommitDate: Wed Sep 24 10:07:52 2008 +0200
because it took time to find and track down the lockup, and it also took
time to come up with the fix. So i definitely wanted a delta fix (we
handle another 150 append-only commits in a 10 day timespan so a delta
fix is the right approach), but messed up its generation.
Ingo
next prev parent reply other threads:[~2008-10-10 17:04 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-09 23:49 [git pull] x86 updates for v2.6.28, phase #2 - PAT updates Ingo Molnar
2008-10-10 16:27 ` Linus Torvalds
2008-10-10 16:45 ` Ingo Molnar
2008-10-10 16:56 ` Ingo Molnar
2008-10-10 17:23 ` Linus Torvalds
2008-10-10 17:04 ` Ingo Molnar [this message]
2008-10-10 17:34 ` Linus Torvalds
2008-10-10 17:41 ` [git pull, take 2] " Ingo Molnar
2008-10-10 18:15 ` Linus Torvalds
2008-10-10 18:33 ` Ingo Molnar
2008-10-10 18:17 ` Ingo Molnar
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=20081010170426.GA5947@elte.hu \
--to=mingo@elte.hu \
--cc=akpm@linux-foundation.org \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.org \
/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.