Re: [dm-devel] Reworking dm-writeboost [was: Re: staging: Adddm-writeboost]
From: Mikulas Patocka
Date: Wed Oct 02 2013 - 20:05:39 EST
On Tue, 1 Oct 2013, Joe Thornber wrote:
> > Alternatively, delaying them will stall the filesystem because it's
> > waiting for said REQ_FUA IO to complete. For example, journal writes
> > in XFS are extremely IO latency sensitive in workloads that have a
> > signifincant number of ordering constraints (e.g. O_SYNC writes,
> > fsync, etc) and delaying even one REQ_FUA/REQ_FLUSH can stall the
> > filesystem for the majority of that barrier_deadline_ms.
>
> Yes, this is a valid concern, but I assume Akira has benchmarked.
> With dm-thin, I delay the REQ_FUA/REQ_FLUSH for a tiny amount, just to
> see if there are any other FUA requests on my queue that can be
> aggregated into a single flush. I agree with you that the target
> should never delay waiting for new io; that's asking for trouble.
>
> - Joe
You could send the first REQ_FUA/REQ_FLUSH request directly to the disk
and aggregate all the requests that were received while you processed the
initial request. This way, you can do request batching without introducing
artifical delays.
Mikulas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/