Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261670AbVCVTFm (ORCPT ); Tue, 22 Mar 2005 14:05:42 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261661AbVCVTFm (ORCPT ); Tue, 22 Mar 2005 14:05:42 -0500 Received: from stat16.steeleye.com ([209.192.50.48]:56543 "EHLO hancock.sc.steeleye.com") by vger.kernel.org with ESMTP id S261659AbVCVTFc (ORCPT ); Tue, 22 Mar 2005 14:05:32 -0500 Subject: Re: [PATCH] - Fusion-MPT much faster as module From: James Bottomley To: "Moore, Eric Dean" Cc: linux-kernel , SCSI Mailing List , Andrew Morton , kenneth.w.chen@intel.com In-Reply-To: <91888D455306F94EBD4D168954A9457C01AEB0A8@nacos172.co.lsil.com> References: <91888D455306F94EBD4D168954A9457C01AEB0A8@nacos172.co.lsil.com> Content-Type: text/plain Date: Tue, 22 Mar 2005 13:05:23 -0600 Message-Id: <1111518323.5520.60.camel@mulgrave> Mime-Version: 1.0 X-Mailer: Evolution 2.0.4 (2.0.4-1) Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1088 Lines: 26 On Tue, 2005-03-22 at 11:40 -0700, Moore, Eric Dean wrote: > History on this: > Between the 3.01.16 and 3.01.18, we introduced new method > to passing command line options to the driver. Some of the > command line options are used for fine tuning dv(domain > validation) in the driver. By accident, these command line options were > wrapped around #ifdef MODULE in the 3.01.18 version of the driver. > What this meant is when the driver is compiled built-in the kernel, > the optimal settings for dv were ignored, thus poor performance. OK, I'll add this to the queue. Could I just point out that if your driver actually printed the results of negotiation, this would have been an awful lot easier to debug. Additionally, if you used the SPI transport class domain validation, the issue wouldn't have arisen in the first place. James - 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/