[dts] [PATCH v1] test_plans: Add enable_package_download_in_ice_driver_test_plan.rst

[Tu, Lijuan]

It's not necessary to use hardware packet generator, IXIA, I think it's fine to use scapy instead, it's more easy to get scapy than IXIA for testing guys.

The package is called DDP package or OS package officially.

> -----Original Message-----
> From: dts [mailto:dts-bounces at dpdk.org] On Behalf Of Peng Yuan
> Sent: Sunday, April 21, 2019 10:05 PM
> To: dts at dpdk.org
> Cc: Peng, Yuan <yuan.peng at intel.com>
> Subject: [dts] [PATCH v1] test_plans: Add
> enable_package_download_in_ice_driver_test_plan.rst
> 
> Add enable_package_download_in_ice_driver_test_plan.rst
> 
> Signed-off-by: Peng Yuan <yuan.peng at intel.com>
> 
> diff --git a/test_plans/enable_package_download_in_ice_driver_test_plan.rst
> b/test_plans/enable_package_download_in_ice_driver_test_plan.rst
> new file mode 100644
> index 0000000..e3a8235
> --- /dev/null
> +++ b/test_plans/enable_package_download_in_ice_driver_test_plan.rst
> @@ -0,0 +1,142 @@
> +.. Copyright (c) <2019>, Intel Corporation
> +         All rights reserved.
> +
> +   Redistribution and use in source and binary forms, with or without
> +   modification, are permitted provided that the following conditions
> +   are met:
> +
> +   - Redistributions of source code must retain the above copyright
> +     notice, this list of conditions and the following disclaimer.
> +
> +   - Redistributions in binary form must reproduce the above copyright
> +     notice, this list of conditions and the following disclaimer in
> +     the documentation and/or other materials provided with the
> +     distribution.
> +
> +   - Neither the name of Intel Corporation nor the names of its
> +     contributors may be used to endorse or promote products derived
> +     from this software without specific prior written permission.
> +
> +   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
> CONTRIBUTORS
> +   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> +   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND
> FITNESS
> +   FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
> +   COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT,
> INDIRECT,
> +   INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
> +   (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
> OR
> +   SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
> +   HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
> CONTRACT,
> +   STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
> +   ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
> ADVISED
> +   OF THE POSSIBILITY OF SUCH DAMAGE.
> +
> +==========================================================
> +Flexible pipeline package processing on CPK NIC mode Tests
> +==========================================================
> +
> +Description
> +===========
> +
> +DPDK PMD is able to load flexible pipeline package file, process the
> +content then program to NIC.
> +
> +This is very important feature, all Classification and Qos functions
> +depends on this.
> +
> +This feature set enabled package downloading to the device. The package
> +is to be in the /lib/firmware/intel/ice/ddp directory and named ice.pkg.
> +The package is shared by the kernel driver and the DPDK PMD.
> +
> +If package download failed, driver need to go to safe mode. This patch
> +set disabled RSS, QINQ, checksum offload in safe mode.
> +
> +The patch set also enabled RSS for UPD/TCP/SCTP+IPV4/IPV6 packets.
[Lijuan]  The description should be convert to DPDK 's perspective, but not a patches' perspective.
> +
> +Prerequisites
> +=============
> +
> +1. Hardware::
> +
> +    Ice 100G*2
[Lijuan] I think any ICE NIC is fine.
> +    enp216s0f0 <---> IXIA_port_0
> +    enp216s0f1 <---> IXIA_port_1
[Lijuan] change to below is more common.
DUT_port_0 <---> Tester_port_0
> +
> +2. Flash the latest firmware for CVL, and update the driver to the
> +   corresponding version.
> +
[Lijuan] I don't think it's necessary to do that, when the E800 series NIC launched, the released firmware will support it.
> +Test case 1: Download the package successfully
> +==============================================
> +
> +1. Put the ice.pkg to /lib/firmware/intel/ice/ddp/ice.pkg,
> +   then reboot the server.
> +
> +2. Start the testpmd::
> +
> +    ./testpmd -c 0x3fe -n 6 -- -i --nb-cores=8 --rxq=8 --txq=8 \
> +    --port-topology=chained
> +
> +   There will be a printing::
> +
> +    download success
> +
> +3. Normal forward
> +
> +   Set forward mode::
> +
> +    testpmd> set mac fwd
> +    testpmd> start
> +
> +   Send packet from IXIA_port_0,
> +   IXIA_port_1 can receive the forwarded packets.
> +   The forward can run normally.
> +
> +4. The RSS function run normally.
> +
> +   set rxonly mode::
> +
> +    testpmd> set mac rxonly
> +    testpmd> start
> +
> +5. Send UPD/TCP/SCTP+IPV4/IPV6 packets with packet generator
> +   with continuous ipv4/ipv6 address or TCP/UDP/SCTP ports,
> +   the packets can be distributed to eight rx queues averagely.
> +
> +Test case 2: failed to download pkg
> +===================================
> +
> +1. Server power on, then put a new ice.pkg to
> +   /lib/firmware/intel/ice/ddp/ice.pkg.
> +   Make sure the new ice.pkg is different with the original one.
> +
> +2. Start testpmd::
> +
> +    ./testpmd -c 0x3fe -n 6 -- -i --nb-cores=8 --rxq=8 --txq=8 \
> +    --port-topology=chained
> +
> +   There will be an error reported::
> +
> +    ice_dev_init(): Failed to load default OS package
> +
> +   The driver need to go to safe mode.
> +
> +3. Normal forward
> +
> +   Set forward mode::
> +
> +    testpmd> set mac fwd
> +    testpmd> start
> +
> +   Send packet from IXIA_port_0,
> +   IXIA_port_1 can receive the forwarded packets.
> +   The forward can run normally.
> +
> +4. The RSS function doesn't work.
> +
> +   set rxonly mode::
> +
> +    testpmd> set mac rxonly
> +    testpmd> start
> +
> +5. Send UPD/TCP/SCTP+IPV4/IPV6 packets with packet generator
> +   with continuous ipv4/ipv6 address or TCP/UDP/SCTP ports,
> +   the packets can be only distributed to rx queue 0.
> --
> 2.14.3