On 3 April 2017 at 11:54, Marek Olšák <maraeo at gmail.com> wrote:
> On Mon, Apr 3, 2017 at 10:27 AM, Nicolai Hähnle <nhaehnle at gmail.com> wrote:
>> From: Junwei Zhang <Jerry.Zhang at amd.com>
>>
>> v2: fix indent
>>
>> Signed-off-by: Junwei Zhang <Jerry.Zhang at amd.com>
>> Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>
>> Reviewed-by: Christian König <christian.koenig at amd.com>
>> ---
>> amdgpu/amdgpu_bo.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/amdgpu/amdgpu_bo.c b/amdgpu/amdgpu_bo.c
>> index f725bfd..8bc1ebd 100644
>> --- a/amdgpu/amdgpu_bo.c
>> +++ b/amdgpu/amdgpu_bo.c
>> @@ -696,21 +696,23 @@ int amdgpu_bo_va_op_raw(amdgpu_device_handle dev,
>> amdgpu_bo_handle bo,
>> uint64_t offset,
>> uint64_t size,
>> uint64_t addr,
>> uint64_t flags,
>> uint32_t ops)
>> {
>> struct drm_amdgpu_gem_va va;
>> int r;
>>
>> - if (ops != AMDGPU_VA_OP_MAP && ops != AMDGPU_VA_OP_UNMAP)
>> + if (ops != AMDGPU_VA_OP_MAP && ops != AMDGPU_VA_OP_UNMAP &&
>> + ops != AMDGPU_VA_OP_REPLACE && ops != AMDGPU_VA_OP_CLEAR)
>> +
>
> Spurious empty line?
>
> Other than that, the series is:
> Reviewed-by: Marek Olšák <marek.olsak at amd.com>
>
> I don't know how Dave feels about pushing this into libdrm when it's
> not in Linus's tree.
>
I'm not Dave, so pardon for dropping in.
AFAICT the idea is that once the feature is within a non-rebase branch
such as drm-next, it's part of the ABI. As such one can use it across
the board - be that in here, Mesa or elsewhere.
That said, the series looks great. Thanks for the updates gents.
Acked-by: Emil Velikov <emil.velikov at collabora.com>
Thanks
Emil
