[PULL] build fix, GC clipping cleanup
Aaron Plattner
aplattner at nvidia.com
Tue Oct 4 12:37:59 PDT 2011
On 10/04/2011 09:33 AM, Keith Packard wrote:
> On Tue, 4 Oct 2011 18:19:54 +0200, Michal Suchanek<hramrach at centrum.cz> wrote:
>
>> Wasn't it just renamed from DestroyPixmap to ReleasePixmap?
>
> The function was renamed, but the screen structure member was not (nor
> should it have been -- it's now actually called when the pixmap is
> destroyed, not just when dereferenced).
>
> Oh, this patch sequence should *definitely* come with an ABI bump;
> there's no way you can change driver source code so it can build with
> either version...
With the old CreatePixmap API, drivers could override the width and
height when calling down to prevent the fb layer from allocating the
pixmap memory, and then plug in its own. It looks like a driver could
do the same thing now by changing the pixmap structure's width and
height to 0 temporarily, but it looks like that will cause the new code
to call malloc(0) and then fail if it returns NULL.
-- Aaron
More information about the xorg-devel
mailing list