On Thu, Jul 01, 2021 at 04:06:09PM +0100, Cian Ferriter wrote: > From: Harry van Haaren <harry.van.haa...@intel.com> > > Split the very large file dpif-netdev.c and the datastructures > it contains into multiple header files. Each header file is > responsible for the datastructures of that component. > > This logical split allows better reuse and modularity of the code, > and reduces the very large file dpif-netdev.c to be more managable. > > Due to dependencies between components, it is not possible to > move component in smaller granularities than this patch. > > To explain the dependencies better, eg: > > DPCLS has no deps (from dpif-netdev.c file) > FLOW depends on DPCLS (struct dpcls_rule) > DFC depends on DPCLS (netdev_flow_key) and FLOW (netdev_flow_key) > THREAD depends on DFC (struct dfc_cache) > > DFC_PROC depends on THREAD (struct pmd_thread) > > DPCLS lookup.h/c require only DPCLS > DPCLS implementations require only dpif-netdev-lookup.h. > - This change was made in 2.12 release with function pointers > - This commit only refactors the name to "private-dpcls.h" > > netdev_flow_key_equal_mf() is renamed to emc_flow_key_equal_mf(). > > Rename functions specific to dpcls from netdev_* namespace to the > dpcls_* namespace, as they are only used by dpcls code. > > 'inline' is added to the dp_netdev_flow_hash() when it is moved > definition to fix a compiler error. > > One valid checkpatch issue with the use of the > EMC_FOR_EACH_POS_WITH_HASH() macro was fixed. > > Signed-off-by: Harry van Haaren <harry.van.haa...@intel.com> > Co-authored-by: Cian Ferriter <cian.ferri...@intel.com> > Signed-off-by: Cian Ferriter <cian.ferri...@intel.com> > > ---
Acked-by: Flavio Leitner <f...@sysclose.org> _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev