All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Devera <devik@cdi.cz>
To: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: David Woodhouse <dwmw2@infradead.org>,
	linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org,
	linux-scsi@vger.kernel.org, lkosewsk@gmail.com,
	Jeff Garzik <jgarzik@pobox.com>
Subject: Re: [PATCH 2.6.15.4 rel.2 1/1] libata: add hotswap to sata_svw
Date: Wed, 29 Nov 2006 03:05:32 +0100	[thread overview]
Message-ID: <456CEAEC.6010802@cdi.cz> (raw)
In-Reply-To: <1164758483.5350.113.camel@localhost.localdomain>

Benjamin Herrenschmidt wrote:
> On Tue, 2006-11-28 at 23:22 +0000, David Woodhouse wrote:
>> On Thu, 2006-02-16 at 16:09 +0100, Martin Devera wrote:
>>> From: Martin Devera <devik@cdi.cz>
>>>
>>> Add hotswap capability to Serverworks/BroadCom SATA controlers. The
>>> controler has SIM register and it selects which bits in SATA_ERROR
>>> register fires interrupt.
>>> The solution hooks on COMWAKE (plug), PHYRDY change and 10B8B decode 
>>> error (unplug) and calls into Lukasz's hotswap framework.
>>> The code got one day testing on dual core Athlon64 H8SSL Supermicro 
>>> MoBo with HT-1000 SATA, SMP kernel and two CaviarRE SATA HDDs in
>>> hotswap bays.
>>>
>>> Signed-off-by: Martin Devera <devik@cdi.cz>
>> What became of this?
> 
> I might be to blame for not testing it... The Xserve I had on my desk
> was too noisy for most of my co-workers so I kept delaying and forgot
> about it.... 
> 
> Also the Xserve I have only has one disk, which makes hotplug testing a
> bit harder :-)

Unfortunately my box with ht1000 is already deployed. Another similar one should
arrive soon so that I'll retest it.
Just now I've VIA based mobo here - and hotswap is NOT working with it ..

Martin

      reply	other threads:[~2006-11-29  2:05 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-02-16 15:09 [PATCH 2.6.15.4 rel.2 1/1] libata: add hotswap to sata_svw Martin Devera
2006-02-16 22:41 ` Benjamin Herrenschmidt
2006-11-28 23:22 ` David Woodhouse
2006-11-29  0:01   ` Benjamin Herrenschmidt
2006-11-29  2:05     ` Martin Devera [this message]

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=456CEAEC.6010802@cdi.cz \
    --to=devik@cdi.cz \
    --cc=benh@kernel.crashing.org \
    --cc=dwmw2@infradead.org \
    --cc=jgarzik@pobox.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=lkosewsk@gmail.com \
    /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.