From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Monjalon <thomas.monjalon@6wind.com>
Subject: Re: [PATCH v3] doc: add performance test guide to the
	linux gsg
Date: Fri, 25 Sep 2015 16:10:54 +0200
Message-ID: <1584750.6FoDJtCokB@xps13>
References: <1443095316-31699-1-git-send-email-qian.q.xu@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7Bit
Cc: dev@dpdk.org
To: Qian Xu <qian.q.xu@intel.com>
Return-path: <dev-bounces@dpdk.org>
Received: from mail-wi0-f176.google.com (mail-wi0-f176.google.com
 [209.85.212.176]) by dpdk.org (Postfix) with ESMTP id 4C4858D3C
 for <dev@dpdk.org>; Fri, 25 Sep 2015 16:12:01 +0200 (CEST)
Received: by wicfx3 with SMTP id fx3so23707657wic.1
 for <dev@dpdk.org>; Fri, 25 Sep 2015 07:12:01 -0700 (PDT)
In-Reply-To: <1443095316-31699-1-git-send-email-qian.q.xu@intel.com>
List-Id: patches and discussions about DPDK <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

2015-09-24 19:48, Qian Xu:
> --- a/doc/guides/linux_gsg/index.rst
> +++ b/doc/guides/linux_gsg/index.rst
> @@ -47,3 +47,4 @@ Contents
>      build_sample_apps
>      enable_func
>      quick_start
> +    perf_test_intel_platform_nic

Why "test"? What about nic_perf_intel_platform?

> --- /dev/null
> +++ b/doc/guides/linux_gsg/perf_test_intel_platform_nic.rst
[...]
> +Grub Boot Parameters
> +~~~~~~~~~~~~~~~~~~~~
> +
> +The following are some recommendations on GRUB boot settings:
> +
> +#. Use the default grub file as a starting point.
> +
> +#. Reserve 1G huge pages via grub configurations. For example to reserve 8 huge pages of 1G size::
> +
> +      default_hugepagesz=1G hugepagesz=1G hugepages=8
> +
> +#. Isolate CPU cores which will be used for DPDK. For example::
> +
> +      isolcpus=2,3,4,5,6,7,8
> +
> +#. If it wants to use VFIO, use the following additional grub parameters::
> +
> +      iommu=pt intel_iommu=on

These paramters can be set with GRUB, yes.
But they are Linux command line parameters.
I suggest to change the title.

[...]
> +4. Bind the test ports to igb_uio. For example bind two ports to a DPDK compatible driver and check the status:

Not speaking about VFIO here suggests that igb_uio gives better performance.
By the way, UIO/VFIO is not used by mlx4.

[...]
> +   Note: Don't use one 2 lcores on the same core

one 2 lcores? looks to be a typo

[...]
> +5. As it is known that a XL710 40G port need at least two queue pairs to achieve best performance,

Is it well known? At least it is said here.

> +8. Configure the traffic to a traffic generator such as IXIA or Spirent.
> +
> +   * Start creating a stream on packet generator, e.g. IXIA.

Not sure mentioning IXIA or Spirent helps here.
Same comment for the (nice) diagram.

Thanks for the documentation effort.