915098 - round-robin texture units during composition

Status: RESOLVED WONTFIX

(Core :: Graphics: Layers, defect)

Description

[Andreas Gal :gal]

We currently only use texture unit 0. Our chipset friends are reporting that their driver doesn't do texture unit renaming internally, so we only utilize a fraction of the available fill rate.

Comment 1

[Andreas Gal :gal]

Attachment: patch

Tested on mac only. This round-robins the texture units and should spread out the use of texture units better during composition. I will ask the silicon partner to test this.

Comment 2

[Andreas Gal :gal]

Can you please test this patch and see whether this improves contention over texture units?

Comment 3

[Milan Sreckovic [:milan] (needinfo for best results)]

(In reply to Andreas Gal :gal from comment #2)
> Can you please test this patch and see whether this improves contention over
> texture units?

This was aimed at CodeAurora?

Comment 4

[Andreas Gal :gal]

Yes. The vendor will test this. We are all a little doubtful we could possibly need this.

Comment 5

[Mandyam Vikram]

On a build that includes the patch: Comment 87  Ed Morley [:edmorley UTC+1] 2013-09-13 07:04:29 PDT, there is no tangible performance gain (which is evident while profiling it with and without this patch).

Comment 6

[Andreas Gal :gal]

Looks like a WONTFIX to me then.