Re: [PATCH 4/7] drm/radeon: Fill out gem_object->resv

2019-07-25 Thread Daniel Vetter
On Tue, Jun 25, 2019 at 10:52:17PM +0200, Sam Ravnborg wrote: > On Tue, Jun 25, 2019 at 10:42:05PM +0200, Daniel Vetter wrote: > > That way we can ditch our gem_prime_res_obj implementation. Since ttm > > absolutely needs the right reservation object all the boilerplate is > > already there and we

Re: [PATCH 4/7] drm/radeon: Fill out gem_object->resv

2019-07-19 Thread Daniel Vetter
On Thu, Jun 27, 2019 at 09:33:50AM +0200, Daniel Vetter wrote: > On Wed, Jun 26, 2019 at 10:23:12AM +0200, Daniel Vetter wrote: > > On Wed, Jun 26, 2019 at 07:10:21AM +, Koenig, Christian wrote: > > > Those patches would become superfluous when merging Gerd's work. > > > > Not entirely, they

Re: [PATCH 4/7] drm/radeon: Fill out gem_object->resv

2019-06-27 Thread Daniel Vetter
On Wed, Jun 26, 2019 at 10:23:12AM +0200, Daniel Vetter wrote: > On Wed, Jun 26, 2019 at 07:10:21AM +, Koenig, Christian wrote: > > Those patches would become superfluous when merging Gerd's work. > > Not entirely, they still remove the gem_prime_res_obj. Setting up > gem_bo.resv is only one

Re: [PATCH 4/7] drm/radeon: Fill out gem_object->resv

2019-06-26 Thread Daniel Vetter
On Wed, Jun 26, 2019 at 07:10:21AM +, Koenig, Christian wrote: > Those patches would become superfluous when merging Gerd's work. Not entirely, they still remove the gem_prime_res_obj. Setting up gem_bo.resv is only one half of what these do here. And yeah I think that single addition can be

Re: [PATCH 4/7] drm/radeon: Fill out gem_object->resv

2019-06-26 Thread Koenig, Christian
Those patches would become superfluous when merging Gerd's work. But I'm not sure if that is going to fly soon or not. Christian. Am 25.06.19 um 22:42 schrieb Daniel Vetter: > That way we can ditch our gem_prime_res_obj implementation. Since ttm > absolutely needs the right reservation object

Re: [PATCH 4/7] drm/radeon: Fill out gem_object->resv

2019-06-25 Thread Sam Ravnborg
On Tue, Jun 25, 2019 at 10:42:05PM +0200, Daniel Vetter wrote: > That way we can ditch our gem_prime_res_obj implementation. Since ttm > absolutely needs the right reservation object all the boilerplate is > already there and we just have to wire it up correctly. > > Note that gem/prime doesn't

[PATCH 4/7] drm/radeon: Fill out gem_object->resv

2019-06-25 Thread Daniel Vetter
That way we can ditch our gem_prime_res_obj implementation. Since ttm absolutely needs the right reservation object all the boilerplate is already there and we just have to wire it up correctly. Note that gem/prime doesn't care when we do this, as long as we do it before the bo is registered and