[Bug 719615] oggdemux: slow seeking on some ogg files

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=719615
  GStreamer | gst-plugins-base | git

Nicolas Dufresne <nicolas.dufresne> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
                 CC|                            |nicolas.dufresne at collabora.
                   |                            |co.uk
     Ever Confirmed|0                           |1

--- Comment #1 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk> 2013-12-01 02:03:00 UTC ---
If someone is interested, perf top shows that we spend all our time in:

   ogg_page_checksum_set()

Backtrace:

#0  0x0000003d92402207 in ogg_page_checksum_set () from /lib64/libogg.so.0
#1  0x0000003d92402c40 in ogg_sync_pageseek () from /lib64/libogg.so.0
#2  0x00007fffd41ea06a in gst_ogg_demux_get_next_page (ogg=0x7fffbc0be120,
og=0x7fffffff8ef0, boundary=287946072, offset=0x7fffffff8f10)
    at gstoggdemux.c:2378
#3  0x00007fffd41ec599 in gst_ogg_demux_do_seek (ogg=0x7fffbc0be120,
segment=0x7fffbc0be2a0, accurate=0, keyframe=0, rchain=0x7fffffff90d0)
    at gstoggdemux.c:2991
#4  0x00007fffd41ed8b6 in gst_ogg_demux_perform_seek_pull (ogg=0x7fffbc0be120,
event=0x23abde0) at gstoggdemux.c:3210

Not sure if it's a regression in libogg or GStreamer, but I'm guessing we
massively call _get_next_page() while doing a binary search. I have very little
knowledge of ogg myself, but maybe that will save someone that knows a bit of
time.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.