* [PATCH] eventdev: fix build with clang 21
@ 2025-11-04 13:08 David Marchand
2025-11-04 14:23 ` Gujjar, Abhinandan S
0 siblings, 1 reply; 2+ messages in thread
From: David Marchand @ 2025-11-04 13:08 UTC (permalink / raw)
To: dev
Cc: stable, Abhinandan Gujjar, Jerin Jacob, Akhil Goyal,
Volodymyr Fialko, Naga Harish K S V
Fix 16-bits formatting issues reported by clang 21 on Fedora 43:
../lib/eventdev/rte_event_crypto_adapter.c:1461:4: error:
format specifies type 'unsigned char' but the argument has type
'uint16_t' (aka 'unsigned short') [-Werror,-Wformat]
1459 | RTE_EDEV_LOG_ERR("Failed to get adapter caps dev %" PRIu8
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1460 | " cdev %" PRIu8, adapter->eventdev_id,
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1461 | adapter->next_cdev_id);
| ^~~~~~~~~~~~~~~~~~~~~~
../lib/eventdev/rte_event_crypto_adapter.c:1592:46: error:
format specifies type 'unsigned char' but the argument has type
'uint16_t' (aka 'unsigned short') [-Werror,-Wformat]
1592 | RTE_EDEV_LOG_ERR("Invalid dev_id=%" PRIu8, cdev_id);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
../lib/eventdev/rte_event_crypto_adapter.c:1613:45: error:
format specifies type 'unsigned char' but the argument has type
'uint16_t' (aka 'unsigned short') [-Werror,-Wformat]
1612 | RTE_EDEV_LOG_ERR("Event vectorization is not supported,"
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1613 | "dev %" PRIu8 " cdev %" PRIu8, dev_id, cdev_id);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
Fixes: c1749bc5ee10 ("eventdev: introduce event cryptodev vector type")
Fixes: 04ed18cd41f1 ("eventdev/crypto: support runtime set/get parameters")
Cc: stable@dpdk.org
Signed-off-by: David Marchand <david.marchand@redhat.com>
---
lib/eventdev/rte_event_crypto_adapter.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/lib/eventdev/rte_event_crypto_adapter.c b/lib/eventdev/rte_event_crypto_adapter.c
index b827a0ffd6..3f9cc7379a 100644
--- a/lib/eventdev/rte_event_crypto_adapter.c
+++ b/lib/eventdev/rte_event_crypto_adapter.c
@@ -1457,7 +1457,7 @@ crypto_adapter_cap_check(struct event_crypto_adapter *adapter)
&caps);
if (ret) {
RTE_EDEV_LOG_ERR("Failed to get adapter caps dev %" PRIu8
- " cdev %" PRIu8, adapter->eventdev_id,
+ " cdev %" PRIu16, adapter->eventdev_id,
adapter->next_cdev_id);
return ret;
}
@@ -1589,7 +1589,7 @@ rte_event_crypto_adapter_vector_limits_get(
RTE_EVENTDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
if (!rte_cryptodev_is_valid_dev(cdev_id)) {
- RTE_EDEV_LOG_ERR("Invalid dev_id=%" PRIu8, cdev_id);
+ RTE_EDEV_LOG_ERR("Invalid dev_id=%" PRIu16, cdev_id);
return -EINVAL;
}
@@ -1610,7 +1610,7 @@ rte_event_crypto_adapter_vector_limits_get(
if (!(cap & RTE_EVENT_CRYPTO_ADAPTER_CAP_EVENT_VECTOR)) {
RTE_EDEV_LOG_ERR("Event vectorization is not supported,"
- "dev %" PRIu8 " cdev %" PRIu8, dev_id, cdev_id);
+ "dev %" PRIu8 " cdev %" PRIu16, dev_id, cdev_id);
return -ENOTSUP;
}
--
2.51.0
^ permalink raw reply [flat|nested] 2+ messages in thread
* RE: [PATCH] eventdev: fix build with clang 21
2025-11-04 13:08 [PATCH] eventdev: fix build with clang 21 David Marchand
@ 2025-11-04 14:23 ` Gujjar, Abhinandan S
0 siblings, 0 replies; 2+ messages in thread
From: Gujjar, Abhinandan S @ 2025-11-04 14:23 UTC (permalink / raw)
To: Marchand, David, dev
Cc: stable, Jerin Jacob, Akhil Goyal, Volodymyr Fialko, Naga Harish K, S V
> -----Original Message-----
> From: David Marchand <david.marchand@redhat.com>
> Sent: Tuesday, November 4, 2025 6:38 PM
> To: dev@dpdk.org
> Cc: stable@dpdk.org; Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>;
> Jerin Jacob <jerinj@marvell.com>; Akhil Goyal <gakhil@marvell.com>;
> Volodymyr Fialko <vfialko@marvell.com>; Naga Harish K, S V
> <s.v.naga.harish.k@intel.com>
> Subject: [PATCH] eventdev: fix build with clang 21
>
> Fix 16-bits formatting issues reported by clang 21 on Fedora 43:
>
> ../lib/eventdev/rte_event_crypto_adapter.c:1461:4: error:
> format specifies type 'unsigned char' but the argument has type
> 'uint16_t' (aka 'unsigned short') [-Werror,-Wformat]
> 1459 | RTE_EDEV_LOG_ERR("Failed to get adapter caps dev %" PRIu8
> | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 1460 | " cdev %" PRIu8, adapter->eventdev_id,
> | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 1461 | adapter->next_cdev_id);
> | ^~~~~~~~~~~~~~~~~~~~~~
>
> ../lib/eventdev/rte_event_crypto_adapter.c:1592:46: error:
> format specifies type 'unsigned char' but the argument has type
> 'uint16_t' (aka 'unsigned short') [-Werror,-Wformat]
> 1592 | RTE_EDEV_LOG_ERR("Invalid dev_id=%" PRIu8, cdev_id);
> | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
>
> ../lib/eventdev/rte_event_crypto_adapter.c:1613:45: error:
> format specifies type 'unsigned char' but the argument has type
> 'uint16_t' (aka 'unsigned short') [-Werror,-Wformat]
> 1612 | RTE_EDEV_LOG_ERR("Event vectorization is not supported,"
> | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 1613 | "dev %" PRIu8 " cdev %" PRIu8, dev_id, cdev_id);
> | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
>
> Fixes: c1749bc5ee10 ("eventdev: introduce event cryptodev vector type")
> Fixes: 04ed18cd41f1 ("eventdev/crypto: support runtime set/get parameters")
> Cc: stable@dpdk.org
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>
Acked-by: abhinandan.gujjar@intel.com
> ---
> lib/eventdev/rte_event_crypto_adapter.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/lib/eventdev/rte_event_crypto_adapter.c
> b/lib/eventdev/rte_event_crypto_adapter.c
> index b827a0ffd6..3f9cc7379a 100644
> --- a/lib/eventdev/rte_event_crypto_adapter.c
> +++ b/lib/eventdev/rte_event_crypto_adapter.c
> @@ -1457,7 +1457,7 @@ crypto_adapter_cap_check(struct
> event_crypto_adapter *adapter)
> &caps);
> if (ret) {
> RTE_EDEV_LOG_ERR("Failed to get adapter caps dev %" PRIu8
> - " cdev %" PRIu8, adapter->eventdev_id,
> + " cdev %" PRIu16, adapter->eventdev_id,
> adapter->next_cdev_id);
> return ret;
> }
> @@ -1589,7 +1589,7 @@ rte_event_crypto_adapter_vector_limits_get(
> RTE_EVENTDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL);
>
> if (!rte_cryptodev_is_valid_dev(cdev_id)) {
> - RTE_EDEV_LOG_ERR("Invalid dev_id=%" PRIu8, cdev_id);
> + RTE_EDEV_LOG_ERR("Invalid dev_id=%" PRIu16, cdev_id);
> return -EINVAL;
> }
>
> @@ -1610,7 +1610,7 @@ rte_event_crypto_adapter_vector_limits_get(
>
> if (!(cap & RTE_EVENT_CRYPTO_ADAPTER_CAP_EVENT_VECTOR)) {
> RTE_EDEV_LOG_ERR("Event vectorization is not supported,"
> - "dev %" PRIu8 " cdev %" PRIu8, dev_id,
> cdev_id);
> + "dev %" PRIu8 " cdev %" PRIu16, dev_id,
> cdev_id);
> return -ENOTSUP;
> }
>
> --
> 2.51.0
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-11-04 14:23 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-11-04 13:08 [PATCH] eventdev: fix build with clang 21 David Marchand
2025-11-04 14:23 ` Gujjar, Abhinandan S
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).