arc: Add support for cooperative sharing of VPX vector registers #148621
Annotations
1 error, 1 warning, and 7 notices
|
Run Compliance Tests
Process completed with exit code 1.
|
|
check-warns:
ClangFormat.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#clang-format for more details.
You may want to run clang-format on this change:
-#define ISR_VPX_LOCK 0
-#define ISR_VPX_UNLOCK 1
+#define ISR_VPX_LOCK 0
+#define ISR_VPX_UNLOCK 1
File:tests/arch/arc/arc_vpx_lock/src/main.c
Line:23
You may want to run clang-format on this change:
- zassert_equal(0, status,
- "Expected return value %d, not %d\n", 0, status);
- zassert_equal(cpu_id, lock_id,
- "Expected lock ID %u, not %u\n", cpu_id, lock_id);
+ zassert_equal(0, status, "Expected return value %d, not %d\n", 0, status);
+ zassert_equal(cpu_id, lock_id, "Expected lock ID %u, not %u\n", cpu_id, lock_id);
File:tests/arch/arc/arc_vpx_lock/src/main.c
Line:135
You may want to run clang-format on this change:
- zassert_equal(0, status,
- "Expected return value %d, not %d\n", 0, status);
- zassert_equal(lock_id, new_lock_id,
- "Expected lock ID %u, not %u\n", lock_id, new_lock_id);
+ zassert_equal(0, status, "Expected return value %d, not %d\n", 0, status);
+ zassert_equal(lock_id, new_lock_id, "Expected lock ID %u, not %u\n", lock_id, new_lock_id);
File:tests/arch/arc/arc_vpx_lock/src/main.c
Line:149
You may want to run clang-format on this change:
- zassert_equal(0, status,
- "Expected return value %d, not %d\n", 0, status);
- zassert_equal(cpu_id, lock_id,
- "Expected lock ID %u, not %u\n", cpu_id, lock_id);
+ zassert_equal(0, status, "Expected return value %d, not %d\n", 0, status);
+ zassert_equal(cpu_id, lock_id, "Expected lock ID %u, not %u\n", cpu_id, lock_id);
/* VPX already locked. */
irq_offload(vpx_isr, (void *)(uintptr_t)ISR_VPX_LOCK);
- zassert_equal(-EBUSY, isr_result, /* Expect failure */
- "Expected return value %d (-EBUSY), not %d\n",
- -EBUSY, isr_result);
+ zassert_equal(-EBUSY, isr_result, /* Expect failure */
+ "Expected return value %d (-EBUSY), not %d\n", -EBUSY, isr_result);
File:tests/arch/arc/arc_vpx_lock/src/main.c
Line:172
You may want to run clang-format on this change:
- zassert_equal(0, isr_result,
- "Expected return value %d, not %d\n", 0, isr_result);
- zassert_equal(cpu_id, isr_vpx_lock_id,
- "Expected lock ID %u, not %u\n", cpu_id, isr_vpx_lock_id);
+ zassert_equal(0, isr_result, "Expected return value %d, not %d\n", 0, isr_result);
+ zassert_equal(cpu_id, isr_vpx_lock_id, "Expected lock ID %u, not %u\n", cpu_id,
+ isr_vpx_lock_id);
File:tests/arch/arc/arc_vpx_lock/src/main.c
Line:183
You may want to run clang-format on this change:
- zassert_equal(0, status,
- "Expected return value %d, not %d\n", 0, status);
- zassert_equal(cpu_id, lock_id,
- "Expected lock ID %u, not %d\n", cpu_id, lock_id);
+ zassert_equal(0, status, "Expected return value %d, not %d\n", 0, status);
+ zassert_equal(cpu_id, lock_id, "Expected lock ID %u, not %d\n", cpu_id, lock_id);
/* The VPX lock has already been taken; expect errors */
status = arc_vpx_lock(&tmp, K_NO_WAIT);
- zassert_equal(-EBUSY, status,
- "Expected return value %d (-EBUSY), not %d\n",
- -EBUSY, status);
+ zassert_equal(-EBUSY, status, "Expected return value %d (-EBUSY), not %d\n", -EBUSY,
+ status);
status = arc_vpx_lock(&tmp, K_MSEC(10));
- zassert_equal(-EAGAIN, status,
- "Expected return value %d (-EAGAIN), not %d\n",
- -EAGAIN, status);
+ zassert_equal(-EAGAIN, status, "Expected return value %d (-EAGAIN), not %d\n", -EAGAIN,
+ status);
/* Verify that unlocking makes it available */
arc_vpx_unlock(lock_id);
status = arc_vpx_lock(&tmp, K_NO_WAIT);
- zassert_equal(0, status,
- "Expected return value %d, not %d\n", 0, status);
- zassert_equal(lock_id, tmp,
- "Expected lock ID %d, not %d\n", lock_id, tmp);
+ zassert_equal(0, status, "Expected return value %d, not %d\n", 0, status);
+ zassert_equal(lock_id, tmp, "Expected lock ID %d, not %d\n", lock_id, tmp);
File:tests/arch/arc/arc_vpx_lock/src/main.c
Line:224
You may want to run clang-format on this change:
- k_threa
|
|
Run Compliance Tests:
tests/arch/arc/arc_vpx_lock/src/main.c#L23
tests/arch/arc/arc_vpx_lock/src/main.c:23
-#define ISR_VPX_LOCK 0
-#define ISR_VPX_UNLOCK 1
+#define ISR_VPX_LOCK 0
+#define ISR_VPX_UNLOCK 1
|
|
Run Compliance Tests:
tests/arch/arc/arc_vpx_lock/src/main.c#L135
tests/arch/arc/arc_vpx_lock/src/main.c:135
- zassert_equal(0, status,
- "Expected return value %d, not %d\n", 0, status);
- zassert_equal(cpu_id, lock_id,
- "Expected lock ID %u, not %u\n", cpu_id, lock_id);
+ zassert_equal(0, status, "Expected return value %d, not %d\n", 0, status);
+ zassert_equal(cpu_id, lock_id, "Expected lock ID %u, not %u\n", cpu_id, lock_id);
|
|
Run Compliance Tests:
tests/arch/arc/arc_vpx_lock/src/main.c#L149
tests/arch/arc/arc_vpx_lock/src/main.c:149
- zassert_equal(0, status,
- "Expected return value %d, not %d\n", 0, status);
- zassert_equal(lock_id, new_lock_id,
- "Expected lock ID %u, not %u\n", lock_id, new_lock_id);
+ zassert_equal(0, status, "Expected return value %d, not %d\n", 0, status);
+ zassert_equal(lock_id, new_lock_id, "Expected lock ID %u, not %u\n", lock_id, new_lock_id);
|
|
Run Compliance Tests:
tests/arch/arc/arc_vpx_lock/src/main.c#L172
tests/arch/arc/arc_vpx_lock/src/main.c:172
- zassert_equal(0, status,
- "Expected return value %d, not %d\n", 0, status);
- zassert_equal(cpu_id, lock_id,
- "Expected lock ID %u, not %u\n", cpu_id, lock_id);
+ zassert_equal(0, status, "Expected return value %d, not %d\n", 0, status);
+ zassert_equal(cpu_id, lock_id, "Expected lock ID %u, not %u\n", cpu_id, lock_id);
/* VPX already locked. */
irq_offload(vpx_isr, (void *)(uintptr_t)ISR_VPX_LOCK);
- zassert_equal(-EBUSY, isr_result, /* Expect failure */
- "Expected return value %d (-EBUSY), not %d\n",
- -EBUSY, isr_result);
+ zassert_equal(-EBUSY, isr_result, /* Expect failure */
+ "Expected return value %d (-EBUSY), not %d\n", -EBUSY, isr_result);
|
|
Run Compliance Tests:
tests/arch/arc/arc_vpx_lock/src/main.c#L183
tests/arch/arc/arc_vpx_lock/src/main.c:183
- zassert_equal(0, isr_result,
- "Expected return value %d, not %d\n", 0, isr_result);
- zassert_equal(cpu_id, isr_vpx_lock_id,
- "Expected lock ID %u, not %u\n", cpu_id, isr_vpx_lock_id);
+ zassert_equal(0, isr_result, "Expected return value %d, not %d\n", 0, isr_result);
+ zassert_equal(cpu_id, isr_vpx_lock_id, "Expected lock ID %u, not %u\n", cpu_id,
+ isr_vpx_lock_id);
|
|
Run Compliance Tests:
tests/arch/arc/arc_vpx_lock/src/main.c#L224
tests/arch/arc/arc_vpx_lock/src/main.c:224
- zassert_equal(0, status,
- "Expected return value %d, not %d\n", 0, status);
- zassert_equal(cpu_id, lock_id,
- "Expected lock ID %u, not %d\n", cpu_id, lock_id);
+ zassert_equal(0, status, "Expected return value %d, not %d\n", 0, status);
+ zassert_equal(cpu_id, lock_id, "Expected lock ID %u, not %d\n", cpu_id, lock_id);
/* The VPX lock has already been taken; expect errors */
status = arc_vpx_lock(&tmp, K_NO_WAIT);
- zassert_equal(-EBUSY, status,
- "Expected return value %d (-EBUSY), not %d\n",
- -EBUSY, status);
+ zassert_equal(-EBUSY, status, "Expected return value %d (-EBUSY), not %d\n", -EBUSY,
+ status);
status = arc_vpx_lock(&tmp, K_MSEC(10));
- zassert_equal(-EAGAIN, status,
- "Expected return value %d (-EAGAIN), not %d\n",
- -EAGAIN, status);
+ zassert_equal(-EAGAIN, status, "Expected return value %d (-EAGAIN), not %d\n", -EAGAIN,
+ status);
/* Verify that unlocking makes it available */
arc_vpx_unlock(lock_id);
status = arc_vpx_lock(&tmp, K_NO_WAIT);
- zassert_equal(0, status,
- "Expected return value %d, not %d\n", 0, status);
- zassert_equal(lock_id, tmp,
- "Expected lock ID %d, not %d\n", lock_id, tmp);
+ zassert_equal(0, status, "Expected return value %d, not %d\n", 0, status);
+ zassert_equal(lock_id, tmp, "Expected lock ID %d, not %d\n", lock_id, tmp);
|
|
Run Compliance Tests:
tests/arch/arc/arc_vpx_lock/src/main.c#L236
tests/arch/arc/arc_vpx_lock/src/main.c:236
- k_thread_create(&helper_thread, helper_stack, STACK_SIZE,
- test_helper_entry, NULL, NULL, NULL,
- priority - 2, 0, K_NO_WAIT);
+ k_thread_create(&helper_thread, helper_stack, STACK_SIZE, test_helper_entry, NULL, NULL,
+ NULL, priority - 2, 0, K_NO_WAIT);
|
Loading