Bug 104609
| Summary: | --profile differences between Linux and Android | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Philip Rogers <pdr> |
| Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED DUPLICATE | ||
| Severity: | Normal | CC: | eric, peter |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Philip Rogers
Due to using two profilers, our --profile output is different on each platform:
Android - SvgCubics
[ perf record: Captured and wrote 6.210 MB /data/perf.data (~271321 samples) ]
59.49% SignalSender libm.so [.] 0x1169c
8.66% SignalSender [kernel] [k] 0xc004fa50
7.10% SignalSender libDumpRenderTree.so [.] WebCore::PathTraversalState::cubicBezierTo(WebCore::FloatPoint const&, WebCore::FloatPoint const&, WebCore::FloatPoint const&)
5.80% SignalSender libDumpRenderTree.so [.] WebCore::CubicBezier::split(WebCore::CubicBezier&, WebCore::CubicBezier&) const
4.45% SignalSender libDumpRenderTree.so [.] WebCore::distanceLine(WebCore::FloatPoint const&, WebCore::FloatPoint const&)
2.57% SignalSender libc.so [.] 0x1fce0
1.41% SignalSender [unknown] [.] 0x6c12764c
0.94% SignalSender libDumpRenderTree.so [.] 0x181f90
0.21% SignalSender libDumpRenderTree.so [.] SkPath::RawIter::next(SkPoint*)
0.20% SignalSender libDumpRenderTree.so [.] WebCore::convertPathPoints(WebCore::FloatPoint*, SkPoint const*, int)
Linux - SvgCubics
Finished: 1.381556 s
56 47.9% 47.9% 56 47.9% WebCore::PathTraversalState::cubicBezierTo
3 2.6% 50.4% 3 2.6% 00007fba7a9055e2
3 2.6% 53.0% 3 2.6% __access
3 2.6% 55.6% 3 2.6% __libc_free
2 1.7% 57.3% 2 1.7% 00007fba7a905417
2 1.7% 59.0% 2 1.7% WebCore::Path::apply
2 1.7% 60.7% 2 1.7% WebCore::SVGResourcesCache::cachedResourcesForRenderObject
2 1.7% 62.4% 2 1.7% WebCore::StyleResolver::collectMatchingRules
2 1.7% 64.1% 2 1.7% __libc_malloc
2 1.7% 65.8% 2 1.7% tcmalloc::FL_Push
It is possible this is due to the platform, but
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Eric Seidel (no email)
Yeah, one uses pprof, the other uses perf. I should make it possible to use perf on either. It's possible pprof could be made to work on Android too (but not until they use tcmalloc... which I don't think they do yet).
Eric Seidel (no email)
These both use perf by default now. :)
*** This bug has been marked as a duplicate of bug 104971 ***