All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vladislav Bolkhovitin <vst@vlnb.net>
To: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Cc: nab@linux-iscsi.org, linux-scsi@vger.kernel.org,
	linux-kernel@vger.kernel.org, michaelc@cs.wisc.edu, hch@lst.de,
	hare@suse.de, James.Bottomley@suse.de, axboe@kernel.dk,
	bharrosh@panasas.com
Subject: Re: [RFC v2 05/21] tcm: Add ConfigFS subsystem backstore infrastructure
Date: Wed, 06 Oct 2010 16:13:55 +0400	[thread overview]
Message-ID: <4CAC6803.9070609@vlnb.net> (raw)
In-Reply-To: <20101006162018R.fujita.tomonori@lab.ntt.co.jp>

FUJITA Tomonori, on 10/06/2010 11:25 AM wrote:
>>> The main issue here is that the user still needs to know the $LPORT_WWPN
>>> before hand (either from looking at a sticker on the card, or via
>>> another method) in order to preform the initial TFO->fabric_make_wwn()
>>> -> $TCM_MOD_make_wwn() operation.  So what we need is a configfs attrib
>>> at the top level TCM fabric group in order to see a list of the
>>> available hardware ports from the specific $TCM_MOD.  What I was
>>> thinking for TCM HW fabric module ports would be to have something like:
>>>
>>> 	/sys/kernel/config/target/$TCM_MOD/hw_ports
>>>
>>> that would walk the struct pci_dev looking for fabric module specific HW
>>> target mode capabilities.  I assume this is what you had in mind for
>>> drivers/scsi/ibmvscsi as well, yes..?
>>
>> Doesn't sound so.
>>
>> I want the driver to create necessary target directories in
>> /sys/kernel/config/target/ibmvscsit/ automatically.
> 
> In addition, I also think that /sys/kernel/config/target/$TCM_MOD
> should show up automatically when I load the module.
> 
> vine:/home/fujita# modprobe iscsi_target_mod
> 
> Then why do I need to create iscsi directory by hand?

Hello Tomonori,

Welcome to the world of configfs-based SCSI target configuration
infrastructure!

Configfs is user space driven and only by user space. This is the main
reason why I from the very beginning have writing that it isn't suitable
for managing SCSI targets.

Needless to say that SCST's sysfs-based configuration infrastructure
doesn't suffer from this limitation and allows creation of entries from
both kernel and user spaces.

Vlad

      parent reply	other threads:[~2010-10-06 12:13 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-22 22:49 [RFC v2 05/21] tcm: Add ConfigFS subsystem backstore infrastructure Nicholas A. Bellinger
2010-10-06  0:17 ` FUJITA Tomonori
2010-10-06  4:30   ` Nicholas A. Bellinger
2010-10-06  7:09     ` FUJITA Tomonori
2010-10-06  7:22       ` Nicholas A. Bellinger
2010-10-06  7:43         ` FUJITA Tomonori
2010-10-06  7:51           ` Nicholas A. Bellinger
2010-10-06  7:25       ` FUJITA Tomonori
2010-10-06  7:43         ` Nicholas A. Bellinger
2010-10-06 12:13         ` Vladislav Bolkhovitin [this message]

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=4CAC6803.9070609@vlnb.net \
    --to=vst@vlnb.net \
    --cc=James.Bottomley@suse.de \
    --cc=axboe@kernel.dk \
    --cc=bharrosh@panasas.com \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=hare@suse.de \
    --cc=hch@lst.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=michaelc@cs.wisc.edu \
    --cc=nab@linux-iscsi.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.