05/07/2018 13:48, Gaetan Rivet:
> +/**
> + * @file
> + *
> + * DPDK device class interface.
> + *
> + * This file exposes API and interfaces of device classes.
What is the difference between "API" and "interfaces"?
I think you need to explain what is a device class.
You can mention examples ethdev and cryptodev.
> +#define RTE_REGISTER_CLASS(nm, cls) \
> +RTE_INIT_PRIO(classinitfn_ ##nm, CLASS); \
If you remove the ; you can avoid next line.
See this patch which removes such useless line:
https://patches.dpdk.org/patch/41228/
> +static void classinitfn_ ##nm(void) \
> +{\
> + (cls).name = RTE_STR(nm);\
Better to insert a space before \
> + rte_class_register(&cls); \
> +}
