Am 16.03.2017 um 10:14 schrieb Zhang, Jerry:
Hi Christian,

Thanks for your info.
Sorry to miss them...

Actually this a serial of messed function patch, I should send them separately 
next time.

If we support PRT with bo=NULL, we may need a new VA op.
So please take a look at patch 4:
    * [PATCH 4/4] amdgpu: add a new bo va function to support NULL bo

I also have a question about current VA op func, why we didn't use the input 
flags?
I have no idea.

So I prepare the patch 2 to use the input flags:
     * [PATCH 2/4] amdgpu: set va flag with input parameter

Nicolai handled both issues by creating a new function where the BO is optional and the flags taken into account.

Please take a look at his patches, they are on the mailing list.

Christian.



Regards,
Jerry (Junwei Zhang)

Linux Base Graphics
SRDC Software Development
_____________________________________


-----Original Message-----
From: Christian König [mailto:[email protected]]
Sent: Thursday, March 16, 2017 17:00
To: Zhang, Jerry; [email protected]
Subject: Re: [PATCH 0/4] [libdrm] support PRT and replace interface

Nicolai already proposed patches for PRT support in libdrm (except the
CLEAR/REPLACE addition).

They look a bit cleaner and where posted first, I think we should use them
instead.

Christian.

Am 16.03.2017 um 04:56 schrieb Junwei Zhang:
* add PRT flag and new VA op
* fix flag setting for current VA op function
      * Why don't we set flag for current VA op function?
* [RFC] implement new VA fun to support NULL bo

Junwei Zhang (4):
    amdgpu: add new VA operations CLEAR and REPLACE
    amdgpu: set va flag with input parameter
    amdgpu: add PRT page attribute
    amdgpu: add a new bo va function to support NULL bo

   amdgpu/amdgpu.h          |  7 +++++++
   amdgpu/amdgpu_bo.c       | 42
++++++++++++++++++++++++++++++++++++++++--
   include/drm/amdgpu_drm.h |  4 ++++
   3 files changed, 51 insertions(+), 2 deletions(-)


_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to