> -----Original Message----- > From: Stephen Hemminger <[email protected]> > Sent: 21 December 2022 00:37 > To: Srikanth Yalavarthi <[email protected]> > Cc: [email protected]; Shivah Shankar Shankar Narayan Rao > <[email protected]>; Jerin Jacob Kollanukkaran > <[email protected]>; Anup Prabhu <[email protected]> > Subject: [EXT] Re: [PATCH v3 0/4] implementation of ML common code > > External Email > > ---------------------------------------------------------------------- > On Tue, 20 Dec 2022 09:52:52 -0800 > Srikanth Yalavarthi <[email protected]> wrote: > > > Machine Learning common code > > ---------------------------- > > > > This patch series implements the common ML code that can be used by ML > > drivers. Common code include functions to convert ML IO type to > > string, IO format type to string, function get size of ML IO type, and > > functions for converting data types from higher precision to lower > > precision and vice-versa. > > > > Data type conversion functions support handling float32, float16, > > bfloat16, uint8, int8, uint16 and int16. Two versions of conversion > > functions are implemented in the series, generic scalar version and > > vector version using Arm NEON intrinsics. When compiling DPDK for > > platform supporting Arm NEON, vector NEON version of the routines > > would be enabled. Compilation would fall back to generic scalar > > versions on platform like x86_64 / PowerPC etc., that don't support Arm > NEON. > > > > > > Srikanth Yalavarthi (4): > > common/ml: add initial files for ML common code > > common/ml: add common utility functions > > common/ml: add scalar type conversion functions > > common/ml: add Arm NEON type conversion routines > > > Ok, but much more is needed. > > Where is the documentation? Documentation related to the functions is part of the header files. Doxygen documentation is not added as the functions / macros part of common/ml is non RTE code.
> Where are the tests? > Where is an example? We don't intend to implement unit tests or examples for the common/ml code as it is intended for driver use only. > > Need a driver that uses it ml/cnxk driver uses the functions part of common/ml code. Driver is pushed as part of a separate patch series http://patches.dpdk.org/project/dpdk/list/?series=26050 http://patches.dpdk.org/project/dpdk/patch/[email protected]/
