public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: Roman Zippel <zippel@linux-m68k.org>
To: Andre Hedrick <andre@linux-ide.org>
Cc: James Bottomley <James.Bottomley@steeleye.com>,
	naveenb@cisco.com, hch@infradead.org,
	SCSI Mailing List <linux-scsi@vger.kernel.org>
Subject: Re: Request for review of Linux iSCSI driver version 4.0.0.1
Date: Sun, 7 Dec 2003 01:37:49 +0100 (CET)	[thread overview]
Message-ID: <Pine.LNX.4.58.0312070034550.26104@serv> (raw)
In-Reply-To: <Pine.LNX.4.10.10312061111220.18839-100000@master.linux-ide.org>

Hi,

On Sat, 6 Dec 2003, Andre Hedrick wrote:

> James:
>
> Thank you for explaining to Roman and company the obvious nature of
> BLOCK on top of SCSI.  This is only a point of proof that I was correct in
> my statements earlier.

In case you missed it, but James was not the only one stating that the
send data is not static and so far it's rather unclear what you're trying
to proof here.

> iSCSI(MC) will corrupt randomly.
>
> iSCSI(ZC) will fail data digests and invoke the retry.

Again, how should the data in a local buffer become corrupted? Where is
that "random corruption" which you don't have to deal with zero copy as
well?

> Same example with iSCSI(ERL = 1), the original 19 good PDU's are buffered
> and an in-connection recovery is executed for the one failed.
>
> Translation:
>
> iSCSI(ERL = 0) 20 * (m + 1), m = I(all postive integers)
>
> iSCSI(ERL = 1) 20 + n, n = I(all postive integers)
>
> m == the number of entire commands retired.
> n == the number of PDU's retried.
>
> For ease, make m == n = 1;
>
> iSCSI(ERL = 0) == 40 PDU's
>
> iSCSI(ERL = 1) == 21 PDU's
>
> The only case where they are equal is if all PDU's in iSCSI(ERL = 1) had
> to be retried.

Andre, maybe you can scare little girls with this, but otherwise this
highly unimpressive, how about you come up with some real data?
It's easily possible to limit the transfer size and so to reduce the
amount of the data which has to be transfered again, which protocol is
used to request a retry of the transfer, is not that important. It would
be a lot more interesting if you could actually show a real perfomance
difference with some real numbers.
Of course ERL1 allows to solve this problem more elegant, but so far you
failed to show any proof that ERL0 is not usable.

> The goal of the exercise is to show that both Roman's and Cisco's
> iSCSI(ERL = 0) initaitors are not in a data integrity class acceptable for
> anyone.

iSCSI works quite fine also without an additional checksum and if one
knows the ip transport is reliable, one can easily live without it.

bye, Roman

  reply	other threads:[~2003-12-07  1:03 UTC|newest]

Thread overview: 99+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-12-01 12:30 Request for review of Linux iSCSI driver version 4.0.0.1 Naveen Burmi
2003-12-01 14:08 ` Naveen Burmi
2003-12-01 18:48   ` Andre Hedrick
2003-12-01 19:23   ` Andre Hedrick
2003-12-01 16:20 ` Roman Zippel
2003-12-01 17:19   ` Scott M. Ferris
2003-12-01 20:06     ` Clay Haapala
2003-12-01 20:31       ` Andre Hedrick
2003-12-01 20:58         ` Clay Haapala
2003-12-02  3:46   ` Andre Hedrick
2003-12-02 12:02     ` Naveen Burmi
2003-12-02 13:57     ` Roman Zippel
2003-12-02 11:56   ` Naveen Burmi
2003-12-02 14:11     ` Roman Zippel
2003-12-02 16:37     ` James Bottomley
2003-12-02 17:42       ` Mike Anderson
2003-12-02 23:55         ` James Bottomley
2003-12-02 23:41       ` Clay Haapala
2003-12-03 14:06       ` Naveen Burmi
2003-12-03 15:09         ` James Bottomley
2003-12-03 17:03           ` Clay Haapala
2003-12-03 17:32             ` James Bottomley
2003-12-03 17:54             ` Mike Anderson
2003-12-03 20:31               ` Clay Haapala
2003-12-03 21:14                 ` James Bottomley
2003-12-03 21:53                   ` Scott M. Ferris
2003-12-03 22:57                 ` Scott M. Ferris
2003-12-03 20:45               ` Clay Haapala
2003-12-03 21:19                 ` James Bottomley
2003-12-11 11:21                   ` Naveen Burmi
2003-12-03 22:15                 ` Scott M. Ferris
2003-12-03 22:32                   ` Clay Haapala
2003-12-03 23:24                 ` Mike Anderson
2003-12-06 19:37       ` Andre Hedrick
2003-12-07  0:37         ` Roman Zippel [this message]
  -- strict thread matches above, loose matches on Subject: below --
