On Wed, Sep 4, 2013 at 9:59 PM, Maarten Lankhorst <maarten.lankho...@canonical.com> wrote: > Op 04-09-13 05:34, Ben Skeggs schreef: >> On Tue, Sep 3, 2013 at 12:31 AM, Maarten Lankhorst >> <maarten.lankho...@canonical.com> wrote: >>> There are a lot of places that allocate multiples of 1000, >>> but do not set alignment correctly and still require this >>> alignment implicitly or explicitly. >> This is wrong. Where are the places you think you need this? >> > All the calls to nouveau_gpuobj_map_vm for example in > core/engine/graph/nvc0.c. > > engctx_create is usually called with alignment = 0x100 too, which seems like > it would break in the fifo_context_attach calls too. All the places I can see with a "too small" alignment are not suballocated, so the backend will force alignment to minimum GPU page size anyway.
If someone wants to go through all these calls and fix them up explicitly too, then, patches welcome. The posted patch, however, isn't the right way to go about this, and I couldn't find an example yet that'd cause an actual breakage. Ben. > > ~Maarten _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
