Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nettest_omni: Remove duplicate variable definitions #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kraj
Copy link

@kraj kraj commented Aug 12, 2020

These defines are already defined in nettest_bsd.c and exported by
nettest_bsd.h this should fix build with -fno-common

Signed-off-by: Khem Raj [email protected]

These defines are already defined in nettest_bsd.c and exported by
nettest_bsd.h this should fix build with -fno-common

Signed-off-by: Khem Raj <[email protected]>
@westurner
Copy link

Would this also solve for "Unable to compile on Fedora 32" #42?

@kraj
Copy link
Author

kraj commented May 31, 2021

Would this also solve for "Unable to compile on Fedora 32" #42?

i think so

cliping added a commit to cliping/avocado-vt that referenced this pull request Jul 9, 2021
When run 'make' command to compile netperf, meet following error:
  gcc     -o netperf netperf.o netlib.o netsh.o nettest_bsd.o nettest_dlpi.o nettest_unix.o nettest_xti.o nettest_sctp.o nettest_sdp.o nettest_omni.o net_uuid.o dscp.o netcpu_none.o netrt_none.o netdrv_none.o netslot_none.o netsys_none.o netsec_none.o  -lm
  /usr/bin/ld: nettest_omni.o:(.bss+0x398): multiple definition of `loc_nodelay'; nettest_bsd.o:(.bss+0x20): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x39c): multiple definition of `rem_nodelay'; nettest_bsd.o:(.bss+0x24): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x3a0): multiple definition of `loc_sndavoid'; nettest_bsd.o:(.bss+0x30): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x3a4): multiple definition of `loc_rcvavoid'; nettest_bsd.o:(.bss+0x34): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x3a8): multiple definition of `rem_sndavoid'; nettest_bsd.o:(.bss+0x38): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x3ac): multiple definition of `rem_rcvavoid'; nettest_bsd.o:(.bss+0x3c): first defined here
  collect2: error: ld returned 1 exit status
  make[3]: *** [Makefile:443: netperf] Error 1

One patch [1] is to fix this issue. According to this patch to
update netperf package.

[1] HewlettPackard/netperf#46

Signed-off-by: lcheng <[email protected]>
cliping added a commit to cliping/avocado-vt that referenced this pull request Jul 22, 2021
When run 'make' command to compile netperf, meet following error:
  gcc     -o netperf netperf.o netlib.o netsh.o nettest_bsd.o nettest_dlpi.o nettest_unix.o nettest_xti.o nettest_sctp.o nettest_sdp.o nettest_omni.o net_uuid.o dscp.o netcpu_none.o netrt_none.o netdrv_none.o netslot_none.o netsys_none.o netsec_none.o  -lm
  /usr/bin/ld: nettest_omni.o:(.bss+0x398): multiple definition of `loc_nodelay'; nettest_bsd.o:(.bss+0x20): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x39c): multiple definition of `rem_nodelay'; nettest_bsd.o:(.bss+0x24): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x3a0): multiple definition of `loc_sndavoid'; nettest_bsd.o:(.bss+0x30): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x3a4): multiple definition of `loc_rcvavoid'; nettest_bsd.o:(.bss+0x34): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x3a8): multiple definition of `rem_sndavoid'; nettest_bsd.o:(.bss+0x38): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x3ac): multiple definition of `rem_rcvavoid'; nettest_bsd.o:(.bss+0x3c): first defined here
  collect2: error: ld returned 1 exit status
  make[3]: *** [Makefile:443: netperf] Error 1

One patch [1] is to fix this issue. According to this patch to
update netperf package.

[1] HewlettPackard/netperf#46

Signed-off-by: lcheng <[email protected]>
ana pushed a commit to ana/avocado-vt that referenced this pull request Aug 9, 2021
When run 'make' command to compile netperf, meet following error:
  gcc     -o netperf netperf.o netlib.o netsh.o nettest_bsd.o nettest_dlpi.o nettest_unix.o nettest_xti.o nettest_sctp.o nettest_sdp.o nettest_omni.o net_uuid.o dscp.o netcpu_none.o netrt_none.o netdrv_none.o netslot_none.o netsys_none.o netsec_none.o  -lm
  /usr/bin/ld: nettest_omni.o:(.bss+0x398): multiple definition of `loc_nodelay'; nettest_bsd.o:(.bss+0x20): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x39c): multiple definition of `rem_nodelay'; nettest_bsd.o:(.bss+0x24): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x3a0): multiple definition of `loc_sndavoid'; nettest_bsd.o:(.bss+0x30): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x3a4): multiple definition of `loc_rcvavoid'; nettest_bsd.o:(.bss+0x34): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x3a8): multiple definition of `rem_sndavoid'; nettest_bsd.o:(.bss+0x38): first defined here
  /usr/bin/ld: nettest_omni.o:(.bss+0x3ac): multiple definition of `rem_rcvavoid'; nettest_bsd.o:(.bss+0x3c): first defined here
  collect2: error: ld returned 1 exit status
  make[3]: *** [Makefile:443: netperf] Error 1

One patch [1] is to fix this issue. According to this patch to
update netperf package.

[1] HewlettPackard/netperf#46

Signed-off-by: lcheng <[email protected]>
sayboras added a commit to sayboras/netperf that referenced this pull request Aug 20, 2022
These defines are already defined in nettest_bsd.c and exported by
nettest_bsd.h this should fix build with -fno-common

Thanks to Khem Raj [email protected]
Relates: HewlettPackard#46
sayboras added a commit to sayboras/netperf that referenced this pull request Aug 20, 2022
These defines are already defined in nettest_bsd.c and exported by
nettest_bsd.h this should fix build with -fno-common

Thanks to Khem Raj [email protected]
Relates: HewlettPackard#46

Signed-off-by: Tam Mach <[email protected]>
jrajahalme added a commit to cilium/packer-ci-build that referenced this pull request Jan 4, 2023
Pull from sayboras/netperf that has the multiple definitions fix
(HewlettPackard/netperf#46).

Signed-off-by: Jarno Rajahalme <[email protected]>
Copy link

@jtaleric jtaleric left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, tested this on ubi9 and was able to successfully build netperf. Without this patch it was unable to build.

jrajahalme added a commit to cilium/packer-ci-build that referenced this pull request Feb 19, 2023
Pull from sayboras/netperf that has the multiple definitions fix
(HewlettPackard/netperf#46).

Signed-off-by: Jarno Rajahalme <[email protected]>
jrajahalme added a commit to cilium/packer-ci-build that referenced this pull request May 12, 2023
Pull from sayboras/netperf that has the multiple definitions fix
(HewlettPackard/netperf#46).

Signed-off-by: Jarno Rajahalme <[email protected]>
@abdhaleegit
Copy link

@jtaleric Can you please merge this PR, with out this netperf build fails for all platform

@PraveenPenguin
Copy link

@kirbychris @GavinB-hpe sorry for the spam can anyone you (not sure if any one of active here if not can you please help us to reach out to right person ) please help us to merge this PR we are kind of stuck here as we did not get maintainer list in repo

@jtaleric
Copy link

@jtaleric Can you please merge this PR, with out this netperf build fails for all platform

I don't have that authority 😢

@PraveenPenguin
Copy link

PraveenPenguin commented Aug 24, 2023

@HewlettPackard @ChrisLynchHPE can you please help here

@ChrisLynchHPE
Copy link
Member

I do not own this repository and have no knowledge on the code. I am unaware who even maintains this repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants