From c608d153c4bc7258c8c97469639bb22c4dcf240e Mon Sep 17 00:00:00 2001 From: Zbigniew Jędrzejewski-Szmek Date: Mar 26 2020 14:27:03 +0000 Subject: Modify the downstream udev rule to use bfq to only apply to disks (cherry picked from commit 437cd52f28d51e1db652206497a41fbe3bab9124) --- diff --git a/464a73411c13596a130a7a8f0ac00ca728e5f69e.patch b/464a73411c13596a130a7a8f0ac00ca728e5f69e.patch deleted file mode 100644 index 4de01c4..0000000 --- a/464a73411c13596a130a7a8f0ac00ca728e5f69e.patch +++ /dev/null @@ -1,40 +0,0 @@ -From 464a73411c13596a130a7a8f0ac00ca728e5f69e Mon Sep 17 00:00:00 2001 -From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= -Date: Wed, 14 Aug 2019 15:57:42 +0200 -Subject: [PATCH] udev: use bfq as the default scheduler - -As requested in https://bugzilla.redhat.com/show_bug.cgi?id=1738828. -Test results are that bfq seems to behave better and more consistently on -typical hardware. The kernel does not have a configuration option to set -the default scheduler, and it currently needs to be set by userspace. - -See the bug for more discussion and links. ---- - rules/60-block-scheduler.rules | 5 +++++ - rules/meson.build | 1 + - 2 files changed, 6 insertions(+) - create mode 100644 rules/60-block-scheduler.rules - -diff --git a/rules/60-block-scheduler.rules b/rules/60-block-scheduler.rules -new file mode 100644 -index 00000000000..480b941761f ---- /dev/null -+++ b/rules/60-block-scheduler.rules -@@ -0,0 +1,5 @@ -+# do not edit this file, it will be overwritten on update -+ -+ACTION=="add", SUBSYSTEM=="block", \ -+ KERNEL=="mmcblk*[0-9]|msblk*[0-9]|mspblk*[0-9]|sd*[!0-9]|sr*", \ -+ ATTR{queue/scheduler}="bfq" -diff --git a/rules/meson.build b/rules/meson.build -index b6a32ba77e2..1da958b4d46 100644 ---- a/rules/meson.build -+++ b/rules/meson.build -@@ -2,6 +2,7 @@ - - rules = files(''' - 60-block.rules -+ 60-block-scheduler.rules - 60-cdrom_id.rules - 60-drm.rules - 60-evdev.rules diff --git a/systemd.spec b/systemd.spec index 2212260..43c8834 100644 --- a/systemd.spec +++ b/systemd.spec @@ -57,7 +57,7 @@ GIT_DIR=../../src/systemd/.git git diffab -M v233..master@{2017-06-15} -- hwdb/[ %endif # https://bugzilla.redhat.com/show_bug.cgi?id=1738828 -Patch0001: https://github.com/keszybz/systemd/commit/464a73411c13596a130a7a8f0ac00ca728e5f69e.patch +Patch0001: use-bfq-scheduler.patch Patch0900: 0002-Revert-units-set-NoNewPrivileges-for-all-long-runnin.patch @@ -708,6 +708,7 @@ fi %changelog * Thu Mar 26 2020 Zbigniew Jędrzejewski-Szmek - 243.8-1 - Update to latest stable version +- Modify the downstream udev rule to use bfq to only apply to disks (#1803500) * Mon Feb 10 2020 Zbigniew Jędrzejewski-Szmek - 243.7-1 - Do not time out when resuming from hibernation (#1705522) diff --git a/use-bfq-scheduler.patch b/use-bfq-scheduler.patch new file mode 100644 index 0000000..9549fb3 --- /dev/null +++ b/use-bfq-scheduler.patch @@ -0,0 +1,41 @@ +From 464a73411c13596a130a7a8f0ac00ca728e5f69e Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= +Date: Wed, 14 Aug 2019 15:57:42 +0200 +Subject: [PATCH] udev: use bfq as the default scheduler + +As requested in https://bugzilla.redhat.com/show_bug.cgi?id=1738828. +Test results are that bfq seems to behave better and more consistently on +typical hardware. The kernel does not have a configuration option to set +the default scheduler, and it currently needs to be set by userspace. + +See the bug for more discussion and links. +--- + rules/60-block-scheduler.rules | 5 +++++ + rules/meson.build | 1 + + 2 files changed, 6 insertions(+) + create mode 100644 rules/60-block-scheduler.rules + +diff --git a/rules/60-block-scheduler.rules b/rules/60-block-scheduler.rules +new file mode 100644 +index 00000000000..480b941761f +--- /dev/null ++++ b/rules/60-block-scheduler.rules +@@ -0,0 +1,6 @@ ++# do not edit this file, it will be overwritten on update ++ ++ACTION=="add", SUBSYSTEM=="block", \ ++ KERNEL=="mmcblk*[0-9]|msblk*[0-9]|mspblk*[0-9]|sd*[!0-9]|sr*", \ ++ DEVTYPE=="disk", \ ++ ATTR{queue/scheduler}="bfq" +diff --git a/rules/meson.build b/rules/meson.build +index b6a32ba77e2..1da958b4d46 100644 +--- a/rules/meson.build ++++ b/rules/meson.build +@@ -2,6 +2,7 @@ + + rules = files(''' + 60-block.rules ++ 60-block-scheduler.rules + 60-cdrom_id.rules + 60-drm.rules + 60-evdev.rules