DPDK usage discussions
 help / color / mirror / Atom feed
* meson options - build vs target machine cpuflags question
@ 2022-03-10 14:46 Sanford, Robert
  2022-03-10 22:54 ` Dan Gora
  0 siblings, 1 reply; 8+ messages in thread
From: Sanford, Robert @ 2022-03-10 14:46 UTC (permalink / raw)
  To: users

Hello All,

We build a DPDK 21.05 app on Intel x86 machines with RDSEED in cpuflags, and may run it on machines *without* RDSEED.
This results in a fatal error ...
ERROR: This system does not support "RDSEED".
Please check that RTE_MACHINE is set correctly.
EAL: FATAL: unsupported cpu type.

I try adding -Dplatform=haswell when running meson, (because gcc man page indicates that haswell is last arch w/o RDSEED) but get the same result.
Until we resolve it, our workaround is changing the error-out in rte_cpu_is_supported() to just print a warning and continue.
We don't have direct access to automated build machines, we go through change request processes, and so we can't rapidly try too many things.

Is there a better meson option, such as machine=haswell, or something else that will work?

Thanks in advance,
Robert Sanford


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

* Re: meson options - build vs target machine cpuflags question
  2022-03-10 14:46 meson options - build vs target machine cpuflags question Sanford, Robert
@ 2022-03-10 22:54 ` Dan Gora
  2022-03-10 22:58   ` Dan Gora
  2022-03-11  1:18   ` Stephen Hemminger
  0 siblings, 2 replies; 8+ messages in thread
From: Dan Gora @ 2022-03-10 22:54 UTC (permalink / raw)
  To: Sanford, Robert; +Cc: users

On Thu, Mar 10, 2022 at 11:46 AM Sanford, Robert <rsanford@akamai.com> wrote:
>
> Hello All,
>
> We build a DPDK 21.05 app on Intel x86 machines with RDSEED in cpuflags, and may run it on machines *without* RDSEED.
> This results in a fatal error ...
> ERROR: This system does not support "RDSEED".
> Please check that RTE_MACHINE is set correctly.
> EAL: FATAL: unsupported cpu type.
>
> I try adding -Dplatform=haswell when running meson, (because gcc man page indicates that haswell is last arch w/o RDSEED) but get the same result.
> Until we resolve it, our workaround is changing the error-out in rte_cpu_is_supported() to just print a warning and continue.
> We don't have direct access to automated build machines, we go through change request processes, and so we can't rapidly try too many things.
>
> Is there a better meson option, such as machine=haswell, or something else that will work?
>
> Thanks in advance,
> Robert Sanford

I tried four times to get a simple fix for this (and the lack of
getentropy() on older glibc) to determine the entropy source at run
time and got nothing but an endless raft of shit and ridiculous
criticisms that I completely gave up trying to contribute to DPDK ever
again.

The DPDK developers think that it's your responsibility to have a
separate build system for each of your target systems and platforms
and that if you don't you're basically a big dummy.

Don't believe me?  Go look through the archives:

"[PATCH 2/2] eal: resolve getentropy at run time for random seed"
"[PATCH v4 2/2] eal: emulate glibc getentropy for initial random seed".

I suggest that you just fork DPDK and use one of those patches and
just maintain a separate DPDK tree.  That's what I did.  It's way
easier than trying to get anything upstream.

thanks
dan

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

* Re: meson options - build vs target machine cpuflags question
  2022-03-10 22:54 ` Dan Gora
@ 2022-03-10 22:58   ` Dan Gora
  2022-03-10 23:39     ` Sanford, Robert
  2022-03-11  1:18   ` Stephen Hemminger
  1 sibling, 1 reply; 8+ messages in thread
From: Dan Gora @ 2022-03-10 22:58 UTC (permalink / raw)
  To: Sanford, Robert; +Cc: users

