Ticket #201 (closed defect: fixed)

Opened 4 years ago

Last modified 2 years ago

Bounce opens and reads too many icons

Reported by: RafaelOrtiz Owned by: RafaelOrtiz
Priority: High Milestone: 0.88
Component: sugar Version: 0.82.x
Severity: Major Keywords:
Cc: Distribution/OS: Unspecified
Bug Status: Needinfo

Description

from  http://dev.laptop.org/ticket/8658

please read important info there and then close it there when finished
in the 2 places...maybe doing the follow up in here.

Change History

  Changed 4 years ago by gregdek

  • priority changed from normal to High
  • distribution set to Unspecified
  • status_field set to New
  • severity set to Major
  • milestone set to 0.84

From the parent bug:

"strace shows that Bounce opens and reads over 100 icons that it clearly doesn't need, from places like /usr/share/icons/sugar/scalable/device/battery-030.icon . (unexplored avenue: other activities may do this as well.) "

This could be an ugly performance issue, and given that we have intractable performance issues like Python Slowness, this could be a big win. Assigning to 0.84 to see if we can have a look, but I'm guessing we'll have to push it to 0.86. Maybe some intrepid soul can do the analysis here.

follow-up: ↓ 3   Changed 4 years ago by wadeb

I'm pretty sure this was happening to *all* activities, not just Bounce. However, I tried the strace myself on 8.2.0 and was unable see the icon loads.

in reply to: ↑ 2   Changed 4 years ago by tomeu

Replying to wadeb:

I'm pretty sure this was happening to *all* activities, not just Bounce. However, I tried the strace myself on 8.2.0 and was unable see the icon loads.

Actually, I saw it happening once but then couldn't reproduce.

I guess this happens because gtk cannot find the icon cache, so recreates it. This may have to do with rainbow. Maybe someone can check?

  Changed 4 years ago by erikos

If it is rainbow specific - one should be able to easily find out by turning off rainbow and compare.

  Changed 4 years ago by tomeu

  • status_field changed from New to Needinfo

  Changed 4 years ago by RafaelOrtiz

  • milestone changed from 0.84 to 0.88

  Changed 3 years ago by erikos

  • owner changed from marcopg to RafaelOrtiz
  • status changed from new to assigned
  • version changed from Unspecified to 0.82.x

Can someone test this again on the XO?

  Changed 2 years ago by RafaelOrtiz

  • status changed from assigned to closed
  • resolution set to fixed

Tested again on the xo, bounce v7, strace not reproducible closing.

Note: See TracTickets for help on using tickets.