Convert depth_dbm_solver.c to use condition variables instead of a busy/sleep loop #8

Closed
shlomif opened this Issue Dec 8, 2016 · 1 comment

Comments

Projects
None yet
1 participant
@shlomif
Owner

shlomif commented Dec 8, 2016

Make a wise use of condition variables in the depth_dbm_solver instead of a busy/usleep() loop to wake up threads for further processing. The basic idea is:

  1. A worker thread will signal the condvar when new data is available in the queue to wake up a single thread.

  2. A thread will broadcast the condvar to terminate the process completely.

Please use a feature branch for that.

@shlomif shlomif added the enhancement label Dec 8, 2016

shlomif added a commit that referenced this issue Dec 8, 2016

Implement condvars instead of usleep .
In the depth_dbm_solver . See:

#8 .

shlomif added a commit that referenced this issue Dec 9, 2016

Implement condvars instead of usleep .
In the depth_dbm_solver . See:

#8 .

shlomif added a commit that referenced this issue Dec 10, 2016

Implement condvars instead of usleep .
In the depth_dbm_solver . See:

#8 .

shlomif added a commit that referenced this issue Dec 12, 2016

Implement condvars instead of usleep .
In the depth_dbm_solver . See:

#8 .

shlomif added a commit that referenced this issue Dec 14, 2016

Implement condvars instead of usleep .
In the depth_dbm_solver . See:

#8 .

shlomif added a commit that referenced this issue Dec 20, 2016

Implement condvars instead of usleep .
In the depth_dbm_solver . See:

#8 .

shlomif added a commit that referenced this issue Dec 20, 2016

Implement condvars instead of usleep .
In the depth_dbm_solver . See:

#8 .

shlomif added a commit that referenced this issue Dec 20, 2016

Implement condvars instead of usleep .
In the depth_dbm_solver . See:

#8 .
@shlomif

This comment has been minimized.

Show comment
Hide comment
@shlomif

shlomif Dec 21, 2016

Owner

Merged into master, so closing.

Owner

shlomif commented Dec 21, 2016

Merged into master, so closing.

@shlomif shlomif closed this Dec 21, 2016

shlomif added a commit that referenced this issue Dec 25, 2016

shlomif added a commit that referenced this issue Jan 27, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment