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 586331 - [cdparanoia] expose cd cache size parameter
[cdparanoia] expose cd cache size parameter
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal enhancement
: 0.10.24
Assigned To: GStreamer Maintainers
GStreamer Maintainers
: 589907 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2009-06-19 01:09 UTC by Jonathan Matthew
Modified: 2009-07-27 21:10 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
add cache-size property (4.31 KB, patch)
2009-06-19 01:31 UTC, Jonathan Matthew
committed Details | Review

Description Jonathan Matthew 2009-06-19 01:09:05 UTC
cdparanoia 10.2 added a function for setting the CD sector cache size, which defaults to 1200, where the previous size was 150.  The larger cache size is good for audio extraction (I assume), but bad for playback.  Playback starts much slower, and since the TOC scan happens in READY->PAUSED, that's much slower too.  It'd be nice to be able to set a lower cache size for playback.
Comment 1 Jonathan Matthew 2009-06-19 01:31:45 UTC
Created attachment 136964 [details] [review]
add cache-size property

This adds a ./configure check for the function used to apply the setting, so it requires cdparanoia 10.2 (see bug 531035).
Comment 2 Tim-Philipp Müller 2009-06-19 21:21:54 UTC
commit 4e1086f93b509d92529d7af5475e3c9780d201c1
Author: Tim-Philipp Müller <tim.muller@collabora.co.uk>
Date:   Fri Jun 19 21:25:54 2009 +0100

    cdparanoia: try to guess a good cache size if it's set to -1
    
    Try to guess from the paranoia-mode setting whether playback or
    ripping is wanted, and use a smaller cache size if we're likely
    to be doing playback, to avoid a long startup delay. Since this
    was the value used in older cdparanoia versions, it should be
    fine in any case. See #586331.

commit 9daee1f4a9195ab040db1a2180e6eece1ee2997e
Author: Jonathan Matthew <jonathan@d14n.org>
Date:   Fri Jun 19 11:27:40 2009 +1000

    cdparanoia: expose cache size setting
    
    This setting was added in cdparanoia 10.2.  The default value is good
    for audio extraction, but lower values (previous versions of cdparanoia
    used 150) are better for realtime playback.
    
    Fixes #586331.
Comment 3 Jonathan Matthew 2009-07-27 21:10:45 UTC
*** Bug 589907 has been marked as a duplicate of this bug. ***