The path_latency prioritizer didn't call get_prio_timeout, so that the
timeout could possibly be 0. Also, it was assuming that the timeout was
in seconds, instead of milliseconds.
Signed-off-by: Benjamin Marzinski
---
libmultipath/prioritizers/path_latency.c | 5 +++--
1 file changed, 3 insertio
This patchset changes how prioritizers set their timeouts, to make them
match how the checker functions work, and also cleans up some minor
timeout issues. I did this to make out timeouts consistent, but if
someone has a good reason to object to that, I don't feel
strongly that it's necessary, and
detect_alua calls prioritizer functions that expect the timeout to be in
milliseconds instead of seconds. Fix this and also respect the
checker_timeout setting.
Signed-off-by: Benjamin Marzinski
---
libmultipath/discovery.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --gi
Multipath's prioritizers previously didn't use the sysfs timeout for
scsi devices, and used a prioritizer specific default timeout (although
in practice, all the prioritizers except hds used 60 seconds). Now
prioritizers deal with timeouts the same way as the checkers. They first
use checker_time i
by the time get_state() is rechecking the sysfs timeout value,
c->timeout has already been set. The only reason why this check exists
is to deal with the possiblity that the sysfs value has changed. If the
sysfs value doesn't exist (which likely means that the device is not a
scsi device), then th
The pull request you sent on Thu, 27 Jul 2023 16:41:38 -0400:
> git://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git
> tags/for-6.5/dm-fixes
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/c75981a1be350f14dbfca56e62bea077dafdad96
Thank you!
--
De
Mike Snitzer writes:
> Thanks for the extra context, but a _big_ elephant in the room for
> this line of discussion is that: the Linux workqueue code has
> basically always been only available for use by GPL'd code. Given
> VDO's historic non-GPL origins, it seems _to me_ that an alternative
>
