On Thu, Sep 29, 2016 at 8:10 AM, Christian König
<[email protected]> 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 <[email protected]>

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

Reply via email to