[neurun] Clearify Scheduler::ESTAndExecTime with comments (#6649)
authorДилшоджон Пошшоев/AI Tools Lab /SRR/Engineer/삼성전자 <d.poshshoev@samsung.com>
Mon, 19 Aug 2019 02:15:33 +0000 (05:15 +0300)
committer이한종/On-Device Lab(SR)/Engineer/삼성전자 <hanjoung.lee@samsung.com>
Mon, 19 Aug 2019 02:15:33 +0000 (11:15 +0900)
Add comments to clearify return values in case of non-parallel-executor

Signed-off-by: Poshshoev Dilshodzhon <d.poshshoev@samsung.com>
runtimes/neurun/core/src/compiler/Scheduler.cc

index bbdab80..6480f2f 100644 (file)
@@ -503,7 +503,11 @@ Scheduler::ESTAndExecTime(const backend::Backend *backend, const model::Operatio
     _backends_avail_time[cpu_backend].erase(it);
   }
 
-  // In case non-parallel executor measure just exec time and data transfer time
+  /* In case non-parallel executor measure just exec time and data transfer time
+   * because EFT(prev_op_ft) is the same for all backends. Since two operations
+   * can't be run simultaneously, finish of running operation must be waited for.
+   * When an operation starts, all backends are free. So, they need time just for
+   * data transfer.*/
   if (!is_parallel_exec)
   {
     VERBOSE(Scheduler::ESTAndExecTime)