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 727765 - pool: Review overall usage of pool and its activation
pool: Review overall usage of pool and its activation
Status: RESOLVED DUPLICATE of bug 728268
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-04-07 15:44 UTC by Nicolas Dufresne (ndufresne)
Modified: 2014-04-15 15:31 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Nicolas Dufresne (ndufresne) 2014-04-07 15:44:53 UTC
Bug 727498 highlights the need of sharing pools that may be used by more then one elements. This not being fully considered in general when dealing with pool.

Related to that, it may means that an active pool could be shared, in which case calls to set_config() will fail, and certain settings ignored. This need for sure more clarification and structure, but base on the analyses we will have to review the usage of pools so we know we keep doing the right thing. Specially for cases where pools cannot grow.

While doing this investigation, it would be nice to have the reflection on how to check the "negotiated" result of calling set_config(). As the doc says, any modification of the requested config should lead to FALSE being returned, though config shall be updated as far as possible. I supposed the intention was for the negotiator to re-read the config it tried to set and check if it should try the next propoosed pool/allocator, or if the changes are acceptable in the context.
Comment 1 Nicolas Dufresne (ndufresne) 2014-04-15 15:31:55 UTC
Sorry, I'm getting confused, but description is better in the other bug.

*** This bug has been marked as a duplicate of bug 728268 ***