From: Chandra Seetharaman <sekharan@us.ibm.com>
To: Mike Christie <michaelc@cs.wisc.edu>
Cc: Mike Anderson <andmike@us.ibm.com>,
dm-devel <dm-devel@redhat.com>,
Alasdair G Kergon <agk@redhat.com>
Subject: Re: dm-multipath based engenio-lsi hardware handler
Date: Thu, 22 Mar 2007 15:37:06 -0800 [thread overview]
Message-ID: <1174606627.14100.7.camel@linuxchandra> (raw)
In-Reply-To: <4602C533.3010908@cs.wisc.edu>
On Thu, 2007-03-22 at 13:04 -0500, Mike Christie wrote:
Mike,
Thanks for your comments.
> Chandra Seetharaman wrote:
>
> Did you see may later revisions on this or Ed's updated handler? In the
> current kernel we do not need to allocate or manage our own bios. That
> was a old old hack from when the request did not have a end io. The hack
> is probably only needed for old distros. It is not needed upstream.
>
> For dm based hw handlers you want to do something more like this:
> http://www.cs.wisc.edu/~michaelc/block/dm/v4/
Looks simpler. Can do it.
But, I am little confused though. I do not see the underlying functions
like dm_scsi_init/destroy_context_pool(), dm_scsi_execute_rq() etc., in
2.6.20/21-rc4 tree. Where is it available ?
Also, dm-emc.c (in 2.6.20/21-rc4) doesn't seem to use that model.
<snip>
> >
> > static void rdac_pg_init(struct hw_handler *hwh, unsigned bypassed,
> > struct path *path)
> > {
> > struct rdac_handler *h = hwh->context;
> >
> > switch (h->lun) {
> > case UNINITIALIZED_LUN:
> > submit_c8_inquiry(h, path);
> > break;
> > case UNSUPPORTED_LUN:
> > dm_pg_init_complete(path, MP_FAIL_PATH);
> > break;
> > default:
> > submit_c9_inquiry(h, path);
> > }
> > }
> >
>
> Why not just have userspace check and pass the LUN to the create
> function? This is not a review comment to do it. I am just asking if it
> is difficult to gather info in userspace and pass it down?
It should be possible to get the inquiry page from user space and
implement what you suggest. But, there is currently no storage device
specific interface (like path checkers) to do this. We have to either
add that interface or hack somewhere else(like path checkers) to do it.
Do you think it is worth it ?
Also, is it possible to get the lun info from user space without getting
the inquiry page ?
chandra
--
----------------------------------------------------------------------
Chandra Seetharaman | Be careful what you choose....
- sekharan@us.ibm.com | .......you may get it.
----------------------------------------------------------------------
next prev parent reply other threads:[~2007-03-22 23:37 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1174589692.22545.33.camel@linuxchandra>
2007-03-22 18:04 ` dm-multipath based engenio-lsi hardware handler Mike Christie
2007-03-22 23:37 ` Chandra Seetharaman [this message]
2007-03-27 17:20 ` Mike Christie
2007-03-21 19:23 Chandra Seetharaman
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=1174606627.14100.7.camel@linuxchandra \
--to=sekharan@us.ibm.com \
--cc=agk@redhat.com \
--cc=andmike@us.ibm.com \
--cc=dm-devel@redhat.com \
--cc=michaelc@cs.wisc.edu \
/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.