Page MenuHomeFreeBSD

Improve outlier latency by implementing batch size for bioq_disksort
ClosedPublic

Authored by imp on Sep 11 2015, 4:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 24 2024, 9:32 PM
Unknown Object (File)
Dec 22 2023, 9:10 PM
Unknown Object (File)
Oct 21 2023, 2:02 AM
Unknown Object (File)
Aug 20 2023, 7:15 AM
Unknown Object (File)
Aug 1 2023, 2:21 PM
Unknown Object (File)
May 10 2023, 8:43 PM
Unknown Object (File)
Mar 5 2023, 5:56 PM
Unknown Object (File)
Feb 17 2023, 4:20 PM

Details

Reviewers
kib
emax
Summary

Implement maximum batch size for the disksort algorithm (also called
max car size in the literature). This limits the number of requests
that are batched together before starting a new batch. It has little
effect on the average service time, but can have a large effect on the
worst service time.

Submitted by: emax@

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 486
Build 486: arc lint + arc unit

Event Timeline

imp retitled this revision from to Improve outlier latency by implementing batch size for bioq_disksort.
imp updated this object.
imp edited the test plan for this revision. (Show Details)
emax added a reviewer: emax.
This revision is now accepted and ready to land.Sep 11 2015, 6:21 PM
kib added a reviewer: kib.

IMO the batch size should be per-queue, but this is probably fine for now.

sys/kern/subr_disk.c
28

I do not like that this knob is under debug OID, is it really debugging facility ?

Also, it makes sense to allow the knob to be managed from the very beginning of the system lifetime, mark it as CTLFAG_RWTUN perhaps.