netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Gallatin <gallatin@myri.com>
To: David Miller <davem@davemloft.net>
Cc: rick.jones2@hp.com, rdreier@cisco.com, jgarzik@pobox.com,
	swise@opengridcomputing.com, divy@chelsio.com, kxie@chelsio.com,
	netdev@vger.kernel.org, open-iscsi@googlegroups.com,
	michaelc@cs.wisc.edu, daisyc@us.ibm.com, wenxiong@us.ibm.com,
	bhua@us.ibm.com, dm@chelsio.com, lee@jasper.es
Subject: Re: [RFC][PATCH 1/1] cxgb3i: cxgb3 iSCSI initiator
Date: Fri, 15 Aug 2008 08:19:30 -0400	[thread overview]
Message-ID: <48A57452.40905@myri.com> (raw)
In-Reply-To: <20080814.152300.127682912.davem@davemloft.net>

David Miller wrote:
 > From: Andrew Gallatin <gallatin@myri.com>
 > Date: Thu, 14 Aug 2008 16:45:57 -0400
 >
 >> Speaking of stateless data placement.  Assume you have a page or set
 >> of pages allocated by a network driver which contain exactly the data
 >> a block driver is interested in to fulfill a read request (eg, the NIC
 >> understands the protocol just well enough to split headers).  Is it
 >> possible in the block driver to simply replace the pages that are
 >> attached to the buf with pages allocated by the network driver?
 >>
 >> It was my impression that the pages associated with a buf are in
 >> fairly magical states and that you cannot replace them.
 >
 > This, if true, would be quite easy to rectify.

That's great news.

The reason I think this is because a few months ago, I tried to make
the AOE block driver zero-copy on read.  I modified the myri10ge
ethernet driver and firmware to split AOE headers, so that the AOE
read data started nicely aligned in a page allocated by myri10ge.
(this obviously only works for for jumbo MTUs..).  This part was
trivial, but I got bogged down trying to figure out how to replace the
buf associated pages in the AOE driver, as they seemed to be in
various states of entanglement with the LRU and page caches.

I'm more of a network driver guy, and my lack of understanding of the
block layer and page cache probably made me give up too quickly.  I'll
dust off this work and ask for help.

Thanks,

Drew

  reply	other threads:[~2008-08-15 12:27 UTC|newest]

Thread overview: 71+ 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: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 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 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
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 [this message]
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-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=48A57452.40905@myri.com \
    --to=gallatin@myri.com \
    --cc=bhua@us.ibm.com \
    --cc=daisyc@us.ibm.com \
    --cc=davem@davemloft.net \
    --cc=divy@chelsio.com \
    --cc=dm@chelsio.com \
    --cc=jgarzik@pobox.com \
    --cc=kxie@chelsio.com \
    --cc=lee@jasper.es \
    --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 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).