[Bug 664778] [souphttpsrc] chunk allocator: memory waste

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=664778
  GStreamer | gst-plugins-good | unspecified

René Stadler <mail> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
                 CC|                            |mail at renestadler.de
     Ever Confirmed|0                           |1

--- Comment #1 from René Stadler <mail at renestadler.de> 2011-11-29 12:06:56 UTC ---
That reminds me. I once wrote a patch about this, but I think it got lost in a
hard drive crash. IIRC we cannot get larger amounts of data from libsoup. But
what we can do is be smarter with our allocations in souphttpsrc. It's
basically as you say, but there is even a worse case: when the network is
really slow, you can also get 0 bytes often (even for the default 4KB buffers).
In that case, the buffer is thrown away and immediately reallocated...

The solution is to keep the buffer around if it's not full yet and keep filling
it up before pushing. That reduces the overhead from re-allocations for zero
reads and pushes larger, fewer buffers into the pipeline.

-- 
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.