Messages in this thread | | | Date | Wed, 21 Aug 2019 13:55:23 +0200 | From | Thierry Reding <> | Subject | Re: [Nouveau] [Intel-gfx] [PATCH v6 08/17] drm/ttm: use gem vma_node |
| |
On Wed, Aug 21, 2019 at 04:33:58PM +1000, Ben Skeggs wrote: > On Wed, 14 Aug 2019 at 20:14, Gerd Hoffmann <kraxel@redhat.com> wrote: > > > > Hi, > > > > > > Changing the order doesn't look hard. Patch attached (untested, have no > > > > test hardware). But maybe I missed some detail ... > > > > > > I came up with something very similar by splitting up nouveau_bo_new() > > > into allocation and initialization steps, so that when necessary the GEM > > > object can be initialized in between. I think that's slightly more > > > flexible and easier to understand than a boolean flag. > > > > Yes, that should work too. > > > > Acked-by: Gerd Hoffmann <kraxel@redhat.com> > Acked-by: Ben Skeggs <bskeggs@redhat.com>
Thanks guys, applied to drm-misc-next.
Thierry [unhandled content-type:application/pgp-signature] | |