From: Suresh Siddha <suresh.b.siddha@intel.com>
To: mingo@elte.hu, hpa@zytor.com, tglx@linutronix.de, arjan@linux.intel.com
Cc: linux-kernel@vger.kernel.org
Subject: [patch 0/7] x86, cpa: cpa related changes to be inline with TLB Application note
Date: Thu, 11 Sep 2008 13:30:48 -0700 [thread overview]
Message-ID: <20080911203048.929515000@linux-os.sc.intel.com> (raw)
TLB Application note[1] says:
"The TLBs may contain both ordinary and large-page translations for a 4-KByte
range of linear addresses. This may occur if software modifies the paging
structures so that the page size used for the address range changes. If the
two translations differ with respect to page frame or attributes (e.g.,
permissions), processor behavior is undefined and may be implementation
specific. The processor may use a page frame or attributes that correspond to
neither translation; it may improperly set or fail to set the dirty bit in the
appropriate paging-structure entry.
Such undefined behavior is problematic because prefetches and memory accesses
that are a result of speculative execution may occur, using the affected range
of linear addresses. It is also problematic if software (including the software
modifying the paging structures) is accessing data or executing code in the
affected range of linear addresses. Software should not write to a
paging-structure entry in a way that would change, for any linear address,
both the page size and either the page frame or attributes."
Currently we violate this at:
a. kernel identity mapping, where large/small pages setup very early in the
boot will be split up/merged into large pages along with attribute changes
during the direct memory mapping init.
b. while doing cpa(), potentially we will split large page and change attribute
both at the same time.
Following patches fixes this behavior.
[1] http://developer.intel.com/design/processor/applnots/317080.pdf
Signed-off-by: Suresh Siddha <suresh.b.siddha@intel.com>
--
next reply other threads:[~2008-09-11 21:19 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-09-11 20:30 Suresh Siddha [this message]
2008-09-11 20:30 ` [patch 1/7] x86, cpa: rename PTE attribute macros for kernel direct mapping in early boot Suresh Siddha
2008-09-11 20:30 ` [patch 2/7] x86, cpa: remove USER permission from the very early identity mapping attribute Suresh Siddha
2008-09-11 20:30 ` [patch 3/7] x86, cpa: make the kernel physical mapping initialization a two pass sequence Suresh Siddha
2008-09-11 20:30 ` [patch 4/7] x86, cpa: dont use large pages for kernel identity mapping with DEBUG_PAGEALLOC Suresh Siddha
2008-09-11 20:30 ` [patch 5/7] x86, cpa: fix taking the pgd_lock with interrupts off Suresh Siddha
2008-09-11 20:30 ` [patch 6/7] x86, cpa: remove cpa pool code Suresh Siddha
2008-09-11 20:30 ` [patch 7/7] x86, cpa: global flush tlb after splitting large page and before doing cpa Suresh Siddha
2008-09-13 17:10 ` Jeremy Fitzhardinge
2008-09-14 15:29 ` [patch 0/7] x86, cpa: cpa related changes to be inline with TLB Application note Ingo Molnar
2008-09-14 16:59 ` Ingo Molnar
2008-09-16 5:44 ` Suresh Siddha
2008-09-14 15:40 ` Arjan van de Ven
2008-09-14 15:52 ` 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=20080911203048.929515000@linux-os.sc.intel.com \
--to=suresh.b.siddha@intel.com \
--cc=arjan@linux.intel.com \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=tglx@linutronix.de \
/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.