[Accel-config] [PATCH v2] accel-config: Do not set group traffic class for hw ver < 0x200

ramesh.thomas at intel.com
Newsgroups dev.linux.lists.accel-config
Message-ID <[email protected]>
From: Ramesh Thomas <ramesh.thomas(a)intel.com>

A default value will be set by driver for hw versions < 0x200

Signed-off-by: Ramesh Thomas <ramesh.thomas(a)intel.com>
---
v2:
Use enum for device versions

 accfg/libaccel_config.h | 5 +++++
 test/libaccfg.c         | 7 +++++++
 2 files changed, 12 insertions(+)

diff --git a/accfg/libaccel_config.h b/accfg/libaccel_config.h
index a74571c..f8b9bf9 100644
--- a/accfg/libaccel_config.h
+++ b/accfg/libaccel_config.h
@@ -27,6 +27,11 @@ extern "C" {
 #define WQ_PRIORITY_LIMIT 15
 #define UUID_ZERO "00000000-0000-0000-0000-000000000000"
 
+enum accfg_device_version {
+	ACCFG_DEVICE_VERSION_1 = 0x100,
+	ACCFG_DEVICE_VERSION_2 = 0x200,
+};
+
 /* no need to save device state */
 enum accfg_device_type {
 	ACCFG_DEVICE_DSA = 0,
diff --git a/test/libaccfg.c b/test/libaccfg.c
index c37b71c..7849d63 100644
--- a/test/libaccfg.c
+++ b/test/libaccfg.c
@@ -186,6 +186,10 @@ static int config_group(struct accfg_ctx *ctx, struct accfg_device *device,
 				group_param->tokens_allowed));
 	SET_ERR(rc, accfg_group_set_use_token_limit(group,
 				group_param->use_token_limit));
+
+	if (accfg_device_get_version(device) < ACCFG_DEVICE_VERSION_2)
+		return rc;
+
 	SET_ERR(rc, accfg_group_set_traffic_class_a(group,
 				group_param->traffic_class_a));
 	SET_ERR(rc, accfg_group_set_traffic_class_b(group,
@@ -217,6 +221,9 @@ static int check_group(struct accfg_ctx *ctx, struct accfg_device *device,
 		return -EINVAL;
 	}
 
+	if (accfg_device_get_version(device) < ACCFG_DEVICE_VERSION_2)
+		return 0;
+
 	if (group_param->traffic_class_a !=
 			accfg_group_get_traffic_class_a(group)) {
 		fprintf(stderr, "%s failed on traffic_class_a\n", __func__);
-- 
2.26.3
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.