[PATCH v2 00/18] drm/ttm: make ttm bo a gem bo subclass

Daniel Vetter daniel at ffwll.ch
Thu Jun 27 15:57:22 UTC 2019


On Thu, Jun 27, 2019 at 02:22:18PM +0200, Gerd Hoffmann wrote:
>   Hi,
> 
> > Imo get maybe another ttm+gem stakeholder to review this (Thomas for vram
> > helpers or Ben for nouveau) and then this can land. I think Thomas
> > Hellstrom tuned down his categorical "nak" to "we'll see where this goes,
> > I might need to jump in and help course-correct".
> 
> Yep, seems he also is on summer vacation now ...
> 
> > btw I also resubmitted my prime resv_obj cleanup series:
> > 
> > https://patchwork.freedesktop.org/series/62735/
> 
> What is the merge plan for this?  Looks like this is pretty much ready
> to go.

Needs an r-b from amd and maybe Ben, then I'll merge that I think.

> > I think the only conflicts with your series here is that the setting of
> > gem_bo.resv that I add in radeon/amdgpu/nouveau can be removed again when
> > your stuff lands.
> 
> Yep, doesn't look too bad.  I can rebase on top of your series, I plan
> to send v3 anyway, I have some smaller (comment) updates pending.

It's just 3 lines we need to remove in the end (the ones that set
gem_bo.resv), and those 3 lines are harmless I think. I think we can just
merge whenever and then do the follow up once both series have landed.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch


More information about the dri-devel mailing list