Overview

Request 592117 accepted

- Enhance IO scheduler tweaking rules to support blk-mq as well
* Set optimal blk-mq schedulers (bfq and mq-deadline for HDD and
SSD respectively)
* Explicitly set CFQ for rotational disks when no `elevator`
is specified
* Verbose comments explaining what is going on
* Rename the rules file since it is not ssd-only anymore

Loading...

Andrei Dziahel's avatar

@fbui the request has been acked by @mwilck, but it's still in new state. Is it stuck?


Andrei Dziahel's avatar

um. @systemd-maintainers anyone?


Franck Bui's avatar

It's just that I haven't received any notification about your new submission, sorry for the delay.


Franck Bui's avatar

I meant that I didn't see that Martin acked your changes.


Andrei Dziahel's avatar

Looks like the review flow has been stuck somehow.



Andrei Dziahel's avatar

Shouldn't it be picked for some staging project BTW?



Andrei Dziahel's avatar

@mwilck the trick here is the elevator cmdline affects non-BLK-MQ config only, that's why it's put after MQ-related part


Andrei Dziahel's avatar

@mwilck OTOH we shouldn't care whether elevator affects BLK-MQ configs or not. Fixing. Thank you again.

Request History
Andrei Dziahel's avatar

develop7 created request

- Enhance IO scheduler tweaking rules to support blk-mq as well
* Set optimal blk-mq schedulers (bfq and mq-deadline for HDD and
SSD respectively)
* Explicitly set CFQ for rotational disks when no `elevator`
is specified
* Verbose comments explaining what is going on
* Rename the rules file since it is not ssd-only anymore


Andrei Dziahel's avatar

develop7 added mwilck as a reviewer

Hi Martin, I've followed your suggestions, please check it out


Franck Bui's avatar

fbui added mwilck as a reviewer

Martin, could you ack this ? thanks.


Martin Wilck's avatar

mwilck accepted review

The syntax 'ENV{elevator}=="*?"' is a bit unusual ("?*" is typically used) but that's just a nitpick. Ack.


Martin Wilck's avatar

mwilck accepted review

The syntax 'ENV{elevator}=="*?"' is a bit unusual ("?*" is typically used) but that's just a nitpick. Ack.


Martin Wilck's avatar

mwilck approved review

The syntax 'ENV{elevator}=="*?"' is a bit unusual ("?*" is typically used) but that's just a nitpick. Ack.


Franck Bui's avatar

fbui accepted request

@develop7: thanks !

openSUSE Build Service is sponsored by