All of lore.kernel.org
 help / color / mirror / Atom feed
From: annie li <annie.li@oracle.com>
To: Wei Liu <wei.liu2@citrix.com>
Cc: 'Steven Haigh' <netwiz@crc.id.au>,
	Ian Campbell <Ian.Campbell@citrix.com>,
	"Palagummi, Siva" <Siva.Palagummi@ca.com>,
	Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
	"msw@amazon.com" <msw@amazon.com>,
	Jacek Milewicz <jacekowski@jacekowski.org>
Subject: Re: Is: SKB_MAX_LEN bites again. Was: Re: bug disabling guest interface
Date: Wed, 13 Mar 2013 14:44:17 +0800	[thread overview]
Message-ID: <51402041.9070106@oracle.com> (raw)
In-Reply-To: <1363119219.29093.112.camel@zion.uk.xensource.com>


On 2013-3-13 4:13, Wei Liu wrote:
> Actually the copy is done with Hypervisor, I don't know how it is
> possible to do coalesce while copying.
>
My understanding is we do coalescing in netback during grant copy. For 
example,

                  gop->source.u.ref = txreq.gref;
                  gop->source.domid = vif->domid;
                  gop->source.offset = txreq.offset;

                  gop->dest.u.gmfn = virt_to_mfn(page_address(page));  
--  here should be the page where we coalesce data into, it is likely 
the page is the one which previous request used.
                  gop->dest.domid = DOMID_SELF;
                  gop->dest.offset = txreq.offset;  -- dest.offset 
should be changed to something which represents the offset where we 
start coalescing data of this request

                  gop->len = txreq.size;  -- len should be the total len 
of data which we coalesce into a page
                  gop->flags = GNTCOPY_source_gref;

With this coalescing, it is possible that we could decrease the total 
frags number of skb in netback.

Thanks
Annie

  parent reply	other threads:[~2013-03-13  6:44 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-08  8:46 bug disabling guest interface Steven Haigh
2013-03-08 19:36 ` Jacek Milewicz
2013-03-08 20:36   ` Is: SKB_MAX_LEN bites again. Was: " Konrad Rzeszutek Wilk
2013-03-08 22:09     ` Jacek Milewicz
2013-03-09  2:19       ` Steven Haigh
2013-03-12 14:49       ` Konrad Rzeszutek Wilk
2013-03-12 14:56         ` Ian Campbell
2013-03-12 15:05           ` Steven Haigh
2013-03-12 15:08             ` Ian Campbell
2013-03-12 22:19               ` James Harper
2013-03-13  4:09                 ` jacek burghardt
2013-03-12 14:49       ` Konrad Rzeszutek Wilk
2013-03-09  2:57     ` Ian Campbell
2013-03-09  3:16       ` Steven Haigh
2013-03-09 12:53       ` Sander Eikelenboom
2013-03-10  4:58         ` ANNIE LI
2013-03-10  4:49       ` ANNIE LI
2013-03-12 11:39         ` Ian Campbell
2013-03-10 19:18       ` Wei Liu
2013-03-12 11:40         ` Ian Campbell
2013-03-12 12:18           ` Wei Liu
2013-03-12 15:07             ` ANNIE LI
2013-03-12 15:25               ` Wei Liu
2013-03-12 20:13                 ` Wei Liu
2013-03-12 21:08                   ` Wei Liu
2013-03-13  6:44                   ` annie li [this message]
2013-03-13  9:43                   ` Ian Campbell
2013-03-13 11:24                     ` Wei Liu
2013-03-13  6:22                 ` annie li
2013-03-13 11:26                   ` Wei Liu
2013-03-10  4:48     ` ANNIE LI
     [not found] <CAHyyzzTtK+LbxOh2r_X0=Zd2FsTxoAiLLaWahv5FY-QZnQxhLQ@mail.gmail.com>
     [not found] ` <CAHyyzzSO0vY3LquDjXVAeV3U9sLnAY28+iADzsOLFkMGcJHLOQ@mail.gmail.com>
     [not found]   ` <CAHyyzzSDqmBNWmyRKFijei8YrJD5i40AJ9mcxxnme9CeCLZX2g@mail.gmail.com>
     [not found]     ` <CAHyyzzQcjzeWv-Z6SBOb8Ra0u6BOMzjKV3Ze482rMmMHkAMccw@mail.gmail.com>
     [not found]       ` <CAHyyzzQsh30y7-4M1xmJc+8p-EcqS0+jaibpTrXxTQi92iGDgw@mail.gmail.com>
2013-04-10 15:28         ` (no subject) jacek burghardt
     [not found]           ` <20130411010344.GY11427@reaktio.net>
2013-04-11  7:31             ` Is: SKB_MAX_LEN bites again. Was: Re: bug disabling guest interface Brian Menges

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=51402041.9070106@oracle.com \
    --to=annie.li@oracle.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=Siva.Palagummi@ca.com \
    --cc=jacekowski@jacekowski.org \
    --cc=konrad.wilk@oracle.com \
    --cc=msw@amazon.com \
    --cc=netwiz@crc.id.au \
    --cc=wei.liu2@citrix.com \
    --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.