From: annie li <annie.li@oracle.com>
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: KonradRzeszutekWilk <konrad@kernel.org>,
xen-devel <xen-devel@lists.xen.org>
Subject: Re: netchannel vs MAX_SKB_FRAGS (Was: Re: [PATCH] xen/netfront: handle compound page fragments on transmit)
Date: Fri, 23 Nov 2012 23:08:48 +0800 [thread overview]
Message-ID: <50AF9180.7020701@oracle.com> (raw)
In-Reply-To: <1353662328.13542.173.camel@zakaz.uk.xensource.com>
On 2012-11-23 17:18, Ian Campbell wrote:
> On Fri, 2012-11-23 at 01:30 +0000, ANNIE LI wrote:
>
>> It is hard to negotiate this between netfront and netback for different
>> packets.
>>
> You wouldn't negotiate for each packet, you would negotiate at start of
> day.
>
> You'd also need some sort of fallback for the case where you end up
> negotiating something smaller than the maximum your upper layer network
> stack might give you. I suppose you'd have to do segmentation somewhere
> along the line. (We have this problem now, and the Linux implementation
> just ignores it and drops the frames.)
What I am thinking is, this negotiation would be implemented during
xenbus communication.
* netback provides its default value in xenstore
* netfront read out this value and compared this value with itself, then
write back the larger one
netback = netfront, use the same value
netback > netfront, use netfront value and netfront does not need
more slots
netback < netfront, use netfront value and netfront does not need to
do segmentation. But if netfront's total frag numbers exceed the max
frag number of dom0(like linux's MAX_SKB_FRAGS), segmentation is needed
for this kind of packets?
* netback read out the negotiated value from xenstore
It seems changing current netback to per-VIF is necessary(persistent
grant benefit from it too), and every different vif in netback maintains
its own value.
Thanks
Annie
next prev parent reply other threads:[~2012-11-23 15:08 UTC|newest]
Thread overview: 92+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-09 13:47 compound skb frag pages appearing in start_xmit Ian Campbell
2012-10-09 13:54 ` Eric Dumazet
2012-10-09 13:54 ` Eric Dumazet
2012-10-09 14:01 ` Eric Dumazet
2012-10-09 14:23 ` Ian Campbell
2012-10-09 14:33 ` Eric Dumazet
2012-10-09 14:33 ` Eric Dumazet
2012-10-09 14:54 ` Ian Campbell
2012-10-09 14:54 ` Ian Campbell
2012-10-09 14:23 ` Ian Campbell
2012-10-09 14:01 ` Eric Dumazet
2012-10-09 14:17 ` Ian Campbell
2012-10-09 14:27 ` Eric Dumazet
2012-10-09 14:27 ` Eric Dumazet
2012-10-09 14:40 ` Ian Campbell
2012-10-09 14:40 ` Ian Campbell
2012-10-09 14:51 ` Eric Dumazet
2012-10-09 14:51 ` Eric Dumazet
2012-10-10 10:13 ` [Xen-devel] " Ian Campbell
2012-10-10 12:24 ` Sander Eikelenboom
2012-10-10 12:29 ` [Xen-devel] " Ian Campbell
2012-10-10 13:31 ` Sander Eikelenboom
2012-10-10 13:31 ` [Xen-devel] " Sander Eikelenboom
2012-10-10 12:29 ` Ian Campbell
2012-10-10 13:09 ` [Xen-devel] " Ian Campbell
2012-10-10 14:49 ` Sander Eikelenboom
2012-10-11 8:02 ` Ian Campbell
2012-10-11 10:00 ` Sander Eikelenboom
2012-10-11 10:05 ` Eric Dumazet
2012-10-11 10:14 ` Ian Campbell
2012-10-11 10:20 ` Sander Eikelenboom
2012-10-11 10:20 ` [Xen-devel] " Sander Eikelenboom
2012-11-15 2:31 ` ANNIE LI
2012-11-19 15:43 ` Sander Eikelenboom
2012-11-20 8:30 ` [Xen-devel] " Stefan Bader
2012-11-20 9:21 ` Ian Campbell
2012-11-20 11:36 ` Ian Campbell
2012-11-20 11:36 ` [Xen-devel] " Ian Campbell
2012-11-21 2:42 ` ANNIE LI
2012-11-21 2:42 ` ANNIE LI
2012-11-20 11:40 ` [PATCH] xen/netfront: handle compound page fragments on transmit Ian Campbell
2012-11-20 11:40 ` Ian Campbell
2012-11-20 12:28 ` Jan Beulich
2012-11-20 12:28 ` [Xen-devel] " Jan Beulich
2012-11-20 13:35 ` Ian Campbell
2012-11-20 13:51 ` Jan Beulich
2012-11-20 14:14 ` Ian Campbell
2012-11-20 14:32 ` Jan Beulich
2012-11-20 15:06 ` Ian Campbell
2012-11-20 15:06 ` [Xen-devel] " Ian Campbell
2012-11-20 15:28 ` Eric Dumazet
2012-11-20 15:28 ` [Xen-devel] " Eric Dumazet
2012-11-20 15:54 ` Ian Campbell
2012-11-20 16:04 ` netchannel vs MAX_SKB_FRAGS (Was: Re: [PATCH] xen/netfront: handle compound page fragments on transmit) Ian Campbell
2012-11-23 1:30 ` ANNIE LI
2012-11-23 2:10 ` James Harper
2012-11-23 6:49 ` ANNIE LI
2012-11-23 9:22 ` Paul Durrant
2012-11-23 9:26 ` James Harper
2012-11-23 9:34 ` Ian Campbell
2012-11-23 9:18 ` Ian Campbell
2012-11-23 15:08 ` annie li [this message]
2012-11-20 15:54 ` [PATCH] xen/netfront: handle compound page fragments on transmit Ian Campbell
2012-11-20 15:44 ` Jan Beulich
2012-11-20 15:44 ` [Xen-devel] " Jan Beulich
2012-11-20 16:14 ` Eric Dumazet
2012-11-20 14:32 ` Jan Beulich
2012-11-20 14:14 ` Ian Campbell
2012-11-20 13:51 ` Jan Beulich
2012-11-20 13:35 ` Ian Campbell
2012-11-20 13:30 ` Stefan Bader
2012-11-20 13:30 ` Stefan Bader
2012-11-20 13:45 ` Sander Eikelenboom
2012-11-20 13:45 ` Sander Eikelenboom
2012-11-20 14:45 ` Eric Dumazet
2012-11-20 14:45 ` Eric Dumazet
2012-11-20 15:05 ` Ian Campbell
2012-11-20 15:05 ` Ian Campbell
2012-11-21 2:52 ` ANNIE LI
2012-11-21 2:52 ` ANNIE LI
2012-11-21 11:09 ` Ian Campbell
2012-11-21 11:09 ` Ian Campbell
2012-11-20 8:30 ` compound skb frag pages appearing in start_xmit Stefan Bader
2012-11-15 2:31 ` ANNIE LI
2012-10-11 10:14 ` Ian Campbell
2012-10-11 10:05 ` Eric Dumazet
2012-10-11 10:00 ` Sander Eikelenboom
2012-10-11 8:02 ` Ian Campbell
2012-10-10 14:49 ` Sander Eikelenboom
2012-10-10 13:09 ` Ian Campbell
2012-10-10 10:13 ` Ian Campbell
2012-10-09 14:17 ` Ian Campbell
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=50AF9180.7020701@oracle.com \
--to=annie.li@oracle.com \
--cc=Ian.Campbell@citrix.com \
--cc=konrad@kernel.org \
--cc=xen-devel@lists.xen.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 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.