All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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.