The previous name is clearly confusing. Signed-off-by: Hitoshi Mitake <mitake.hito...@lab.ntt.co.jp> --- lib/work.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/lib/work.c b/lib/work.c index 6933e1a..25cf964 100644 --- a/lib/work.c +++ b/lib/work.c @@ -65,7 +65,7 @@ struct worker_info { size_t nr_threads; /* protected by uatomic primitives */ - size_t nr_workers; + size_t nr_queued_work; /* we cannot shrink work queue till this time */ uint64_t tm_end_of_protection; @@ -109,7 +109,7 @@ static inline uint64_t wq_get_roof(struct worker_info *wi) static bool wq_need_grow(struct worker_info *wi) { - if (wi->nr_threads < uatomic_read(&wi->nr_workers) && + if (wi->nr_threads < uatomic_read(&wi->nr_queued_work) && wi->nr_threads * 2 <= wq_get_roof(wi)) { wi->tm_end_of_protection = get_msec_time() + WQ_PROTECTION_PERIOD; @@ -125,7 +125,7 @@ static bool wq_need_grow(struct worker_info *wi) */ static bool wq_need_shrink(struct worker_info *wi) { - if (uatomic_read(&wi->nr_workers) < wi->nr_threads / 2) + if (uatomic_read(&wi->nr_queued_work) < wi->nr_threads / 2) /* we cannot shrink work queue during protection period. */ return wi->tm_end_of_protection <= get_msec_time(); @@ -201,7 +201,7 @@ void queue_work(struct work_queue *q, struct work *work) { struct worker_info *wi = container_of(q, struct worker_info, q); - uatomic_inc(&wi->nr_workers); + uatomic_inc(&wi->nr_queued_work); pthread_mutex_lock(&wi->pending_lock); if (wq_need_grow(wi)) @@ -235,7 +235,7 @@ static void worker_thread_request_done(int fd, int events, void *data) list_del(&work->w_list); work->done(work); - uatomic_dec(&wi->nr_workers); + uatomic_dec(&wi->nr_queued_work); } } } @@ -404,5 +404,5 @@ bool work_queue_empty(struct work_queue *q) { struct worker_info *wi = container_of(q, struct worker_info, q); - return uatomic_read(&wi->nr_workers) == 0; + return uatomic_read(&wi->nr_queued_work) == 0; } -- 1.7.10.4 -- sheepdog mailing list sheepdog@lists.wpkg.org http://lists.wpkg.org/mailman/listinfo/sheepdog