All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Milburn <dmilburn@redhat.com>
To: "Frank D. Cringle" <fdc-lide@cutlass.ping.de>
Cc: linux-ide@vger.kernel.org
Subject: Re: AHCI backplane errors in 2.6.27
Date: Wed, 19 Nov 2008 13:17:49 -0600	[thread overview]
Message-ID: <4924665D.4090300@redhat.com> (raw)
In-Reply-To: <87abbw9hm1.fsf@cutlass.ping.de>

[-- Attachment #1: Type: text/plain, Size: 1160 bytes --]

Frank D. Cringle wrote:
> Lazy <lazy404@gmail.com> writes:
>> When booting 2.6.27-rc1 kernel on out intel SR1530HSH ( Quad core Xeon X3320)
>> Sata controller is ICH9r running in AHCI mode with newest bios.
>>
>> After loading ahci driver backplane leds go amber which indicates a
>> non fatal error according to my intel resseller.
> 
> I see the same effect on a "FUJITSU SIEMENS PRIMERGY TX150 S6" which
> also has a ICH9r, running with the fake megaraid disabled in the bios.
> This has 4 hotswap drive bays. When I boot from drive 0, the remaining
> drives all blink their orange fault led.

ahci_sw_activity_blink should only be changing the state of
the activity led only when emp->activity has changed.

 From the sata ahci spec for Byte 2-3 of the led message

The LED bit locations are:
        Bits 2:0 - Activity LED (may be driven by hardware)
        Bits 5:3 - Vendor Specific LED (e.g. locate)
        Bits 8:6 - Vendor Specific LED (e.g. fault)
        Bits 15:9 - Reserved

Does this patch (against 2.6.27) make a difference?

Thanks,
David

> 
> It is possible to suppress the blinking with the boot parameter
> ahci.ahci_em_messages=0.
> 


[-- Attachment #2: linux-2.6-ahci-blink.patch --]
[-- Type: text/x-patch, Size: 524 bytes --]

diff -Nurp linux-2.6.27.orig/drivers/ata/ahci.c linux-2.6.27/drivers/ata/ahci.c
--- linux-2.6.27.orig/drivers/ata/ahci.c	2008-10-09 18:13:53.000000000 -0400
+++ linux-2.6.27/drivers/ata/ahci.c	2008-11-19 12:56:04.000000000 -0500
@@ -1220,7 +1220,7 @@ static void ahci_sw_activity_blink(unsig
 	unsigned long led_message = emp->led_state;
 	u32 activity_led_state;
 
-	led_message &= 0xffff0000;
+	led_message &= 0x00070000;
 	led_message |= ap->port_no | (link->pmp << 8);
 
 	/* check to see if we've had activity.  If so,

  reply	other threads:[~2008-11-19 19:18 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-18 15:33 AHCI backplane errors in 2.6.27 Lazy
2008-11-19  9:42 ` Frank D. Cringle
2008-11-19 19:17   ` David Milburn [this message]
2008-11-19 20:21     ` Frank D. Cringle

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=4924665D.4090300@redhat.com \
    --to=dmilburn@redhat.com \
    --cc=fdc-lide@cutlass.ping.de \
    --cc=linux-ide@vger.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.