| Message ID | 20210811033702.368488-1-damien.lemoal@wdc.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | IO priority fixes and improvements | expand |
On 2021/08/11 12:37, Damien Le Moal wrote: > This series fixes problems with IO priority values handling and cleans > up several macro names and code for clarity. Jens, Any comment on this ? > > Changes from v3: > * Split former patch 2 into patches 2, 3 and 4 to facilitate review and > have more descriptive commit titles. > * In patch 5, keep IOPRIO_BE_NR as an alias for the new IOPRIO_NR_LEVELS > macro. Change this patch title and commit message accordingly. > * In patch 6, define IOPRIO_BE_NORM as an alias of IOPRIO_NORM. > > Changes from v2: > * Fixed typo in a comment in patch 3 > * Added reviewed-by tags > > Changes from v1: > * Added patch 4 to unify the default priority value used in various > places. > * Fixed patch 2 as suggested by Bart: remove extra parenthesis and move > ioprio_valid() from the uapi header to the kernel header. > * In patch 2, add priority value masking. > > Damien Le Moal (6): > block: bfq: fix bfq_set_next_ioprio_data() > block: improve ioprio class description comment > block: change ioprio_valid() to an inline function > block: fix IOPRIO_PRIO_CLASS() and IOPRIO_PRIO_VALUE() macros > block: Introduce IOPRIO_NR_LEVELS > block: fix default IO priority handling > > block/bfq-iosched.c | 10 +++++----- > block/bfq-iosched.h | 4 ++-- > block/bfq-wf2q.c | 6 +++--- > block/ioprio.c | 9 ++++----- > drivers/nvme/host/lightnvm.c | 2 +- > fs/f2fs/sysfs.c | 2 +- > include/linux/ioprio.h | 17 ++++++++++++++++- > include/uapi/linux/ioprio.h | 34 ++++++++++++++++++++-------------- > 8 files changed, 52 insertions(+), 32 deletions(-) >
On 8/10/21 9:36 PM, Damien Le Moal wrote: > This series fixes problems with IO priority values handling and cleans > up several macro names and code for clarity. Applied for 5.15 - note that I dropped the lightnvm change from 6/6, as it isn't strictly needed and lightnvm is deleted from the 5.15 drivers branch anyway.
On 2021/08/18 22:24, Jens Axboe wrote: > On 8/10/21 9:36 PM, Damien Le Moal wrote: >> This series fixes problems with IO priority values handling and cleans >> up several macro names and code for clarity. > > Applied for 5.15 - note that I dropped the lightnvm change from 6/6, > as it isn't strictly needed and lightnvm is deleted from the 5.15 > drivers branch anyway. OK. Thanks !