<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""> <div><blockquote type="cite" class=""><div class="">On 18 Jul 2017, at 17:54, Pavel Grunt <<a href="mailto:pgrunt@redhat.com" class="">pgrunt@redhat.com</a>> wrote:</div> <div class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">On Tue, 2017-07-18 at 17:04 +0200, Christophe de Dinechin wrote:<br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">On 14 Jul 2017, at 14:27, Christophe de Dinechin <<a href="mailto:dinechin@redhat.com" class="">dinechin@redhat.com</a>> wrote: <blockquote type="cite" class=""> This looks better than my fix. Can’t ack right now (I did not build and test it yet), but that looks like a reasonable approach <blockquote type="cite" class="">On 14 Jul 2017, at 14:24, Pavel Grunt <<a href="mailto:pgrunt@redhat.com" class="">pgrunt@redhat.com</a>> wrote: This also fixes the huge memore leak reported by Christophe. Steps to reproduce: 1. ./configure --disable-pulse --enable-gstaudio 2. connect to a vm streaming video with no audio playing 3. see the memory grow really fast It seems to be due to the fact that the "fake audio" channel updates the mmtime in a way that schedules the frames to the future ((unsigned) 0 - delay) and then gstreamer starts to queue the frames (they should be played later) Pavel On Fri, 2017-07-14 at 13:19 +0200, Pavel Grunt wrote: <blockquote type="cite" class="">The fake channel has been introduced to get the audio volume by starting the gstreamer's audio pipeline and querring its volume info (see commit aa8d044417bbf60685f59163b874ecb4f157c3c9). Hovewer starting the pipeline updates the mmtime as a side effect. This may cause a (big) delay in displaying a video stream. Because the fake channel is only needed to get the volume info, make sure to not update the mm-time with it. Reported-by: Christophe de Dinechin <<a href="mailto:dinechin@redhat.com" class="">dinechin@redhat.com</a>> --- src/spice-gstaudio.c | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/spice-gstaudio.c b/src/spice-gstaudio.c index 014c3a5..715f824 100644 --- a/src/spice-gstaudio.c +++ b/src/spice-gstaudio.c @@ -38,6 +38,7 @@ struct stream { GstElement *sink; guint rate; guint channels; + gboolean fake; /* fake channel just for getting info about audio (volume) */ }; struct _SpiceGstaudioPrivate { @@ -264,6 +265,8 @@ static gboolean update_mmtime_timeout_cb(gpointer data) SpiceGstaudioPrivate *p = gstaudio->priv; GstQuery *q; + g_return_val_if_fail(!p->playback.fake, TRUE); + q = gst_query_new_latency(); if (gst_element_query(p->playback.pipe, q)) { gboolean live; @@ -326,7 +329,7 @@ cleanup: if (p->playback.pipe) gst_element_set_state(p->playback.pipe, GST_STATE_PLAYING); - if (p->mmtime_id == 0) { + if (!p->playback.fake && p->mmtime_id == 0) { update_mmtime_timeout_cb(gstaudio); p->mmtime_id = g_timeout_add_seconds(1, update_mmtime_timeout_cb, gstaudio); } @@ -569,7 +572,6 @@ static gboolean spice_gstaudio_get_playback_volume_info_finish(SpiceAudio *audio GstElement *e; gboolean lmute; gdouble vol; - gboolean fake_channel = FALSE; GTask *task = G_TASK(res); g_return_val_if_fail(g_task_is_valid(task, audio), FALSE); @@ -584,9 +586,9 @@ static gboolean spice_gstaudio_get_playback_volume_info_finish(SpiceAudio *audio if (p->playback.sink == NULL || p->playback.channels == 0) { SPICE_DEBUG("PlaybackChannel not created yet, force start"); + p->playback.fake = TRUE; /* In order to get system volume, we start the pipeline */ playback_start(NULL, SPICE_AUDIO_FMT_S16, 2, 48000, audio); - fake_channel = TRUE; } if (GST_IS_BIN(p->playback.sink)) @@ -604,9 +606,10 @@ static gboolean spice_gstaudio_get_playback_volume_info_finish(SpiceAudio *audio } g_object_unref(e); - if (fake_channel) { + if (p->playback.fake) { SPICE_DEBUG("Stop faked PlaybackChannel"); playback_stop(SPICE_GSTAUDIO(audio)); + p->playback.fake = FALSE; } if (mute != NULL) { </blockquote></blockquote></blockquote> This fixes the issue I was seeing with a black screen. </blockquote><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Does it fix the memory usage problem for you? it does for me.<br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""></div></blockquote><div> </div><div>It fixes the case under light load, yes. It does not fix the issue under heavy load. For this one, I need the other patch (the one that resets the queue length. To me, that patch was not about that problem, though, and I did not expect it to address memory growth under load, so I’m still satisfied.</div><div> </div><div> </div><div>Christophe</div> <blockquote type="cite" class=""><div class=""><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Pavel<br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class=""> Acked-by: Christophe de Dinechin <<a href="mailto:dinechin@redhat.com" class="">dinechin@redhat.com</a>> <blockquote type="cite" class=""> _______________________________________________ Spice-devel mailing list <a href="mailto:Spice-devel@lists.freedesktop.org" class="">Spice-devel@lists.freedesktop.org</a> <a href="https://lists.freedesktop.org/mailman/listinfo/spice-devel" class="">https://lists.freedesktop.org/mailman/listinfo/spice-devel</a></blockquote></blockquote></div></blockquote></div> </body></html>