On Fri, Feb 20, 2009 at 15:27, Sitsofe Wheeler [off-list ref] wrote:

On Fri, Feb 20, 2009 at 02:23:38PM +0100, Kay Sievers wrote:

quoted

On Fri, Feb 20, 2009 at 14:12, Sitsofe Wheeler [off-list ref] wrote:

quoted

I tried
SUBSYSTEM=="block", ATTRS{model}=="ASUS-PHISON *", ATTR{queue/rotational}="0"
but udevtest /sys/block/sda/ doesn't list it as firing (udev 117-8)...

I does here:
  udev_rules_apply_to_event: ATTR '/sys/devices/ ...
/sdb/queue/rotational' writing '0'

  $grep . /sys/class/block/*/queue/rotational
  /sys/class/block/sda/queue/rotational:1
  /sys/class/block/sdb/queue/rotational:0

Maybe "udevtest" is not showing it for you, or udev 117 is too old and
does not work that way.

After you mentioned this I tried again and seemingly I was mistaken - it
does work as you described.

Great.

I had to make a slight rule tweak to stop
it matching on partitions though:

SUBSYSTEM=="block", KERNEL=="[sh]d[a-z]", ATTRS{model}=="ASUS-PHISON *", ATTR{queue/rotational}="0"

Newer kernels have DEVTYPE in the envent environment, and:
  ENV{DEVTYPE}=="disk"
can be used, which is easier and more reliable.

Also the whitespace after "...PHISON *" does not need to be matched.
If the match value does not end in whitespace itself, udev will match
against the value with all trailing whitespace ignored.

Kay