18/04/2017 14:25, Shreyansh Jain: > Hi Olivier, > From: Olivier MATZ [mailto:[email protected]] > > On Thu, 13 Apr 2017 14:32:07 +0530, Shreyansh Jain > > <[email protected]> wrote: > > > Signed-off-by: Shreyansh Jain <[email protected]> > > > > Fixes: 9a8e9b57f544 ("mempool: move ring handler as a driver") > > > > Acked-by: Olivier Matz <[email protected]> > > > > > --- > > > > > > 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

