[PATCH 2/7] drm/amdgpu: update the shadow PD together with the real one

Alex Deucher alexdeucher at gmail.com
Thu Sep 29 16:51:14 UTC 2016


On Thu, Sep 29, 2016 at 8:10 AM, Christian König
<deathsimple at vodafone.de> wrote:
> Am 29.09.2016 um 11:18 schrieb zhoucm1:
>>
>>
>>
>> On 2016年09月29日 15:52, Christian König wrote:
>>>
>>> -    if (count)
>>> +    if (count) {
>>> +        if (vm->page_directory-shadow)
>>
>> Forget compile fix to amend?
>
>
> Ups, no that actually compiles because we have a local variable named
> "shadow" (and by coincident actually does what it should do).
>
> Anyway clearly a typo. Can I get your rb for the series?

With that fixed, this patch is:
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

>
> Regards,
> Christian.
>
>
>>
>> Regards,
>> David Zhou
>>>
>>> + amdgpu_vm_do_set_ptes(&params, last_shadow, last_pt,
>>> +                          count, incr, AMDGPU_PTE_VALID);
>>
>>
>> _______________________________________________
>> amd-gfx mailing list
>> amd-gfx at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>
>
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


More information about the amd-gfx mailing list