* Re: [Scst-devel] [ANNOUNCE]: ConfigFS enabled Generic Target Mode andiSCSI Target Stack on v2.6.27-rc7
[not found] ` <48E4FE98.7050409@vlnb.net>
@ 2008-10-02 21:47 ` Nicholas A. Bellinger
0 siblings, 0 replies; only message in thread
From: Nicholas A. Bellinger @ 2008-10-02 21:47 UTC (permalink / raw)
To: Vladislav Bolkhovitin
Cc: Sufficool, Stanley, scst-devel, Linux-iSCSI.org Target Dev, LKML,
linux-scsi
On Thu, 2008-10-02 at 21:02 +0400, Vladislav Bolkhovitin wrote:
> Sufficool, Stanley wrote:
> >
> >> -----Original Message-----
> >> From: Nicholas A. Bellinger [mailto:nab@linux-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
^ permalink raw reply [flat|nested] only message in thread