From: "Eric Barton" <eeb@bartonsoftware.com>
To: "'David Miller'" <davem@davemloft.net>
Cc: <netdev@vger.kernel.org>
Subject: RE: PATCH zero-copy send completion callback
Date: Tue, 17 Oct 2006 01:53:02 +0100 [thread overview]
Message-ID: <019c01c6f186$99d0d6b0$0281a8c0@ebpc> (raw)
In-Reply-To: <20061016.135222.78711520.davem@davemloft.net>
David,
> Also, the correct mailing list to get to the networking developers
> is netdev@vger.kernel.org. "linux-net" is for users.
Noted.
> Finally, I very much doubt you have much chance getting this
> change in, the infrastructure is implemented in a very ad-hoc
> fashion and it takes into consideration none of the potential
> other users of such a thing.
Are you referring to the absence of a callback argument other than the
callback descriptor itself? It seemed natural to me to contain the
descriptor in whatever state the higher-level protocol associates with the
message it's sending, and to derive this from the descriptor address in the
callback.
If this isn't what you mean, could you explain? I'm not at all religious
about it.
> And these days we're trying to figure
> out how to eliminate skbuff and skb_shared_info struct members
> whereas you're adding 16-bytes of space on 64-bit platforms.
Do you think the general concept of a zero-copy completion callback is
useful?
If so, do you have any ideas about how to do it more economically? It's 2
pointers rather than 1 to avoid forcing an unnecessary packet boundary
between successive zero-copy sends. But I guess that might not be hugely
significant since you're generally sending many pages when zero-copy is
needed for performance. Also, (please correct me if I'm wrong) I didn't
think this would push the allocation over to the next entry in
'malloc_sizes'.
Cheers,
Eric
next parent reply other threads:[~2006-10-17 0:52 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20061016.135222.78711520.davem@davemloft.net>
2006-10-17 0:53 ` Eric Barton [this message]
2006-10-17 9:01 ` PATCH zero-copy send completion callback Eric Dumazet
2006-10-17 12:23 ` Eric Barton
2006-10-17 21:45 ` David Miller
2006-10-17 11:19 ` Evgeniy Polyakov
2006-10-17 12:50 Eric Barton
2006-10-17 13:13 ` Evgeniy Polyakov
[not found] <20061017094643.GA28926@infradead.org>
2006-10-17 12:27 ` Eric Barton
-- strict thread matches above, loose matches on Subject: below --
2006-10-16 18:21 Eric Barton
2006-10-16 17:25 Eric Barton
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='019c01c6f186$99d0d6b0$0281a8c0@ebpc' \
--to=eeb@bartonsoftware.com \
--cc=davem@davemloft.net \
--cc=netdev@vger.kernel.org \
/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