Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754727Ab1DOPgm (ORCPT ); Fri, 15 Apr 2011 11:36:42 -0400 Received: from ironport2-out.teksavvy.com ([206.248.154.181]:54374 "EHLO ironport2-out.pppoe.ca" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754497Ab1DOPgk (ORCPT ); Fri, 15 Apr 2011 11:36:40 -0400 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: ApIBADxlqE1Ld/sX/2dsb2JhbAAMhEOrGqoSkSiBKYNNeASUKw X-IronPort-AV: E=Sophos;i="4.64,219,1301889600"; d="scan'208";a="106518292" Message-ID: <4DA86606.308@teksavvy.com> Date: Fri, 15 Apr 2011 11:36:38 -0400 From: Mark Lord User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-GB; rv:1.9.2.15) Gecko/20110303 Thunderbird/3.1.9 MIME-Version: 1.0 To: Bruce Stenning CC: "linux-kernel@vger.kernel.org" , "linux-ide@vger.kernel.org" , Justin Maggard , Tejun Heo Subject: Re: sata_mv port lockup on hotplug (kernel 2.6.38.2) References: <4D9CD275.9000002@teksavvy.com> <4D9FACC9.7020200@teksavvy.com> <4DA5BCF3.5080205@teksavvy.com> <4DA79134.1010508@teksavvy.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1328 Lines: 39 On 11-04-15 09:21 AM, Bruce Stenning wrote: .. > sata_mv 0000:00:04.0: Gen-II 32 slots 4 ports SCSI mode IRQ via INTx .. > > I had noticed occasional messages similar to the following earlier in the week: > > ata4.00: limiting speed to UDMA/100:PIO4 Ugh. Tejun: that's just plain silly. Why would we EVER EVER EVER fall back to "PIO" for SATA ??? > I am building with CONFIG_ARCH_SUPPORTS_MSI=y, but also with CONFIG_PCI_MSI > not set. > > With all this discussion of SATA link speed, I ought to say that we limit our > SATA links to 1.5Gbps with the following kernel parameter: > > libata.force=1.5g Mmm... I suppose it could be getting stuck at 1.5g, and not transitioning back to "auto detect" or something. That part of sata_mv involves a bit of voo-doo, because I was reluctant to cut/paste the 200 lines or so of generic code just for the sake of changing five lines. Perhaps the generic code has changed since this was originally working, and now no longer always works for sata_mv.. I suppose you have a good reason for using "libata.force=1.5g" there? Cheers -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/