[Bug 772841] udpsrc: high CPU usage at high packet rate

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=772841

--- Comment #41 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
(In reply to Mathieu Duponchelle from comment #37)

> * At least on its own, the proposed patch incorrectly blocked on
> receive_message calls, as the udpsrc socket is blocking. We can not make it
> blocking, at least not by default, as it might be used by other code such as
> udpsink when sending; it would be possible to follow two different code
> paths depending on the socket's refcount, but that's complicated and IMO
> unrelated to the bufferpool work.

Maybe add a property for that? It seems like a valuable optimization (in a
separate patch), that would prevent one syscall per packet if packets are
arriving fast.

> * I added support for larger than MTU messages, by falling back to a larger
> memory, very similar to what we had before, except we no longer cache the
> allocator / allocation params

As Nicolas mentioned already this breaks the pool. We need to do something more
clever here. Maybe only ever use a bufferpool if MTU is set, and don't set it
by default. If MTU is set we always allocate that much and don't need to do any
magic with different memories.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.