[<prev] [next>] [day] [month] [year] [list]
Message-Id: <1222984066.15764.131.camel@haakon2.linux-iscsi.org>
Date: Thu, 02 Oct 2008 14:47:46 -0700
From: "Nicholas A. Bellinger" <nab@...ux-iscsi.org>
To: Vladislav Bolkhovitin <vst@...b.net>
Cc: "Sufficool, Stanley" <ssufficool@....sbcounty.gov>,
scst-devel@...ts.sourceforge.net,
"Linux-iSCSI.org Target Dev"
<linux-iscsi-target-dev@...glegroups.com>,
LKML <linux-kernel@...r.kernel.org>,
linux-scsi <linux-scsi@...r.kernel.org>
Subject: Re: [Scst-devel] [ANNOUNCE]: ConfigFS enabled Generic Target Mode
andiSCSI Target Stack on v2.6.27-rc7
On Thu, 2008-10-02 at 21:02 +0400, Vladislav Bolkhovitin wrote:
> Sufficool, Stanley wrote:
> >
> >> -----Original Message-----
> >> From: Nicholas A. Bellinger [mailto:nab@...ux-iscsi.org]
> >> Sent: Monday, September 29, 2008 12:22 PM
> >> To: LKML; linux-scsi; Linux-iSCSI.org Target Dev; SCST-Devel; iet-dev
> >> Cc: FUJITA Tomonori; Mike Christie; Vladislav Bolkhovitin;
> >> Greg KH; Joel Becker; Jerome Martin; James Bottomley; H. Peter Anvin
> >> Subject: [Scst-devel] [ANNOUNCE]: ConfigFS enabled Generic
> >> Target Mode andiSCSI Target Stack on v2.6.27-rc7
> >>
> >>
<SNIP>
> >>
> >> target:~# tree $CONFIGFS
> >> /sys/kernel/config/
> >> `-- target
> >> |-- core
> >> | |-- fileio_0
> >> | | |-- file_object
> >> | | | |-- dev_control
> >> | | | |-- dev_enable
> >> | | | `-- dev_info
> >> | | `-- hba_info
> >> | |-- iblock_0
> >> | | |-- hba_info
> >> | | `-- lvm_test0
> >> | | |-- dev_control
> >> | | |-- dev_enable
> >> | | `-- dev_info
> >> | |-- pscsi_0
> >> | | |-- hba_info
> >> | | `-- sdd
> >> | | |-- dev_control
> >> | | |-- dev_enable
> >> | | `-- dev_info
> >> | |-- rd_dr_0
> >> | | |-- hba_info
> >> | | `-- ramdisk0
> >> | | |-- dev_control
> >> | | |-- dev_enable
> >> | | `-- dev_info
> >> | `-- rd_mcp_0
> >> | |-- hba_info
> >> | `-- ramdisk0
> >> | |-- dev_control
> >> | |-- dev_enable
> >> | `-- dev_info
> >> |-- iscsi
> >> | |-- iqn.2003-01.org.linux-iscsi.target.i686:sn.e475ed6fcdd0
> >> | | `-- tpgt_1
> >> | | |-- lun
> >> | | | |-- lun_0
> >> | | | | |-- lio_west_port ->
> >> ../../../../../../target/core/iblock_0/lvm_test0
> >> | | | | |-- port_control
> >> | | | | `-- port_info
> >> | | | `-- lun_1
> >> | | | |-- lio_east_port ->
> >> ../../../../../../target/core/pscsi_0/sdd
> >> | | | |-- port_control
> >> | | | `-- port_info
> >> | | |-- np
> >> | | | `-- 172.16.201.137:3260
> >> | | | `-- portal_info
> >> | | |-- tpg_control
> >> | | `-- tpg_enable
> >> | `-- lio_version
> >> `-- version
> >>
> >> 22 directories, 29 files
> >>
> >> --nab
> >
> > My 2 cents as a user of the current SCST infrastructure and with no clue
> > how ConfigFS works:
> >
> > If ConfigFS is able to isolate the settings and "switches" to individual
> > directories, do we really need the prefixes on the switches?
> >
> > dev_control/dev_enable/dev_info = control/enable/info since we know we
> > are in a device config directory. Same for port, hba, etc?
> >
> > This would IMHO enable simpler development of an all in one user tool
> > for managing all these switches and configs.
>
> Yes, I think, you're right. Additionally, no existing entries in sysfs
> has prefixes like that. So, it would be better to remove them.
>
Sounds good. Here is the commit:
http://git.kernel.org/?p=linux/kernel/git/nab/lio-core-2.6.git;a=commit;h=374e8ace84ca58490be22d881f017d48c6742b50
Thanks for your comments!
--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