On Thu, Mar 10, 2022 at 7:54 PM Dan Gora <dg@adax.com> wrote:
>
> On Thu, Mar 10, 2022 at 11:46 AM Sanford, Robert <rsanford@akamai.com> wrote:
> >
> > Hello All,
> >
> > We build a DPDK 21.05 app on Intel x86 machines with RDSEED in cpuflags, and may run it on machines *without* RDSEED.
> > This results in a fatal error ...
> > ERROR: This system does not support "RDSEED".
> > Please check that RTE_MACHINE is set correctly.
> > EAL: FATAL: unsupported cpu type.
> >
> > I try adding -Dplatform=haswell when running meson, (because gcc man page indicates that haswell is last arch w/o RDSEED) but get the same result.
> > Until we resolve it, our workaround is changing the error-out in rte_cpu_is_supported() to just print a warning and continue.
> > We don't have direct access to automated build machines, we go through change request processes, and so we can't rapidly try too many things.
> >
> > Is there a better meson option, such as machine=haswell, or something else that will work?
> >
> > Thanks in advance,
> > Robert Sanford
>
> I tried four times to get a simple fix for this (and the lack of
> getentropy() on older glibc) to determine the entropy source at run
> time and got nothing but an endless raft of shit and ridiculous
> criticisms that I completely gave up trying to contribute to DPDK ever
> again.
>
> The DPDK developers think that it's your responsibility to have a
> separate build system for each of your target systems and platforms
> and that if you don't you're basically a big dummy.
>
> Don't believe me?  Go look through the archives:
>
> "[PATCH 2/2] eal: resolve getentropy at run time for random seed"
> "[PATCH v4 2/2] eal: emulate glibc getentropy for initial random seed".

Sorry, those are the getentropy() fixes.  This is the fix to determine
RDSEED at runtime:

"[PATCH 0/2] eal: choose initial PRNG seed source at runtime"

It was acked but I don't think that it was ever merged.

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

* Re: meson options - build vs target machine cpuflags question
  2022-03-10 22:58   ` Dan Gora
@ 2022-03-10 23:39     ` Sanford, Robert
  2022-03-11  0:00       ` Dan Gora
  0 siblings, 1 reply; 8+ messages in thread
From: Sanford, Robert @ 2022-03-10 23:39 UTC (permalink / raw)
  To: Dan Gora; +Cc: users

Hi Dan,
Thanks for the response. It is a lot of work and patience to get a change accepted. I admire all the guys who do it on a regular basis.

My problem turns out to be twofold: 1) Build machine has a CPU flag that the target does not have, and I was building "native".
2) Since I was building native, I was enabling RDSEED instructions even though the CPU doesn't support them. Fortunately, getentropy(3) works in our env.

I found the answer, by digging into the meson.build files, especially in config/ and config/x86/, and the resulting files created in the build dir.
Passing -Dplatform=haswell to meson did almost nothing AFAICS, still "native".
When I switch to -Dmachine=haswell, it has the desired effect on both build.ninja (generating -march=haswell for gcc)
and rte_build_config.h (disabling RDSEED).

Regards,
Robert Sanford

-----Original Message-----
From: Dan Gora <dg@adax.com>
Date: Thursday, March 10, 2022 at 5:59 PM
To: "Sanford, Robert" <rsanford@akamai.com>
Cc: "users@dpdk.org" <users@dpdk.org>
Subject: Re: meson options - build vs target machine cpuflags question

On Thu, Mar 10, 2022 at 7:54 PM Dan Gora <dg@adax.com> wrote:
>
> On Thu, Mar 10, 2022 at 11:46 AM Sanford, Robert <rsanford@akamai.com> wrote:
> >
> > Hello All,
> >
> > We build a DPDK 21.05 app on Intel x86 machines with RDSEED in cpuflags, and may run it on machines *without* RDSEED.
> > This results in a fatal error ...
> > ERROR: This system does not support "RDSEED".
> > Please check that RTE_MACHINE is set correctly.
> > EAL: FATAL: unsupported cpu type.
> >
> > I try adding -Dplatform=haswell when running meson, (because gcc man page indicates that haswell is last arch w/o RDSEED) but get the same result.
> > Until we resolve it, our workaround is changing the error-out in rte_cpu_is_supported() to just print a warning and continue.
> > We don't have direct access to automated build machines, we go through change request processes, and so we can't rapidly try too many things.
> >
> > Is there a better meson option, such as machine=haswell, or something else that will work?
> >
> > Thanks in advance,
> > Robert Sanford
>
> I tried four times to get a simple fix for this (and the lack of
> getentropy() on older glibc) to determine the entropy source at run
> time and got nothing but an endless raft of shit and ridiculous
> criticisms that I completely gave up trying to contribute to DPDK ever
> again.
>
> The DPDK developers think that it's your responsibility to have a
> separate build system for each of your target systems and platforms
> and that if you don't you're basically a big dummy.
>
> Don't believe me?  Go look through the archives:
>
> "[PATCH 2/2] eal: resolve getentropy at run time for random seed"
> "[PATCH v4 2/2] eal: emulate glibc getentropy for initial random seed".

Sorry, those are the getentropy() fixes.  This is the fix to determine
RDSEED at runtime:

"[PATCH 0/2] eal: choose initial PRNG seed source at runtime"

It was acked but I don't think that it was ever merged.


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

* Re: meson options - build vs target machine cpuflags question
  2022-03-10 23:39     ` Sanford, Robert
