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? So I prepare the patch 2 to use the input flags: * [PATCH 2/4] amdgpu: set va flag with input parameter Regards, Jerry (Junwei Zhang) Linux Base Graphics SRDC Software Development _____________________________________ > -----Original Message----- > From: Christian König [mailto:deathsimple at vodafone.de] > Sent: Thursday, March 16, 2017 17:00 > To: Zhang, Jerry; amd-gfx at lists.freedesktop.org > 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(-) > >