Integrate hdaps_protect for hard disk protection

Bug #256817 reported by Jaime
16
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: linux-image

Hi,

I have a Thinkpad T60 with integrated accelerometers, fully supported in Hardy 64 bits (8.04.1) by using hdaps_ec.ko (tp_smapi)

The problem is that the kernel is not patched to also nable /sys/block/sda/queue/protect in order to park heads while moving the laptop (I think thats the original intention for hdaps_ec, apart for games).

Currently to enable this I have to manually patch the kernel and rebuild the image and modules, using the attached patches.

It would be interesting to include it in a future update.

Revision history for this message
Jaime (jaime-lopez) wrote :
Revision history for this message
Jaime (jaime-lopez) wrote :
Revision history for this message
Friedemann Schorer (friedemann-schorer) wrote :

The latest protect patch as well as some comments for the release can be found on the net, e.g. here:

http://news.gmane.org/group/gmane.linux.drivers.hdaps.devel/last=/force_load=t

I strongly support the inclusion of the disk protect patch!

Thanks in advance for your work!

Revision history for this message
Jaime (jaime-lopez) wrote :

The issue is BIG. During the last month I managed to break two hard disks and could be related with the lack of support for this, and have been seriously thinking about going back to Windows only for this reason.

Finally I've managed to rebuild kernel packages patched for hdaps parking support and now is working OK. I left documented steps in http://www.thinkwiki.org/wiki/Installing_Ubuntu_8.04_(Hardy_Heron)_on_a_ThinkPad_T60#Active_Protection_System_.28Reduced_Power_Version.29 and uploaded some binary debs (amd64) in megaupload.com

I hope kernel developers take note and include this in further releases.

Revision history for this message
Peter Meiser (meiser79) wrote :

The disk shock protection support will be included in 2.6.28.

Elias Oltmanns, the author of this patch, backported his patch to 2.6.27-rc8. It can be found here: http://article.gmane.org/gmane.linux.drivers.hdaps.devel/1379

But as the sysfs interface changed, a new hdapsd is needed. There's already a request to add it to the Debian archive: http://mentors.debian.net/debian/pool/main/h/hdapsd/

Would be great to see it in Intrepid.

Revision history for this message
Jaime (jaime-lopez) wrote :

In Ubuntu Intrepid, hdaps_ec.ko is gone, only hdaps.ko which is not compatible with my thinkpads (T60 & X61). Yep. Great.

Revision history for this message
Miguel Martinez (el-quark) wrote :

Hi Jaime,

You may try building tp_smapi from source. It requires installing linux-source, unpacking the tarball at /usr/src and then making a symlink to /lib/modules/$(uname -r)/source

More detailed instructions here:
http://www.thinkwiki.org/wiki/Tp_smapi#Installation_on_older_Ubuntu.2FDebian

Link to the latest tp_smapi:
http://sourceforge.net/project/showfiles.php?group_id=1212

You should be more lucky than I am, since tp_smapi currently doesn't work on Centrino 2 thinkpads (x200s, T400 and T500 confirmed)

Revision history for this message
Miguel Martinez (el-quark) wrote :

By the way, bug #297213 was opened due to missing hdaps_ec in Intrepid kernels.

Revision history for this message
Jaime (jaime-lopez) wrote : Re: [Bug 256817] Re: Integrate hdaps_protect for hard disk protection

Ok, thanks for the tip I'll try ASAP

(Gracias, Miguel...!)

On Sat, Dec 13, 2008 at 9:04 PM, Miguel Martinez <email address hidden> wrote:

> By the way, bug #297213 was opened due to missing hdaps_ec in Intrepid
> kernels.
>
> --
> Integrate hdaps_protect for hard disk protection
> https://bugs.launchpad.net/bugs/256817
> You received this bug notification because you are a direct subscriber
> of the bug.
>

Revision history for this message
Jaime (jaime-lopez) wrote :

Building tp_smapi is not a solution for X61s (Centrino 2 too)
Is there any hdaps.ko equivalent available for Centrino 2 Thinkpads?

Revision history for this message
Leann Ogasawara (leannogasawara) wrote :

[This is an automated message. Apologies if it has reached you inappropriately.]

This bug was reported against the linux-meta package when it likely should have been reported against the linux package instead. We are automatically transitioning this to the linux kernel package so that the appropriate teams are notified and made aware of this issue. Thanks.

affects: linux-meta (Ubuntu) → linux (Ubuntu)
Revision history for this message
Chris Jones (cmsj) wrote :

Is this bug still relevant? Hasn't the disk head protection stuff been merged upstream in the latest kernels? (and thus karmic)

tags: added: review-request
Revision history for this message
Peter Meiser (meiser79) wrote :

The functionality is merged upstream since kernel 2.6.28.

Changed in linux (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.