[Pkg-sysvinit-devel] Re: [Pkg-sysvinit-commits] r816 - in sysvinit/trunk/debian: . patches

[Henrique de Moraes Holschuh]

On Fri, 04 Aug 2006, Petter Reinholdtsen wrote:
> Log:
>   * Make sure SATA disks are powered down as well as IDE disks.  Patch
>     from Sebastian Reichelt.  (Closes: #348172)

I am *not* at all against this patch.  But should we not be raising hell
with the Debian kernel team to fix the &#@$&@#$ kernel to do SATA and SCSI
device shutdown properly, also?

Halt needing to power down disks is just a workaround for a broken kernel,
really.  It won't work always, it is easy to see a lot of border conditions
on this patch.  The first one is the extremely stupid idea that there might
be only sda..sdh in a system, and the other is that the devices are named
/dev/sd* in the first place (not true, depends on local admin config for
udev).

I have a patch from SuSE for kernel SATA device powerdown, that should be
easy for the kernel people to extend for the entire SCSI chain, if they want
to.

What do you guys think?

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh