lkml.org 
[lkml]   [2016]   [Nov]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 3/4] blk-mq: implement hybrid poll mode for sync O_DIRECT
From
Date
On 11/01/2016 03:05 PM, Jens Axboe wrote:
> +static void blk_mq_poll_hybrid_sleep(struct request_queue *q,
> + struct request *rq)
> +{
> + struct hrtimer_sleeper hs;
> + ktime_t kt;
> +
> + if (!q->poll_nsec || test_bit(REQ_ATOM_POLL_SLEPT, &rq->atomic_flags))
> + return;
> +
> + set_bit(REQ_ATOM_POLL_SLEPT, &rq->atomic_flags);
> +
> + /*
> + * This will be replaced with the stats tracking code, using
> + * 'avg_completion_time / 2' as the pre-sleep target.
> + */
> + kt = ktime_set(0, q->poll_nsec);
> +
> + hrtimer_init_on_stack(&hs.timer, CLOCK_MONOTONIC, HRTIMER_MODE_REL);
> + hrtimer_set_expires(&hs.timer, kt);
> +
> + hrtimer_init_sleeper(&hs, current);
> + do {
> + if (test_bit(REQ_ATOM_COMPLETE, &rq->atomic_flags))
> + break;
> + set_current_state(TASK_INTERRUPTIBLE);
> + hrtimer_start_expires(&hs.timer, HRTIMER_MODE_REL);
> + if (hs.task)
> + io_schedule();
> + hrtimer_cancel(&hs.timer);
> + } while (hs.task && !signal_pending(current));
> +
> + __set_current_state(TASK_RUNNING);
> + destroy_hrtimer_on_stack(&hs.timer);
> +}

Hello Jens,

Will avg_completion_time/2 be a good choice for a polling interval if an
application submits requests of varying sizes, e.g. 4 KB and 64 KB?

Can avg_completion_time be smaller than the context switch time? If so,
do you think any other thread will be able to do any useful work after
the timer has been started and before the timer has expired?

Thanks,

Bart.

\
 
 \ /
  Last update: 2016-11-03 15:18    [W:0.208 / U:0.044 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site