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

tests: smp: cancel CONFIG_MP_NUM_CPUS limit of some testcases #33547

Merged
merged 2 commits into from
Apr 27, 2021

Conversation

enjiamai
Copy link
Collaborator

@enjiamai enjiamai commented Mar 22, 2021

This PR includes 2 commits:
The first commit try to remove CONFIG_MP_NUM_CPUS=1 configuration for the test of condvar,
sysmutex and semaphore, in order to test SMP condition more.

The second commit makes some changes on codvar test cases to fit SMP.

  1. Re-init the mutex to prevent affection from previous test cases.
  2. Shorter the test case execution time.
    Fixes qemu_cortex_a53_smp and qemu_x86_64 failed in tests/kernel/condvar/condvar while enabling for SMP  #33558.

Signed-off-by: Enjia Mai enjiax.mai@intel.com

@github-actions github-actions bot added area: Kernel area: Tests Issues related to a particular existing or missing test labels Mar 22, 2021
@enjiamai enjiamai requested a review from LeiW000 March 22, 2021 05:51
@enjiamai enjiamai force-pushed the remove_some_limit_cpu_nums branch from efc0f28 to 6bc7633 Compare March 27, 2021 14:42
@enjiamai enjiamai force-pushed the remove_some_limit_cpu_nums branch from 6bc7633 to ae169e8 Compare March 28, 2021 18:06
@enjiamai enjiamai force-pushed the remove_some_limit_cpu_nums branch 2 times, most recently from 6fccbc1 to a49f227 Compare April 25, 2021 07:51
Try to remove CONFIG_MP_NUM_CPUS=1 configuration for the test of
condvar, sysmutex and semaphore, in order to test SMP condition more.

Signed-off-by: Enjia Mai <enjiax.mai@intel.com>
@enjiamai enjiamai force-pushed the remove_some_limit_cpu_nums branch from a49f227 to e28bbef Compare April 25, 2021 08:01
@enjiamai enjiamai force-pushed the remove_some_limit_cpu_nums branch from e28bbef to 57e05a3 Compare April 25, 2021 12:13
Make some change on two codvar test cases to fit testing under SMP,
and shorter the test cases execution time.

Fixes zephyrproject-rtos#33558.

Signed-off-by: Enjia Mai <enjiax.mai@intel.com>
@enjiamai enjiamai force-pushed the remove_some_limit_cpu_nums branch from 57e05a3 to 9340965 Compare April 25, 2021 12:17
@nashif nashif merged commit d3747db into zephyrproject-rtos:master Apr 27, 2021
@enjiamai enjiamai deleted the remove_some_limit_cpu_nums branch April 28, 2021 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Kernel area: Tests Issues related to a particular existing or missing test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

qemu_cortex_a53_smp and qemu_x86_64 failed in tests/kernel/condvar/condvar while enabling for SMP
5 participants