Re: [dpdk-dev] [PATCH 5/7] net/mrvl: add extra error logs

[Tomasz Duszynski <tdu@semihalf.com>]

On Fri, Dec 01, 2017 at 11:32:15AM +0800, Jianbo Liu wrote:
> The 11/30/2017 14:32, Tomasz Duszynski wrote:
> > Add extra error logs in a few places.
> >
> > Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
> > ---
> >  drivers/net/mrvl/mrvl_ethdev.c | 9 ++++++---
> >  1 file changed, 6 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/net/mrvl/mrvl_ethdev.c b/drivers/net/mrvl/mrvl_ethdev.c
> > index 92cc283..ed97831 100644
> > --- a/drivers/net/mrvl/mrvl_ethdev.c
> > +++ b/drivers/net/mrvl/mrvl_ethdev.c
> > @@ -431,8 +431,10 @@ mrvl_dev_start(struct rte_eth_dev *dev)
> >       priv->bpool_min_size = priv->nb_rx_queues * MRVL_BURST_SIZE * 2;
> >
> >       ret = pp2_ppio_init(&priv->ppio_params, &priv->ppio);
> > -     if (ret)
> > +     if (ret) {
> > +             RTE_LOG(ERR, PMD, "Failed to init ppio\n");
> >               return ret;
> > +     }
> >
> >       /*
> >        * In case there are some some stale uc/mc mac addresses flush them
> > @@ -467,8 +469,8 @@ mrvl_dev_start(struct rte_eth_dev *dev)
> >       if (mrvl_qos_cfg) {
> >               ret = mrvl_start_qos_mapping(priv);
> >               if (ret) {
> > -                     pp2_ppio_deinit(priv->ppio);
> > -                     return ret;
> > +                     RTE_LOG(ERR, PMD, "Failed to setup QoS mapping\n");
> > +                     goto out;
> >               }
> >       }
> >
>
> Do you need to print error log as well if mrvl_dev_set_link_up fails?

Right, extra log can be added for consistency.

>
> > @@ -478,6 +480,7 @@ mrvl_dev_start(struct rte_eth_dev *dev)
> >
> >       return 0;
> >  out:
> > +     RTE_LOG(ERR, PMD, "Failed to start device\n");
> >       pp2_ppio_deinit(priv->ppio);
> >       return ret;
> >  }
> > --
> > 2.7.4
> >
>
> --
> IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

--
- Tomasz Duszyński