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

sanitycheck failed issue in parallel running. #21637

Closed
chen-png opened this issue Jan 2, 2020 · 2 comments · Fixed by #21684
Closed

sanitycheck failed issue in parallel running. #21637

chen-png opened this issue Jan 2, 2020 · 2 comments · Fixed by #21684
Assignees
Labels
area: Sanitycheck Sanitycheck has been renamed to Twister bug The issue is a bug, or the PR is fixing a bug Duplicate This issue is a duplicate of another issue (please specify) priority: low Low impact/importance bug

Comments

@chen-png
Copy link
Collaborator

chen-png commented Jan 2, 2020

Describe the bug
use the sanitycheck to run all tests with four boards, it failed with error "generated an exception: [Errno 24] Too many open files", i found this error is caused by the code "log_out_fp = open(self.log, "wt")" in DeviceHandler, the system limits the max open files to 1024.
then I modified the /etc/secruity/limits.conf to set the limited open files to 2048, and run sanitycheck again, it failed with "ValueError: filedescriptor out of range in select()", caused by code" readable, _, _ = select.select(readlist, [], [], self.timeout)

To Reproduce
Steps to reproduce the behavior:

  1. connect four boars to pc and run all tests use sanitycheck
  2. sanitycheck --generate-hardware-map map4.yml
  3. sanitycheck --hardware-map map4.yml --device-testing -T tests -v
  4. See error

Screenshots or console output

Exception in thread Thread-635:
Traceback (most recent call last):
File "/usr/lib64/python3.6/threading.py", line 916, in _bootstrap_inner
self.run()
File "/usr/lib64/python3.6/threading.py", line 864, in run
self._target(*self._args, **self._kwargs)
File "/home/chenpeng/zephyrproject/zephyr/scripts/sanitycheck", line 628, in monitor_serial
readable, _, _ = select.select(readlist, [], [], self.timeout)
ValueError: filedescriptor out of range in select()

INFO - 666/720 mimxrt1050_evk tests/arch/arm/arm_interrupt/arch.interrupt.no_optimizations FAILED N/A (device 8.871s)
ERROR - see: sanity-out/mimxrt1050_evk/tests/arch/arm/arm_interrupt/arch.interrupt.no_optimizations/build.log
Exception in thread Thread-636:
Traceback (most recent call last):
File "/usr/lib64/python3.6/threading.py", line 916, in _bootstrap_inner
self.run()
File "/usr/lib64/python3.6/threading.py", line 864, in run
self._target(*self._args, **self._kwargs)
File "/home/chenpeng/zephyrproject/zephyr/scripts/sanitycheck", line 628, in monitor_serial
readable, _, _ = select.select(readlist, [], [], self.timeout)
ValueError: filedescriptor out of range in select()

Environment (please complete the following information):

  • OS: fedora28
  • Toolchain: zephyr-sdk-0.10.3
  • Commit ID: 4e135d7
@chen-png chen-png added the bug The issue is a bug, or the PR is fixing a bug label Jan 2, 2020
@aescolar aescolar added area: Sanitycheck Sanitycheck has been renamed to Twister priority: low Low impact/importance bug labels Jan 2, 2020
@aescolar
Copy link
Member

aescolar commented Jan 2, 2020

CC @nashif

@nashif nashif self-assigned this Jan 2, 2020
@nashif
Copy link
Member

nashif commented Jan 2, 2020

duplicate of #20974

@nashif nashif added the Duplicate This issue is a duplicate of another issue (please specify) label Jan 2, 2020
nashif added a commit to nashif/zephyr that referenced this issue Jan 4, 2020
Make sure we close the pipe file handles after we are done, otherwise we
will end up with too many open file descriptors and crash...

Fixes zephyrproject-rtos#20974
Fixes zephyrproject-rtos#21637

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
nashif added a commit that referenced this issue Jan 6, 2020
Make sure we close the pipe file handles after we are done, otherwise we
will end up with too many open file descriptors and crash...

Fixes #20974
Fixes #21637

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Sanitycheck Sanitycheck has been renamed to Twister bug The issue is a bug, or the PR is fixing a bug Duplicate This issue is a duplicate of another issue (please specify) priority: low Low impact/importance bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants