public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: "Nicholas A. Bellinger" <nab@linux-iscsi.org>
To: Joel Becker <Joel.Becker@oracle.com>
Cc: "Linux-iSCSI.org Target Dev"
	<linux-iscsi-target-dev@googlegroups.com>,
	linux-scsi <linux-scsi@vger.kernel.org>,
	SCST-Devel <scst-devel@lists.sourceforge.net>
Subject: ConfigFS + Target Mode Engine API discussion
Date: Wed, 10 Sep 2008 18:42:46 -0700	[thread overview]
Message-ID: <1221097366.27831.216.camel@haakon2.linux-iscsi.org> (raw)
In-Reply-To: <20080910233107.GC23864@mail.oracle.com>

On Wed, 2008-09-10 at 16:31 -0700, Joel Becker wrote:
> On Wed, Sep 10, 2008 at 03:59:07PM -0700, Nicholas A. Bellinger wrote:
> > Whoops, forgot to CC you boss. :-)
> 
> 	FYI, I'll be at both KSummit and Plumbers.
> 
> Joel
> 

Great, I look forward to catching up.. :-)

Btw, I started working on the target related configfs this past weekend,
and stuck on one particular item wrt the API of a target mode fabric
plugin with a configfs enabled target engine under
$CONFIGFS/target/$FABRIC

I am making a default group under $CONFIGFS/target/core that will be
used for the generic target engine for storage object registration and
device object tuning related to target mode export, etc.  From there, I
expect the target fabric modules that are being loaded to call the
following API routines at modprobe time from a previously allocated
struct configfs_subsystem:

extern struct config_group *target_fabric_configfs_init (void)
{
        struct configfs_subsystem *subsys;

        subsys = &target_core_subsystem[0];
        return(&subsys->su_group);
}

extern struct config_group *target_fabric_configfs_register (
        struct config_group *su_cg,
        char *name,
        struct config_item_type *fabric_cit)
{

	if (!(fabric_cg = kzalloc(struct config_group, GFP_NORMAL))) {
                printk(KERN_ERR "Unable to allocate fabric_cg\n");
                ERR_PTR(-ENOMEM);
        }

        config_group_init_type_name(fabric_cg, name, fabric_cit);

}

So I am thinking about the following question:  What would be the
preferred method for calling ct_group_ops->make_group() in order to
create the $CONFIGFS/target/$FABRIC struct config_item directly from
target_fabric_configfs_register() call?  From there, the config group
hanging off $CONFIGFS/target/$FABRIC will be fabric dependent and
providing their own groups, items, depends, from the passed *fabric_cit.
How do I "simulate" a mkdir(2) configfs -> make_group() call coming from
the fabric module itself..?   This would be assuming that both mkdir(2)
and rmdir(2) would be *NOT* be able to be called directly on
$CONFIGFS/target/$FABRIC, and only the rmmod $FABRIC_MOD will call
target_fabric_configfs_unregister() and -> drop_item() to remove the
$FABRIC plugin from $CONFIGFS/target/

The configfs enabled generic target engine will need similar
functionality to simulate mkdir(2) -> make_group() and rmdir(2) ->
drop_item() in order for autoregistration of storage objects using echo
$STORAGE_OBJECT > $CONFIGFS/target/$FABRIC/$FABRIC_ENDPOINT/lun_0 to
setup the mapping as I envision.

Thanks Joel!

--nab





       reply	other threads:[~2008-09-11  1:42 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1221087547.27831.165.camel@haakon2.linux-iscsi.org>
     [not found] ` <20080910233107.GC23864@mail.oracle.com>
2008-09-11  1:42   ` Nicholas A. Bellinger [this message]
2008-09-11  2:13     ` ConfigFS + Target Mode Engine API discussion Joel Becker
2008-09-11  4:29       ` Nicholas A. Bellinger
2008-09-11  6:38         ` Joel Becker
2008-09-11  8:58           ` Nicholas A. Bellinger
2008-09-12 16:24             ` Nicholas A. Bellinger
2008-09-12 16:30               ` Nicholas A. Bellinger
2008-09-12 20:15             ` Joel Becker
2008-09-12 22:27               ` Nicholas A. Bellinger
2008-09-13  4:49                 ` Joel Becker
2008-09-13 19:22                   ` Nicholas A. Bellinger
2008-09-14  1:56                     ` Nicholas A. Bellinger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1221097366.27831.216.camel@haakon2.linux-iscsi.org \
    --to=nab@linux-iscsi.org \
    --cc=Joel.Becker@oracle.com \
    --cc=linux-iscsi-target-dev@googlegroups.com \
    --cc=linux-scsi@vger.kernel.org \
    --cc=scst-devel@lists.sourceforge.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox