xpra icon
Bug tracker and wiki

Opened 2 months ago

Closed 8 weeks ago

#2349 closed enhancement (wontfix)

cache gstreamer config

Reported by: Antoine Martin Owned by: Antoine Martin
Priority: major Milestone: 3.0
Component: sound Version: 2.5.x
Keywords: Cc:

Description

Related to #2341 and #2347. Similar to #2345.

Instead of running xpra _sound_query every time we run the client or server, we could cache the results in a text file and just reload it.

We would need to ensure that the timestamp on this file is newer than the gstreamer plugins directory, so that we query it again whenever plugins are added or removed.
This is similar to what gstreamer does with its plugin registry (now found in .cache/gstreamer-1.0/) - can we re-use this mechanism without incurring the same startup cost?

Attachments (1)

cache-sound-query.patch (12.1 KB) - added by Antoine Martin 8 weeks ago.
cache sound query data

Download all attachments as: .zip

Change History (2)

Changed 8 weeks ago by Antoine Martin

Attachment: cache-sound-query.patch added

cache sound query data

comment:1 Changed 8 weeks ago by Antoine Martin

Resolution: wontfix
Status: newclosed

The patch above implements sound query caching, but there are problems:

  • complexity
  • still takes 100ms because we have to run gst-inspect-1.0 to ensure that the cache is up-to-date (best case, this can take longer and may break..)
  • the sound query output includes some pulseaudio things, and those may get updated too - we just don't know when the cache needs to be invalidated

So, I don't think we should be doing this..

Note: See TracTickets for help on using tickets.