linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
To: Jeff Garzik <jeff@garzik.org>
Cc: Jonathan Corbet <corbet@lwn.net>,
	Boaz Harrosh <bharrosh@panasas.com>,
	James Bottomley <James.Bottomley@hansenpartnership.com>,
	Zach Brown <zach.brown@oracle.com>,
	Chris Mason <chris.mason@oracle.com>, Tejun Heo <tj@kernel.org>,
	linux-scsi <linux-scsi@vger.kernel.org>
Subject: Re: LSF Papers online?
Date: Tue, 14 Apr 2009 03:24:59 +0200	[thread overview]
Message-ID: <200904140324.59657.bzolnier@gmail.com> (raw)
In-Reply-To: <49E3BBB9.4040100@garzik.org>

On Tuesday 14 April 2009 00:24:57 Jeff Garzik wrote:
> Bartlomiej Zolnierkiewicz wrote:
> > I've started reading it and immediately noticed a thing which made by day. :-)
> > 
> > Sorry if it will sound off-topic or undiplomatic but it is the best occasion
> > to straighten up some facts:
> > 
> >  "Discussion then moved on to the current status of getting libata out of
> >   SCSI: we have had several successes, notably timer handling and pieces of
> >   error handling have moved up to block. Unfortunately, the current progress
> >   has reached the point where it's being impeded by the legacy IDE subsystem
> > 
> > Heh, you can also blame the lack of world peace on the legacy IDE subsystem.
> > 
> > I wonder who came up with this ridiculous excuse (I'm sure it wasn't James!).

It was you!? :)

> > The thing is that during last _five_ years almost nothing was done in this
> > direction.  Despite the fact that it was #1 condition under which the whole
> > code has been merged.  Sorry to say it but it seems like the whole merge
> > strategy was to over-promise things now and worry about delivery later.
> 
> Yet, shockingly, users have been happily using libata despite all these 
> horrors.

That was not the issue raised:

If you think that you can take a "I will deliver later" credit from the
developers community and later cover it up by "this is still my goal, I
just need to find some suckers to do it for me" and think that you won by
fooling people you're sadly mistaken and will most likely have a reality
check one day (not from me, I really don't care that much to waste my
precious time on proving you wrong).

Having corporate backing will save you only to some point, I would suggest
you to look at the current situation in area quite close to our area where
we have another project manager doing heavy mumbo-jumbo on a three years
behind the schedule, untested and already technically obsolete project.

> > To make things worse all the "successes" quoted above are nothing else
> > like back-ridding on block layer and SCSI improvements which were done by
> > non-libata developers.
> 
> False.  Tejun authored many of the changesets getting timer and error 
> handling "moved up the stack."

OK, I stand corrected:

s/all/vast majority of/

> >   which is still relying on some very old fields and undocumented behavior
> >   of the block layer, since the next step is to simplify the block to low level
> > 
> > When it comes to block layer interactions the legacy IDE subsystem is just
> > another "dumb" (== very simple) block layer driver.
> 
> Hardly.  The IDE driver has all sorts of special cases that no other 
> block driver has.  One must roll dice to see which of rq->special, 
> ->buffer, ->data and ->sense is filled in, and at what times. Is 
> ->buffer, ->data, etc. pointing to a buffer... or an opaque kernel data 
> structure?  None of this is clear or documented.

I fail to see how it stops you from working on moving libata out of SCSI,
also Tejun had patches cleaning it for some time now.

> REQ_TYPE_ATA_* is still around.  Overall the consistency of request 
> handling across the IDE class drivers is low.  ide-tape sticks out like 
> a sore thumb with its use of current_nr_sectors.

ditto

What is the relation between REQ_TYPE_ATA_* or ide-tape and moving
libata out of SCSI?

> IDE's interactions with the block layer are quite complex and opaque, 
> compared to other block drivers.

They are certainly more complex than most block layer drivers but there
is very little magic left + Tejun and Borislav are taking care of it
(there were also a lot of good work done by FUJITA and Boaz in the past).

More importantly how exactly does this stop you from working on moving
libata out of SCSI?

To me it looks like some silly behind-somebody's back blame shifting...

Thanks,
Bart

  reply	other threads:[~2009-04-14  1:22 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-13 12:53 LSF Papers online? Boaz Harrosh
2009-04-13 13:58 ` James Bottomley
2009-04-13 14:42   ` Boaz Harrosh
2009-04-13 14:51     ` James Bottomley
2009-04-13 15:19     ` Chris Mason
2009-04-13 15:44       ` Boaz Harrosh
2009-04-13 16:45         ` Theodore Tso
2009-04-13 18:11     ` Jonathan Corbet
2009-04-13 20:05       ` Nicholas A. Bellinger
2009-04-13 21:40       ` Bartlomiej Zolnierkiewicz
2009-04-13 21:49         ` Bartlomiej Zolnierkiewicz
2009-04-13 22:24         ` Jeff Garzik
2009-04-14  1:24           ` Bartlomiej Zolnierkiewicz [this message]
2009-04-14 10:14             ` Jeff Garzik
2009-04-14 14:54               ` Bartlomiej Zolnierkiewicz
2009-04-14 15:40                 ` Jeff Garzik
2009-04-14 16:54                 ` Alan Cox
2009-04-14 22:09                   ` Bartlomiej Zolnierkiewicz
2009-04-14 22:49                     ` James Bottomley
2009-04-15  1:39                       ` Robert Hancock
2009-04-15  3:58                         ` James Bottomley
2009-04-15  8:30                           ` Alan Cox
2009-04-16  6:31                         ` Grant Grundler
2009-04-16 16:37                           ` James Bottomley
2009-04-16 17:45                             ` Matthew Wilcox
2009-04-14 23:14                     ` Jeff Garzik
2009-04-15  9:28                     ` Alan Cox
2009-04-15 13:38                       ` Bartlomiej Zolnierkiewicz
2009-04-15 14:56                         ` Alan Cox
2009-04-16 16:01                           ` Bartlomiej Zolnierkiewicz
2009-04-14  3:30         ` Tejun Heo
2009-04-14 14:47           ` Bartlomiej Zolnierkiewicz
2009-04-16 21:36 ` Grant Grundler
2009-04-17  4:44   ` Martin K. Petersen
2009-04-18  4:06     ` Grant Grundler
2009-04-19 11:00   ` Boaz Harrosh

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=200904140324.59657.bzolnier@gmail.com \
    --to=bzolnier@gmail.com \
    --cc=James.Bottomley@hansenpartnership.com \
    --cc=bharrosh@panasas.com \
    --cc=chris.mason@oracle.com \
    --cc=corbet@lwn.net \
    --cc=jeff@garzik.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=tj@kernel.org \
    --cc=zach.brown@oracle.com \
    /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).