Skip to content

some tiny fixes#4

Merged
niehao100 merged 2 commits intostepfun-ai:mainfrom
fengidri:main
Aug 1, 2025
Merged

some tiny fixes#4
niehao100 merged 2 commits intostepfun-ai:mainfrom
fengidri:main

Conversation

@fengidri
Copy link
Contributor

@fengidri fengidri commented Aug 1, 2025

  1. fix undefined reference to symbol 'shm_open@@GLIBC_2.2.5'
  2. fix miss optimus.ops from tests/benchmark/bmk_comm_latency_multiserver.py

Remove unused packet.

Signed-off-by: Xuan Zhuo <xuanzhuo@linux.alibaba.com>
This project uses the librt interfaces, such as the shm_open family of
functions. In some Linux distributions, librt has already been merged
into glibc, so the rt library is not explicitly needed. However, in some
other distributions, linking errors may occur, such as the one shown
below. By adding -lrt during linking, we can ensure better compatibility
across different systems. This does not cause any problems on
distributions where librt is already included in glibc, as the required
symbols are still accessible.

/usr/bin/ld: ../../libaf.a(van.cc.o): undefined reference to symbol 'shm_open@@GLIBC_2.2.5'
/usr/bin/ld: /usr/lib64/librt.so.1: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status

Signed-off-by: Xuan Zhuo <xuanzhuo@linux.alibaba.com>
@zhouyu-sunny
Copy link
Collaborator

cc @niehao100 @ymjiang

@niehao100
Copy link
Collaborator

lgtm

1 similar comment
@niehao100
Copy link
Collaborator

lgtm

@niehao100 niehao100 merged commit 091940c into stepfun-ai:main Aug 1, 2025
1 check passed
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.

3 participants