Skip to content
Snippets Groups Projects

Fixes to the threadpool

Merged Fixes to the threadpool
threadpool_task_plots2 into master
Merged Peter W. Draper requested to merge threadpool_task_plots2 into master

Don't free the tp->threads struct when not allocated (1 thread). Also set the function when logging one thread (otherwise we don't get the function names in log).

More testing to check 1 thread behaviour and different numbers of threads. Tweak output to make progress more obvious.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
src/threadpool.c
268 268 if (tp->num_threads == 1) {
269 269 map_function(map_data, N, extra_data);
270 270 #ifdef SWIFT_DEBUG_THREADPOOL
271 tp->map_function = map_function;
  • Peter W. Draper Added 19 commits:

    Added 19 commits:

    • 12fb2217...a590b4d9 - 18 commits from branch master
    • 5b51ae91 - Merge remote-tracking branch 'origin/master' into threadpool_task_plots2
  • Pedro Gonnet
    Pedro Gonnet @nnrw56
    Developer

    LGTM.

  • Pedro Gonnet Reassigned to @pdraper

    Reassigned to @pdraper

  • Peter W. Draper Added 1 commit:

    Added 1 commit:

  • Peter W. Draper Status changed to merged

    Status changed to merged

  • Peter W. Draper mentioned in commit 47ddcd03

    mentioned in commit 47ddcd03

    • Please register or sign in to reply