[
https://issues.apache.org/jira/browse/PROTON-2231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17240977#comment-17240977
]
Andrew Stitcher commented on PROTON-2231:
-----------------------------------------
[~rkieley] Thank you for the debugging. However I think this is a misdiagnosis
of the issue:
The documentation for pn_listener_close() has nothing to say about how many
time it may be called so it may be called as many time as you like, except that
it will only close the one listener! However you are not allowed to call
pn_listener_close() after you have received the PN_LISTEN_CLOSE event for the
listener.
I think it is a totally legitimate test to call pn_listener_close() multiple
times for the listener. And the epoll implementation seems to fare correctly
under this test. So I'd say this has pointed out an actual bug in the libuv
proactor, not a bug in threadercizer.
Did I miss something or misunderstand?
> Assertion fail on macOS with libuv in c-threaderciser test
> ----------------------------------------------------------
>
> Key: PROTON-2231
> URL: https://issues.apache.org/jira/browse/PROTON-2231
> Project: Qpid Proton
> Issue Type: Bug
> Components: proton-c
> Affects Versions: proton-c-0.32.0
> Reporter: Jiri Daněk
> Assignee: Roddie Kieley
> Priority: Major
> Labels: freebsd, macOS
>
> As described on PROTON-2225, the test fails with assertion error. It is
> currently disabled on macOS for this reason (in .travis.yml).
> {noformat}
> 6: Test command: /usr/local/opt/python/libexec/bin/python
> "/Users/travis/build/jiridanek/qpid-proton/scripts/env.py" "--"
> "/Users/travis/build/jiridanek/qpid-proton/build/c/tests/c-threaderciser"
> 6: Test timeout computed to be: 1500
> 6: threaderciser start: threads=8, time=1, actions=[listen, close-listen,
> connect, close-connect, wake, timeout, cancel-timeout]
> 6: Assertion failed: (p->active > 0), function remove_active_lh, file
> /Users/travis/build/jiridanek/qpid-proton/c/src/proactor/libuv.c, line 392.
> 6/31 Test #6: c-threaderciser ..................***Failed 0.18 sec
> {noformat}
> If the test is meant to stay disabled for a longer time, it will have to be
> disabled in CMakeLists.txt, so that users compiling the project do not run it
> accidentally.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
