Skip to content

Commit

Permalink
tests: ThriftTest: adjust resources for SSL test
Browse files Browse the repository at this point in the history
Since the file descriptor leak was plugged in the previous
commit, we can reduce memory requirements for this test.

Also, we can keep the stack sentinal as default.

Lastly, use `qemu_riscv64` instead of `qemu_riscv64_smp`
for now, as the latter seems to be experiencing weird issues.

Signed-off-by: Chris Friedt <[email protected]>
  • Loading branch information
cfriedt committed Jan 20, 2023
1 parent e6b46ca commit ae52a98
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 13 deletions.
11 changes: 7 additions & 4 deletions tests/lib/thrift/ThriftTest/prj.conf
Original file line number Diff line number Diff line change
Expand Up @@ -23,13 +23,16 @@ CONFIG_TEST_RANDOM_GENERATOR=y
CONFIG_NET_TEST=y
CONFIG_NET_DRIVERS=y
CONFIG_NET_LOOPBACK=y
# pthread_cond_wait() triggers sentinel for some reason
CONFIG_STACK_SENTINEL=n

# default stack sizes are not enough
CONFIG_ZTEST_STACK_SIZE=8192
CONFIG_MAIN_STACK_SIZE=2048
CONFIG_SYSTEM_WORKQUEUE_STACK_SIZE=2048
CONFIG_MAIN_STACK_SIZE=4096
CONFIG_SYSTEM_WORKQUEUE_STACK_SIZE=8192
CONFIG_THRIFTTEST_SERVER_STACK_SIZE=8192
CONFIG_NET_TCP_WORKQ_STACK_SIZE=4096
CONFIG_NET_MGMT_EVENT_STACK_SIZE=4096
CONFIG_IDLE_STACK_SIZE=4096
CONFIG_NET_RX_STACK_SIZE=8192

# Network address config
CONFIG_NET_IPV4=y
Expand Down
16 changes: 7 additions & 9 deletions tests/lib/thrift/ThriftTest/testcase.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ common:
- mps2_an385
- qemu_cortex_a53
- qemu_riscv32
- qemu_riscv64_smp
- qemu_riscv64
- qemu_x86_64
tests:
thrift.ThriftTest.newlib.binaryProtocol:
Expand All @@ -34,19 +34,17 @@ tests:
- CONFIG_MBEDTLS=y
- CONFIG_MBEDTLS_PEM_CERTIFICATE_FORMAT=y
- CONFIG_MBEDTLS_ENABLE_HEAP=y
- CONFIG_MBEDTLS_HEAP_SIZE=60000
- CONFIG_MBEDTLS_HEAP_SIZE=48000
- CONFIG_MBEDTLS_SSL_MAX_CONTENT_LEN=2048
- CONFIG_NET_SOCKETS_TLS_MAX_CONTEXTS=6
- CONFIG_NET_SOCKETS_SOCKOPT_TLS=y
- CONFIG_NET_TCP=y
- CONFIG_NET_SOCKETS=y
- CONFIG_NET_BUF_TX_COUNT=64
- CONFIG_NET_PKT_TX_COUNT=64
- CONFIG_NET_BUF_RX_COUNT=64
- CONFIG_NET_PKT_RX_COUNT=64
- CONFIG_POSIX_MAX_FDS=64
integration_platforms:
- mps2_an385
- CONFIG_NET_BUF_TX_COUNT=20
- CONFIG_NET_PKT_TX_COUNT=20
- CONFIG_NET_BUF_RX_COUNT=20
- CONFIG_NET_PKT_RX_COUNT=20
- CONFIG_POSIX_MAX_FDS=16
tags: newlib
# thrift.hello.native:
# arch_allow: posix
Expand Down

0 comments on commit ae52a98

Please sign in to comment.