758845 - Enable Skia on B2G

Status: RESOLVED DUPLICATE of bug 905227

(Core :: Graphics, defect)

Description

[Michael Wu [:mwu]]

Attachment: Enable Skia on B2G

B2G is close enough to Android that this works largely out of the box.

cjones, what do you think?

Comment 1

[Joe Drew (not getting mail)]

This will strictly make your binary bigger, fwiw. We don't use Skia for anything except plugins on Android right now.

Comment 2

[Michael Wu [:mwu]]

Well when bug 725119 is landed then.

Comment 3

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Comment on attachment 627421 [details] [diff] [review]
Enable Skia on B2G

s/Enable Skia/Enable building Skia/

Comment 4

[Andrew Overholt [:overholt]]

bjacob told me that he thinks this is already enabled.  George, can you confirm?

Comment 5

[Joe Drew (not getting mail)]

We shouldn't bother enabling Skia on B2G unless we have a reason. It's slower on many things on Android.

Comment 6

[Michael Wu [:mwu]]

(In reply to Andrew Overholt [:overholt] from comment #4)
> bjacob told me that he thinks this is already enabled.  George, can you
> confirm?

This isn't enabled AFAICT.

Comment 7

[George Wright (:gw280) (needinfo me!)]

It shouldn't be enabled, as far as I know.

Comment 8

[Peter Chang[:pchang]]

Attachment: add skia configuration for b2g on m-c

Added skia configuration on b2g but didn't turn on skia by default.

Comment 9

[Peter Chang[:pchang]]

Attachment: add skia configuration for b2g on m-c v2

Rebase to latest inbound

Comment 10

[Peter Chang[:pchang]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/bbd6b352f01c

Comment 11

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/bbd6b352f01c

Comment 12

[Michael Wu [:mwu]]

Peter, can you back out? We weren't planning to build skia until we actually start using it.

Comment 13

[Andreas Gal :gal]

How far away are we from turning on skia on mobile?

Comment 14

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Does it cause problems to build it even before we start using it?

Comment 15

[Michael Wu [:mwu]]

No, but we previously agreed not to turn it on until it would actually be used. No need to increase our codesize until we need it.

Comment 16

[Andreas Gal :gal]

I am still much more interested in when we will start using this.

Comment 17

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

I was going to suggest that if it's buildable and mostly works, that we turn it on even with known issues.

Comment 18

[Peter Chang[:pchang]]

(In reply to Michael Wu [:mwu] from comment #15)
> No, but we previously agreed not to turn it on until it would actually be
> used. No need to increase our codesize until we need it.

I just check the binary size when build with/without skia configuration.
It only increases about 460 KB and overall memory increase about 4 MB.

In my opinion, it is still acceptable.
And it could reduce the works for people who want to try skiaGL and they could easily change the preference to enable it.

For skiaGL on B2g, I could run the guimark2 and cut the rope with better performance.
For other benchmarks, we may have the CPU bottleneck on "flush" or memory not enough issues.

Later I will check any other testing items failed on B2G with SkiaGL enabled.

[libxul file size]
peter@pchang-desktop:~/b2g_debug_leo/objdir-b2g_central_in/dist$ ls -lk b2g/libxul.so
-rwxrwxr-x 1 peter peter 21789 Jul 26 10:42 b2g/libxul.so

peter@pchang-desktop:~/b2g_debug_leo/objdir-b2g_central_in/dist$ ls -lk b2g_noskia/libxul.so
-rwxrwxr-x 1 peter peter 21329 Jul 26 10:31 b2g_noskia/libxul.so


[b2g-info noskia]
            Total 384.9 MB
     Used - cache 107.0 MB
  B2G procs (PSS)  99.6 MB
    Non-B2G procs   7.4 MB
     Free + cache 277.9 MB
             Free 181.5 MB
            Cache  96.4 MB

[b2g-info skia]
            Total 384.9 MB
     Used - cache 109.4 MB
  B2G procs (PSS) 102.3 MB
    Non-B2G procs   7.1 MB
     Free + cache 275.5 MB
             Free 177.7 MB
            Cache  97.9 MB

Comment 19

[Andreas Gal :gal]

Ok, so 4MB memory is _a lot_. Thats 4% of the total available memory or so. That is definitely not ok. Why does skia take up so much memory disabled?

Comment 20

[Peter Chang[:pchang]]

(In reply to Andreas Gal :gal from comment #19)
> Ok, so 4MB memory is _a lot_. Thats 4% of the total available memory or so.
> That is definitely not ok. Why does skia take up so much memory disabled?

I just re-check the memory usage of b2g with/without skia configuration.
And the following are the average data for rebooting to home page 6 times.

I could only see the big difference on cache part, for B2G or non B2G process are almost the same.

Any idea to profile more detail?

[With skia configuration]
            Total	384.9
     Used - cache	104.28
  B2G procs (PSS)	99.14
    Non-B2G procs	5.14
     Free+ cache	280.6
             Free	220.34
            Cache	60.24

[Without skia configuration]
            Total	384.9
     Used - cache	104.34
  B2G procs (PSS)	99.16
    Non-B2G procs	5.18
     Free+ cache	280.54
             Free	215.36
            Cache	65.2

[Raw data]
http://www.pastebin.mozilla.org/2714268

Comment 21

[Andreas Gal :gal]

More cached is probably fine. I think that just means we transitioned more data through the cache. The  memory is still available if needed. pchang, when will we officially enable skia in b2g?

Comment 22

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

It's good on Android, this is the first step in enabling it in b2g.  We expect the stability to get better once bug 858914 lands, but we need to code in the build so that we can pref it on and off easier. Peter is landing this in part so that Benoit can work on this at the same time.
We do expect caching to be a big part of the SkiaGL though, so the extra memory usage is something we need to sort out.

Comment 23

[Benoit Jacob [:bjacob] (mostly away)]

Attachment: Enable Skia/GL on B2G

This patch flips the preferences enabling Skia/GL on B2G... we want to first fix regressions before we land that. (Will file as blockers of this bug).

Comment 24

[Benoit Jacob [:bjacob] (mostly away)]

Try push of graphics branch (Friday) + enabling skia/GL prefs on B2G:
https://tbpl.mozilla.org/?tree=Try&rev=af9e1082e710

Comment 25

[Benoit Jacob [:bjacob] (mostly away)]

Argh, no tests... trying with "emulator" even though trychooser says "no tests yet".
https://tbpl.mozilla.org/?tree=Try&rev=6169407755d2