[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1300229797.28255.317.camel@haakon2.linux-iscsi.org>
Date: Tue, 15 Mar 2011 15:56:37 -0700
From: "Nicholas A. Bellinger" <nab@...ux-iscsi.org>
To: Christoph Hellwig <hch@....de>
Cc: linux-scsi <linux-scsi@...r.kernel.org>,
linux-kernel <linux-kernel@...r.kernel.org>,
James Bottomley <James.Bottomley@...senPartnership.com>,
Mike Christie <michaelc@...wisc.edu>,
Hannes Reinecke <hare@...e.de>,
FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>,
Boaz Harrosh <bharrosh@...asas.com>,
Stephen Rothwell <sfr@...b.auug.org.au>,
Douglas Gilbert <dgilbert@...erlog.com>
Subject: Re: [RFC-v2 07/12] iscsi-target: Add CHAP Authentication support
using libcrypto
On Tue, 2011-03-15 at 11:24 +0100, Christoph Hellwig wrote:
> On Mon, Mar 14, 2011 at 04:57:04AM -0700, Nicholas A. Bellinger wrote:
> > From: Nicholas Bellinger <nab@...ux-iscsi.org>
> >
> > This patch adds support for libcrypto md5 based iSCSI CHAP authentication
> > support for iscsi_target_mod. This includes support for mutual and one-way
> > NodeACL authentication for SessionType=Normal and SessionType=Discovery
> > via /sys/kernel/config/target/iscsi.
> >
> > Signed-off-by: Nicholas A. Bellinger <nab@...ux-iscsi.org>
> > ---
> > drivers/target/iscsi/iscsi_auth_chap.c | 502 ++++++++++++++++++++++++++++++++
> > drivers/target/iscsi/iscsi_auth_chap.h | 33 ++
> > 2 files changed, 535 insertions(+), 0 deletions(-)
> > create mode 100644 drivers/target/iscsi/iscsi_auth_chap.c
> > create mode 100644 drivers/target/iscsi/iscsi_auth_chap.h
>
> Why aren't these files named iscsi_target_* like most others? If you prefix
> all files in a module please do it consistently, or alternatively just
> drop the prefix altogether.
>
This where shared amounst initiator code at one point.. Renaming to
iscsi_target_auth now.
Thanks,
--nab
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists