You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/09/26 11:20:54 UTC
[GitHub] [incubator-nuttx] CV-Bowen opened a new pull request, #7188: vfs/fs_poll: not clear POLLIN event if POLLHUP or POLLERR set
CV-Bowen opened a new pull request, #7188:
URL: https://github.com/apache/incubator-nuttx/pull/7188
Signed-off-by: wangbowen6 <wa...@xiaomi.com>
## Summary
usrsocktest failed:
```
nsh> usrsocktest
Starting unit-tests...
Testing group "char_dev" =>
Group "char_dev": [OK]
Testing group "no_daemon" =>
Group "no_daemon": [OK]
Testing group "basic_daemon" =>
Group "basic_daemon": [OK]
Testing group "basic_connect" =>
Group "basic_connect": [OK]
Testing group "basic_connect_delay" =>
Group "basic_connect_delay": [OK]
Testing group "no_block_connect" =>
Group "no_block_connect": [OK]
Testing group "basic_send" =>
Group "basic_send": [OK]
Testing group "no_block_send" =>
Group "no_block_send": [OK]
Testing group "block_send" =>
Group "block_send": [OK]
Testing group "no_block_recv" =>
Group "no_block_recv": [OK]
Testing group "block_recv" =>
Group "block_recv": [OK]
Testing group "remote_disconnect" =>
[TEST ASSERT FAILED!]
In function "remote_disconnect_poll":
line 688: Assertion `(ssize_t)((pfd.revents & (0x01))) == (ssize_t)(((0x01)))' failed.
got value: 0
should be: 1
Group "remote_disconnect": [FAILED]
Testing group "basic_setsockopt" =>
Group "basic_setsockopt": [OK]
Testing group "basic_getsockopt" =>
Group "basic_getsockopt": [OK]
Testing group "basic_getsockname" =>
Group "basic_getsockname": [OK]
Testing group "wake_with_signal" =>
Group "wake_with_signal": [OK]
Testing group "multithread" =>
Group "multithread": [OK]
Unit-test groups done... OK:16, FAILED:1, TOTAL:17
```
In poll_nofity(), should't clear POLLIN if POLLERR or POLLOUT set.
## Impact
All the poll operation.
## Testing
sim:usrsocktest pss, sim:ostest pass.
--
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: commits-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] CV-Bowen commented on pull request #7188: vfs/fs_poll: not clear POLLIN event if POLLHUP or POLLERR set
Posted by GitBox <gi...@apache.org>.
CV-Bowen commented on PR #7188:
URL: https://github.com/apache/incubator-nuttx/pull/7188#issuecomment-1258010844
The CI Failed, seems unrelated to this PR, rebase to HEAD and trigger the CI.
--
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: commits-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] masayuki2009 merged pull request #7188: vfs/fs_poll: not clear POLLIN event if POLLHUP or POLLERR set
Posted by GitBox <gi...@apache.org>.
masayuki2009 merged PR #7188:
URL: https://github.com/apache/incubator-nuttx/pull/7188
--
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: commits-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org