patches for DPDK stable branches
 help / color / mirror / Atom feed
* [dpdk-stable] [dpdk-dev] [PATCH] bus/dpaa: fix compilation issue with meson build
@ 2018-06-04  2:52 Jerin Jacob
  2018-06-05  6:20 ` Shreyansh Jain
  0 siblings, 1 reply; 3+ messages in thread
From: Jerin Jacob @ 2018-06-04  2:52 UTC (permalink / raw)
  To: dev; +Cc: hemant.agrawal, thomas, bruce.richardson, Jerin Jacob, stable

ccache gcc -Idrivers/drivers@@tmp_rte_pmd_dpaa_sec@sta -Idrivers
In file included from ../drivers/bus/dpaa/include/fsl_usd.h:11,
                 from ../drivers/crypto/dpaa_sec/dpaa_sec.c:27:
../drivers/bus/dpaa/include/compat.h:53:
 error: "__packed" redefined [-Werror]
 #define __packed __rte_packed

In file included from /usr/include/bsd/string.h:39,
from ../lib/librte_eal/common/include/rte_string_fns.h:71,
from ../drivers/crypto/dpaa_sec/dpaa_sec.c:25:
/usr/include/bsd/sys/cdefs.h:120: note: this is the location
of the previous definition
#  define __packed __attribute__((__packed__))

Cc: stable@dpdk.org
Fixes: 39f373cf015a ("bus/dpaa: add compatibility and helper macros")

Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
---
 drivers/bus/dpaa/include/compat.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/bus/dpaa/include/compat.h b/drivers/bus/dpaa/include/compat.h
index e4b570214..0b49ed5e4 100644
--- a/drivers/bus/dpaa/include/compat.h
+++ b/drivers/bus/dpaa/include/compat.h
@@ -50,7 +50,9 @@
 /* Required compiler attributes */
 #define __maybe_unused	__rte_unused
 #define __always_unused	__rte_unused
+#ifndef __packed
 #define __packed	__rte_packed
+#endif
 #define noinline	__attribute__((noinline))
 
 #define L1_CACHE_BYTES 64
-- 
2.17.1

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-stable] [dpdk-dev] [PATCH] bus/dpaa: fix compilation issue with meson build
  2018-06-04  2:52 [dpdk-stable] [dpdk-dev] [PATCH] bus/dpaa: fix compilation issue with meson build Jerin Jacob
@ 2018-06-05  6:20 ` Shreyansh Jain
  2018-06-21  8:25   ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Shreyansh Jain @ 2018-06-05  6:20 UTC (permalink / raw)
  To: Jerin Jacob; +Cc: dev, hemant.agrawal, thomas, bruce.richardson, stable

On 6/4/2018 8:22 AM, Jerin Jacob wrote:
> ccache gcc -Idrivers/drivers@@tmp_rte_pmd_dpaa_sec@sta -Idrivers
> In file included from ../drivers/bus/dpaa/include/fsl_usd.h:11,
>                   from ../drivers/crypto/dpaa_sec/dpaa_sec.c:27:
> ../drivers/bus/dpaa/include/compat.h:53:
>   error: "__packed" redefined [-Werror]
>   #define __packed __rte_packed
> 
> In file included from /usr/include/bsd/string.h:39,
> from ../lib/librte_eal/common/include/rte_string_fns.h:71,
> from ../drivers/crypto/dpaa_sec/dpaa_sec.c:25:
> /usr/include/bsd/sys/cdefs.h:120: note: this is the location
> of the previous definition
> #  define __packed __attribute__((__packed__))
> 
> Cc: stable@dpdk.org
> Fixes: 39f373cf015a ("bus/dpaa: add compatibility and helper macros")
> 
> Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
> ---

Thanks Jerin.
Somehow, this didn't appear on my build environment (maybe because LIB 
BSD is not enabled?). But, the change looks harmless.

Acked-by: Shreyansh Jain <shreyansh.jain@nxp.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-stable] [dpdk-dev] [PATCH] bus/dpaa: fix compilation issue with meson build
  2018-06-05  6:20 ` Shreyansh Jain
@ 2018-06-21  8:25   ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2018-06-21  8:25 UTC (permalink / raw)
  To: Jerin Jacob; +Cc: stable, Shreyansh Jain, dev, hemant.agrawal, bruce.richardson

05/06/2018 08:20, Shreyansh Jain:
> On 6/4/2018 8:22 AM, Jerin Jacob wrote:
> > ccache gcc -Idrivers/drivers@@tmp_rte_pmd_dpaa_sec@sta -Idrivers
> > In file included from ../drivers/bus/dpaa/include/fsl_usd.h:11,
> >                   from ../drivers/crypto/dpaa_sec/dpaa_sec.c:27:
> > ../drivers/bus/dpaa/include/compat.h:53:
> >   error: "__packed" redefined [-Werror]
> >   #define __packed __rte_packed
> > 
> > In file included from /usr/include/bsd/string.h:39,
> > from ../lib/librte_eal/common/include/rte_string_fns.h:71,
> > from ../drivers/crypto/dpaa_sec/dpaa_sec.c:25:
> > /usr/include/bsd/sys/cdefs.h:120: note: this is the location
> > of the previous definition
> > #  define __packed __attribute__((__packed__))
> > 
> > Cc: stable@dpdk.org
> > Fixes: 39f373cf015a ("bus/dpaa: add compatibility and helper macros")
> > 
> > Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
> > ---
> 
> Thanks Jerin.
> Somehow, this didn't appear on my build environment (maybe because LIB 
> BSD is not enabled?). But, the change looks harmless.
> 
> Acked-by: Shreyansh Jain <shreyansh.jain@nxp.com>

I missed this patch and sent a more complete one recently:

	https://patches.dpdk.org/patch/41314/

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-06-21  8:25 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-04  2:52 [dpdk-stable] [dpdk-dev] [PATCH] bus/dpaa: fix compilation issue with meson build Jerin Jacob
2018-06-05  6:20 ` Shreyansh Jain
2018-06-21  8:25   ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).