Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757614Ab1CBVi7 (ORCPT ); Wed, 2 Mar 2011 16:38:59 -0500 Received: from mail.linux-iscsi.org ([67.23.28.174]:32837 "EHLO linux-iscsi.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754924Ab1CBVi6 (ORCPT ); Wed, 2 Mar 2011 16:38:58 -0500 Subject: Re: [RFC 12/12] iscsi-target: Add Makefile/Kconfig and update TCM top level From: "Nicholas A. Bellinger" To: Randy Dunlap Cc: linux-scsi , linux-kernel , Christoph Hellwig , Mike Christie , Hannes Reinecke , FUJITA Tomonori , James Bottomley , Boaz Harrosh , Stephen Rothwell , Douglas Gilbert In-Reply-To: <20110301223251.11c95eed.rdunlap@xenotime.net> References: <1299036841-23677-1-git-send-email-nab@linux-iscsi.org> <1299036841-23677-13-git-send-email-nab@linux-iscsi.org> <20110301223251.11c95eed.rdunlap@xenotime.net> Content-Type: text/plain Date: Wed, 02 Mar 2011 13:32:11 -0800 Message-Id: <1299101531.5401.313.camel@haakon2.linux-iscsi.org> Mime-Version: 1.0 X-Mailer: Evolution 2.22.3.1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3035 Lines: 85 On Tue, 2011-03-01 at 22:32 -0800, Randy Dunlap wrote: > On Tue, 1 Mar 2011 19:34:01 -0800 Nicholas A. Bellinger wrote: > > > From: Nicholas Bellinger > > > > Add Makefile/Kconfig and update drivers/target/[Makefile,Kconfig] > > to include the fabric module. > > > > igned-off-by: Nicholas A. Bellinger > > --- > > drivers/target/Kconfig | 1 + > > drivers/target/Makefile | 1 + > > drivers/target/iscsi/Kconfig | 17 +++++++++++++++++ > > drivers/target/iscsi/Makefile | 20 ++++++++++++++++++++ > > 4 files changed, 39 insertions(+), 0 deletions(-) > > create mode 100644 drivers/target/iscsi/Kconfig > > create mode 100644 drivers/target/iscsi/Makefile > > > > diff --git a/drivers/target/Kconfig b/drivers/target/Kconfig > > index 387d293..798749a 100644 > > --- a/drivers/target/Kconfig > > +++ b/drivers/target/Kconfig > > @@ -30,5 +30,6 @@ config TCM_PSCSI > > passthrough access to Linux/SCSI device > > > > source "drivers/target/tcm_loop/Kconfig" > > +source "drivers/target/iscsi/Kconfig" > > > > endif > > diff --git a/drivers/target/Makefile b/drivers/target/Makefile > > index 60028fe..b038b7d 100644 > > --- a/drivers/target/Makefile > > +++ b/drivers/target/Makefile > > @@ -24,3 +24,4 @@ obj-$(CONFIG_TCM_PSCSI) += target_core_pscsi.o > > > > # Fabric modules > > obj-$(CONFIG_TCM_LOOP_FABRIC) += tcm_loop/ > > +obj-$(CONFIG_ISCSI_TARGET) += iscsi/ > > diff --git a/drivers/target/iscsi/Kconfig b/drivers/target/iscsi/Kconfig > > new file mode 100644 > > index 0000000..d1eaec4 > > --- /dev/null > > +++ b/drivers/target/iscsi/Kconfig > > @@ -0,0 +1,17 @@ > > +config ISCSI_TARGET > > + tristate "Linux-iSCSI.org iSCSI Target Mode Stack" > > + select CRYPTO > > + select CRYPTO_CRC32C > > + select CRYPTO_CRC32C_INTEL > > CRYPTO_CRC32C_INTEL depends on X86. so is ISCSI_TARGET only for X86, > or is this kconfig just mucked up? > > Hi Randy, The kernel code itself that is specific to using the SSE v4.2 instruction for CRC32C offload are using #ifdef CONFIG_X86 stubs in iscsi_target_login.c:iscsi_login_setup_crypto(), and !CONFIG_X86 will default to using the unoptimized 1x8 slicing soft CRC32C code. This particular piece of logic has been tested on powerpc and arm and is funcitoning as expected from the kernel level using the arch independent soft code. On the kbuild side, I do see the following warning on !CONFIG_X86: warning: (LIO_TARGET) selects CRYPTO_CRC32C_INTEL which has unmet direct dependencies (CRYPTO && X86) I looking at trying to fix this at one point, but was unable to determine a method for adding a CONFIG_$ARCH condition to an individual 'select BAR' section of 'config FOO'.. How would you recommend handling this case..? Thanks, --nab -- 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/