[EXT] Re: [PATCH 20.11 2/2] dpaa2: fix build with RTE_LIBRTE_IEEE1588

[Christian Ehrhardt]

On Fri, Apr 1, 2022 at 7:29 AM Vanshika Shukla <vanshika.shukla at nxp.com>
wrote:

> Hi,
> I have reviewed the issue. 4690a6114ff6993afc5c005fc340af0b1e4dbe94 commit
> (
> https://github.com/DPDK/dpdk/commit/4690a6114ff6993afc5c005fc340af0b1e4dbe94)
> which is a new feature that is not required to be backported on the stable
> branch declares the variable that is causing issue here. I have a patch
> ready to fix this issue. Please confirm if I can submit the patch on
> stable at dpdk.org mailing list?
>

Yes you can send it there, here a copy&paste from our call for backports
about the preferred style.

A backport should contain a reference to the DPDK main branch commit

in it's commit message in the following fashion:

    [ upstream commit <commit's dpdk main branch SHA-1 checksum> ]



 For example:


https://git.dpdk.org/dpdk-stable/commit/?h=18.11&id=d90e6ae6f936ecdc2fd3811ff9f26aec7f3c06eb


 When sending the backported patch, please indicate the target branch in
the
 subject line, as we have multiple branches, for example:

     [PATCH 20.11] foo/bar: fix baz



 With git format-patch, this can be achieved by appending the parameter:

     --subject-prefix='PATCH 20.11'

Keeping Luca on CC when submitting the patch will also help.


Regards,
> Vanshika
>
> -----Original Message-----
> From: Luca Boccassi <bluca at debian.org>
> Sent: Thursday, 31 March 2022 5:56 pm
> To: dpdk stable <stable at dpdk.org>
> Cc: Vanshika Shukla <vanshika.shukla at nxp.com>; Lingli Chen <
> linglix.chen at intel.com>; Hemant Agrawal <hemant.agrawal at nxp.com>
> Subject: [EXT] Re: [PATCH 20.11 2/2] dpaa2: fix build with
> RTE_LIBRTE_IEEE1588
>
> Caution: EXT Email
>
> On Tue, 29 Mar 2022 at 15:11, Luca Boccassi <bluca at debian.org> wrote:
> >
> > On Tue, 2022-03-29 at 13:09 +0100, luca.boccassi at gmail.com wrote:
> > > From: Luca Boccassi <bluca at debian.org>
> > >
> > > ../drivers/net/dpaa2/dpaa2_rxtx.c: In function ‘dpaa2_dev_rx’:
> > > ../drivers/net/dpaa2/dpaa2_rxtx.c:863:4: error: ‘priv’ undeclared
> (first use in this function); did you mean ‘ldiv’?
> > >     priv->rx_timestamp =
> > >     ^~~~
> > >     ldiv
> > >
> > > Bugzilla ID: 982
> > >
> > > Fixes: 1d10966a22ca ("net/dpaa2: fix timestamping for IEEE1588")
> > >
> > > Signed-off-by: Luca Boccassi <bluca at debian.org>
> > > ---
> > >  drivers/net/dpaa2/dpaa2_rxtx.c | 3 +++
> > >  1 file changed, 3 insertions(+)
> > >
> > > diff --git a/drivers/net/dpaa2/dpaa2_rxtx.c
> > > b/drivers/net/dpaa2/dpaa2_rxtx.c index ecd004f9fa..23e193f86c 100644
> > > --- a/drivers/net/dpaa2/dpaa2_rxtx.c
> > > +++ b/drivers/net/dpaa2/dpaa2_rxtx.c
> > > @@ -772,6 +772,9 @@ dpaa2_dev_rx(void *queue, struct rte_mbuf **bufs,
> uint16_t nb_pkts)
> > >       const struct qbman_fd *fd;
> > >       struct qbman_pull_desc pulldesc;
> > >       struct rte_eth_dev_data *eth_data = dpaa2_q->eth_data;
> > > +#if defined(RTE_LIBRTE_IEEE1588)
> > > +     struct dpaa2_dev_priv *priv = eth_data->dev_private; #endif
> > >
> > >
> > >       if (unlikely(!DPAA2_PER_LCORE_DPIO)) {
> > >               ret = dpaa2_affine_qbman_swp();
> >
> > FYI Vanshika, Hemant and Lingli
>
> Ping - this is a blocker for the 20.11.5 LTS release, would it be possible
> to have a quick review from NXP?
>
> Thanks!
>


-- 
Christian Ehrhardt
Staff Engineer, Ubuntu Server
Canonical Ltd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mails.dpdk.org/archives/stable/attachments/20220401/a321d234/attachment.htm>