From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boaz Harrosh Subject: Re: [PATCH 17/18] osd: Kconfig file for in-tree builds Date: Wed, 07 Jan 2009 10:48:42 +0200 Message-ID: <49646C6A.3020304@panasas.com> References: <49636507.8060808@panasas.com> <1231252344-24792-1-git-send-email-bharrosh@panasas.com> <4963978D.8060001@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Return-path: Received: from gw-ca.panasas.com ([66.104.249.162]:18633 "EHLO laguna.int.panasas.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751856AbZAGIsr (ORCPT ); Wed, 7 Jan 2009 03:48:47 -0500 In-Reply-To: <4963978D.8060001@oracle.com> Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: Randy Dunlap Cc: James Bottomley , linux-scsi , open-osd ml , Andrew Morton Randy Dunlap wrote: > Boaz Harrosh wrote: >> Kconfig file for the drivers/scsi/osd subdirectory. >> Adds the following config items: >> config SCSI_OSD_INITIATOR >> config SCSI_OSD_ULD >> config SCSI_OSD_DPRINT_SENSE >> config SCSI_OSD_DEBUG >> >> Signed-off-by: Boaz Harrosh >> Reviewed-by: Benny Halevy >> --- >> drivers/scsi/osd/Kconfig | 53 ++++++++++++++++++++++++++++++++++++++++++++++ >> 1 files changed, 53 insertions(+), 0 deletions(-) >> create mode 100644 drivers/scsi/osd/Kconfig >> >> diff --git a/drivers/scsi/osd/Kconfig b/drivers/scsi/osd/Kconfig >> new file mode 100644 >> index 0000000..861b5ce >> --- /dev/null >> +++ b/drivers/scsi/osd/Kconfig >> @@ -0,0 +1,53 @@ >> +# >> +# Kernel configuration file for the OSD scsi protocol >> +# >> +# Copyright (C) 2008 Panasas Inc. All rights reserved. >> +# >> +# Authors: >> +# Boaz Harrosh >> +# Benny Halevy >> +# >> +# This program is free software; you can redistribute it and/or modify >> +# it under the terms of the GNU General Public version 2 License as >> +# published by the Free Software Foundation >> +# >> +# FIXME: SCSI_OSD_INITIATOR should select CONFIG (HMAC) SHA1 somehow. >> +# How is it done properly? > > Probably similar to how net/sctp/Kconfig does it: > > select CRYPTO > select CRYPTO_HMAC > select CRYPTO_SHA1 > It is not yet relevant, I have not submitted the security code. But thanks, that looks doable. I have a question? When you do select, you must manually select the fixture you need plus all it's dependencies and their dependencies recursively, parents first? >> +# >> + >> +config SCSI_OSD_INITIATOR >> + tristate "OSD-Initiator library" >> + depends on SCSI >> + help >> + Enable the OSD-Initiator library (libosd.ko). >> + NOTE: You must also select CRYPTO_SHA1 + CRYPTO_HMAC and their >> + dependencies > > All of the kconfig help text indentation is incorrect. :( > It should all be tab + 2 spaces. > >> + >> +config SCSI_OSD_ULD >> + tristate "OSD Upper Level driver" >> + depends on SCSI_OSD_INITIATOR >> + help >> + Build a SCSI upper layer driver that exports /dev/osdX devices >> + to user-mode for testing and controlling OSD devices. It is also >> + needed by exofs, for mounting an OSD based file system. > > OSD-based > >> + >> +config SCSI_OSD_DPRINT_SENSE >> + int "(0-2) When sense is returned, DEBUG print all sense descriptors" >> + default 1 >> + depends on SCSI_OSD_INITIATOR >> + help >> + When a CHECK_CONDITION status is returned from a target, and a >> + sense-buffer is retrieved, turning this on will dump a full >> + sense-decoding message. Setting to 2 will also print recoverable >> + errors that might be regularly returned for some filesystem >> + operations. >> + >> +config SCSI_OSD_DEBUG >> + bool "Compile All OSD modules with lots of DEBUG prints" >> + default n >> + depends on SCSI_OSD_INITIATOR >> + help >> + OSD Code is populated with lots of OSD_DEBUG(..) printouts to >> + dmesg. Enable this if you found a bug and you want to help us >> + track the problem (see also MAINTAINERS). Setting this will also >> + force SCSI_OSD_DPRINT_SENSE=2. > > Thanks Randy, my first Kconfig, sorry. Should have read Documentation. I'll post a ver2 as reply to original patch. Boaz