From: Tejun Heo <htejun@gmail.com>
To: Matthew Wilcox <matthew@wil.cx>
Cc: Grant Grundler <grundler@google.com>,
James Bottomley <James.Bottomley@hansenpartnership.com>,
linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org
Subject: Re: libata / scsi separation
Date: Thu, 11 Dec 2008 00:33:48 +0900 [thread overview]
Message-ID: <493FE15C.4060907@gmail.com> (raw)
In-Reply-To: <20081210152445.GW25548@parisc-linux.org>
Hello, Matthew.
Matthew Wilcox wrote:
> I think what Tejun means is add a new command, say READ_SG which would
> transfer a page of LBA ranges (see how TRIM works for details), then the
> drive would do a single transfer which contained the data from all those
> ranges.
Yes, I did.
> This would work, but (ignoring the political / standardisation efforts
> required to make this happen), this is just a cop-out. When networking
> people were first faced with gigabit, they tried the same thing (oooh,
> 9000 byte packets, oooh, TCP Offload, etc), all in the name of passing
> larger amounts of data to the card in a single transaction so they
> didn't have to fix their per-transaction overheads.
>
> Users weren't interested. They wanted to keep sending 1500 byte packets
> (because most equipment couldn't handle jumbo frames) and they wanted
> netfilter and SACK and all the other goodies that the Linux networking
> stack offered and the card's TCP stack didn't.
>
> We should stop denying that users actually want to do 4k IOs and just
> get on with fixing the storage stack to cope with lots of them.
And yeap we definitely should try to do that too but I don't think
RW_SG would be as useless as jumbo frames (much less compatibility
problem and no loss of functionality), and the actual hardware
overhead of issuing separate commands for each 4k segment is way
higher than anything we do along the block and low level driver layers
in terms of IO access, host bus and ATA (or SAS) bus overhead.
Thanks.
--
tejun
next prev parent reply other threads:[~2008-12-10 15:33 UTC|newest]
Thread overview: 65+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-12-03 1:38 [PATCH] remove ide-scsi FUJITA Tomonori
2008-12-03 10:06 ` Christoph Hellwig
2008-12-03 13:31 ` Willem Riede
2008-12-03 13:55 ` Matthew Wilcox
2008-12-03 14:02 ` Alan Cox
2008-12-03 15:09 ` James Bottomley
2008-12-06 6:12 ` Pete Zaitcev
2008-12-06 14:06 ` Bartlomiej Zolnierkiewicz
2008-12-06 14:51 ` Bartlomiej Zolnierkiewicz
2008-12-06 15:06 ` Alan Cox
2008-12-06 16:29 ` Bartlomiej Zolnierkiewicz
2008-12-06 15:25 ` Willem Riede
2008-12-06 15:59 ` Bartlomiej Zolnierkiewicz
2008-12-06 17:00 ` Dan Noé
2008-12-06 21:41 ` Bartlomiej Zolnierkiewicz
2008-12-06 22:24 ` Alan Cox
2008-12-06 22:52 ` Sergei Shtylyov
2008-12-06 23:02 ` Alan Cox
2008-12-06 23:19 ` Sergei Shtylyov
2008-12-06 23:32 ` Alan Cox
2008-12-07 0:08 ` Sergei Shtylyov
2008-12-07 11:40 ` Alan Cox
2008-12-07 14:46 ` Sergei Shtylyov
2008-12-07 15:04 ` James Bottomley
2008-12-07 15:21 ` Sergei Shtylyov
2008-12-09 22:21 ` libata / scsi separation Matthew Wilcox
2008-12-09 22:38 ` James Bottomley
2008-12-10 3:37 ` Matthew Wilcox
2008-12-10 1:54 ` Tejun Heo
2008-12-10 2:29 ` Grant Grundler
2008-12-10 2:47 ` Tejun Heo
2008-12-10 3:23 ` Grant Grundler
2008-12-10 3:44 ` Tejun Heo
2008-12-10 15:24 ` Matthew Wilcox
2008-12-10 15:33 ` Tejun Heo [this message]
2008-12-10 16:01 ` Matthew Wilcox
2008-12-10 17:11 ` Grant Grundler
2008-12-10 17:21 ` Grant Grundler
2008-12-07 0:19 ` [PATCH] remove ide-scsi Sergei Shtylyov
2008-12-07 9:59 ` Sergei Shtylyov
2008-12-07 10:41 ` Sergei Shtylyov
2008-12-09 21:41 ` Matthew Wilcox
2008-12-10 17:46 ` Sergei Shtylyov
2008-12-06 23:28 ` Jeff Garzik
2008-12-06 23:42 ` Sergei Shtylyov
2008-12-06 23:48 ` Jeff Garzik
2008-12-07 3:36 ` Yinghai Lu
2008-12-07 4:17 ` Jeff Garzik
2008-12-07 5:07 ` Yinghai Lu
2008-12-07 11:00 ` Sergei Shtylyov
2008-12-09 19:59 ` Mark Lord
2008-12-09 20:07 ` Jeff Garzik
2008-12-09 21:04 ` James Bottomley
2008-12-06 23:45 ` Bartlomiej Zolnierkiewicz
2008-12-06 23:50 ` Jeff Garzik
2008-12-06 23:40 ` Bartlomiej Zolnierkiewicz
2008-12-06 23:51 ` Alan Cox
2008-12-07 0:56 ` Bartlomiej Zolnierkiewicz
2008-12-07 1:14 ` Alan Cox
2008-12-07 10:32 ` Sergei Shtylyov
2008-12-06 23:51 ` Jeff Garzik
2008-12-06 22:33 ` Al Viro
2008-12-06 23:13 ` Bartlomiej Zolnierkiewicz
2008-12-06 23:17 ` Willem Riede
2008-12-07 0:09 ` Al Viro
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=493FE15C.4060907@gmail.com \
--to=htejun@gmail.com \
--cc=James.Bottomley@hansenpartnership.com \
--cc=grundler@google.com \
--cc=linux-ide@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=matthew@wil.cx \
/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).