[PATCH 04/36] dix: introduce gpu screens. (v3)

Dave Airlie airlied at gmail.com
Tue Jul 3 01:17:24 PDT 2012


On Mon, Jul 2, 2012 at 9:07 PM, Adam Jackson <ajax at nwnk.net> wrote:
> 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.

It doesn't get used until hotplug time.

>
> 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.

Oh good point, will revise to be saner.

Dave.


More information about the xorg-devel mailing list