@ 2022-03-11  0:00       ` Dan Gora
  0 siblings, 0 replies; 8+ messages in thread
From: Dan Gora @ 2022-03-11  0:00 UTC (permalink / raw)
  To: Sanford, Robert; +Cc: users

On Thu, Mar 10, 2022 at 8:39 PM Sanford, Robert <rsanford@akamai.com> wrote:
>
> Hi Dan,
> Thanks for the response. It is a lot of work and patience to get a change accepted. I admire all the guys who do it on a regular basis.
>
> My problem turns out to be twofold: 1) Build machine has a CPU flag that the target does not have, and I was building "native".
> 2) Since I was building native, I was enabling RDSEED instructions even though the CPU doesn't support them. Fortunately, getentropy(3) works in our env.
>
> I found the answer, by digging into the meson.build files, especially in config/ and config/x86/, and the resulting files created in the build dir.
> Passing -Dplatform=haswell to meson did almost nothing AFAICS, still "native".
> When I switch to -Dmachine=haswell, it has the desired effect on both build.ninja (generating -march=haswell for gcc)
> and rte_build_config.h (disabling RDSEED).

yeah I recall that the meson makefiles had a couple of changes that
needed to be done.  The patch worked by compiling the code with the
RDSEED instructions regardless of whether or not the build platform
supported them, but then checking the CPU flags at run time to see if
it could be accessed or not.  But that should work too, you just can't
access RDSEED on platforms which support it, but whatever..
getentropy() will work.

thanks
dan

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

* Re: meson options - build vs target machine cpuflags question
  2022-03-10 22:54 ` Dan Gora
  2022-03-10 22:58   ` Dan Gora
@ 2022-03-11  1:18   ` Stephen Hemminger
  2022-03-11  2:45     ` Dan Gora
  1 sibling, 1 reply; 8+ messages in thread
From: Stephen Hemminger @ 2022-03-11  1:18 UTC (permalink / raw)
  To: Dan Gora; +Cc: Sanford, Robert, users

On Thu, 10 Mar 2022 19:54:48 -0300
Dan Gora <dg@adax.com> wrote:

> On Thu, Mar 10, 2022 at 11:46 AM Sanford, Robert <rsanford@akamai.com> wrote:
> >
> > Hello All,
> >
> > We build a DPDK 21.05 app on Intel x86 machines with RDSEED in cpuflags, and may run it on machines *without* RDSEED.
> > This results in a fatal error ...
> > ERROR: This system does not support "RDSEED".
> > Please check that RTE_MACHINE is set correctly.
> > EAL: FATAL: unsupported cpu type.
> >
> > I try adding -Dplatform=haswell when running meson, (because gcc man page indicates that haswell is last arch w/o RDSEED) but get the same result.
> > Until we resolve it, our workaround is changing the error-out in rte_cpu_is_supported() to just print a warning and continue.
> > We don't have direct access to automated build machines, we go through change request processes, and so we can't rapidly try too many things.
> >
> > Is there a better meson option, such as machine=haswell, or something else that will work?
> >
> > Thanks in advance,
> > Robert Sanford  
> 
> I tried four times to get a simple fix for this (and the lack of
> getentropy() on older glibc) to determine the entropy source at run
> time and got nothing but an endless raft of shit and ridiculous
> criticisms that I completely gave up trying to contribute to DPDK ever
> again.
> 
> The DPDK developers think that it's your responsibility to have a
> separate build system for each of your target systems and platforms
> and that if you don't you're basically a big dummy.
> 
> Don't believe me?  Go look through the archives:
> 
> "[PATCH 2/2] eal: resolve getentropy at run time for random seed"
> "[PATCH v4 2/2] eal: emulate glibc getentropy for initial random seed".
> 
> I suggest that you just fork DPDK and use one of those patches and
> just maintain a separate DPDK tree.  That's what I did.  It's way
> easier than trying to get anything upstream.
> 
> thanks
> dan

The ideal solution would be able to use gcc function attributes and dyanmic
linking to do this at runtime - ie multiversion library.  But this opens a set
of different problems like compiler versions and testing.

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

* Re: meson options - build vs target machine cpuflags question
  2022-03-11  1:18   ` Stephen Hemminger
