Am 22.01.2018 um 13:56 schrieb Tom St Denis: > Previously if a PTE was hit with V=0 the decoder would stop. > Now it continues but only if you're doing a --vm-decode. > > Signed-off-by: Tom St Denis <tom.stdenis at amd.com> > Reported-by: Christian König <christian.koenig at amd.com> Acked-by: Christian König <christian.koenig at amd.com> > --- > src/lib/read_vram.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/src/lib/read_vram.c b/src/lib/read_vram.c > index 32b024529b12..491a55f69d49 100644 > --- a/src/lib/read_vram.c > +++ b/src/lib/read_vram.c > @@ -241,7 +241,7 @@ static int umr_access_vram_vi(struct umr_asic *asic, uint32_t vmid, > (int)pte_fields.valid, > (int)pte_fields.system); > > - if (!pte_fields.valid) > + if (pdst && !pte_fields.valid) > return -1; > > // compute starting address > @@ -266,7 +266,7 @@ static int umr_access_vram_vi(struct umr_asic *asic, uint32_t vmid, > (int)pte_fields.valid, > (int)pte_fields.system); > > - if (!pte_fields.valid) > + if (pdst && !pte_fields.valid) > return -1; > > // compute starting address > @@ -575,7 +575,7 @@ pde_is_pte: > if (!pte_fields.system) > pte_fields.page_base_addr -= vm_fb_offset; > > - if (!pte_fields.prt && !pte_fields.valid) > + if (pdst && !pte_fields.prt && !pte_fields.valid) > return -1; > > // compute starting address > @@ -625,7 +625,7 @@ pde_is_pte: > (int)pte_fields.valid, > (int)pte_fields.system); > > - if (!pte_fields.valid) > + if (pdst && !pte_fields.valid) > return -1; > > // compute starting address
