From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Snitzer Subject: Re: [patch 2/2] dm-delay: add third flush class Date: Tue, 17 Apr 2018 15:08:20 -0400 Message-ID: <20180417190820.GA29125@redhat.com> References: <20180416223312.122871155@debian.vm> <20180416223610.350132415@debian.vm> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20180416223610.350132415@debian.vm> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: Mikulas Patocka Cc: dm-devel@redhat.com, "Alasdair G. Kergon" , Zdenek Kabelac List-Id: dm-devel.ids On Mon, Apr 16 2018 at 6:33pm -0400, Mikulas Patocka wrote: > This patch adds a new class for dm-delay that delays flush requests. > Previously, flushes were delayed as writes, but it caused problems if the > user needed to create a device with one or few slow sectors for the > purpose of testing - all flushes would be forwarded to this device and > delayed, and that skews the test results. This patch allows to select 0 > delay for flushes. > > Signed-off-by: Mikulas Patocka The flush device shouldn't ever be allowed to be different than the write device should it? Also, what does an offset even mean in the context of flush? Pretty awkward really. I get that you've factored out the ctr code and are just reusing it for flush; and that in practice these knobs won't get used (or flush_device won't be different than write_device).. but I'm just not following why we want to expose flush_offset and flush_device at all. Mike