bevy icon indicating copy to clipboard operation
bevy copied to clipboard

disable gpu preprocessing on android with Adreno 730 GPU and earilier

Open Litttlefish opened this issue 1 year ago • 7 comments

Objective

Fix #14146

Solution

Expansion of #13323 , excluded Adreno 730 and earlier.

Testing

Tested on android device(Adreno 730) that used to crash

Litttlefish avatar Jul 06 '24 09:07 Litttlefish

This limits too much because 740 is proven to work https://github.com/bevyengine/bevy/pull/13323#issuecomment-2105665001, and 750 works for me.

eero-lehtinen avatar Jul 06 '24 10:07 eero-lehtinen

This limits too much because 740 is proven to work #13323 (comment), and 750 works for me.

I've changed it to only disble on 6xx and 730 now

Litttlefish avatar Jul 06 '24 10:07 Litttlefish

There are also 702, 710, 720, 725 that I think are safe to assume don't work either because they are older. 732 and 735 probably don't either. https://en.wikipedia.org/wiki/Adreno#Adreno_700_series

eero-lehtinen avatar Jul 06 '24 10:07 eero-lehtinen

So I would check that the second number is 3 or less.

eero-lehtinen avatar Jul 06 '24 10:07 eero-lehtinen

So I would check that the second number is 3 or less.

My assumption is GPU earlier than 730(probably, since there Adreno 740 is used for 8Gen2 series) doesn't support it, so I used this for now

But Adreno number naming is not that simple, like 720 is used in 7Gen3, which probably should support it(newer), while 725 used in 7Gen2 is the same to 730 with lower frequency🤔needs more testing to confirm

Litttlefish avatar Jul 06 '24 11:07 Litttlefish

Yeah looks like 732 and 735 are very new as well

eero-lehtinen avatar Jul 06 '24 11:07 eero-lehtinen

Yeah looks like 732 and 735 are very new as well

Anyways I excluded 720, since it probably supports gpu preprocessing, I'll let it for others to test

Litttlefish avatar Jul 06 '24 11:07 Litttlefish