From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754717AbdGUUyV (ORCPT ); Fri, 21 Jul 2017 16:54:21 -0400 Received: from mout.kundenserver.de ([212.227.126.135]:63169 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751996AbdGUUyS (ORCPT ); Fri, 21 Jul 2017 16:54:18 -0400 From: Arnd Bergmann To: "James E.J. Bottomley" , "Martin K. Petersen" Cc: Arnd Bergmann , stable@vger.kernel.org, Hannes Reinecke , Finn Thain , Johannes Thumshirn , James Smart , linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH, RESEND] scsi: lpfc: fix linking against modular NVMe support Date: Fri, 21 Jul 2017 22:53:46 +0200 Message-Id: <20170721205401.3374331-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K0:aJQh8hkTxgsOedbPPl+h9pFO9xpoTJoONCsB+vD/zOwIWoahP9z X18cMVZi0FkJ/wnsSHJqpQ29p4rhbnERwr7zSc9xmPhj4PhBMXmX2t/nkAaJf5zqDZ74X0g LvEKdZzAKpuht1EUjUuIhYuZW1Q1TUPm4GpsCVPG+wrWI4TOIItEf6LJq+Sr0MGmaCVZY2R t877goig9g55nZgkze1/Q== X-UI-Out-Filterresults: notjunk:1;V01:K0:6IDh2NrlAsc=:hYgJD8E94t8vJf8LwtAkHl qhAuFC0OvQ9Elk51aCd9O8i/Fsbh+nk3KXB4WOJDzSdE/VfLQQrJPwSB2KrUw4piTB2QvetPO 6mhpd3t2Y0YxGhUUYS4UC5vsXIJ4/xBoHfE29fj2GQjfG3tGDpSHnV6xS8khOxX1DFecENVEM pcqra+1Q4BZ/YN96YT/8PIHgLDBhSc5n/JNcF4DYR6nSz+t3FAewQp/874qGXldu+z02UgzUf 8DnDpy5iLPL8MKf3wHScmP7x77kOUicpbdQUyMK7IAncCTcBcOQvcQU9kiMEaN9MjjhxoeARy z8UYSy7oeDOWAf96dPCFoTYFlQm7SnusrZtkE4hy0blPT3HH1wZgGUWg5Pp82rV3nCckJQOvk Ed+AuSBwSpe1mRg3q5GtXrMh3RnEf86dLSOXzMyE5dprTlpvLrEfQBoX7Vxsvq/JsJYJjoEwk e4HKRQUKynDesMJXIGwmXt/BqMKbM/LPjN/wlnJEbpiFLHBEToCmKwPSZnqxmK2f0UaKWDg7/ 2JvbSVHy86mExnv1bcoqTV+BWVRKWei1FcrML843xaGYxeKkC0mf0BcvgSHZ2GsCB4rwher3H 8Uw0rcmRtk4au67T1LDT84kyjJHbqm731r+qIA1Gf37BbZtqOhR6Uu+9KyTjmQerOpo856bvI 8erPtzlCWPZl80vluMcqCGb6LRuHNQNY3vby0Oxsz4FQjiBPHTTQJOdaM9yBKQqqSFqnvQbbG 6yCHWcVvZ0NHm+5kDF0k1DE9l82aZ0FfIfXxkA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org When LPFC is built-in but NVMe is a loadable module, we fail to link the kernel: drivers/scsi/built-in.o: In function `lpfc_nvme_create_localport': (.text+0x156a82): undefined reference to `nvme_fc_register_localport' drivers/scsi/built-in.o: In function `lpfc_nvme_destroy_localport': (.text+0x156eaa): undefined reference to `nvme_fc_unregister_remoteport' We can avoid this either by forcing lpfc to be a module, or by disabling NVMe support in this case. This implements the former. Fixes: 7d7080335f8d ("scsi: lpfc: Finalize Kconfig options for nvme") Cc: stable@vger.kernel.org Link: https://patchwork.kernel.org/patch/9636569/ Signed-off-by: Arnd Bergmann --- I originally sent the patch in March 2013, the day after the broken patch made it into Linux-next. Unfortunately there was some misunderstanding about it (James Smart thought the broken patch was the correct version), and my fix never got merged. --- drivers/scsi/Kconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig index d384f4f86c26..f4538d7a3016 100644 --- a/drivers/scsi/Kconfig +++ b/drivers/scsi/Kconfig @@ -1230,6 +1230,8 @@ config SCSI_LPFC tristate "Emulex LightPulse Fibre Channel Support" depends on PCI && SCSI depends on SCSI_FC_ATTRS + depends on NVME_TARGET_FC || NVME_TARGET_FC=n + depends on NVME_FC || NVME_FC=n select CRC_T10DIF ---help--- This lpfc driver supports the Emulex LightPulse -- 2.9.0