All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yuanhan Liu <yuanhan.liu@linux.intel.com>
To: "Michal Orsák" <xorsak01@stud.fit.vutbr.cz>
Cc: Thomas Monjalon <thomas.monjalon@6wind.com>,
	Jan Viktorin <viktorin@rehivetech.com>,
	dev@dpdk.org
Subject: Re: [dpdk-stable] [PATCH 1/2] net/virtio: fix performance regression due to TSO enabling
Date: Mon, 30 Jan 2017 21:30:47 +0800	[thread overview]
Message-ID: <20170130133047.GI20916@yliu-dev.sh.intel.com> (raw)
In-Reply-To: <a12848b4-76ef-29bc-f512-81bd8c1b9b76@stud.fit.vutbr.cz>

On Mon, Jan 16, 2017 at 12:26:41PM +0100, Michal Orsák wrote:
> >>>>>>For such workload, I don't think it would behaviour worse on ARM.
> >>>>>No reply yet; I will treat it as no objections, and please shout out if any.
> >>>>>
> >>>>>Both applied to dpdk-next-virtio.
> >>>>>
> >>>>>	--yliu
> >>>>Hello,
> >>>>
> >>>>
> >>>>currently I am running short of time. If you have any test prepared which i
> >>>>can just ran, please send me a link.
> >>>No link, but you could try:
> >>>
> >>>- a typical PVP test
> >>>
> >>>- a txonly test: running txonly fwd mode in guest PMD while running
> >>>   rxonly in fwd mode.
> >>>
> >>>The second is a micro test, thus I saw way bigger boost.
> >>>
> >>>When are you available for the testing, btw?
> >>25.1.2017+
> >Okay, I will hold on a while to apply them.
> Ok, I will send you results when I have them.

Again, no reply yet. I'm appying them to dpdk-next-virtio. I really
don't think it could perform worse in ARM, as it removes a costly
cache invalidation operation (which should be more expensive than the
instruction cycles).

OTOH, again, testing is welcome, if it's later proved to be worse on
ARM (which I highly doubt), I could revert them.

	--yliu

  parent reply	other threads:[~2017-01-30 13:28 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-11  4:27 [PATCH 0/2] net/virtio: optimize virtio net header reset Yuanhan Liu
2017-01-11  4:27 ` [PATCH 1/2] net/virtio: fix performance regression due to TSO enabling Yuanhan Liu
2017-01-11  7:59   ` Maxime Coquelin
2017-01-11  8:08     ` Yuanhan Liu
2017-01-11  8:22     ` Olivier MATZ
2017-01-11 14:51   ` [dpdk-stable] " Thomas Monjalon
2017-01-12  2:30     ` Yuanhan Liu
2017-01-12 15:02       ` Jan Viktorin
2017-01-13  6:13         ` Yuanhan Liu
2017-01-16  7:12           ` Yuanhan Liu
     [not found]             ` <46569522-b2c3-2a33-9111-049b73c79760@stud.fit.vutbr.cz>
     [not found]               ` <20170116111256.GA11439@yliu-dev.sh.intel.com>
     [not found]                 ` <8e8178c6-caa2-1b6e-10a0-c83820868db5@stud.fit.vutbr.cz>
2017-01-16 11:21                   ` Yuanhan Liu
     [not found]                     ` <a12848b4-76ef-29bc-f512-81bd8c1b9b76@stud.fit.vutbr.cz>
2017-01-30 13:30                       ` Yuanhan Liu [this message]
2017-01-30 13:54                         ` Maxime Coquelin
2017-01-30 14:10                           ` Yuanhan Liu
2017-01-11  4:27 ` [PATCH 2/2] net/virtio: optimize header reset on any layout Yuanhan Liu
2017-01-11  8:01   ` Maxime Coquelin

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=20170130133047.GI20916@yliu-dev.sh.intel.com \
    --to=yuanhan.liu@linux.intel.com \
    --cc=dev@dpdk.org \
    --cc=thomas.monjalon@6wind.com \
    --cc=viktorin@rehivetech.com \
    --cc=xorsak01@stud.fit.vutbr.cz \
    /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.