mirror of
https://github.com/xen-project/xen.git
synced 2026-10-28 02:01:24 +01:00
x86/HVM: purge dubious lastpage diagnostic
Quoting b5d8b03db1 ("x86/shadow: Drop dubious lastpage diagnostic"):
"This is a global variable (actually 3, one per GUEST_PAGING_LEVEL), operated
on using atomics only (with no regard to what else shares the same cacheline),
which emits a diagnostic (in debug builds only) without changing any program
behaviour.
It is presumably left-over debugging, as it interlinks the behaviour of all
vCPUs in chronological order. Based on the read-only p2m types, this
diagnostic can be tripped by entirely legitimate guest behaviour."
The same applies here (it's only a single variable of course).
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
This commit is contained in:
1 parent
068314888c
commit
31270f11a9
1 file changed
+1
-10
+1
-10
@@ -3354,16 +3354,7 @@ static enum hvm_translation_result __hvm_copy(
|
||||
memcpy(buf, p, count);
|
||||
hvmemul_write_cache(v, gfn_to_gaddr(gfn) | pgoff, buf, count);
|
||||
}
|
||||
else if ( p2m_is_discard_write(p2mt) )
|
||||
{
|
||||
static unsigned long lastpage;
|
||||
|
||||
if ( xchg(&lastpage, gfn_x(gfn)) != gfn_x(gfn) )
|
||||
dprintk(XENLOG_G_DEBUG,
|
||||
"%pv attempted write to read-only gfn %#lx (mfn=%#"PRI_mfn")\n",
|
||||
v, gfn_x(gfn), mfn_x(page_to_mfn(page)));
|
||||
}
|
||||
else
|
||||
else if ( !p2m_is_discard_write(p2mt) )
|
||||
{
|
||||
if ( buf )
|
||||
memcpy(p, buf, count);
|
||||
|
||||