Skip to content

Update granular lock test framework#65

Merged
rohitmadan07 merged 1 commit into
granular_lock_test_setupfrom
review_granular_lock_test_setup
May 21, 2025
Merged

Update granular lock test framework#65
rohitmadan07 merged 1 commit into
granular_lock_test_setupfrom
review_granular_lock_test_setup

Conversation

@chinglee-iot

Copy link
Copy Markdown
Owner

Description

  • Use other kernel branch for unit test compiler error. These fixes should be merged back to feature branch.
  • Fix smp unit test introduced by granular lock feature change
  • Adding granular_lock_timer test group

Test Steps

Filename Line Coverage Functions Branches
event_groups.c 76.0% (187/246) 72.7% (16/22) 65.5% (72/110)
list.c 100.0% (41/41) 100.0% (5/5) 100.0% (6/6)
queue.c 89.0% (753/846) 92.6% (50/54) 94.2% (458/486)
stream_buffer.c 84.4% (345/409) 80.0% (28/35) 83.9% (198/236)
tasks.c 95.5% (1539/1612) 97.1% (100/103) 89.0% (1020/1146)
timers.c 91.3% (264/289) 91.4% (32/35) 65.4% (83/127)

Checklist:

  • I have tested my changes. No regression in existing tests.
  • I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

* Use other kernel branch for unit test compiler error. These fixes should be merged back to feature branch.
* Fix smp unit test introduced by granular lock feature change
* Adding granular_lock_timer test group
@rohitmadan07 rohitmadan07 merged commit b00f7af into granular_lock_test_setup May 21, 2025
11 of 40 checks passed
rohitmadan07 added a commit that referenced this pull request May 22, 2025
* Adding granular lock nullfication utest

* Update test case for critical nesting problem in xQueueReceive

* Update granular lock functional tests for interrupt status

* Add more test cases to cover the following combination

  - xQueueReceive vs vTaskDelete
    - Causes issue via: vTaskPlaceOnEventList ? prvAddCurrentTaskToDelayedList
  - xQueueReceive vs vTaskSuspend
    - Causes issue via: vTaskPlaceOnEventList ? prvAddCurrentTaskToDelayedList
  - xQueueSend vs vTaskDelete
    - Causes issue via: vTaskPlaceOnEventList ? prvAddCurrentTaskToDelayedList
  - xQueueSend vs vTaskSuspend
    - Causes issue via: vTaskPlaceOnEventList ? prvAddCurrentTaskToDelayedList
  - xEventGroupWaitBits vs vTaskDelete
    - Causes issue via: vTaskPlaceOnUnorderedEventList ? prvAddCurrentTaskToDelayedList
  - xEventGroupWaitBits vs vTaskSuspend
    - Causes issue via: vTaskPlaceOnUnorderedEventList ? prvAddCurrentTaskToDelayedList
  - Semaphore take (xSemaphoreTake) vs vTaskDelete
    - Causes issue via: vTaskPlaceOnEventList ? prvAddCurrentTaskToDelayedList
  - Semaphore take (xSemaphoreTake) vs vTaskSuspend
    - Causes issue via: vTaskPlaceOnEventList ? prvAddCurrentTaskToDelayedList
  - Semaphore give (xSemaphoreGive) vs vTaskDelete
    - Causes issue via: vTaskPlaceOnEventList ? prvAddCurrentTaskToDelayedList
  - Semaphore give (xSemaphoreGive) vs vTaskSuspend
    - Causes issue via: vTaskPlaceOnEventList ? prvAddCurrentTaskToDelayedList

* Add declaration of port data group critical section

* Modify smp Makefile

* Fix failing smp tests

* Separate tests based on timers

* Update granular lock test framework (#65)

* Use other kernel branch for unit test compiler error. These fixes should be merged back to feature branch.
* Fix smp unit test introduced by granular lock feature change
* Adding granular_lock_timer test group

* Revert some makefile changes

* Revert unecessary changes

* Add newline at end of file

---------

Co-authored-by: Ching-Hsin,Lee <chinglee@amazon.com>
Co-authored-by: Ubuntu <ubuntu@ip-172-31-34-119.us-west-2.compute.internal>
Co-authored-by: chinglee-iot <61685396+chinglee-iot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants