18/04/2017 14:25, Shreyansh Jain: > Hi Olivier, > From: Olivier MATZ [mailto:olivier.m...@6wind.com] > > On Thu, 13 Apr 2017 14:32:07 +0530, Shreyansh Jain > > <shreyansh.j...@nxp.com> wrote: > > > Signed-off-by: Shreyansh Jain <shreyansh.j...@nxp.com> > > > > Fixes: 9a8e9b57f544 ("mempool: move ring handler as a driver") > > > > Acked-by: Olivier Matz <olivier.m...@6wind.com> > > > > > --- > > > > > > Context: > > > From the discussion in [1], it was observed that application should > > > have a default pool already linked even in case of shared builds. > > > Ring is especially important because packet mbuf creation API refer to > > > ring_mp_mc as default handler. > > > > > > Documentation for this is pending. > > > > > > [1] http://dpdk.org/ml/archives/dev/2017-April/063819.html > > > > Maybe this comment could go in the patch. > > I can send another version of the patch with some part of the above > documentation within the patch commit.
Moved the mempool ring line below the mempool line, Added the explanations, and Applied, thanks
