* [dpdk-stable] [PATCH 4/5] examples/l2fwd-cat: fix build on FreeBSD
[not found] <20190409092933.55356-1-bruce.richardson@intel.com>
@ 2019-04-09 9:29 ` Bruce Richardson
2019-04-09 9:56 ` [dpdk-stable] [dpdk-dev] " David Marchand
2019-04-09 10:40 ` Luca Boccassi
0 siblings, 2 replies; 8+ messages in thread
From: Bruce Richardson @ 2019-04-09 9:29 UTC (permalink / raw)
To: dev; +Cc: Bruce Richardson, stable
The definition of CPU_AND differs from Linux to BSD, so we need to use
RTE_CPU_AND instead.
Fixes: f6baccbc2b3b ("examples/l2fwd-cat: add sample application for PQoS CAT and CDP")
Cc: stable@dpdk.org
Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
examples/l2fwd-cat/cat.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/examples/l2fwd-cat/cat.c b/examples/l2fwd-cat/cat.c
index a6081e676..502c6b327 100644
--- a/examples/l2fwd-cat/cat.c
+++ b/examples/l2fwd-cat/cat.c
@@ -345,7 +345,7 @@ check_cpus_overlapping(void)
for (i = 0; i < m_config_count; i++) {
for (j = i + 1; j < m_config_count; j++) {
- CPU_AND(&mask,
+ RTE_CPU_AND(&mask,
&m_config[i].cpumask,
&m_config[j].cpumask);
--
2.20.1
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH 4/5] examples/l2fwd-cat: fix build on FreeBSD
2019-04-09 9:29 ` [dpdk-stable] [PATCH 4/5] examples/l2fwd-cat: fix build on FreeBSD Bruce Richardson
@ 2019-04-09 9:56 ` David Marchand
2019-04-09 10:03 ` Bruce Richardson
2019-04-09 10:40 ` Luca Boccassi
1 sibling, 1 reply; 8+ messages in thread
From: David Marchand @ 2019-04-09 9:56 UTC (permalink / raw)
To: Bruce Richardson; +Cc: dev, dpdk stable
On Tue, Apr 9, 2019 at 11:30 AM Bruce Richardson <bruce.richardson@intel.com>
wrote:
> The definition of CPU_AND differs from Linux to BSD, so we need to use
> RTE_CPU_AND instead.
>
> Fixes: f6baccbc2b3b ("examples/l2fwd-cat: add sample application for PQoS
> CAT and CDP")
> Cc: stable@dpdk.org
This creates a dependency on backporting c3568ea37670 ("eal: restrict
control threads to startup CPU affinity") which introduced the RTE_CPU_AND
macro.
--
David Marchand
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH 4/5] examples/l2fwd-cat: fix build on FreeBSD
2019-04-09 9:56 ` [dpdk-stable] [dpdk-dev] " David Marchand
@ 2019-04-09 10:03 ` Bruce Richardson
2019-04-09 10:34 ` David Marchand
0 siblings, 1 reply; 8+ messages in thread
From: Bruce Richardson @ 2019-04-09 10:03 UTC (permalink / raw)
To: David Marchand; +Cc: dev, dpdk stable
On Tue, Apr 09, 2019 at 11:56:04AM +0200, David Marchand wrote:
> On Tue, Apr 9, 2019 at 11:30 AM Bruce Richardson
> <[1]bruce.richardson@intel.com> wrote:
>
> The definition of CPU_AND differs from Linux to BSD, so we need to
> use
> RTE_CPU_AND instead.
> Fixes: f6baccbc2b3b ("examples/l2fwd-cat: add sample application for
> PQoS CAT and CDP")
> Cc: [2]stable@dpdk.org
>
> This creates a dependency on backporting c3568ea37670 ("eal: restrict
> control threads to startup CPU affinity") which introduced the
> RTE_CPU_AND macro.
> --
> David Marchand
>
Shall I drop the stable reference from the v2, then?
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH 4/5] examples/l2fwd-cat: fix build on FreeBSD
2019-04-09 10:03 ` Bruce Richardson
@ 2019-04-09 10:34 ` David Marchand
2019-04-09 10:40 ` Bruce Richardson
2019-04-09 10:40 ` Luca Boccassi
0 siblings, 2 replies; 8+ messages in thread
From: David Marchand @ 2019-04-09 10:34 UTC (permalink / raw)
To: Bruce Richardson; +Cc: dev, dpdk stable
On Tue, Apr 9, 2019 at 12:03 PM Bruce Richardson <bruce.richardson@intel.com>
wrote:
> On Tue, Apr 09, 2019 at 11:56:04AM +0200, David Marchand wrote:
> > On Tue, Apr 9, 2019 at 11:30 AM Bruce Richardson
> > <[1]bruce.richardson@intel.com> wrote:
> >
> > The definition of CPU_AND differs from Linux to BSD, so we need to
> > use
> > RTE_CPU_AND instead.
> > Fixes: f6baccbc2b3b ("examples/l2fwd-cat: add sample application for
> > PQoS CAT and CDP")
> > Cc: [2]stable@dpdk.org
> >
> > This creates a dependency on backporting c3568ea37670 ("eal: restrict
> > control threads to startup CPU affinity") which introduced the
> > RTE_CPU_AND macro.
> > --
> > David Marchand
> >
> Shall I drop the stable reference from the v2, then?
>
We can backport in 18.11, as I would expect c3568ea37670 to be backported.
The question is more what we want to do with 17.11.
We could backport only the macro bits from this patch if needed.
--
David Marchand
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH 4/5] examples/l2fwd-cat: fix build on FreeBSD
2019-04-09 10:34 ` David Marchand
@ 2019-04-09 10:40 ` Bruce Richardson
2019-04-09 10:40 ` Luca Boccassi
1 sibling, 0 replies; 8+ messages in thread
From: Bruce Richardson @ 2019-04-09 10:40 UTC (permalink / raw)
To: David Marchand; +Cc: dev, dpdk stable
On Tue, Apr 09, 2019 at 12:34:06PM +0200, David Marchand wrote:
> On Tue, Apr 9, 2019 at 12:03 PM Bruce Richardson
> <[1]bruce.richardson@intel.com> wrote:
>
> On Tue, Apr 09, 2019 at 11:56:04AM +0200, David Marchand wrote:
> > On Tue, Apr 9, 2019 at 11:30 AM Bruce Richardson
> > <[1][2]bruce.richardson@intel.com> wrote:
> >
> > The definition of CPU_AND differs from Linux to BSD, so we
> need to
> > use
> > RTE_CPU_AND instead.
> > Fixes: f6baccbc2b3b ("examples/l2fwd-cat: add sample
> application for
> > PQoS CAT and CDP")
> > Cc: [2][3]stable@dpdk.org
> >
> > This creates a dependency on backporting c3568ea37670 ("eal:
> restrict
> > control threads to startup CPU affinity") which introduced the
> > RTE_CPU_AND macro.
> > --
> > David Marchand
> >
> Shall I drop the stable reference from the v2, then?
>
> We can backport in 18.11, as I would expect c3568ea37670 to be
> backported.
> The question is more what we want to do with 17.11.
> We could backport only the macro bits from this patch if needed.
> --
Yes. I'll leave the stable cc and then each maintainer can decide
themselves on backport. I'll include note in the patch (below cutline) to
call out dependency explicitly.
/Bruce
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH 4/5] examples/l2fwd-cat: fix build on FreeBSD
2019-04-09 10:34 ` David Marchand
2019-04-09 10:40 ` Bruce Richardson
@ 2019-04-09 10:40 ` Luca Boccassi
2019-04-09 11:09 ` David Marchand
1 sibling, 1 reply; 8+ messages in thread
From: Luca Boccassi @ 2019-04-09 10:40 UTC (permalink / raw)
To: David Marchand, Bruce Richardson; +Cc: dev, dpdk stable
On Tue, 2019-04-09 at 12:34 +0200, David Marchand wrote:
> On Tue, Apr 9, 2019 at 12:03 PM Bruce Richardson <
> bruce.richardson@intel.com
> >
> wrote:
>
> > On Tue, Apr 09, 2019 at 11:56:04AM +0200, David Marchand wrote:
> > > On Tue, Apr 9, 2019 at 11:30 AM Bruce Richardson
> > > <[1]
> > > bruce.richardson@intel.com
> > > > wrote:
> > >
> > > The definition of CPU_AND differs from Linux to BSD, so we
> > > need to
> > > use
> > > RTE_CPU_AND instead.
> > > Fixes: f6baccbc2b3b ("examples/l2fwd-cat: add sample
> > > application for
> > > PQoS CAT and CDP")
> > > Cc: [2]
> > > stable@dpdk.org
> > >
> > >
> > > This creates a dependency on backporting c3568ea37670 ("eal:
> > > restrict
> > > control threads to startup CPU affinity") which introduced the
> > > RTE_CPU_AND macro.
> > > --
> > > David Marchand
> > >
> >
> > Shall I drop the stable reference from the v2, then?
> >
>
> We can backport in 18.11, as I would expect c3568ea37670 to be
> backported.
>
> The question is more what we want to do with 17.11.
> We could backport only the macro bits from this patch if needed.
If c3568 is not destined for 17.11.x, if you are up for doing the extra
work you can send the macro-only change as an individual patch to
stable@dpdk.org (do not cc dev) and use --subject-prefix='PATCH 17.11'
- alternatively, the 17.11 maintainer can simply opt to not pick up
this patch.
--
Kind regards,
Luca Boccassi
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH 4/5] examples/l2fwd-cat: fix build on FreeBSD
2019-04-09 9:29 ` [dpdk-stable] [PATCH 4/5] examples/l2fwd-cat: fix build on FreeBSD Bruce Richardson
2019-04-09 9:56 ` [dpdk-stable] [dpdk-dev] " David Marchand
@ 2019-04-09 10:40 ` Luca Boccassi
1 sibling, 0 replies; 8+ messages in thread
From: Luca Boccassi @ 2019-04-09 10:40 UTC (permalink / raw)
To: Bruce Richardson, dev; +Cc: stable
On Tue, 2019-04-09 at 10:29 +0100, Bruce Richardson wrote:
> The definition of CPU_AND differs from Linux to BSD, so we need to
> use
> RTE_CPU_AND instead.
>
> Fixes: f6baccbc2b3b ("examples/l2fwd-cat: add sample application for
> PQoS CAT and CDP")
> Cc:
> stable@dpdk.org
>
>
> Signed-off-by: Bruce Richardson <
> bruce.richardson@intel.com
> >
> ---
> examples/l2fwd-cat/cat.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Acked-by: Luca Boccassi <bluca@debian.org>
--
Kind regards,
Luca Boccassi
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH 4/5] examples/l2fwd-cat: fix build on FreeBSD
2019-04-09 10:40 ` Luca Boccassi
@ 2019-04-09 11:09 ` David Marchand
0 siblings, 0 replies; 8+ messages in thread
From: David Marchand @ 2019-04-09 11:09 UTC (permalink / raw)
To: Luca Boccassi; +Cc: Bruce Richardson, dev, dpdk stable
On Tue, Apr 9, 2019 at 12:40 PM Luca Boccassi <bluca@debian.org> wrote:
> On Tue, 2019-04-09 at 12:34 +0200, David Marchand wrote:
> > On Tue, Apr 9, 2019 at 12:03 PM Bruce Richardson <
> > bruce.richardson@intel.com
> > >
> > wrote:
> >
> > > On Tue, Apr 09, 2019 at 11:56:04AM +0200, David Marchand wrote:
> > > > On Tue, Apr 9, 2019 at 11:30 AM Bruce Richardson
> > > > <[1]
> > > > bruce.richardson@intel.com
> > > > > wrote:
> > > >
> > > > The definition of CPU_AND differs from Linux to BSD, so we
> > > > need to
> > > > use
> > > > RTE_CPU_AND instead.
> > > > Fixes: f6baccbc2b3b ("examples/l2fwd-cat: add sample
> > > > application for
> > > > PQoS CAT and CDP")
> > > > Cc: [2]
> > > > stable@dpdk.org
> > > >
> > > >
> > > > This creates a dependency on backporting c3568ea37670 ("eal:
> > > > restrict
> > > > control threads to startup CPU affinity") which introduced the
> > > > RTE_CPU_AND macro.
> > > > --
> > > > David Marchand
> > > >
> > >
> > > Shall I drop the stable reference from the v2, then?
> > >
> >
> > We can backport in 18.11, as I would expect c3568ea37670 to be
> > backported.
> >
> > The question is more what we want to do with 17.11.
> > We could backport only the macro bits from this patch if needed.
>
> If c3568 is not destined for 17.11.x, if you are up for doing the extra
> work you can send the macro-only change as an individual patch to
> stable@dpdk.org (do not cc dev) and use --subject-prefix='PATCH 17.11'
> - alternatively, the 17.11 maintainer can simply opt to not pick up
> this patch.
>
Not hard to achieve from my pov, the relevant bits are just this, I can
send it if the patch is selected.
@@ -23,10 +23,18 @@
#define LCORE_ID_ANY UINT32_MAX /**< Any lcore. */
#if defined(__linux__)
- typedef cpu_set_t rte_cpuset_t;
+typedef cpu_set_t rte_cpuset_t;
+#define RTE_CPU_AND(dst, src1, src2) CPU_AND(dst, src1, src2)
#elif defined(__FreeBSD__)
#include <pthread_np.h>
- typedef cpuset_t rte_cpuset_t;
+typedef cpuset_t rte_cpuset_t;
+#define RTE_CPU_AND(dst, src1, src2) do \
+{ \
+ cpuset_t tmp; \
+ CPU_COPY(src1, &tmp); \
+ CPU_AND(&tmp, src2); \
+ CPU_COPY(&tmp, dst); \
+} while (0)
#endif
/**
--
David Marchand
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-04-09 11:09 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20190409092933.55356-1-bruce.richardson@intel.com>
2019-04-09 9:29 ` [dpdk-stable] [PATCH 4/5] examples/l2fwd-cat: fix build on FreeBSD Bruce Richardson
2019-04-09 9:56 ` [dpdk-stable] [dpdk-dev] " David Marchand
2019-04-09 10:03 ` Bruce Richardson
2019-04-09 10:34 ` David Marchand
2019-04-09 10:40 ` Bruce Richardson
2019-04-09 10:40 ` Luca Boccassi
2019-04-09 11:09 ` David Marchand
2019-04-09 10:40 ` Luca Boccassi
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).