x86: Stop setting the physical address in finalizePhysical.
authorGabe Black <gabeblack@google.com>
Fri, 15 May 2020 23:29:20 +0000 (16:29 -0700)
committerGabe Black <gabeblack@google.com>
Sat, 16 May 2020 03:13:49 +0000 (03:13 +0000)
commit4c7e6ce894d5979a7c0a21f7fe7ad752546fefcb
tree2bd411899e62fde0cefd5294709eb79300276ce0
parentde944f8a832ebdd55b03b42f72a329abb53ef98e
x86: Stop setting the physical address in finalizePhysical.

The physical address has already been set (it's read earlier in the
function), and so doesn't need to be set again. Reading the virtual
address can cause an assert if the virtual address had never been set in
the first place, for example when an access comes from KVM which might
give you an access to complete which is based on a physical address
only.

Change-Id: Ic46a40b1a94235538b5bd53065e5019273b3d3f3
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/29172
Maintainer: Jason Lowe-Power <power.jg@gmail.com>
Reviewed-by: Pouya Fotouhi <pfotouhi@ucdavis.edu>
Tested-by: kokoro <noreply+kokoro@google.com>
src/arch/x86/tlb.cc