From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Lord Subject: Re: sata_mv & pmp support Date: Wed, 16 Apr 2008 08:37:46 -0400 Message-ID: <4805F31A.3010105@rtr.ca> References: <47FA4AFF.2010209@gmail.com> <48003C04.4040508@garzik.org> <48023462.1050305@rtr.ca> <4803C429.6070201@rtr.ca> <4803C850.9010901@rtr.ca> <48040DC8.1050907@gmail.com> <48050904.1040808@rtr.ca> <48050CA1.5010800@rtr.ca> <48052E08.5040306@gmail.com> <480531DB.3020602@rtr.ca> <48055E7C.1060004@rtr.ca> <48056016.6020900@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from rtr.ca ([76.10.145.34]:1320 "EHLO mail.rtr.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755678AbYDPMhs (ORCPT ); Wed, 16 Apr 2008 08:37:48 -0400 In-Reply-To: <48056016.6020900@gmail.com> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Tejun Heo Cc: Jeff Garzik , IDE/ATA development list Tejun Heo wrote: > Mark Lord wrote: .. >> I've just noticed that, if I unplug/replug the host-side SATA cable >> on the Sil3726 PM, and *then* "insmod sata_mv", the PM is not found. >> >> If I then simply do "rmmod sata_mv ; insmod sata_mv", then the PM is >> found. >> >> This happens with the original hardreset code from Jeff, >> as well as the updated sata_mv code that I've posted earlier. >> So it is not anything I've broken (recently :) ). >> >> Very strange.. I wonder if it also happens with the Marvell PM, >> except I don't have one here to test with at the moment. >> >> Ever heard of anything strange like that? > > IIRC, ICH8 ahci + 4726 fails the initial reset sequence because the PMP > comes up after the hardreset code times out, which triggers another EH > iteration which works out fine. If the problem is caused by the PMP not > responding fast enough, it should cause a hotplug event afterwards. > Maybe there's a race condition where hotplug events can be lost? .. Ah.. perhaps that could be it. I've temporarily disabled the hotplug detection in sata_mv, because it is broken at present and interfered with PMP support during earlier testing. It is next on the list of things to fix (this week, hopefully) as part of the IRQ/EH handling overhaul in sata_mv. But strange that the driver always finds it on a simple rmmod/insmod reload. ??