From 2a71e0c1a675e4f30f02c03dd0325944b393c434 Mon Sep 17 00:00:00 2001 From: Trevor Woerner Date: Fri, 12 Jan 2024 01:54:26 -0500 Subject: [PATCH 3/3] BmapCopy.py: tweak suggested udev rule Both bdi/max_ratio and queue/scheduler are only valid for whole block devices, not individual partitions. Therefore, add a ENV{DEVTYPE}!="partition", to the suggested udev rule so that bmaptool doesn't try to check every partition of the block device, just the whole device. Otherwise the following will appear in the logs: Jan 10 01:30:31 localhost (udev-worker)[10399]: sdk1: /etc/udev/rules.d/60-bmaptool-optimizations.rules:5 Failed to write ATTR{/sys/devices/pci0000:00/0000:00:14.0/usb2/2-3/2-3.1/2-3.1.2/2-3.1.2:1.0/host14/target14:0:0/14:0:0:0/block/sdk/sdk1/bdi/min_ratio}, ignoring: No such file or directory Jan 10 01:30:31 localhost (udev-worker)[10399]: sdk1: /etc/udev/rules.d/60-bmaptool-optimizations.rules:5 Failed to write ATTR{/sys/devices/pci0000:00/0000:00:14.0/usb2/2-3/2-3.1/2-3.1.2/2-3.1.2:1.0/host14/target14:0:0/14:0:0:0/block/sdk/sdk1/bdi/max_ratio}, ignoring: No such file or directory Jan 10 01:30:31 localhost (udev-worker)[10399]: sdk1: /etc/udev/rules.d/60-bmaptool-optimizations.rules:5 Failed to write ATTR{/sys/devices/pci0000:00/0000:00:14.0/usb2/2-3/2-3.1/2-3.1.2/2-3.1.2:1.0/host14/target14:0:0/14:0:0:0/block/sdk/sdk1/queue/scheduler}, ignoring: No such file or directory [... and so on for every partition on your block device ...] Upstream-Status: Submitted [https://github.com/intel/bmap-tools/pull/131] Signed-off-by: Trevor Woerner --- bmaptools/BmapCopy.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/bmaptools/BmapCopy.py b/bmaptools/BmapCopy.py index b1e8e0fcbdb7..a4c1177246a9 100644 --- a/bmaptools/BmapCopy.py +++ b/bmaptools/BmapCopy.py @@ -906,7 +906,8 @@ class BmapBdevCopy(BmapCopy): "\n" 'ACTION=="add", SUBSYSTEMS=="usb", ATTRS{idVendor}=="xxxx", ' 'ATTRS{idProduct}=="xxxx", TAG+="uaccess"\n' - 'SUBSYSTEMS=="usb", ATTRS{idVendor}=="xxxx", ' + 'SUBSYSTEMS=="usb", ENV{DEVTYPE}!="partition", ' + 'ATTRS{idVendor}=="xxxx", ' 'ATTRS{idProduct}=="xxxx", ATTR{bdi/min_ratio}="0", ' 'ATTR{bdi/max_ratio}="1", ATTR{queue/scheduler}="none"\n' "\n" -- 2.43.0.76.g1a87c842ece3