Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1527945

Re: [PATCH V4 07/15] blk-throttle: make throtl_slice tunable

From Tejun Heo <tj@kernel.org>
Newsgroups linux.kernel
Subject Re: [PATCH V4 07/15] blk-throttle: make throtl_slice tunable
Date 2016-11-22 22:30 +0100
Message-ID <sGqNA-4cd-7@gated-at.bofh.it> (permalink)
References <sDxVf-4Bm-3@gated-at.bofh.it> <sDxVf-4Bm-17@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hello,

On Mon, Nov 14, 2016 at 02:22:14PM -0800, Shaohua Li wrote:
> throtl_slice is important for blk-throttling. A lot of stuffes depend on
> it, for example, throughput measurement. It has 100ms default value,
> which is not appropriate for all disks. For example, for SSD we might
> use a smaller value to make the throughput smoother. This patch makes it
> tunable.

It bothers me a bit because time slice doesn't mean anything inherent
to throttling.  It really is an implementation detail - throttling can
be implemented at per-operation level without time slice involved at
all.  It's okay to expose the knob if necessary but the meaning of the
knob is almost completely arbitrary to users (as it has no inherent
meaning).

Thanks.

-- 
tejun

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH V4 07/15] blk-throttle: make throtl_slice tunable Shaohua Li <shli@fb.com> - 2016-11-14 23:30 +0100
  Re: [PATCH V4 07/15] blk-throttle: make throtl_slice tunable Tejun Heo <tj@kernel.org> - 2016-11-22 22:30 +0100
    Re: [PATCH V4 07/15] blk-throttle: make throtl_slice tunable Tejun Heo <tj@kernel.org> - 2016-11-23 22:20 +0100

csiph-web