linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: James Bottomley <James.Bottomley@suse.de>
To: Gennadiy Nerubayev <parakie@gmail.com>
Cc: Vladislav Bolkhovitin <vst@vlnb.net>,
	scst-devel <scst-devel@lists.sourceforge.net>,
	linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org
Subject: Re: [Scst-devel] Fwd: Re: linuxcon 2010...
Date: Mon, 23 Aug 2010 11:59:00 -0500	[thread overview]
Message-ID: <1282582740.11194.17.camel@mulgrave.site> (raw)
In-Reply-To: <AANLkTin9STqUz440fCOEWW=YmedVQtQWU2JLO2wZCt5j@mail.gmail.com>

On Sun, 2010-08-22 at 18:10 -0400, Gennadiy Nerubayev wrote: 
> On Sat, Aug 21, 2010 at 4:38 PM, James Bottomley
> <James.Bottomley@suse.de> wrote:
> > On Sat, 2010-08-21 at 22:51 +0400, Vladislav Bolkhovitin wrote:
> >> James Bottomley, on 08/19/2010 12:43 AM wrote:
> >> >>>> 1. What don't you like in the transition path for users from STGT to
> >> >>>> SCST, which I proposed:
> >> >>>>
> >> >>>>     - The only people which would be affected by replacing of STGT by SCST
> >> >>>> would be users of ibmvstgt. Other STGT users would not notice it at all.
> >> >>>> Thus, we should update ibmvstgt for SCST. If ibmvstgt updated for SCST,
> >> >>>> the update for its users would be just writing of a simple scstadmin's
> >> >>>> config file.
> >> >>>>
> >> >>>>     - STGT doesn't have backend drivers, which SCST doesn't have, so
> >> >>>> there's nothing to worry here. At max, AIO support should be added to
> >> >>>> fileio_tgt.
> >> >>>>
> >> >>>>     - STGT user space targets can use SCST backend via scst_local module.
> >> >>>> Scst_local module is ready and work very well.
> >> >>>>
> >> >>>> The result would be very clear without any obsolete mess.
> >> >>>
> >> >>> So does that get us up to being a drop in replacement?  I think you're
> >> >>> saying that even with all of this, at least the VSCSI part will need
> >> >>> updating, so the answer seems to be "no".
> >> >>
> >> >> Sorry, I can't understand, "no" for which? For the whole transition
> >> >> path, or just until there is a patch for ibmvstgt to become ibmvscst?
> >> >
> >> > No to the question "does that get us up to being a drop in replacement
> >> > [for STGT]?"
> >>
> >> I'm sorry again, I did my best, but still can't understand. What you
> >> wrote looks for me too ambiguous. My English must be too bad..
> >>
> >> Could elaborate more for what the "no" is, please? What don't you like
> >> in the plan I suggested?
> >
> > No it isn't a plan that gives us a drop in replacement for STGT.  I
> > didn't say migration path to random userspace target, I said reuse of
> > existing code.
> 
> Hi James,
> 
> (disclaimer: I'm a hoi polloi SCST user)
> 
> I'm not sure if I understand why there is a need for a replacement
> target to reuse existing code, and would definitely appreciate a brief
> explanation or a pointer to an earlier one.

The best thread on the topic is this massive one:

http://marc.info/?t=120109820100005

I want replacement because evidence suggests that multiple things doing
the same thing don't get as much attention as a single one.  We need to
support STGT because it's the one that has the in-kernel user base.
Just breaking them constitutes an ABI problem under the new kernel
rules.

> But even that aside, I'm
> curious if the criteria for what a replacement target must have for
> (at least potential) inclusion into the kernel were ever clearly
> outlined in the past. If they were, then there probably would have
> been things like interested contenders, deadlines, feature
> comparisons, code reviews, and so on, right?

Yes, in that thread.

My basic conclusion was that there's no incredible discriminator between
LIO and STGT (although there are reams written on which performs better
in which circumsances, is useful for clustering, supports ALUA, etc.
each with partisans for the features).  If the two communities can't
work together (as seems to be the case) and I have to choose one, I'll
go by what helps me which, as I've said before, are:

     1. That it would be a drop in replacement for STGT (our current
        in-kernel target mode driver), since he only wanted a single
        SCSI target infrastructure. 
        
     2. That it used a modern sysfs based control and configuration
        plane. 
        
     3. That the code was reviewed as clean enough for inclusion.


> Now, I can't claim familiarity with the kernel development process, or
> any "political" workings in it. The aforementioned however would seem
> like a logical way of doing this since I assume that for whatever
> reason, there is a strict limit to only one generic SCSI target in the
> Linux kernel, and obviously as per this thread the current one is
> being replaced.

Well, my preference would be to keep STGT.  However, I indicated to both
target infrastructures that if they could satisfy the above, I'd be OK
with replacing STGT, so I'm not about to go back on that after causing
quite a large amount of work.

James



  reply	other threads:[~2010-08-23 16:59 UTC|newest]

Thread overview: 94+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-16 16:20 Fwd: Re: [Scst-devel] linuxcon 2010 Vladislav Bolkhovitin
2010-08-17 20:30 ` James Bottomley
2010-08-18 17:52   ` Vladislav Bolkhovitin
2010-08-18 20:43     ` James Bottomley
2010-08-21 18:51       ` Vladislav Bolkhovitin
2010-08-21 20:38         ` James Bottomley
2010-08-22 22:10           ` [Scst-devel] Fwd: " Gennadiy Nerubayev
2010-08-23 16:59             ` James Bottomley [this message]
2010-08-23 17:44               ` Bart Van Assche
2010-08-23 17:58                 ` James Bottomley
2010-08-23 20:11                   ` Bart Van Assche
2010-08-23 20:21                     ` James Bottomley
2010-08-23 19:40               ` Vladislav Bolkhovitin
2010-08-23 20:38                 ` James Bottomley
2010-08-24 10:32                   ` Bart Van Assche
2010-08-24 13:01                   ` Chris Weiss
2010-08-24 19:53                   ` Vladislav Bolkhovitin
2010-08-23 19:40             ` Vladislav Bolkhovitin
  -- strict thread matches above, loose matches on Subject: below --
2010-08-18 14:58 Chetan Loke
2010-08-18 15:11 ` James Bottomley
     [not found]   ` <AANLkTimJGxn=5kEMH68XVWqFcYG3vpfLjLjZpFGqhG_4@mail.gmail.com>
2010-08-18 15:30     ` Bart Van Assche
2010-08-18 16:04   ` Chetan Loke
2010-08-18 16:18     ` James Bottomley
2010-08-18 17:50       ` Vladislav Bolkhovitin
2010-08-19  1:18         ` jack wang
2010-08-19 21:20           ` Dirk Meister
2010-08-19 22:29             ` Nicholas A. Bellinger
2010-08-21 18:42               ` Vladislav Bolkhovitin
2010-08-21 20:25                 ` Nicholas A. Bellinger
2010-08-24 18:08                   ` Vladislav Bolkhovitin
2010-08-21 20:43                 ` James Bottomley
2010-08-22  7:39                   ` Bart Van Assche
2010-08-22 20:29                     ` James Bottomley
2010-08-23 13:47                       ` Joe Landman
2010-08-23 15:12                         ` Bart Van Assche
     [not found]                         ` <AANLkTim-M6dfLvJQnbieFqZCGG33E+-i+u_soCq2p9f1@mail.gmail.com>
2010-08-23 16:07                           ` Chetan Loke
2010-08-23 18:03                             ` Chetan Loke
2010-08-24  7:25                               ` Pasi Kärkkäinen
2010-08-24 14:55                                 ` Chetan Loke
2010-08-23 19:41                       ` Vladislav Bolkhovitin
2010-08-24 14:41                   ` Vladislav Bolkhovitin
2010-08-24 14:51                     ` Chris Weiss
2010-08-24 14:56                       ` Matthew Wilcox
2010-08-25 22:20                       ` Konrad Rzeszutek Wilk
2010-08-25 22:45                         ` Ted Ts'o
2010-08-24 14:57                     ` James Bottomley
2010-08-24 19:48                       ` Vladislav Bolkhovitin
2010-08-24 21:23                         ` Nicholas A. Bellinger
2010-08-26 20:11                           ` Vladislav Bolkhovitin
2010-08-26 21:23                             ` Nicholas A. Bellinger
2010-08-28 17:32                               ` Vladislav Bolkhovitin
2010-08-28 20:47                                 ` Nicholas A. Bellinger
2010-08-30 20:47                                   ` Vladislav Bolkhovitin
2010-08-30 21:46                                     ` Nicholas A. Bellinger
2010-09-02 19:38                                       ` Vladislav Bolkhovitin
2010-09-02 20:25                                         ` Nicholas A. Bellinger
2010-09-05 20:18                                           ` Dmitry Torokhov
2010-09-05 21:50                                             ` Nicholas A. Bellinger
2010-09-05 23:13                                               ` Mark Deneen
2010-09-06  0:12                                                 ` Nicholas A. Bellinger
2010-09-06  0:58                                                   ` Mark Deneen
2010-09-06  1:34                                                     ` Nicholas A. Bellinger
2010-09-06  5:04                                                   ` Dmitry Torokhov
2010-09-05 23:41                                               ` Dmitry Torokhov
2010-09-05 23:59                                                 ` Nicholas A. Bellinger
2010-09-06  4:56                                                   ` Dmitry Torokhov
2010-09-06 10:39                                                   ` James Bottomley
2010-09-06 11:02                                                     ` Bart Van Assche
2010-09-06 11:27                                                       ` James Bottomley
2010-09-06 15:26                                                         ` Vladislav Bolkhovitin
2010-09-06 21:47                                                     ` Vladislav Bolkhovitin
2010-09-06 21:55                                                       ` Nicholas A. Bellinger
2010-09-06 22:14                                                         ` david
2010-09-07  0:44                                                         ` Dmitry Torokhov
2010-09-07  3:45                                                           ` Chetan Loke
2010-09-07  6:15                                                             ` Bart Van Assche
2010-09-07  6:08                                                           ` Bart Van Assche
2010-09-07  6:26                                                             ` Dmitry Torokhov
2010-09-07  6:29                                                             ` Hannes Reinecke
2010-09-07  6:45                                                               ` Bart Van Assche
2010-09-07 13:20                                                                 ` Vladislav Bolkhovitin
2010-09-07 20:14                                                           ` Vladislav Bolkhovitin
2010-09-07 20:14                                                         ` Vladislav Bolkhovitin
2010-09-06 21:16                                               ` Greg KH
2010-09-06 17:28                                           ` Chetan Loke
2010-09-06 21:52                                           ` Vladislav Bolkhovitin
2010-08-20 13:46           ` Ruben Laban
2010-08-18 17:51       ` Chetan Loke
2010-08-18 16:19   ` Bart Van Assche
2010-08-18 16:28   ` Joe Landman
2010-08-18 17:52     ` Vladislav Bolkhovitin
2010-08-18 15:12 ` Chetan Loke
2010-08-18 17:52 ` Vladislav Bolkhovitin
2010-08-20 17:40 Ari Lemmke

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=1282582740.11194.17.camel@mulgrave.site \
    --to=james.bottomley@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=parakie@gmail.com \
    --cc=scst-devel@lists.sourceforge.net \
    --cc=vst@vlnb.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;
as well as URLs for NNTP newsgroup(s).