From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mout.gmx.net (mout.gmx.net [212.227.17.21]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 9A0C91A0BF5 for ; Fri, 22 May 2015 21:40:00 +1000 (AEST) Date: Fri, 22 May 2015 13:33:34 +0200 From: Sebastian Herbszt To: Alexey Kardashevskiy Cc: james.smart@emulex.com, linuxppc-dev@lists.ozlabs.org, Benjamin Herrenschmidt , Paul Mackerras , "James E . J . Bottomley" , linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, Sebastian Herbszt Subject: Re: [PATCH kernel] commit 4fbdf9cb ("lpfc: Fix for lun discovery issue with saturn adapter.") Message-ID: <20150522133334.000064c9@localhost> In-Reply-To: <555C0595.9060507@ozlabs.ru> References: <1430209582-23925-1-git-send-email-aik@ozlabs.ru> <5548E2C1.3070209@emulex.com> <20150505234659.00003ab7@localhost> <555C0595.9060507@ozlabs.ru> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Alexey Kardashevskiy wrote: > On 05/06/2015 07:46 AM, Sebastian Herbszt wrote: > > James Smart wrote: > >> > >> Reviewed-By: James Smart > >> > >> > >> Alexey, Sebastian, > >> > >> Yes - this section needs to be reverted. This patch is good. > >> > >> -- james s > > > > Reviewed-by: Sebastian Herbszt > > > Unfortunately, just this revert is not enough, it fixed one of my machines > but I have another machine with "Emulex Corporation Saturn-X: LightPulse > Fibre Channel Host Adapter (rev 03)" which does not boot - the booting > process stops at "[ OK ] Reached target Basic System." > > Any quick idea what else to revert between 4.0 and current upstream before > I dig further? Thanks! Is the other machine also POWER7? I tested the revert on x86 and it fixed my issue: 4.1-rc4 (lpfc 10.5.0.0) - broken 4.0 + lpfc 10.5.0.0 - broken 4.1-rc4 (lpfc 10.5.0.0) + revert - ok 4.0 + lpfc 10.5.0.0 + revert - ok Sebastian