You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by GitBox <gi...@apache.org> on 2021/03/30 19:05:32 UTC
[GitHub] [trafficserver] Johanhen opened a new issue #7656: Build fails on FreeBSD 13+
Johanhen opened a new issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656
The build of traffic server 8.1.1 (and probably other versions as well fails on FreeBSD 13.x and newer.
As noted by Brian in the mailing list
FreeBSD 13 added eventfd(2) [1], which means that the check on HAVE_EVENTFD in EventNotify.cc [2] can no longer assume that if you have eventfd(2) you also have epoll.
1. https://github.com/freebsd/freebsd-src/commit/7a202823aa54ba18c485bdbcf355269bcfee1ab9#diff-7ea785358f7ef658d52a5d985ea53f03c09cd0bfee8e6a7403063774d3dc58a8
2. https://github.com/apache/trafficserver/blob/3d266379b816227928171d1f07de747f6a9a254b/src/tscore/EventNotify.cc#L34
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] bgaff edited a comment on issue #7656: Build fails on FreeBSD 13+
Posted by GitBox <gi...@apache.org>.
bgaff edited a comment on issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656#issuecomment-811158893
Yah sorry you need to do the same for the header file:
```
sed -Ei 's/#ifdef HAVE_EVENTFD/#if defined(HAVE_EVENTFD) \&\& TS_USE_EPOLL == 1' src/tscore/EventNotify.cc
sed -Ei 's/#ifdef HAVE_EVENTFD/#if defined(HAVE_EVENTFD) \&\& TS_USE_EPOLL == 1' include/tscore/EventNotify.h
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] bgaff commented on issue #7656: Build fails on FreeBSD 13+
Posted by GitBox <gi...@apache.org>.
bgaff commented on issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656#issuecomment-815266574
Yah I'll do it
On Wed, Apr 7, 2021, 14:04 Leif Hedstrom ***@***.***> wrote:
> Someone want to make a PR for this? :-)
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/trafficserver/issues/7656#issuecomment-815262049>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AAHHNGM2GV36JRQCPPW4FVLTHTCFPANCNFSM42CXW36Q>
> .
>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] bgaff commented on issue #7656: Build fails on FreeBSD 13+
Posted by GitBox <gi...@apache.org>.
bgaff commented on issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656#issuecomment-811158893
Yah sorry you need to do the same for the header file:
```
sed -Ei 's/#ifdef HAVE_EVENTFD/#if defined(HAVE_EVENTFD) \&\& TS_USE_EPOLL == 1' src/tscore/EventNotify.{cc,h}
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] bgaff commented on issue #7656: Build fails on FreeBSD 13+
Posted by GitBox <gi...@apache.org>.
bgaff commented on issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656#issuecomment-824499444
Sending pull request right now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] bgaff commented on issue #7656: Build fails on FreeBSD 13+
Posted by GitBox <gi...@apache.org>.
bgaff commented on issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656#issuecomment-811143735
Sorry my bad the autoconf script always defines TS_USE_EPOLL, you need to do:
The easiest fix is:
```
sed -Ei 's/#ifdef HAVE_EVENTFD/#if defined(HAVE_EVENTFD) \&\& TS_USE_EPOLL == 1' src/tscore/EventNotify.cc
```
That should fix it, but that whole class is kinda ridiculous anyway. Using eventfd + epoll in this way is overkill, the ink_condwait methods should be preferred for many reasons as they solve the same problem:
1. ink_condwait and friends will wrap into pthread_cond_(timed)wait which are as the name implies posix compliant.
2. When using pthread_cond_wait In the case of a signal before a wait no syscall is ever required on most platforms, ie. linux uses futex(2) on contention but it will check the state in userspace before making a syscall.
3. Using eventfd(2) in this way doesn't make sense as you're never acutally using the FD with anything other than this epoll fd.
4. Even if you choose to use eventfd(2) in this way it should probably use poll(2) as it's posix compliant, whereas epoll is linux specific.
Eventfds make sense in situations where you have an existing pollfd / epollfd / etc where you want to tack on some event notification, when you're creating an eventfd to use only with poll/epoll/etc there usually is a better way.
My suggestion is: just yank out all of this eventfd / epoll code and make it use ink_cond_wait/ink_cond_timedwait and be done with it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] Johanhen commented on issue #7656: Build fails on FreeBSD 13+
Posted by GitBox <gi...@apache.org>.
Johanhen commented on issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656#issuecomment-811258463
On 31/03/2021 17:31, Brian Geffon wrote:
>
> Yah sorry you need to do the same for the header file:
>
> |sed -Ei 's/#ifdef HAVE_EVENTFD/#if defined(HAVE_EVENTFD) \&\&
> TS_USE_EPOLL == 1' src/tscore/EventNotify.{cc,h} |
>
>
It compiles now,
sed -i '' 's/#ifdef HAVE_EVENTFD/#if defined(HAVE_EVENTFD) \&\&
TS_USE_EPOLL == 1/' work/trafficserver-8.1.1/src/tscore/EventNotify.cc
sed -i '' 's/#ifdef HAVE_EVENTFD/#if defined(HAVE_EVENTFD) \&\&
TS_USE_EPOLL == 1/' work/trafficserver-8.1.1/include/tscore/EventNotify.h
did the trick, thank you!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] masaori335 closed issue #7656: Build fails on FreeBSD 13+
Posted by GitBox <gi...@apache.org>.
masaori335 closed issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] bgaff commented on issue #7656: Build fails on FreeBSD 13+
Posted by GitBox <gi...@apache.org>.
bgaff commented on issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656#issuecomment-810517643
The easiest fix:
```
sed -Ei 's/#ifdef HAVE_EVENTFD/#if defined(HAVE_EVENTFD) \&\& defined(TS_USE_EPOLL)' src/tscore/EventNotify.cc
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] zwoop commented on issue #7656: Build fails on FreeBSD 13+
Posted by GitBox <gi...@apache.org>.
zwoop commented on issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656#issuecomment-815262049
Someone want to make a PR for this? :-)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] Johanhen commented on issue #7656: Build fails on FreeBSD 13+
Posted by GitBox <gi...@apache.org>.
Johanhen commented on issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656#issuecomment-810958283
Unfortunately i still receive that error after the replacement off #ifdef HAVE_EVENTFD by #if defined(HAVE_EVENTFD) && defined(TS_USE_EPOLL)
libtool: compile: c++ -DHAVE_CONFIG_H -I. -I../../include -D_GLIBCXX_USE_C99 -D_GLIBCXX_USE_C99_MATH -D_GLIBCXX_USE_C99_MATH_TR1 -Dfreebsd -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/eventsystem -I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/net -I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/aio -I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/hostdb -I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/cache -I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/utils -I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/dns -I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/include -I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/include/records -I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/lib -I/usr/local/include -I/usr/local/include/tcl8.6 -I/usr/ports/ww
w/trafficserver/work/trafficserver-8.1.1/lib/yamlcpp/include -isystem /usr/local/include -D_GNU_SOURCE -I/usr/include -DOPENSSL_NO_SSL_INTERN -I/usr/local/include -std=c++17 -g -pipe -Wall -Wno-deprecated-declarations -Qunused-arguments -Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -fno-strict-aliasing -Wno-invalid-offsetof -mcx16 -O2 -pipe -fstack-protector-strong -isystem /usr/local/include -fno-strict-aliasing -isystem /usr/local/include -c EventNotify.cc -fPIC -DPIC -o .libs/EventNotify.o
EventNotify.cc:37:10: fatal error: 'sys/epoll.h' file not found
#include <sys/epoll.h>
^~~~~~~~~~~~~
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] diizzyy commented on issue #7656: Build fails on FreeBSD 13+
Posted by GitBox <gi...@apache.org>.
diizzyy commented on issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656#issuecomment-823895781
@bgaff
Any progress regarding this issue?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] diizzyy commented on issue #7656: Build fails on FreeBSD 13+
Posted by GitBox <gi...@apache.org>.
diizzyy commented on issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656#issuecomment-825933641
Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] Johanhen commented on issue #7656: Build fails on FreeBSD 13+
Posted by GitBox <gi...@apache.org>.
Johanhen commented on issue #7656:
URL: https://github.com/apache/trafficserver/issues/7656#issuecomment-811154948
On 31/03/2021 17:11, Brian Geffon wrote:
>
> Sorry my bad the autoconf script always defines TS_USE_EPOLL, you need
> to do:
>
> The easiest fix is:
>
> |sed -Ei 's/#ifdef HAVE_EVENTFD/#if defined(HAVE_EVENTFD) \&\&
> TS_USE_EPOLL == 1' src/tscore/EventNotify.cc |
>
> That should fix it, but that whole class is kinda ridiculous anyway.
> Using eventfd + epoll in this way is overkill, the ink_condwait
> methods should be preferred for many reasons as they solve the same
> problem:
>
> 1. ink_condwait and friends will wrap into pthread_cond_(timed)wait
> which are as the name implies posix compliant.
> 2. When using pthread_cond_wait In the case of a signal before a wait
> no syscall is ever required on most platforms, ie. linux uses
> futex(2) on contention but it will check the state in userspace
> before making a syscall.
> 3. Using eventfd(2) in this way doesn't make sense as you're never
> acutally using the FD with anything other than this epoll fd.
> 4. Even if you choose to use eventfd(2) in this way it should
> probably use poll(2) as it's posix compliant, whereas epoll is
> linux specific.
>
> Eventfds make sense in situations where you have an existing pollfd /
> epollfd / etc where you want to tack on some event notification, when
> you're creating an eventfd to use only with poll/epoll/etc there
> usually is a better way.
>
> My suggestion is: just yank out all of this eventfd / epoll code and
> make it use ink_cond_wait/ink_cond_timedwait and be done with it.
>
I now get the following error.
libtool: compile: c++ -DHAVE_CONFIG_H -I. -I../../include
-D_GLIBCXX_USE_C99 -D_GLIBCXX_USE_C99_MATH -D_GLIBCXX_USE_C99_MATH_TR1
-Dfreebsd -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1
-I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/eventsystem
-I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/net
-I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/aio
-I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/hostdb
-I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/cache
-I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/utils
-I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/iocore/dns
-I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/include
-I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/include/records
-I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/lib
-I/usr/local/include -I/usr/local/include/tcl8.6
-I/usr/ports/www/trafficserver/work/trafficserver-8.1.1/lib/yamlcpp/include
-isystem /usr/local/include -D_GNU_SOURCE -I/usr/include
-DOPENSSL_NO_SSL_INTERN -I/usr/local/include -std=c++17 -g -pipe -Wall
-Wno-deprecated-declarations -Qunused-arguments -Wextra
-Wno-ignored-qualifiers -Wno-unused-parameter -fno-strict-aliasing
-Wno-invalid-offsetof -mcx16 -O2 -pipe -fstack-protector-strong -isystem
/usr/local/include -fno-strict-aliasing -isystem /usr/local/include -c
EventNotify.cc -fPIC -DPIC -o .libs/EventNotify.o
EventNotify.cc:58:18: error: use of undeclared identifier 'm_cond'
ink_cond_init(&m_cond);
^
EventNotify.cc:59:19: error: use of undeclared identifier 'm_mutex'
ink_mutex_init(&m_mutex);
^
EventNotify.cc:75:20: error: use of undeclared identifier 'm_cond'
ink_cond_signal(&m_cond);
^
EventNotify.cc:102:18: error: use of undeclared identifier 'm_cond'
ink_cond_wait(&m_cond, &m_mutex);
^
EventNotify.cc:102:27: error: use of undeclared identifier 'm_mutex'
ink_cond_wait(&m_cond, &m_mutex);
^
EventNotify.cc:143:30: error: use of undeclared identifier 'm_cond'
return ink_cond_timedwait(&m_cond, &m_mutex, &abstime);
^
EventNotify.cc:143:39: error: use of undeclared identifier 'm_mutex'
return ink_cond_timedwait(&m_cond, &m_mutex, &abstime);
^
EventNotify.cc:153:22: error: use of undeclared identifier 'm_mutex'
ink_mutex_acquire(&m_mutex);
^
EventNotify.cc:163:33: error: use of undeclared identifier 'm_mutex'
return ink_mutex_try_acquire(&m_mutex);
^
EventNotify.cc:173:22: error: use of undeclared identifier 'm_mutex'
ink_mutex_release(&m_mutex);
^
EventNotify.cc:183:21: error: use of undeclared identifier 'm_cond'
ink_cond_destroy(&m_cond);
^
EventNotify.cc:184:22: error: use of undeclared identifier 'm_mutex'
ink_mutex_destroy(&m_mutex);
^
In file included from EventNotify.cc:30:
../../include/tscore/EventNotify.h:49:7: warning: private field
'm_event_fd' is not used [-Wunused-private-field]
int m_event_fd;
^
../../include/tscore/EventNotify.h:50:7: warning: private field
'm_epoll_fd' is not used [-Wunused-private-field]
int m_epoll_fd;
^
2 warnings and 12 errors generated.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org