Dave Airlie <[email protected]> writes: > > if (pScrPriv->provider) > total_providers++; > + xorg_list_for_each_entry(iter, &pScreen->unattached_list, > unattached_head) { > + pScrPriv = rrGetScrPriv(iter); > + total_providers += pScrPriv->provider ? 1 : 0; > + } > > pScrPriv = rrGetScrPriv(pScreen);
Probably should use a different variable to hold the GPU screen private pointer? That way you wouldn't have to re-fetch after this loop. Otherwise, this change looks fine to me. Reviewed-by: Keith Packard <[email protected]> -- [email protected]
pgpKRzhF1Swhj.pgp
Description: PGP signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
