All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vladislav Bolkhovitin <vst@vlnb.net>
To: David Miller <davem@davemloft.net>
Cc: open-iscsi@googlegroups.com, rdreier@cisco.com,
	rick.jones2@hp.com, jgarzik@pobox.com,
	swise@opengridcomputing.com, kxie@chelsio.com,
	netdev@vger.kernel.org, michaelc@cs.wisc.edu, daisyc@us.ibm.com,
	wenxiong@us.ibm.com, bhua@us.ibm.com, dm@chelsio.com,
	leedom@chelsio.com, linux-scsi@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [RFC][PATCH 1/1] cxgb3i: cxgb3 iSCSI initiator
Date: Thu, 14 Aug 2008 22:27:49 +0400	[thread overview]
Message-ID: <48A47925.3000409@vlnb.net> (raw)
In-Reply-To: <20080813.132319.175666454.davem@davemloft.net>

[-- Attachment #1: Type: text/plain, Size: 1392 bytes --]

David Miller wrote:
> From: Vladislav Bolkhovitin <vst@vlnb.net>
> Date: Wed, 13 Aug 2008 22:35:34 +0400
> 
>> This is because the target sends data in a zero-copy manner, so its
>> CPU is capable to deal with the load, but on the initiator there are
>> additional data copies from skb's to page cache and from page cache
>> to application.
> 
> If you've actually been reading at all what I've been saying in this
> thread you'll see that I've described a method to do this copy
> avoidance in a completely stateless manner.
> 
> You don't need to implement a TCP stack in the card in order to do
> data placement optimizations.  They can be done completely stateless.

Sure, I read what you wrote before writing (although, frankly, didn't 
get the idea). But I don't think that overall it would be as efficient 
as full hardware offload. See my reply to Jeff Garzik about that.

> Also, large portions of the cpu overhead are transactional costs,
> which are significantly reduced by existing technologies such as
> LRO.

The test used Myricom Myri-10G cards (myri10ge driver), which support 
LRO. And from ethtool -S output I conclude it was enabled. Just in case, 
I attached it, so you can recheck me.

Thus, apparently, LRO doesn't make a fundamental difference. Maybe this 
particular implementation isn't too efficient, I don't know. I don't 
have enough information for that.

Vlad


[-- Attachment #2: ethtool_initiator.txt --]
[-- Type: text/plain, Size: 1498 bytes --]

NIC statistics:
     rx_packets: 471090527
     tx_packets: 175404246
     rx_bytes: 683684492944
     tx_bytes: 636200696592
     rx_errors: 0
     tx_errors: 0
     rx_dropped: 0
     tx_dropped: 0
     multicast: 0
     collisions: 0
     rx_length_errors: 0
     rx_over_errors: 0
     rx_crc_errors: 0
     rx_frame_errors: 0
     rx_fifo_errors: 0
     rx_missed_errors: 0
     tx_aborted_errors: 0
     tx_carrier_errors: 0
     tx_fifo_errors: 0
     tx_heartbeat_errors: 0
     tx_window_errors: 0
     rx_skbs: 0
     alloc_order: 0
     builtin_fw: 0
     napi: 1
     tx_boundary: 4096
     WC: 2
     irq: 1268
     MSI: 1
     MSIX: 0
     read_dma_bw_MBs: 1575
     write_dma_bw_MBs: 1375
     read_write_dma_bw_MBs: 2406
     serial_number: 320283
     watchdog_resets: 0
     link_changes: 2
     link_up: 1
     dropped_link_overflow: 0
     dropped_link_error_or_filtered: 0
     dropped_pause: 0
     dropped_bad_phy: 0
     dropped_bad_crc32: 0
     dropped_unicast_filtered: 0
     dropped_multicast_filtered: 0
     dropped_runt: 0
     dropped_overrun: 0
     dropped_no_small_buffer: 0
     dropped_no_big_buffer: 479
     ----------- slice ---------: 0
     tx_pkt_start: 176354843
     tx_pkt_done: 176354843
     tx_req: 474673372
     tx_done: 474673372
     rx_small_cnt: 19592127
     rx_big_cnt: 462319631
     wake_queue: 0
     stop_queue: 0
     tx_linearized: 0
     LRO aggregated: 481899984
     LRO flushed: 43071334
     LRO avg aggr: 11
     LRO no_desc: 0

  reply	other threads:[~2008-08-14 18:27 UTC|newest]

Thread overview: 77+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-30  0:19 [RFC][PATCH 1/1] cxgb3i: cxgb3 iSCSI initiator Karen Xie
2008-07-30 18:15 ` Shyam_Iyer
2008-07-30 18:37   ` Karen Xie
2008-07-30 19:21 ` Roland Dreier
2008-07-30 19:35 ` Jeff Garzik
2008-07-30 21:35   ` Roland Dreier
2008-08-01  0:51     ` Divy Le Ray
2008-08-07 18:45       ` Divy Le Ray
2008-08-07 20:07         ` Mike Christie
2008-08-08 18:09         ` Steve Wise
2008-08-08 22:15           ` Jeff Garzik
2008-08-08 22:15             ` Jeff Garzik
2008-08-08 22:20             ` Jeff Garzik
2008-08-09  7:28             ` David Miller
2008-08-09 14:04               ` Steve Wise
2008-08-10  5:14               ` Roland Dreier
2008-08-10  5:47                 ` David Miller
2008-08-10  6:34                   ` Herbert Xu
2008-08-10  6:34                     ` Herbert Xu
2008-08-10 17:57                   ` Steve Wise
2008-08-11 16:09                   ` Roland Dreier
2008-08-11 21:09                     ` David Miller
2008-08-11 21:37                       ` Roland Dreier
2008-08-11 21:51                         ` David Miller
2008-08-11 23:20                       ` Steve Wise
2008-08-11 23:45                         ` Divy Le Ray
2008-08-12  0:22                         ` David Miller
2008-08-10  5:12             ` Roland Dreier
2008-08-10  5:46               ` David Miller
2008-08-11 16:07                 ` Roland Dreier
2008-08-11 21:08                   ` David Miller
2008-08-11 21:39                     ` Roland Dreier
2008-08-11 21:52                       ` David Miller
2008-08-11 18:13                 ` Rick Jones
2008-08-11 21:12                   ` David Miller
2008-08-11 21:41                     ` Roland Dreier
2008-08-11 21:53                       ` David Miller
2008-08-12 21:57                         ` Divy Le Ray
2008-08-12 22:01                           ` David Miller
2008-08-12 22:02                           ` David Miller
2008-08-12 22:21                             ` Divy Le Ray
2008-08-13  1:57                               ` Herbert Xu
2008-08-13  1:57                                 ` Herbert Xu
2008-08-13 18:35                               ` Vladislav Bolkhovitin
2008-08-13 19:29                                 ` Jeff Garzik
2008-08-13 20:13                                   ` David Miller
2008-08-14 18:24                                   ` Vladislav Bolkhovitin
2008-08-14 21:59                                     ` Nicholas A. Bellinger
2008-08-13 20:23                                 ` David Miller
2008-08-14 18:27                                   ` Vladislav Bolkhovitin [this message]
2008-08-14 18:30                                     ` Vladislav Bolkhovitin
2008-08-13 21:27                         ` Roland Dreier
2008-08-13 22:08                           ` David Miller
2008-08-13 23:03                             ` Roland Dreier
2008-08-13 23:12                               ` David Miller
2008-08-14  1:26                                 ` Tom Tucker
2008-08-14  1:37                                   ` David Miller
2008-08-14  1:52                                     ` Steve Wise
2008-08-14  2:05                                       ` David Miller
2008-08-14  2:44                                         ` Steve Wise
2008-08-14  1:57                                     ` Tom Tucker
2008-08-14  2:07                                       ` David Miller
2008-08-14  2:09                                   ` David Miller
2008-08-14 20:45                     ` Andrew Gallatin
2008-08-14 22:23                       ` David Miller
2008-08-15 12:19                         ` Andrew Gallatin
2008-08-10  6:24               ` Herbert Xu
2008-08-10  6:24                 ` Herbert Xu
2008-08-10  9:19               ` Alan Cox
2008-08-10 12:49                 ` Jeff Garzik
2008-08-10 14:54                   ` James Bottomley
     [not found]                     ` <1218380086.3418.21.camel-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2008-08-11 16:50                       ` Mike Christie
2008-08-11 16:50                         ` Mike Christie
2008-07-31  1:24   ` Karen Xie
2008-07-31  1:24     ` Karen Xie
2008-07-31 12:45     ` Boaz Harrosh
2008-07-31 12:33 ` 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=48A47925.3000409@vlnb.net \
    --to=vst@vlnb.net \
    --cc=bhua@us.ibm.com \
    --cc=daisyc@us.ibm.com \
    --cc=davem@davemloft.net \
    --cc=dm@chelsio.com \
    --cc=jgarzik@pobox.com \
    --cc=kxie@chelsio.com \
    --cc=leedom@chelsio.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=michaelc@cs.wisc.edu \
    --cc=netdev@vger.kernel.org \
    --cc=open-iscsi@googlegroups.com \
    --cc=rdreier@cisco.com \
    --cc=rick.jones2@hp.com \
    --cc=swise@opengridcomputing.com \
    --cc=wenxiong@us.ibm.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 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.