From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Bottomley Subject: Re: LSI MegaRAID not recognised correctly in 64-bit. 2.6.29.6 Date: Mon, 21 Sep 2009 15:34:04 +0000 Message-ID: <1253547244.4579.4.camel@mulgrave.site> References: <4AB48254.4000404@attglobal.net> <20090919165404D.fujita.tomonori@lab.ntt.co.jp> <4AB4F792.8020308@attglobal.net> <20090921225015B.fujita.tomonori@lab.ntt.co.jp> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: Received: from cantor2.suse.de ([195.135.220.15]:54546 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751092AbZIUPeI (ORCPT ); Mon, 21 Sep 2009 11:34:08 -0400 In-Reply-To: <20090921225015B.fujita.tomonori@lab.ntt.co.jp> Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: FUJITA Tomonori Cc: stunnel@attglobal.net, matthew@wil.cx, linux-scsi@vger.kernel.org, bo.yang@lsi.com On Mon, 2009-09-21 at 22:51 +0900, FUJITA Tomonori wrote: > On Sat, 19 Sep 2009 08:24:02 -0700 > Eddie wrote: > > > FUJITA Tomonori wrote: > > > Thanks, now the problem is clear; the driver uses 64bit dma_mask but > > > the hardware is not capable of 64bit dma according to James. Seems > > > that hardware lies about the dma capability. > > > > > > Can you try the following patch? > > > > > > > > > diff --git a/drivers/scsi/megaraid/megaraid_mbox.c b/drivers/scsi/megaraid/megaraid_mbox.c > > > index 234f0b7..9ead856 100644 > > > --- a/drivers/scsi/megaraid/megaraid_mbox.c > > > +++ b/drivers/scsi/megaraid/megaraid_mbox.c > > > @@ -888,6 +888,8 @@ megaraid_init_mbox(adapter_t *adapter) > > > if (((magic64 == HBA_SIGNATURE_64_BIT) && > > > ((adapter->pdev->subsystem_device != > > > PCI_SUBSYS_ID_MEGARAID_SATA_150_6) && > > > + (adapter->pdev->device != > > > + PCI_DEVICE_ID_AMI_MEGARAID3) && > > > (adapter->pdev->subsystem_device != > > > PCI_SUBSYS_ID_MEGARAID_SATA_150_4))) || > > > (adapter->pdev->vendor == PCI_VENDOR_ID_LSI_LOGIC && > > > > > > > > OK, that appears to have done the trick. > > Thanks for the confirmation. > > James, here's a patch in the proper format. Got it ... I'd like to wait on confirmation from LSI on this ... I actually said the Megaraid3 wasn't 64 bit based on the 64 bit exception table, but if it really has the firmware marker that says it's 64 bit, we should make sure the manufacturers agree that it needs to be added to the exception table. James