On Tue, Oct 20, 2015 at 01:21:37PM -0700, Yoni Fogel wrote: > Also fixed a bug in many of them where if the rte_malloc of > the TAILQ fails, then we return a pointer to some arbitrary > existing struct. > > Signed-off-by: Yoni Fogel <yrobot at amazon.com>
Thanks, looks like good fixes. However, I think for fixes like these they are better as one-fix-per-patch, because in this case we have the title referring to one fix, while the comment log describes a completely separate fix. Can you perhaps do a V2 as a two-patch set? /Bruce > --- > lib/librte_acl/rte_acl.c | 53 > +++++++++++++++++++++------------------ > lib/librte_hash/rte_cuckoo_hash.c | 6 +++-- > lib/librte_hash/rte_fbk_hash.c | 5 +++- > lib/librte_lpm/rte_lpm.c | 5 +++- > lib/librte_lpm/rte_lpm6.c | 5 +++- > 5 files changed, 44 insertions(+), 30 deletions(-) >
