After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 748370 - Should cancel all thumbnailing when going to the player
Should cancel all thumbnailing when going to the player
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: Movie player
3.16.x
Other Linux
: Normal normal
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2015-04-23 14:37 UTC by Bastien Nocera
Modified: 2015-04-24 15:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
grilo: Limit ourselves to running one thumbnailer at a time (1.45 KB, patch)
2015-04-24 15:27 UTC, Bastien Nocera
committed Details | Review
grilo: Make thumbnail reading cancellable (810 bytes, patch)
2015-04-24 15:27 UTC, Bastien Nocera
committed Details | Review

Description Bastien Nocera 2015-04-23 14:37:58 UTC
Instead of simply stopping new thumbnailers from starting, which will leave at most 5 thumbnailers running in the background, killing performance.
Comment 1 Bastien Nocera 2015-04-24 15:27:03 UTC
The following fixes have been pushed:
ee959d9 grilo: Limit ourselves to running one thumbnailer at a time
Comment 2 Bastien Nocera 2015-04-24 15:27:07 UTC
Created attachment 302300 [details] [review]
grilo: Limit ourselves to running one thumbnailer at a time

Our thumbnailer currently doesn't have support for using hardware
decoding, so it's pretty CPU intensive. On my machine (i7 laptop with an
SSD), one thumbnailer would take about 40% of CPU. This causes both lag
when navigating the videos list, and carries on hampering video playback
if thumbnailing is started before playback.

This is good enough to mark this bug as fixed:
https://bugzilla.gnome.org/show_bug.cgi?id=748370

though a better solution would be to cancel thumbnailing in
totem_grilo_pause(), make sure that
gnome_desktop_thumbnail_path_for_uri() is cancellable, and try to
distinguish in totem-grilo.c between cancellation because the player is
getting shut down, and cancellation because we want all the resources to
go to the player.
Comment 3 Bastien Nocera 2015-04-24 15:27:12 UTC
Created attachment 302301 [details] [review]
grilo: Make thumbnail reading cancellable