On Fri, Apr 1, 2022 at 7:29 AM Vanshika Shukla 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@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 ] 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 > Sent: Thursday, 31 March 2022 5:56 pm > To: dpdk stable > Cc: Vanshika Shukla ; Lingli Chen < > linglix.chen@intel.com>; Hemant Agrawal > 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 wrote: > > > > On Tue, 2022-03-29 at 13:09 +0100, luca.boccassi@gmail.com wrote: > > > From: Luca Boccassi > > > > > > ../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 > > > --- > > > 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