2015-05-29 17:26, Maciej Gajdzica: > From: Pawel Wodkowski <pawelx.wodkowski at intel.com> > > This patch add following features: > 1. Make entry variable length - remove 64 byte limit. > 2. Add line continue character '\' to make multiline values possible. > > This patch also fixes following issues: > 1. BUG: section were allocated as a block of memory. In rte_cfgfile_close() > free() was wrongly called on each section. This bug is fixed implicitly > by implementing new section allocation scheme. > 2. Protect against multiple sections/entries in sections with the same > name.
Would it be possible to have separate patches for these 4 changes? > > Implications: > 1. Change in struct rte_cfgfile_entry deffinition. > 2. Change in rte_cfgfile_section_entries() declaration. > > Signed-off-by: Pawel Wodkowski <pawelx.wodkowski at intel.com> [...] > --- a/lib/librte_cfgfile/rte_cfgfile_version.map > +++ b/lib/librte_cfgfile/rte_cfgfile_version.map > @@ -13,3 +13,11 @@ DPDK_2.0 { > > local: *; > }; > + > +DPDK_2.1 { > + global: > + > + rte_cfgfile_section_entries; > + > + local: *; > +} Syntax error. I should be +} DPDK_2.0; The example qos_sched must be updated, as it is now based on librte_cfgfile. Thanks