Bug 221954 - cam iosched: Call cam_iosched_limiter_init() after ios->current is set to the default
Summary: cam iosched: Call cam_iosched_limiter_init() after ios->current is set to the...
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: kern (show other bugs)
Version: CURRENT
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-scsi (Nobody)
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2017-08-31 10:17 UTC by Fabian Keil
Modified: 2021-10-03 08:36 UTC (History)
4 users (show)

See Also:


Attachments
cam iosched: Call cam_iosched_limiter_init() after ios->current is set to the default (1.09 KB, patch)
2017-08-31 10:17 UTC, Fabian Keil
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fabian Keil 2017-08-31 10:17:30 UTC
Created attachment 185946 [details]
cam iosched: Call cam_iosched_limiter_init() after ios->current is set to the default

The attached patch lets cam_iosched_iop_stats_init() call cam_iosched_limiter_init()
after ios->current is set to the default.

Previously ios->current was set to 0 until the first
cam_iosched_cl_maybe_steer() call.

Obtained from: ElectroBSD
Comment 1 commit-hook freebsd_committer freebsd_triage 2017-09-20 21:26:55 UTC
A commit references this bug:

Author: imp
Date: Wed Sep 20 21:26:01 UTC 2017
New revision: 323832
URL: https://svnweb.freebsd.org/changeset/base/323832

Log:
  cam iosched: Call cam_iosched_limiter_init() after ios->current is set to the default

  Previously ios->current was set to 0 until the first
  cam_iosched_cl_maybe_steer() call.

  PR: 221954
  Obtained from: ElectroBSD
  Submitted by: Fabian Keil
  Differential Revision: https://reviews.freebsd.org/D12349

Changes:
  head/sys/cam/cam_iosched.c
Comment 2 Ed Maste freebsd_committer freebsd_triage 2018-12-04 14:13:14 UTC
MFC plans?
Comment 3 Piotr Pawel Stefaniak freebsd_committer freebsd_triage 2021-09-30 21:26:28 UTC
FreeBSD 12 has the change, 11 reaches EoL today.