From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id D608F2B82 for ; Tue, 23 Feb 2016 11:19:18 +0100 (CET) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga104.fm.intel.com with ESMTP; 23 Feb 2016 02:19:17 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,488,1449561600"; d="scan'208";a="892546448" Received: from irsmsx151.ger.corp.intel.com ([163.33.192.59]) by orsmga001.jf.intel.com with ESMTP; 23 Feb 2016 02:19:16 -0800 Received: from irsmsx103.ger.corp.intel.com ([169.254.3.30]) by IRSMSX151.ger.corp.intel.com ([169.254.4.72]) with mapi id 14.03.0248.002; Tue, 23 Feb 2016 10:19:15 +0000 From: "Mcnamara, John" To: Jan Medala , "dev@dpdk.org" Thread-Topic: [dpdk-dev] [PATCH v3 1/4] ena: Amazon ENA documentation Thread-Index: AQHRbacjCxzk8DN6A0eZkxSeZ6ogxp85ZP/g Date: Tue, 23 Feb 2016 10:19:14 +0000 Message-ID: References: <1456169211-18867-1-git-send-email-jan@semihalf.com> <1456169211-18867-2-git-send-email-jan@semihalf.com> In-Reply-To: <1456169211-18867-2-git-send-email-jan@semihalf.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ctpclassification: CTP_IC x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiN2U0NzdiOTgtYTE2NC00YzBhLThjNGItYWUwY2RmOGU0YzViIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE1LjkuNi42IiwiVHJ1c3RlZExhYmVsSGFzaCI6ImJRanJiWVhCVUw2MzQwVkVIdzBIeGNWbjliZ0sweXZhdWM2V1orSUdEbGs9In0= x-originating-ip: [163.33.239.182] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: Evgeny Schemeilin , "matua@amazon.com" Subject: Re: [dpdk-dev] [PATCH v3 1/4] ena: Amazon ENA documentation X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 23 Feb 2016 10:19:19 -0000 > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Jan Medala > Sent: Monday, February 22, 2016 7:27 PM > To: dev@dpdk.org > Cc: matua@amazon.com; Evgeny Schemeilin > Subject: [dpdk-dev] [PATCH v3 1/4] ena: Amazon ENA documentation Hi, Thanks for this. For reference the DPDK Documentation Guidelines are here: http://dpdk.org/doc/guides/contributing/documentation.html Some general comments below: > --- > doc/guides/nics/ena.rst | 238 You will need to add the new doc to the index file in the same dir in order for it to build and be included in the overall docs: doc/guides/nics/index.rst You can build and test the docs as follows: make -j doc-guides-html firefox build/doc/html/guides/nics/ena.html =20 make -j doc-guides-pdf mupdf build/doc/pdf/guides/nics.pdf > +ENA Poll Mode Driver > +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D The underline should be the same length as the title and should be followed by one blank line to separate it from the text. Most titles in the doc need to be fixed like this. > +ENA PMD is the DPDK poll-mode driver for the Amazon Elastic Network > +Adapter (ENA) family. Maybe better as: The ENA PMD is a DPDK poll-mode driver for the Amazon Elastic Network Adapter (ENA) family. Also, you should insert a short description of ENA or link to external documentation here for people not familiar with ENA. > + > +Overview > +-------- > +The ENA driver exposes a lightweight management interface with a > +minimal set of memory mapped registers and extendable command set s/extendable/an extendable/ > +through an Admin Queue. > + > +The driver supports a wide range of ENA adapter, is link-speed s/adapter/adapters/ > +independent (i.e., the same driver is used for 10GbE, 25GbE, 40GbE, > +etc.), and it negotiates and supports extendable feature set. s/extendable/an extendable/ > + > +Refer to ena_admin_defs.h for the list of supported Get/Set Feature > +properties. Use ```` quotes for filenames, variable, paths, etc, here and elsewhere: ``= ena_admin_defs.h`` > + This is the requested size of receive/transmit queues, while the > actual size will be the minimum between the requested size and the maxima= l > receive/transmit supported by the device. This line, and the previous one, are very long in relation to the rest of t= he lines in the document. Best to make it consistent. > +How to build the suite? > +----------------------- > +The build instructions for the DPDK suite should as follows: > + > +By default the ENA PMD library will be built into the DPDK library. > + > +For configuring and using UIO and VFIO frameworks, please refer the > +documentation that comes with DPDK suite. I'd suggest replacing this section with something like: Building DPDK ------------- =20 See the :ref:`DPDK Getting Started Guide for Linux ` for instructions on how to build DPDK. =20 By default the ENA PMD library will be built into the DPDK library. =20 For configuring and using UIO and VFIO frameworks, also refer to the "Getting Started Guide". > + > +Supported ENA adapters > +---------------------------- > +Current ENA PMD supports the following ENA adapters including: > + > +- 1d0f:ec20 - ENA VF > +- 1d0f:1ec2 - ENA LLQ VF Quote the addresses here: - ``1d0f:ec20`` - ENA VF - ``1d0f:1ec2`` - ENA LLQ VF > + > +Supported Operating Systems > +--------------------------- > +Any Linux distribution fulfilling the conditions described in > +Dependencies section of DPDK documentation. s/DPDK/the DPDK/ and maybe add a link. > + > +Supported features > +------------------ > +- Jumbo frames up to 9K > +- Port Hardware Statistics > +- IPv4/TCP/UDP checksum offload > +- TSO offload > +- Multiple receive and transmit queues > +- RSS > +- Low Latency Queue for Tx > + > +Known bugs and Unsupported features in this release > +--------------------------------------------------- It would better to just call this section "Unsupported features". > +Contact Information > +------------------- > +Any questions or bugs should be reported to DPDK community and to the > +ENA PMD > +maintainers: > + > +- Jan Medala > +- Jakub Palider > +- Netanel Belgazal > +- Evgeny Schemeilin I would suggest omitting this section and just adding all of these names to the MAINTAINERS file (if not already done in another patch). Thanks, John.