tizenaudio-sink: optimize amount of 'max_request' 29/134229/4
authorKimJeongYeon <jeongyeon.kim@samsung.com>
Thu, 15 Jun 2017 09:19:19 +0000 (18:19 +0900)
committerKimJeongYeon <jeongyeon.kim@samsung.com>
Thu, 22 Jun 2017 05:06:45 +0000 (14:06 +0900)
[Version] 5.0.154
[Profile] Common
[Issue Type] Latency improvement

tizenaudio-sink consume maximum 32ms every requesting from driver.
Therefore, I assume that the 'max_request' is able to reduce until 32ms.
This patch related with 'loopback: apply official patchset'.
(Commit: 8c9407a79e8254f3d85f1cc51ad0c6ab48199250)

Signed-off-by: KimJeongYeon <jeongyeon.kim@samsung.com>
Change-Id: Iaf34b94ef3472e5d998437f236847291b6afbff5

packaging/pulseaudio-modules-tizen.spec
src/module-tizenaudio-sink.c

index 7870a671dc93591f4acc82f323ec315a1f93f111..4a4e69ca6a1f13563725d01fb0b953be281a6dd6 100644 (file)
@@ -1,6 +1,6 @@
 Name:             pulseaudio-modules-tizen
 Summary:          Pulseaudio modules for Tizen
-Version:          5.0.153
+Version:          5.0.154
 Release:          0
 Group:            Multimedia/Audio
 License:          LGPL-2.1+
index 75ec05609a9351f58144b77fa91f964d076485be..1df1d8d6c758d7bc87675432fee1b218c8f60fc4 100644 (file)
@@ -260,7 +260,7 @@ static void sink_update_requested_latency_cb(pa_sink *s) {
 
     nbytes = pa_usec_to_bytes(u->block_usec, &s->sample_spec);
     pa_sink_set_max_rewind_within_thread(s, nbytes);
-    pa_sink_set_max_request_within_thread(s, nbytes);
+    pa_sink_set_max_request_within_thread(s, nbytes / 2);  /* max_request 32ms seems enough */
 }
 
 static void process_rewind(struct userdata *u, pa_usec_t now) {
@@ -527,7 +527,7 @@ int pa__init(pa_module*m) {
 
     nbytes = pa_usec_to_bytes(u->block_usec, &u->sink->sample_spec);
     pa_sink_set_max_rewind(u->sink, 0);
-    pa_sink_set_max_request(u->sink, nbytes);
+    pa_sink_set_max_request(u->sink, nbytes / 2);  /* max_request 32ms seems enough */
 
     if (!(u->thread = pa_thread_new("tizenaudio-sink", thread_func, u))) {
         pa_log_error("Failed to create thread.");