[dpdk-dev] [pull-request] next-eventdev 20.11 RC2
Thomas Monjalon
thomas at monjalon.net
Tue Nov 3 11:04:16 CET 2020
02/11/2020 16:57, McDaniel, Timothy:
> From: David Marchand <david.marchand at redhat.com>
> > On Mon, Nov 2, 2020 at 3:31 PM Jerin Jacob Kollanukkaran
> > <jerinj at marvell.com> wrote:
> > >
> > > The following changes since commit
> > 79d69c6dcf0debea38ac258d230e2f8c93e5ad12:
> > >
> > > mbuf: remove seqn field (2020-10-31 22:14:44 +0100)
> > >
> > > are available in the Git repository at:
> > >
> > > http://dpdk.org/git/next/dpdk-next-eventdev
> > >
> > > for you to fetch changes up to
> > e06cd1ddfeac30b4926a52a69336b4f25a9cd209:
> > >
> > > eventdev: check input parameter for dump op (2020-11-02 19:16:51 +0530)
> > >
> > > ----------------------------------------------------------------
> > > David Marchand (1):
> > > eventdev: check input parameter for dump op
> >
> > About this patch, could we squash some change in the dlb/dlb2 drivers?
> > I guess Thomas could do it while pulling if nobody objects?
> >
> >
> > diff --git a/drivers/event/dlb/dlb_xstats.c b/drivers/event/dlb/dlb_xstats.c
> > index 89186d506e..5f4c590307 100644
> > --- a/drivers/event/dlb/dlb_xstats.c
> > +++ b/drivers/event/dlb/dlb_xstats.c
> > @@ -921,11 +921,6 @@ dlb_eventdev_dump(struct rte_eventdev *dev, FILE *f)
> > struct dlb_hw_dev *handle;
> > int i;
> >
> > - if (f == NULL) {
> > - DLB_LOG_ERR("Invalid file pointer\n");
> > - return;
> > - }
> > -
> > dlb = dlb_pmd_priv(dev);
> >
> > if (dlb == NULL) {
> > diff --git a/drivers/event/dlb2/dlb2_xstats.c b/drivers/event/dlb2/dlb2_xstats.c
> > index a274779fee..8c3c3cda94 100644
> > --- a/drivers/event/dlb2/dlb2_xstats.c
> > +++ b/drivers/event/dlb2/dlb2_xstats.c
> > @@ -972,11 +972,6 @@ dlb2_eventdev_dump(struct rte_eventdev *dev, FILE
> > *f)
> > struct dlb2_hw_dev *handle;
> > int i;
> >
> > - if (f == NULL) {
> > - DLB2_LOG_ERR("Invalid file pointer\n");
> > - return;
> > - }
> > -
> > dlb2 = dlb2_pmd_priv(dev);
> >
> > if (dlb2 == NULL) {
> >
> >
> >
> > --
> > David Marchand
>
> I am fine with Thomas pulling these changes in.
OK will do, after moving David's patch at first position.
More information about the dev
mailing list