2003-12-11 15:47 N.C.Krishna Murthy
2003-12-12 12:48 ` Request " Matthew Wilcox
2003-12-12 15:29   ` N.C.Krishna Murthy
2003-12-13  2:46   ` Andre Hedrick
2003-12-11  0:12 Pat LaVarre
2003-11-21 11:40 Shashi Kiran T.R
2003-11-21 17:56 ` Patrick Mansfield
2003-10-23 12:04 Krishna Murthy
2003-10-27 15:39 ` Christoph Hellwig
2003-10-29 13:23   ` Surekha.PC
2003-10-29 13:45     ` 'Christoph Hellwig'
2003-10-29 17:28       ` Mike Christie
2003-10-29 18:45         ` Clay Haapala
2003-10-29 19:01           ` Mike Christie
2003-10-29 19:17             ` Clay Haapala
2003-10-29 19:33               ` Mike Christie
2003-10-30 23:42                 ` Andre Hedrick
2003-10-30 13:34         ` jd
2003-11-11 11:56       ` Naveen Burmi
2003-11-11 17:36         ` 'Christoph Hellwig'
2003-11-19 12:40           ` Krishna Murthy
2003-11-19 12:49             ` Matthew Wilcox
2003-11-19 13:38             ` 'Christoph Hellwig'
2003-11-11 17:40         ` James Bottomley
2003-11-06  9:42   ` Sachin Mhatre (smhatre)
2003-11-06 10:09     ` 'Christoph Hellwig'
2003-11-07  8:55       ` Douglas Gilbert
2003-11-07  9:30         ` 'Christoph Hellwig'
2003-11-10 17:43         ` Patrick Mansfield
2003-11-06 11:10     ` Andre Hedrick
2003-11-06 11:14       ` 'Christoph Hellwig'
2003-11-13 14:30   ` Sachin Mhatre (smhatre)
2003-11-13 14:54     ` Matthew Wilcox
2003-11-19 13:04   ` Sachin Mhatre (smhatre)
2003-11-19 13:10     ` 'Christoph Hellwig'
2003-11-19 14:48   ` Naveen Burmi
2003-11-19 14:48     ` Christoph Hellwig
2003-11-19 15:19       ` Naveen Burmi
2003-11-19 15:20         ` Christoph Hellwig
2003-12-01 12:10       ` Krishna Murthy
2003-12-01 15:22         ` James Bottomley
2003-12-04 12:30           ` N.C.Krishna Murthy
2003-12-05 15:33             ` James Bottomley
2003-12-05 17:03               ` Brian King
2003-12-08 15:06               ` N.C.Krishna Murthy
2003-12-08 15:46                 ` Scott M. Ferris
2003-12-10 15:01                   ` N.C.Krishna Murthy
2003-12-10 16:50                     ` Scott M. Ferris
2003-12-11 14:48               ` N.C.Krishna Murthy
2003-12-01 15:27         ` Christoph Hellwig
2003-12-02  5:52           ` N.C.Krishna Murthy
2003-12-02 16:59             ` Patrick Mansfield
2003-11-19 17:17     ` Patrick Mansfield
2003-11-20 13:32       ` Naveen Burmi
2003-11-20 13:34         ` Christoph Hellwig
2003-11-20 14:53           ` Naveen Burmi
2003-11-21 16:42   ` Clay Haapala
2003-11-21 17:32     ` Matthew Wilcox
2003-11-21 18:18       ` Clay Haapala
2003-11-26 13:41         ` Christoph Hellwig
2003-11-24  6:09   ` Surekha.PC
2003-11-24  7:48     ` 'Christoph Hellwig'
2003-11-24 20:45       ` Patrick Mansfield
2003-11-26 13:45         ` 'Christoph Hellwig'
2003-12-11 12:31       ` Naveen Burmi

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=Pine.LNX.4.58.0312070034550.26104@serv \
    --to=zippel@linux-m68k.org \
    --cc=James.Bottomley@steeleye.com \
    --cc=andre@linux-ide.org \
    --cc=hch@infradead.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=naveenb@cisco.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