@ 2022-03-11  2:45     ` Dan Gora
  2022-03-11  3:01       ` Stephen Hemminger
  0 siblings, 1 reply; 8+ messages in thread
From: Dan Gora @ 2022-03-11  2:45 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: Sanford, Robert, users

On Thu, Mar 10, 2022 at 10:19 PM Stephen Hemminger
<stephen@networkplumber.org> wrote:
>
> The ideal solution would be able to use gcc function attributes and dyanmic
> linking to do this at runtime - ie multiversion library.  But this opens a set
> of different problems like compiler versions and testing.

Detecting RDSEED at run time doesn't require any of that, but working
around the possible absence of getentropy() does.  However none of my
proposed solutions, including dynamic linking was deemed suitable.. or
something, so I gave up.

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

* Re: meson options - build vs target machine cpuflags question
  2022-03-11  2:45     ` Dan Gora
@ 2022-03-11  3:01       ` Stephen Hemminger
  0 siblings, 0 replies; 8+ messages in thread
From: Stephen Hemminger @ 2022-03-11  3:01 UTC (permalink / raw)
  To: Dan Gora; +Cc: Sanford, Robert, users

On Thu, 10 Mar 2022 23:45:39 -0300
Dan Gora <dg@adax.com> wrote:

> On Thu, Mar 10, 2022 at 10:19 PM Stephen Hemminger
> <stephen@networkplumber.org> wrote:
> >
> > The ideal solution would be able to use gcc function attributes and dyanmic
> > linking to do this at runtime - ie multiversion library.  But this opens a set
> > of different problems like compiler versions and testing.  
> 
> Detecting RDSEED at run time doesn't require any of that, but working
> around the possible absence of getentropy() does.  However none of my
> proposed solutions, including dynamic linking was deemed suitable.. or
> something, so I gave up.

I don't blame you build systems suck.

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

end of thread, other threads:[~2022-03-11  3:01 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-10 14:46 meson options - build vs target machine cpuflags question Sanford, Robert
2022-03-10 22:54 ` Dan Gora
2022-03-10 22:58   ` Dan Gora
2022-03-10 23:39     ` Sanford, Robert
2022-03-11  0:00       ` Dan Gora
2022-03-11  1:18   ` Stephen Hemminger
2022-03-11  2:45     ` Dan Gora
2022-03-11  3:01       ` Stephen Hemminger

DPDK usage discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://inbox.dpdk.org/users/0 users/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 users users/ http://inbox.dpdk.org/users \
		users@dpdk.org
	public-inbox-index users

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.users


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git