All of lore.kernel.org
 help / color / mirror / Atom feed
From: Xavier <shiningxc@gmail.com>
To: Tejun Heo <tj@kernel.org>
Cc: linux-ide@vger.kernel.org, Kel Modderman <kel@otaku42.de>
Subject: Re: storage fixup laptop model dependent ?
Date: Fri, 18 Dec 2009 14:22:58 +0100	[thread overview]
Message-ID: <91752840912180522h32077795yba6180d6b839af64@mail.gmail.com> (raw)
In-Reply-To: <4B2712FE.10905@kernel.org>

On Tue, Dec 15, 2009 at 5:39 AM, Tejun Heo <tj@kernel.org> wrote:
> Hello,
>
> On 12/14/2009 09:29 PM, Xavier wrote:
>>> I am just wondering : does it really matter in which laptop that disk
>>> is used ? As laptop disk can be changed/replaced easily, this also
>>> sounds strange to me, but there might be a good reason I am missing :)
>
> OEMs sometimes load specialized firmwares to drives and BIOS may
> configure APM differently according to drive model, so it kind of
> matters.
>

Ah ok, good to know.

>> Was it the right place for this information ?
>> I added the two people who committed to storage-fixup git, just in case.
>
> storage-fixup is at best a stop-gap measure until something better and
> more intelligent comes along.  It might be able to serve as
> documentation later on too.  I don't think it would be wise to
> configure APM to certain value after matching only the drive model.
> That's too wide.  A good solution would be...
>
> * Build database of load cycle limits and useable APM values on drive
>  models.
>
> * Monitor load cycle count by smart commands and if it continues to
>  increase at an excessive rate, warn the user and configure higher
>  APM value.
>

Yes, that sounds quite good and reasonable to me.

> If you replaced the drive yourself, putting hdparm command in one of
> boot scripts should do it for now.  :-(
>

Well for now, I just edited storage-fixup.conf [1] but I could indeed
just run hdparm directly.

Since this is not the original disk, you think it does not deserve
being in upstream storage-fixup.conf as an additional rule [2] ?
Note that the original disk was under warranty and that the new wd
drive was shipped by dell.

[1]
rule dell-xps-m1530
dmi system-manufacturer		Dell Inc.
dmi system-product-name		XPS M1530
#ata model			SAMSUNG HM*I
ata model			WDC WD*BEVT*
act hdparm -B 254 $DEV

[2]
rule dell-xps-m1530-2
dmi system-manufacturer		Dell Inc.
dmi system-product-name		XPS M1530
ata model			WDC WD*BEVT*
act hdparm -B 254 $DEV

  reply	other threads:[~2009-12-18 13:23 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-03 13:33 storage fixup laptop model dependent ? Xavier
2009-12-14 12:29 ` Xavier
2009-12-15  4:39   ` Tejun Heo
2009-12-18 13:22     ` Xavier [this message]
2009-12-21  4:34       ` Tejun Heo
2009-12-22  0:43         ` Robert Hancock
2010-01-11  8:11         ` Tejun Heo
2010-02-17 23:30 ` Xavier Chantry

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=91752840912180522h32077795yba6180d6b839af64@mail.gmail.com \
    --to=shiningxc@gmail.com \
    --cc=kel@otaku42.de \
    --cc=linux-ide@vger.kernel.org \
    --cc=tj@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.