[PATCH 04/36] dix: introduce gpu screens. (v3)
Adam Jackson
ajax at nwnk.net
Mon Jul 2 13:07:04 PDT 2012
On 7/2/12 6:12 AM, Dave Airlie wrote:
> +void
> +RemoveGPUScreen(ScreenPtr pScreen)
> +{
> + int idx, j;
> + if (!pScreen->isGPU)
> + return;
> +
> + idx = pScreen->myNum - GPU_SCREEN_OFFSET;
> + for (j = idx; j < screenInfo.numGPUScreens - 1; j++) {
> + screenInfo.gpuscreens[j] = screenInfo.gpuscreens[j + 1];
> + }
> + screenInfo.numGPUScreens--;
> +
> + free(pScreen);
> +
> +}
I don't see this getting called from the xf86 DDX, at least not in 6/36.
Even if it were, it's not symmetric with how xf86's "remove" path works.
There you change ->scrnIndex to match the current slot in the array,
here you're not doing that with ->myNum. That seems like an error.
- ajax
More information about the xorg-devel
mailing list