- Acc - lists.linaro.org

[PATCH 00/10] uadk_engine: bugfix and cleanup

by Zhiqi Song

This patch set including the following modify: 1. Some optimization of ECC and Digest algorithms to improve the performance of them. 2. Some bugfix and cleanup of the code. 3. Fixup the README.md info about enviroment variable. Zhiqi Song (10): uadk_engine: bugfix null dereference in abnormal conditions digest: cleanup magic number in digest dh: cleanup redundant macro definition ecc: bugfix about repeated alg and dev queries ecc: bugfix the conversion of sm2 pubkey point uadk_engine: modify static check problem ecc: optimize sm2 ctrl process to improve performance digest: remove soft ctx allocation in digest init process ecc: optimize sm2 update session process uadk_engine: fix openssl config info README.md | 16 ++++---- src/uadk_async.c | 22 +++++++++-- src/uadk_async.h | 7 +++- src/uadk_dh.c | 1 - src/uadk_digest.c | 98 +++++++++++++++++++++++++++-------------------- src/uadk_ec.c | 79 ++++++++++++++++++++++++++++---------- src/uadk_ecx.c | 23 +++++++++-- src/uadk_pkey.c | 66 +++++++++++++++++++------------ src/uadk_pkey.h | 13 ++++++- src/uadk_rsa.c | 18 +++++++++ src/uadk_sm2.c | 87 ++++++++++++++++++++++++++++++----------- 11 files changed, 303 insertions(+), 127 deletions(-) -- 2.30.0

2 years, 2 months

1
10
0 0

[PATCH 0/7] uadk/zlibwrapper - misc bugfixes

by Yang Shen

Some bugfixes and clean up for UADK. Yang Shen (7): uadk/zlibwrapper - some misc clean up uadk/zlibwrapper - fix the return value for deflate/inflate uadk/zlibwrapper - add parameters check for APIs uadk/zlibwrapper - remove duplication for deflate and inflate uadk/zlibwrapper - fix wrong windowsize uadk/zlibwrapper - rename zlibwrapper APIs uadk/wd - fix error shmid for multiprocess wd_zlibwrapper.h => include/wd_zlibwrapper.h | 30 +-- wd_util.c | 84 +++---- wd_zlibwrapper.c | 217 +++++++++---------- 3 files changed, 133 insertions(+), 198 deletions(-) rename wd_zlibwrapper.h => include/wd_zlibwrapper.h (68%) -- 2.24.0

2 years, 2 months

1
7
0 0

[PATCH 1/4] uadk: bugfix some framework issues

by Longfang Liu

1. Separate the advanced properties of the device from the driver class properties. 2. Fix some internal implementation bugs. 3. Update some comments. Signed-off-by: Longfang Liu <liulongfang(a)huawei.com> --- include/wd.h | 6 +++++ include/wd_alg.h | 47 ++++++++++++++++++++++++++++++++++++- include/wd_util.h | 8 ++++--- wd.c | 19 ++++++++------- wd_alg.c | 59 +++++++++++++++++++++++++++++------------------ wd_sched.c | 2 ++ wd_util.c | 33 +++++++++++++++++--------- 7 files changed, 126 insertions(+), 48 deletions(-) diff --git a/include/wd.h b/include/wd.h index 21af7ff..301bf1b 100644 --- a/include/wd.h +++ b/include/wd.h @@ -582,11 +582,17 @@ bool wd_need_info(void); struct wd_capability { char alg_name[CRYPTO_MAX_ALG_NAME]; char drv_name[CRYPTO_MAX_ALG_NAME]; + bool available; int priority; + int calc_type; struct wd_capability *next; }; +/** + * wd_get_alg_cap() - Get the algorithm information supported + * in the current system. + */ struct wd_capability *wd_get_alg_cap(void); void wd_release_alg_cap(struct wd_capability *head); diff --git a/include/wd_alg.h b/include/wd_alg.h index e25e191..a1b4c4f 100644 --- a/include/wd_alg.h +++ b/include/wd_alg.h @@ -31,6 +31,13 @@ enum alg_priority { * @drv_name: name of the current device driver * @alg_name: name of the algorithm supported by the driver * @priority: priority of the type of algorithm supported by the driver + * the larger the value of priority, the higher the priority of the driver, + * it will be used first when selecting a driver. + * soft calculation can be defined as 0. + * hard calculation can be defined as a value above 100. + * instruction acceleration can define a higher value according to + * the performance situation, such as 400. + * @calc_type: the calculation method of algorithm supported by the driver * @queue_num: number of device queues required by the device to * execute the algorithm task * @op_type_num: number of modes in which the device executes the @@ -52,6 +59,7 @@ struct wd_alg_driver { const char *drv_name; const char *alg_name; int priority; + int calc_type; int queue_num; int op_type_num; int priv_size; @@ -64,25 +72,62 @@ struct wd_alg_driver { int (*get_usage)(void *param); }; +/** + * wd_alg_driver_register() - Register a device driver. + * @wd_alg_driver: a device driver that supports an algorithm. + * + * Return the execution result, non-zero means error code. + */ int wd_alg_driver_register(struct wd_alg_driver *drv); void wd_alg_driver_unregister(struct wd_alg_driver *drv); +/** + * @alg_name: name of the algorithm supported by the driver + * @drv_name: name of the current device driver + * @available: Indicates whether the current driver still has resources available + * @priority: priority of the type of algorithm supported by the driver + * @calc_type: the calculation method of algorithm supported by the driver + * @refcnt: the number of times the algorithm driver is being cited by the task + * + * @drv: device Drivers Supporting Algorithms + * @next: pointer to the next node of the algorithm linked list + */ struct wd_alg_list { const char *alg_name; const char *drv_name; - bool available; + bool available; int priority; + int calc_type; int refcnt; struct wd_alg_driver *drv; struct wd_alg_list *next; }; +/** + * wd_request_drv() - Apply for an algorithm driver. + * @alg_name: task algorithm name. + * @hw_mask: the flag of shield hardware device drivers. + * + * Returns the applied algorithm driver, non means error. + */ struct wd_alg_driver *wd_request_drv(const char *alg_name, bool hw_mask); void wd_release_drv(struct wd_alg_driver *drv); +/** + * wd_drv_alg_support() - Check the algorithms supported by the driver. + * @alg_name: task algorithm name. + * @drv: a device driver that supports an algorithm. + * + * Return check result. + */ bool wd_drv_alg_support(const char *alg_name, struct wd_alg_driver *drv); + +/** + * wd_enable_drv() - Re-enable use of the current device driver. + * @drv: a device driver that supports an algorithm. + */ void wd_enable_drv(struct wd_alg_driver *drv); void wd_disable_drv(struct wd_alg_driver *drv); diff --git a/include/wd_util.h b/include/wd_util.h index a730f36..b20466f 100644 --- a/include/wd_util.h +++ b/include/wd_util.h @@ -420,9 +420,11 @@ static inline void wd_alg_clear_init(enum wd_status *status) /** * wd_ctx_param_init() - Initialize the current device driver according * to the obtained queue resource and the applied driver. - * @config: device resources requested by the current algorithm. - * @driver: device driver for the current algorithm application. - * @drv_priv: the parameter pointer of the current device driver. + * @ctx_params: queue configuration parameters required for device initialization. + * @user_ctx_params: user-configured queue context parameters. + * @ctx_set_num: the number of user-configured queue contexts. + * @driver: device drivers that support current business algorithms. + * @max_op_type: number of device queue types that support algorithmic services. * * Return 0 if succeed and other error number if fail. */ diff --git a/wd.c b/wd.c index b6286b5..a049035 100644 --- a/wd.c +++ b/wd.c @@ -886,7 +886,7 @@ char *wd_ctx_get_dev_name(handle_t h_ctx) void wd_release_alg_cap(struct wd_capability *head) { struct wd_capability *cap_pnext = head; - struct wd_capability *cap_node = NULL; + struct wd_capability *cap_node; while (cap_pnext) { cap_node = cap_pnext; @@ -903,30 +903,30 @@ struct wd_capability *wd_get_alg_cap(void) struct wd_alg_list *head = wd_get_alg_head(); struct wd_alg_list *pnext = head->next; struct wd_capability *cap_head = NULL; - struct wd_capability *cap_node = NULL; struct wd_capability *cap_pnext = NULL; - int i = 0; + struct wd_capability *cap_node; while (pnext) { cap_node = calloc(1, sizeof(struct wd_capability)); - if (!cap_head) { + if (!cap_node) { WD_ERR("fail to alloc wd capability head\n"); goto alloc_err; } strcpy(cap_node->alg_name, pnext->alg_name); strcpy(cap_node->drv_name, pnext->drv_name); + cap_node->available = pnext->available; cap_node->priority = pnext->priority; + cap_node->calc_type = pnext->calc_type; cap_node->next = NULL; - cap_pnext->next = cap_node; - cap_pnext = cap_node; pnext = pnext->next; - if (i == 0) { + if (!cap_pnext) { cap_head = cap_node; - cap_pnext = cap_head; + cap_pnext = cap_node; } - i++; + cap_pnext->next = cap_node; + cap_pnext = cap_node; } return cap_head; @@ -935,4 +935,3 @@ alloc_err: wd_release_alg_cap(cap_head); return NULL; } - diff --git a/wd_alg.c b/wd_alg.c index 5e4edaf..a802f60 100644 --- a/wd_alg.c +++ b/wd_alg.c @@ -47,6 +47,28 @@ static bool wd_check_accel_dev(const char *dev_name) return false; } +static bool wd_alg_check_available(int calc_type, const char *dev_name) +{ + bool ret = false; + + switch (calc_type) { + case UADK_ALG_SOFT: + break; + /* Should find the CPU if not support CE */ + case UADK_ALG_CE_INSTR: + break; + /* Should find the CPU if not support SVE */ + case UADK_ALG_SVE_INSTR: + break; + /* Check if the current driver has device support */ + case UADK_ALG_HW: + ret = wd_check_accel_dev(dev_name); + break; + } + + return ret; +} + int wd_alg_driver_register(struct wd_alg_driver *drv) { struct wd_alg_list *new_alg; @@ -65,21 +87,16 @@ int wd_alg_driver_register(struct wd_alg_driver *drv) new_alg->alg_name = drv->alg_name; new_alg->drv_name = drv->drv_name; new_alg->priority = drv->priority; + new_alg->calc_type = drv->calc_type; new_alg->drv = drv; new_alg->refcnt = 0; new_alg->next = NULL; - if (drv->priority == UADK_ALG_HW) { - /* If not find dev, remove this driver node */ - new_alg->available = wd_check_accel_dev(drv->drv_name); - if (!new_alg->available) { - free(new_alg); - WD_ERR("failed to find alg driver's device!\n"); - return -WD_ENODEV; - } - } else { - /* Should find the CPU if not support SVE or CE */ - new_alg->available = true; + new_alg->available = wd_alg_check_available(drv->calc_type, drv->drv_name); + if (!new_alg->available) { + free(new_alg); + WD_ERR("failed to find alg driver's device!\n"); + return -WD_ENODEV; } pthread_mutex_lock(&mutex); @@ -103,7 +120,8 @@ void wd_alg_driver_unregister(struct wd_alg_driver *drv) while (pnext) { if (!strcmp(drv->alg_name, pnext->alg_name) && !strcmp(drv->drv_name, pnext->drv_name) && - drv->priority == pnext->priority) { + drv->priority == pnext->priority && + drv->calc_type == pnext->calc_type) { break; } npre = pnext; @@ -162,19 +180,14 @@ void wd_enable_drv(struct wd_alg_driver *drv) while (pnext) { if (!strcmp(drv->alg_name, pnext->alg_name) && !strcmp(drv->drv_name, pnext->drv_name) && - drv->priority == pnext->priority) { + drv->priority == pnext->priority && + drv->calc_type == pnext->calc_type) { break; } pnext = pnext->next; } - if (drv->priority == UADK_ALG_HW) { - /* If not find dev, remove this driver node */ - pnext->available = wd_check_accel_dev(drv->drv_name); - } else { - /* Should find the CPU if not support SVE or CE */ - pnext->available = true; - } + pnext->available = wd_alg_check_available(drv->calc_type, drv->drv_name); pthread_mutex_unlock(&mutex); } @@ -190,7 +203,8 @@ void wd_disable_drv(struct wd_alg_driver *drv) while (pnext) { if (!strcmp(drv->alg_name, pnext->alg_name) && !strcmp(drv->drv_name, pnext->drv_name) && - drv->priority == pnext->priority) { + drv->priority == pnext->priority && + drv->calc_type == pnext->calc_type) { break; } pnext = pnext->next; @@ -217,7 +231,7 @@ struct wd_alg_driver *wd_request_drv(const char *alg_name, bool hw_mask) pthread_mutex_lock(&mutex); while (pnext) { /* hw_mask true mean not to used hardware dev */ - if (hw_mask && pnext->drv->priority == UADK_ALG_HW) { + if (hw_mask && pnext->drv->calc_type == UADK_ALG_HW) { pnext = pnext->next; continue; } @@ -262,4 +276,3 @@ void wd_release_drv(struct wd_alg_driver *drv) select_node->refcnt--; pthread_mutex_unlock(&mutex); } - diff --git a/wd_sched.c b/wd_sched.c index 9a52c4d..79f57fb 100644 --- a/wd_sched.c +++ b/wd_sched.c @@ -368,6 +368,7 @@ static int sched_none_poll_policy(handle_t h_sched_ctx, while (loop_times > 0) { /* Default use ctx 0 */ + loop_times--; ret = sched_ctx->poll_func(0, 1, &poll_num); if ((ret < 0) && (ret != -EAGAIN)) return ret; @@ -409,6 +410,7 @@ static int sched_single_poll_policy(handle_t h_sched_ctx, while (loop_times > 0) { /* Default async mode use ctx 0 */ + loop_times--; ret = sched_ctx->poll_func(0, 1, &poll_num); if ((ret < 0) && (ret != -EAGAIN)) return ret; diff --git a/wd_util.c b/wd_util.c index dab4fc8..f990eb5 100644 --- a/wd_util.c +++ b/wd_util.c @@ -79,7 +79,7 @@ struct acc_alg_item { }; static struct acc_alg_item alg_options[] = { - {"zlib", "zlib-deflate"}, + {"zlib", "zlib"}, {"gzip", "gzip"}, {"deflate", "deflate"}, {"lz77_zstd", "lz77_zstd"}, @@ -131,6 +131,10 @@ static struct acc_alg_item alg_options[] = { {"sha512", "digest"}, {"sha512-224", "digest"}, {"sha512-256", "digest"}, + {"cmac(aes)", "digest"}, + {"gmac(aes)", "digest"}, + {"xcbc-mac-96(aes)", "digest"}, + {"xcbc-prf-128(aes)", "digest"}, {"", ""} }; @@ -1851,9 +1855,10 @@ int wd_init_param_check(struct wd_ctx_config *config, struct wd_sched *sched) static void wd_get_alg_type(const char *alg_name, char *alg_type) { + int alg_nums = ARRAY_SIZE(alg_options); int i; - for (i = 0; i < ARRAY_SIZE(alg_options); i++) { + for (i = 0; i < alg_nums; i++) { if (strcmp(alg_name, alg_options[i].name) == 0) { (void)strcpy(alg_type, alg_options[i].algtype); break; @@ -1893,7 +1898,7 @@ static int file_check_valid(char *lib_file) static int wd_alg_init_fallback(struct wd_alg_driver *fb_driver) { if (!fb_driver->init) { - WD_ERR("soft sec driver have no init interface.\n"); + WD_ERR("soft acc driver have no init interface.\n"); return -WD_EINVAL; } @@ -1905,7 +1910,7 @@ static int wd_alg_init_fallback(struct wd_alg_driver *fb_driver) static void wd_alg_uninit_fallback(struct wd_alg_driver *fb_driver) { if (!fb_driver->exit) { - WD_ERR("soft sec driver have no exit interface.\n"); + WD_ERR("soft acc driver have no exit interface.\n"); return; } @@ -1959,7 +1964,7 @@ void wd_alg_uninit_driver(struct wd_ctx_config_internal *config, driver->exit(priv); /* Ctx config just need clear once */ - if (driver->priority == UADK_ALG_HW) + if (driver->calc_type == UADK_ALG_HW) wd_clear_ctx_config(config); if (driver->fallback) @@ -1967,7 +1972,7 @@ void wd_alg_uninit_driver(struct wd_ctx_config_internal *config, if (priv) { free(priv); - priv = NULL; + *drv_priv = NULL; } } @@ -2052,7 +2057,7 @@ int wd_get_lib_file_path(char *lib_file, char *lib_path, bool is_dir) len = strlen(file_path) - 1; for (i = len; i >= 0; i--) { if (file_path[i] == '/') { - memset(&file_path[i], 0, PATH_STR_SIZE - i + 1); + memset(&file_path[i], 0, PATH_STR_SIZE - i); break; } } @@ -2116,7 +2121,7 @@ void *wd_dlopen_drv(const char *cust_lib_dir) if (ret < 0) goto free_node; - node->dlhandle = dlopen(lib_path, RTLD_NOW); + node->dlhandle = dlopen(lib_path, RTLD_GLOBAL | RTLD_NOW); if (!node->dlhandle) { free(node); /* there are many other files need to skip */ @@ -2183,6 +2188,9 @@ struct wd_alg_driver *wd_alg_drv_bind(int task_type, char *alg_name) } } break; + default: + WD_ERR("task type error.\n"); + return NULL; } return set_driver; @@ -2400,7 +2408,7 @@ static int wd_alg_ctx_init(struct wd_init_attrs *attrs) op_type_num = ctx_params->op_type_num; ctx_set_num = wd_get_ctx_numbers(*ctx_params, op_type_num); if (!ctx_set_num || !op_type_num) { - WD_ERR("invalid: ctx_set_num is %d, op_type_num is %d!\n", + WD_ERR("invalid: ctx_set_num is %u, op_type_num is %u!\n", ctx_set_num, op_type_num); ret = -WD_EINVAL; goto out_freelist; @@ -2486,7 +2494,7 @@ int wd_alg_attrs_init(struct wd_init_attrs *attrs) return -WD_EINVAL; if (attrs->driver) - driver_type = attrs->driver->priority; + driver_type = attrs->driver->calc_type; switch (driver_type) { case UADK_ALG_SOFT: @@ -2555,6 +2563,10 @@ int wd_alg_attrs_init(struct wd_init_attrs *attrs) ret = alg_init_func(ctx_config, alg_sched); if (ret) goto out_pre_init; + break; + default: + WD_ERR("driver type error: %d\n", driver_type); + return -WD_EINVAL; } return 0; @@ -2580,4 +2592,3 @@ void wd_alg_attrs_uninit(struct wd_init_attrs *attrs) } wd_sched_rr_release(alg_sched); } - -- 2.33.0

2 years, 3 months

1
3
0 0

[PATCH] uacce: use filep->f_mapping to replace inode->i_mapping

by Zhangfei Gao

The inode can be different in a container, for example, a docker and host both open the same uacce parent device, which uses the same uacce struct but different inode, so uacce->inode is not enough. What's worse, when docker stops, the inode will be destroyed as well, causing use-after-free in uacce_remove. So use q->filep->f_mapping to replace uacce->inode->i_mapping. Signed-off-by: Weili Qian <qianweili(a)huawei.com> Signed-off-by: Zhangfei Gao <zhangfei.gao(a)linaro.org> --- drivers/misc/uacce/uacce.c | 16 +++++++++------- include/linux/uacce.h | 4 ++-- 2 files changed, 11 insertions(+), 9 deletions(-) diff --git a/drivers/misc/uacce/uacce.c b/drivers/misc/uacce/uacce.c index 346bd7cf2e94..740ace422baa 100644 --- a/drivers/misc/uacce/uacce.c +++ b/drivers/misc/uacce/uacce.c @@ -166,8 +166,8 @@ static int uacce_fops_open(struct inode *inode, struct file *filep) init_waitqueue_head(&q->wait); filep->private_data = q; - uacce->inode = inode; q->state = UACCE_Q_INIT; + q->private_data = filep; mutex_init(&q->mutex); list_add(&q->list, &uacce->queues); mutex_unlock(&uacce->mutex); @@ -574,12 +574,6 @@ void uacce_remove(struct uacce_device *uacce) if (!uacce) return; - /* - * unmap remaining mapping from user space, preventing user still - * access the mmaped area while parent device is already removed - */ - if (uacce->inode) - unmap_mapping_range(uacce->inode->i_mapping, 0, 0, 1); /* * uacce_fops_open() may be running concurrently, even after we remove @@ -589,6 +583,8 @@ void uacce_remove(struct uacce_device *uacce) mutex_lock(&uacce->mutex); /* ensure no open queue remains */ list_for_each_entry_safe(q, next_q, &uacce->queues, list) { + struct file *filep = q->private_data; + /* * Taking q->mutex ensures that fops do not use the defunct * uacce->ops after the queue is disabled. @@ -597,6 +593,12 @@ void uacce_remove(struct uacce_device *uacce) uacce_put_queue(q); mutex_unlock(&q->mutex); uacce_unbind_queue(q); + + /* + * unmap remaining mapping from user space, preventing user still + * access the mmaped area while parent device is already removed + */ + unmap_mapping_range(filep->f_mapping, 0, 0, 1); } /* disable sva now since no opened queues */ diff --git a/include/linux/uacce.h b/include/linux/uacce.h index 0a81c3dfd26c..64b800b74436 100644 --- a/include/linux/uacce.h +++ b/include/linux/uacce.h @@ -86,6 +86,7 @@ enum uacce_q_state { * @state: queue state machine * @pasid: pasid associated to the mm * @handle: iommu_sva handle returned by iommu_sva_bind_device() + * @private_data: private data for saving filep */ struct uacce_queue { struct uacce_device *uacce; @@ -97,6 +98,7 @@ struct uacce_queue { enum uacce_q_state state; u32 pasid; struct iommu_sva *handle; + void *private_data; }; /** @@ -114,7 +116,6 @@ struct uacce_queue { * @mutex: protects uacce operation * @priv: private pointer of the uacce * @queues: list of queues - * @inode: core vfs */ struct uacce_device { const char *algs; @@ -130,7 +131,6 @@ struct uacce_device { struct mutex mutex; void *priv; struct list_head queues; - struct inode *inode; }; #if IS_ENABLED(CONFIG_UACCE) -- 2.39.2 (Apple Git-143)

2 years, 3 months

3
6
0 0

[PATCH V4 0/6] add init2 and dynamic load for digest/aead agls

by Hao Fang

compress/cipher/rsa/ecc already support the init2 and dynamic load, now add support for digest/aead algs. --- v3->v4: 1)fix bug alg match for init2. v2->v3: 1)add alg match for init2. v1->v2: 1)fix bug for aead init testcase. 2)tinyfix for code style. --- Hao Fang (6): uadk/digest: introduce the init2 interface for digest uadk/aead: introduce the init2 interface for aead uadk/digest: adapt the module dynamic load for the digest algs uadk/aead: adadpt the module dynamic load for aead algs uadk/tools: add init2 test for digest algs uadk/tool: add init2 test for aead algs drv/hisi_sec.c | 105 +++++++++---- include/drv/wd_aead_drv.h | 26 ---- include/drv/wd_digest_drv.h | 26 ---- include/wd_aead.h | 28 ++++ include/wd_digest.h | 28 ++++ libwd_crypto.map | 6 + uadk_tool/test/test_sec.c | 249 ++++++++++++++++++++++++++++--- wd_aead.c | 288 +++++++++++++++++++++++++++--------- wd_digest.c | 286 ++++++++++++++++++++++++++--------- 9 files changed, 794 insertions(+), 248 deletions(-) -- 2.30.0

2 years, 4 months

1
6
0 0

[PATCH V3 0/6] add init2 and dynamic load for digest/aead agls

by Hao Fang

compress/cipher/rsa/ecc already support the init2 and dynamic load, now add support for digest/aead algs. --- v2->v3: 1)add alg match for init2. v1->v2: 1)fix bug for aead init testcase. 2)tinyfix for code style. --- Hao Fang (6): uadk/digest: introduce the init2 interface for digest uadk/aead: introduce the init2 interface for aead uadk/digest: adapt the module dynamic load for the digest algs uadk/aead: adadpt the module dynamic load for aead algs uadk/tools: add init2 test for digest algs uadk/tool: add init2 test for aead algs drv/hisi_sec.c | 105 +++++++++---- include/drv/wd_aead_drv.h | 26 ---- include/drv/wd_digest_drv.h | 26 ---- include/wd_aead.h | 28 ++++ include/wd_digest.h | 28 ++++ libwd_crypto.map | 6 + uadk_tool/test/test_sec.c | 249 ++++++++++++++++++++++++++++--- wd_aead.c | 288 +++++++++++++++++++++++++++--------- wd_digest.c | 286 ++++++++++++++++++++++++++--------- 9 files changed, 794 insertions(+), 248 deletions(-) -- 2.30.0

2 years, 4 months

1
6
0 0

[PATCH v2] crypto/uadk: set queue pair in dev_configure

by Zhangfei Gao

By default, uadk only alloc two queues for each algorithm, which will impact performance. Set queue pair number as required in dev_configure. The default max queue pair number is 8, which can be modified via para: max_nb_queue_pairs Example: sudo dpdk-test-crypto-perf -l 0-10 --vdev crypto_uadk,max_nb_queue_pairs=10 -- --devtype crypto_uadk --optype cipher-only --buffer-sz 8192 Signed-off-by: Zhangfei Gao <zhangfei.gao(a)linaro.org> --- doc/guides/cryptodevs/uadk.rst | 27 +++++++++++++++++++ drivers/crypto/uadk/uadk_crypto_pmd.c | 18 +++++++++++-- drivers/crypto/uadk/uadk_crypto_pmd_private.h | 1 + 3 files changed, 44 insertions(+), 2 deletions(-) diff --git a/doc/guides/cryptodevs/uadk.rst b/doc/guides/cryptodevs/uadk.rst index 9af6b88a5a..053622f7bf 100644 --- a/doc/guides/cryptodevs/uadk.rst +++ b/doc/guides/cryptodevs/uadk.rst @@ -110,3 +110,30 @@ Test steps sudo dpdk-test --vdev=crypto_uadk --log-level=6 RTE>>cryptodev_uadk_autotest RTE>>quit + + +Initialization +-------------- + +To use the PMD in an application, user must: + +* Call rte_vdev_init("crypto_uadk") within the application. + +* Use --vdev="crypto_uadk" in the EAL options, which will call rte_vdev_init() internally. + +The following parameters (all optional) can be provided in the previous two calls: + +* max_nb_queue_pairs: Specify the maximum number of queue pairs in the device (8 by default). + The max value of max_nb_queue_pairs can be queried from the device property available_instances. + Property available_instances value may differ from the devices and platforms. + Allocating queue pairs bigger than available_instances will fail. + +Example: + +.. code-block:: console + + cat /sys/class/uacce/hisi_sec2-2/available_instances + 256 + + sudo dpdk-test-crypto-perf -l 0-10 --vdev crypto_uadk,max_nb_queue_pairs=10 \ + -- --devtype crypto_uadk --optype cipher-only --buffer-sz 8192 diff --git a/drivers/crypto/uadk/uadk_crypto_pmd.c b/drivers/crypto/uadk/uadk_crypto_pmd.c index 4f729e0f07..824383512e 100644 --- a/drivers/crypto/uadk/uadk_crypto_pmd.c +++ b/drivers/crypto/uadk/uadk_crypto_pmd.c @@ -357,8 +357,15 @@ static const struct rte_cryptodev_capabilities uadk_crypto_v2_capabilities[] = { /* Configure device */ static int uadk_crypto_pmd_config(struct rte_cryptodev *dev __rte_unused, - struct rte_cryptodev_config *config __rte_unused) + struct rte_cryptodev_config *config) { + char env[128]; + + /* set queue pairs num via env */ + sprintf(env, "sync:%d@0", config->nb_queue_pairs); + setenv("WD_CIPHER_CTX_NUM", env, 1); + setenv("WD_DIGEST_CTX_NUM", env, 1); + return 0; } @@ -434,7 +441,7 @@ uadk_crypto_pmd_info_get(struct rte_cryptodev *dev, if (dev_info != NULL) { dev_info->driver_id = dev->driver_id; dev_info->driver_name = dev->device->driver->name; - dev_info->max_nb_queue_pairs = 128; + dev_info->max_nb_queue_pairs = priv->max_nb_qpairs; /* No limit of number of sessions */ dev_info->sym.max_nb_sessions = 0; dev_info->feature_flags = dev->feature_flags; @@ -1015,6 +1022,7 @@ uadk_cryptodev_probe(struct rte_vdev_device *vdev) struct uadk_crypto_priv *priv; struct rte_cryptodev *dev; struct uacce_dev *udev; + const char *input_args; const char *name; udev = wd_get_accel_dev("cipher"); @@ -1030,6 +1038,9 @@ uadk_cryptodev_probe(struct rte_vdev_device *vdev) if (name == NULL) return -EINVAL; + input_args = rte_vdev_device_args(vdev); + rte_cryptodev_pmd_parse_input_args(&init_params, input_args); + dev = rte_cryptodev_pmd_create(name, &vdev->device, &init_params); if (dev == NULL) { UADK_LOG(ERR, "driver %s: create failed", init_params.name); @@ -1044,6 +1055,7 @@ uadk_cryptodev_probe(struct rte_vdev_device *vdev) RTE_CRYPTODEV_FF_SYMMETRIC_CRYPTO; priv = dev->data->dev_private; priv->version = version; + priv->max_nb_qpairs = init_params.max_nb_queue_pairs; rte_cryptodev_pmd_probing_finish(dev); @@ -1078,4 +1090,6 @@ static struct cryptodev_driver uadk_crypto_drv; RTE_PMD_REGISTER_VDEV(UADK_CRYPTO_DRIVER_NAME, uadk_crypto_pmd); RTE_PMD_REGISTER_CRYPTO_DRIVER(uadk_crypto_drv, uadk_crypto_pmd.driver, uadk_cryptodev_driver_id); +RTE_PMD_REGISTER_PARAM_STRING(UADK_CRYPTO_DRIVER_NAME, + "max_nb_queue_pairs=<int>"); RTE_LOG_REGISTER_DEFAULT(uadk_crypto_logtype, INFO); diff --git a/drivers/crypto/uadk/uadk_crypto_pmd_private.h b/drivers/crypto/uadk/uadk_crypto_pmd_private.h index 9075f0f058..5a7dbff117 100644 --- a/drivers/crypto/uadk/uadk_crypto_pmd_private.h +++ b/drivers/crypto/uadk/uadk_crypto_pmd_private.h @@ -67,6 +67,7 @@ struct uadk_crypto_priv { bool env_cipher_init; bool env_auth_init; enum uadk_crypto_version version; + unsigned int max_nb_qpairs; } __rte_cache_aligned; extern int uadk_crypto_logtype; -- 2.39.2 (Apple Git-143)

2 years, 4 months

1
0
0 0

[PATCH] crypto/uadk: set queue pair in dev_configure

by Zhangfei Gao

By default, uadk only alloc two queues for each algorithm, which will impact performance. Set queue pair number as required in dev_configure. The default max queue pair number is 8, which can be modified via para: max_nb_queue_pairs Example: sudo dpdk-test-crypto-perf -l 0-10 --vdev crypto_uadk,max_nb_queue_pairs=10 -- --devtype crypto_uadk --optype cipher-only --buffer-sz 8192 lcore id Buf Size Burst Size Gbps Cycles/Buf 3 8192 32 7.5226 871.19 7 8192 32 7.5225 871.20 1 8192 32 7.5225 871.20 4 8192 32 7.5224 871.21 5 8192 32 7.5224 871.21 10 8192 32 7.5223 871.22 9 8192 32 7.5223 871.23 2 8192 32 7.5222 871.23 8 8192 32 7.5222 871.23 6 8192 32 7.5218 871.28 Signed-off-by: Zhangfei Gao <zhangfei.gao(a)linaro.org> --- drivers/crypto/uadk/uadk_crypto_pmd.c | 19 +++++++++++++++++-- drivers/crypto/uadk/uadk_crypto_pmd_private.h | 1 + 2 files changed, 18 insertions(+), 2 deletions(-) diff --git a/drivers/crypto/uadk/uadk_crypto_pmd.c b/drivers/crypto/uadk/uadk_crypto_pmd.c index 4f729e0f07..34aae99342 100644 --- a/drivers/crypto/uadk/uadk_crypto_pmd.c +++ b/drivers/crypto/uadk/uadk_crypto_pmd.c @@ -357,8 +357,15 @@ static const struct rte_cryptodev_capabilities uadk_crypto_v2_capabilities[] = { /* Configure device */ static int uadk_crypto_pmd_config(struct rte_cryptodev *dev __rte_unused, - struct rte_cryptodev_config *config __rte_unused) + struct rte_cryptodev_config *config) { + char env[128]; + + /* set queue pairs num via env */ + sprintf(env, "sync:%d@0", config->nb_queue_pairs); + setenv("WD_CIPHER_CTX_NUM", env, 1); + setenv("WD_DIGEST_CTX_NUM", env, 1); + return 0; } @@ -434,7 +441,7 @@ uadk_crypto_pmd_info_get(struct rte_cryptodev *dev, if (dev_info != NULL) { dev_info->driver_id = dev->driver_id; dev_info->driver_name = dev->device->driver->name; - dev_info->max_nb_queue_pairs = 128; + dev_info->max_nb_queue_pairs = priv->max_nb_qpairs; /* No limit of number of sessions */ dev_info->sym.max_nb_sessions = 0; dev_info->feature_flags = dev->feature_flags; @@ -1015,6 +1022,7 @@ uadk_cryptodev_probe(struct rte_vdev_device *vdev) struct uadk_crypto_priv *priv; struct rte_cryptodev *dev; struct uacce_dev *udev; + const char *input_args; const char *name; udev = wd_get_accel_dev("cipher"); @@ -1030,6 +1038,9 @@ uadk_cryptodev_probe(struct rte_vdev_device *vdev) if (name == NULL) return -EINVAL; + input_args = rte_vdev_device_args(vdev); + rte_cryptodev_pmd_parse_input_args(&init_params, input_args); + dev = rte_cryptodev_pmd_create(name, &vdev->device, &init_params); if (dev == NULL) { UADK_LOG(ERR, "driver %s: create failed", init_params.name); @@ -1044,6 +1055,7 @@ uadk_cryptodev_probe(struct rte_vdev_device *vdev) RTE_CRYPTODEV_FF_SYMMETRIC_CRYPTO; priv = dev->data->dev_private; priv->version = version; + priv->max_nb_qpairs = init_params.max_nb_queue_pairs; rte_cryptodev_pmd_probing_finish(dev); @@ -1078,4 +1090,7 @@ static struct cryptodev_driver uadk_crypto_drv; RTE_PMD_REGISTER_VDEV(UADK_CRYPTO_DRIVER_NAME, uadk_crypto_pmd); RTE_PMD_REGISTER_CRYPTO_DRIVER(uadk_crypto_drv, uadk_crypto_pmd.driver, uadk_cryptodev_driver_id); +RTE_PMD_REGISTER_PARAM_STRING(UADK_CRYPTO_DRIVER_NAME, + "max_nb_queue_pairs=<int> " + "socket_id=<int>"); RTE_LOG_REGISTER_DEFAULT(uadk_crypto_logtype, INFO); diff --git a/drivers/crypto/uadk/uadk_crypto_pmd_private.h b/drivers/crypto/uadk/uadk_crypto_pmd_private.h index 9075f0f058..5a7dbff117 100644 --- a/drivers/crypto/uadk/uadk_crypto_pmd_private.h +++ b/drivers/crypto/uadk/uadk_crypto_pmd_private.h @@ -67,6 +67,7 @@ struct uadk_crypto_priv { bool env_cipher_init; bool env_auth_init; enum uadk_crypto_version version; + unsigned int max_nb_qpairs; } __rte_cache_aligned; extern int uadk_crypto_logtype; -- 2.39.2 (Apple Git-143)

2 years, 4 months

2
2
0 0

[PATCH V2 0/6] add init2 and dynamic load for digest/aead agls

by Hao Fang

compress/cipher/rsa/ecc already support the init2 and dynamic load, now add support for digest/aead algs. --- v1->v2: 1)fix bug for aead init testcase. 2)tinyfix for code style. --- Hao Fang (6): uadk/digest: introduce the init2 interface for digest uadk/aead: introduce the init2 interface for aead uadk/digest: adapt the module dynamic load for the digest algs uadk/aead: adadpt the module dynamic load for aead algs uadk/tools: add init2 test for digest algs uadk/tool: add init2 test for aead algs drv/hisi_sec.c | 105 +++++++++---- include/drv/wd_aead_drv.h | 26 ---- include/drv/wd_digest_drv.h | 26 ---- include/wd_aead.h | 28 ++++ include/wd_digest.h | 28 ++++ libwd_crypto.map | 6 + uadk_tool/test/test_sec.c | 249 ++++++++++++++++++++++++++++--- wd_aead.c | 288 +++++++++++++++++++++++++++--------- wd_digest.c | 286 ++++++++++++++++++++++++++--------- 9 files changed, 794 insertions(+), 248 deletions(-) -- 2.30.0

2 years, 4 months

1
6
0 0

uadk scheduler proposal review

by Zhangfei Gao

Hi,  Pls help review uadk scheduler proposal More info 【腾讯文档】uadk scheduler proposal https://docs.qq.com/doc/DRWxHeEFpVm9qVHJa workers。","range":{"gcpBegin":409,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":409,"len":55},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":464,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":464,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":464,"len":1},"property":{"pStyle":{"val":"ablt93"},"numPr":{"ilvl":{"val":0},"numId":{"val":"6u41ua"}},"pBdr":{},"snapToGrid":{"val":false},"jc":{"val":"STJcWml_left"},"taskPr":{},"rPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":"Normal"},{"operationType":"InsertText","param":{"text":"scheduler驱动接管所有ops，再根据policy，分发给workers。","range":{"gcpBegin":465,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":465,"len":40},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":505,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":505,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":505,"len":1},"property":{"pStyle":{"val":"ablt93"},"numPr":{"ilvl":{"val":0},"numId":{"val":"6u41ua"}},"pBdr":{},"snapToGrid":{"val":false},"jc":{"val":"STJcWml_left"},"taskPr":{},"rPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":"Normal"},{"operationType":"InsertText","param":{"text":"最简单是直接转发给默认worker，当worker_num=1，忽略policy.","range":{"gcpBegin":506,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":506,"len":41},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":547,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":547,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":547,"len":1},"property":{"pStyle":{"val":"ablt93"},"numPr":{"ilvl":{"val":0},"numId":{"val":"6u41ua"}},"pBdr":{},"snapToGrid":{"val":false},"jc":{"val":"STJcWml_left"},"taskPr":{},"rPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":"Normal"},{"operationType":"InsertText","param":{"text":"alloc_scheduler(default_worker)传默认worker, 后续再解析环境变量，config文件或者参数传递，增加其他worker。","range":{"gcpBegin":548,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":548,"len":78},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":626,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":626,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":626,"len":1},"property":{"pStyle":{"val":"ablt93"},"numPr":{"ilvl":{"val":0},"numId":{"val":"6u41ua"}},"pBdr":{},"snapToGrid":{"val":false},"jc":{"val":"STJcWml_left"},"taskPr":{},"rPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":"Normal"},{"operationType":"InsertText","param":{"text":"加速器驱动不用主动注册到uadk，因为需要dlopen->probe，parse scheduler worker参数的时候再解析，类似qemu，","range":{"gcpBegin":627,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":627,"len":73},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":700,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":700,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":700,"len":1},"property":{"pStyle":{"val":"ablt93"},"numPr":{"ilvl":{"val":0},"numId":{"val":"6u41ua"}},"pBdr":{"bottom":{}},"snapToGrid":{"val":false},"jc":{"val":"STJcWml_left"},"taskPr":{},"rPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":"Normal"},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":701,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":701,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":701,"len":1},"property":{"pStyle":{"val":"ablt93"},"pBdr":{},"snapToGrid":{"val":false},"jc":{"val":"STJcWml_left"},"taskPr":{},"rPr":{},"numPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":"Normal"},{"operationType":"InsertText","param":{"text":"before: app -> crypto/compress api -> ops -> driver ops","range":{"gcpBegin":702,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":702,"len":56},"property":{"b":{"val":false},"i":{"val":false},"strike":{"val":false},"spacing":{"val":0},"u":{"val":"STUnderline_none"},"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":758,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":758,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":758,"len":1},"property":{"numPr":{},"snapToGrid":{"val":true},"spacing":{"line":240}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"now: app -> crypto/compress api -> ops -> scheduler ops -> policy -> driver a ops","range":{"gcpBegin":759,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":759,"len":81},"property":{"b":{"val":false},"i":{"val":false},"strike":{"val":false},"spacing":{"val":0},"u":{"val":"STUnderline_none"},"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":840,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":840,"len":1},"property":{"b":{"val":false},"i":{"val":false},"strike":{"val":false},"spacing":{"val":0},"u":{"val":"STUnderline_none"},"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":840,"len":1},"property":{"pStyle":{"val":"ablt93"},"numPr":{},"pBdr":{},"snapToGrid":{"val":false},"jc":{"val":"STJcWml_left"},"taskPr":{},"rPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":"Normal"},{"operationType":"InsertText","param":{"text":"\t\t\t\t\t\t\t\t\t\t\t\t\t -> driver b ops","range":{"gcpBegin":841,"len":0}},&…" data-version="3.0.0" data-hash="f4b78a37065c67af61f432e7019a7b2f"> Background uadk targets to support more accelerators, including cpu instruction (isa ce and isa sve) and other vendors. Required from Fanghao Goal 1.  How to support these accelerators in the uadk framework, now uadk only support HiSilicon accelerator. 2.  How to switch between these accelerators to get full performance. Proposal 大概的想法，参考dpdk 1.  scheduler也是一个驱动，采用同样的驱动接口ops 2.  必选的驱动，buildin，简化动态加载 3.  该驱动权限更高，接管所有ops， 4.  scheduler驱动可带多个workers，即真正的加速器驱动，scheduler -> workers。 5.  scheduler驱动接管所有ops，再根据policy，分发给workers。 6.  最简单是直接转发给默认worker，当worker_num=1，忽略policy. 7.  alloc_scheduler(default_worker)传默认worker, 后续再解析环境变量，config文件或者参数传递，增加其他worker。 8.  加速器驱动不用主动注册到uadk，因为需要dlopen->probe，parse scheduler worker参数的时候再解析，类似qemu， before: app -> crypto/compress api -> ops -> driver ops now: app -> crypto/compress api -> ops -> scheduler ops -> policy -> driver a ops -> driver b ops adapter","range":{"gcpBegin":4,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":4,"len":21},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"SettingOperation","param":{"property":{"abstractNum":{"bgss1a":{"abstractNumId":"bgss1a","lvl":{"0":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_decimal"},"lvlText":{"val":"%1."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":336,"hanging":336}},"rPr":{"rFonts":{}},"ilvl":0},"1":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_lowerLetter"},"lvlText":{"val":"%2."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":776,"hanging":336}},"ilvl":1,"rPr":{"rFonts":{}}},"2":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_lowerRoman"},"lvlText":{"val":"%3."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":1216,"hanging":336}},"ilvl":2,"rPr":{"rFonts":{}}},"3":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_decimal"},"lvlText":{"val":"%4."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":1656,"hanging":336}},"ilvl":3,"rPr":{"rFonts":{}}},"4":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_lowerLetter"},"lvlText":{"val":"%5."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":2096,"hanging":336}},"ilvl":4,"rPr":{"rFonts":{}}},"5":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_lowerRoman"},"lvlText":{"val":"%6."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":2536,"hanging":336}},"ilvl":5,"rPr":{"rFonts":{}}},"6":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_decimal"},"lvlText":{"val":"%7."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":2976,"hanging":336}},"ilvl":6,"rPr":{"rFonts":{}}},"7":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_lowerLetter"},"lvlText":{"val":"%8."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":3416,"hanging":336}},"ilvl":7,"rPr":{"rFonts":{}}},"8":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_lowerRoman"},"lvlText":{"val":"%9."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":3856,"hanging":336}},"ilvl":8,"rPr":{"rFonts":{}}}},"lvl_dict_op":{"opType":"Update"}}},"abstractNum_dict_op":{"opType":"Update"},"num":{"uok3bl":{"abstractNumId":{"val":"bgss1a"},"numId":"uok3bl"}},"num_dict_op":{"opType":"Update"}},"settingType":"numbering"},"propertyType":"NumberingProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":25,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":25,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":25,"len":1},"property":{"numPr":{"ilvl":{"val":0},"numId":{"val":"uok3bl"}},"pBdr":{},"snapToGrid":{"val":true},"spacing":{"before":0,"after":0,"line":240},"taskPr":{},"ind":{},"jc":{"val":"STJcWml_left"}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"shengyang: request_ctx, 没有加速器会失败，用户程序忽略错误，用户程序看下。","range":{"gcpBegin":26,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":26,"len":49},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":75,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":75,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":75,"len":1},"property":{"numPr":{"ilvl":{"val":0},"numId":{"val":"uok3bl"}},"pBdr":{},"snapToGrid":{"val":true},"spacing":{"before":0,"after":0,"line":240},"taskPr":{},"ind":{},"jc":{"val":"STJcWml_left"}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"wd_xxx_init->alloc_sheduler时需要缺省worker, 如何定，直接hardcode，或者wd_xxx_init解析环境？","range":{"gcpBegin":76,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":76,"len":73},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":149,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":149,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":149,"len":1},"property":{"numPr":{"ilvl":{"val":0},"numId":{"val":"uok3bl"}},"pBdr":{},"snapToGrid":{"val":true},"spacing":{"before":0,"after":0,"line":240},"taskPr":{},"ind":{},"jc":{"val":"STJcWml_left"}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"alloc_scheduler woker[8] 简单用数组，还是链表之类，不限制个数","range":{"gcpBegin":150,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":150,"len":43},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":193,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":193,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":193,"len":1},"property":{"numPr":{"ilvl":{"val":0},"numId":{"val":"uok3bl"}},"pBdr":{},"snapToGrid":{"val":true},"spacing":{"before":0,"after":0,"line":240},"taskPr":{},"ind":{},"jc":{"val":"STJcWml_left"}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"worker相同是否要支持。","range":{"gcpBegin":194,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":194,"len":14},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":208,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":208,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":208,"len":1},"property":{"numPr":{"ilvl":{"val":0},"numId":{"val":"uok3bl"}},"pBdr":{},"snapToGrid":{"val":true},"spacing":{"before":0,"after":0,"line":240},"taskPr":{},"ind":{},"jc":{"val":"STJcWml_left"}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":""}],"subStory":[],"srcGlobalPadId":"300000000$ElGxAiVojTrZ","copyStart":2292}" data-version="3.0.0" data-hash="6a0b89629c5baee33b29d681d3ded575"> TBD 1.  longfang, 名字->adapter 2.  shengyang: request_ctx, 没有加速器会失败，用户程序忽略错误，用户程序看下。 3.  wd_xxx_init->alloc_sheduler时需要缺省worker, 如何定，直接hardcode，或者wd_xxx_init解析环境？ 4.  alloc_scheduler woker[8] 简单用数组，还是链表之类，不限制个数 5.  worker相同是否要支持。 alloc_scheduler","range":{"gcpBegin":513,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":513,"len":30},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":543,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":543,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":543,"len":1},"property":{"pBdr":{},"snapToGrid":{"val":true},"spacing":{"before":0,"after":0,"line":240},"ind":{"left":776,"leftChars_i":true},"numPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":544,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":544,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":544,"len":1},"property":{"pBdr":{},"snapToGrid":{"val":true},"spacing":{"before":0,"after":0,"line":240},"ind":{"left":776},"taskPr":{},"numPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"为简化, worker 暂时只增加不减少，初始化的时候，parse条件，一步到位，","range":{"gcpBegin":545,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":545,"len":41},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":586,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":586,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":586,"len":1},"property":{"numPr":{"ilvl":{"val":0},"numId":{"val":"0tju7e"}},"pBdr":{},"snapToGrid":{"val":true},"spacing":{"before":0,"after":0,"line":240},"taskPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"先不考虑动态减少worker的情况。后面看情况","range":{"gcpBegin":587,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":587,"len":23},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":610,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":610,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":610,"len":1},"property":{"pBdr":{},"snapToGrid":{"val":true},"spacing":{"before":0,"after":0,"line":240},"ind":{"left":0,"firstLineChars":200},"numPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":611,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":611,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":611,"len":1},"property":{"pBdr":{},"snapToGrid":{"val":true},"spacing":{"before":0,"after":0,"line":240},"ind":{"left":776},"taskPr":{},"numPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"为简化，暂时不提供动态修改参数的api，诸如threshold.","range":{"gcpBegin":612,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":612,"len":32},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":644,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":644,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":644,"len":1},"property":{"numPr":{"ilvl":{"val":0},"numId":{"val":"0tju7e"}},"snapToGrid":{"val":true},"spacing":{"before":0,"after":0,"line":240},"taskPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":645,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":645,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":645,"len":1},"property":{"pStyle":{"val":"ablt93"},"pBdr":{},"snapToGrid":{"val":false},"jc":{"val":"STJcWml_left"},"taskPr":{},"rPr":{},"numPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":"Normal"},{"operationType":"InsertText","param":{"text":"Example:","range":{"gcpBegin":646,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":646,"len":8},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":654,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":654,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":654,"len":1},"property":{"pStyle":{"val":"bdfjvo"},"pBdr":{},"taskPr":{},"rPr":{},"numPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":"heading 3"},{"operationType":"InsertText","param":{"text":"参数传递","range":{"gcpBegin":655,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":655,"len":4},"property":{"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"SettingOperation","param":{"property":{"abstractNum":{"wvis5d":{"abstractNumId":"wvis5d","lvl":{"0":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_decimal"},"lvlText":{"val":"%1."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":336,"hanging":336}},"rPr":{},"ilvl":0},"1":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_lowerLetter"},"lvlText":{"val":"%2."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":776,"hanging":336}},"ilvl":1},"2":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_lowerRoman"},"lvlText":{"val":"%3."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":1216,"hanging":336}},"ilvl":2},"3":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_decimal"},"lvlText":{"val":"%4."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":1656,"hanging":336}},"ilvl":3},"4":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_lowerLetter"},"lvlText":{"val":"%5."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":2096,"hanging":336}},"ilvl":4},"5":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_lowerRoman"},"lvlText":{"val":"%6."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":2536,"hanging":336}},"ilvl":5},"6":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_decimal"},"lvlText":{"val":"%7."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":2976,"hanging":336}},"ilvl":6},"7":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_lowerLetter"},"lvlText":{"val":"%8."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":3416,"hanging":336}},"ilvl":7},"8":{"start":{"val":1},"numFmt":{"val":"STNumberFormat_lowerRoman"},"lvlText":{"val":"%9."},"lvlJc":{"val":"STJcWml_left"},"pPr":{"ind":{"left":3856,"hanging":336}},"ilvl":8}},"lvl_dict_op":{"opType":"Update"}}},"abstractNum_dict_op":{"opType":"Update"},"num":{"nrbh2k":{"abstractNumId":{"val":"wvis5d"},"numId":"nrbh2k"}},"num_dict_op":{"opType":"Update"}},"settingType":"numbering"},"propertyType":"NumberingProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":659,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":659,"len":1},"property":{"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":659,"len":1},"property":{"pStyle":{"val":"ablt93"},"numPr":{"ilvl":{"val":0},"numId":{"val":"nrbh2k"}},"pBdr":{},"taskPr":{},"rPr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":"Normal"},{"operationType":"InsertText","param":{"text":"... --vdev \"crypto_aesni_mb0,name=aesni_mb_1\" --vdev \"crypto_aesni_mb1,name=aesni_mb_2\" --vdev \"crypto_scheduler,worker=aesni_mb_1,worker=aesni_mb_2\" ...","range":{"gcpBegin":660,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":660,"len":153},"property":{"rFonts":{"ascii":"SFMono-Regular, Menlo, Monaco, Consolas, \"Liberation Mono\", \"Courier New\", Courier, monospace","hAnsi":"SFMono-Regular, Menlo, Monaco, Consolas, \"Liberation Mono\", \"Courier New\", Courier, monospace","eastAsia":"SFMono-Regular, Menlo, Monaco, Consolas, \"Liberation Mono\", \"Courier New\", Courier, monospace","cs":"SFMono-Regular, Menlo, Monaco, Consolas, \"Liberation Mono\", \"Courier New\", Courier, monospace"},"i":{"val":false},"strike":{"val":false},"color":{"val":"888888"},"sz":{"val":180},"u":{"val":"STUnderline_none"},"shd":{"val":"STShd_clear","color":"STHexColorAuto_auto","fill":"F8F8F8"},"author":"p.144115215818243108"},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"InsertText","param":{"text":"\r","range":{"gcpBegin":813,"len":0}},"builtinStyleName":""},{"operationType":"ModifyRunProp","param":{"range":{"gcpBegin":813,"len":1},"property":{"rFonts":{"ascii":"SFMono-Regular, Menlo, Monaco, Consolas, \"Liberation Mono\", \"Courier New\", Courier, monospace","hAnsi":"SFMono-Regular, Menlo, Monaco, Consolas, \"Liberation Mono\", \"Courier New\", Courier, monospace","eastAsia":"SFMono-Regular, Menlo, Monaco, Consolas, \"Liberation Mono\", \"Courier New\", Courier, monospace","cs":"SFMono-Regular, Menlo, Monaco, Consolas, \"Liberation Mono\", \"Courier New\", Courier, monospace"},"i":{"val":false},"strike":{"val":false},"color":{"val":"888888"},"sz":{"val":180},"u":{"val":"STUnderline_none"},"shd":{"val":"STShd_clear","color":"STHexColorAuto_auto","fill":"F8F8F8"},"author":"p.144115215818243108","isPlaceholder":true},"mode":1},"propertyType":"RunProperty","builtinStyleName":""},{"operationType":"ModifyParagraphProp","param":{"range":{"gcpBegin":813,"len":1},"property":{"numPr":{},"pBdr":{}},"mode":1},"propertyType":"ParagraphProperty","builtinStyleName":""}],"subStory":[],"srcGlobalPadId":"300000000$ElGxAiVojTrZ","copyStart":4718}" data-version="3.0.0" data-hash="03cbb5d001222496f5029b26b3fd3d59"> Advantage no big uapi change required one api for setting para. (env, conf, para setting) Disadvantage / Limitation 1.  要求前提：所有驱动ops相同, comp, crypto，未来指令加速驱动？ struct wd_alg_driver { const char *drv_name; const char *alg_name; ～ int (*init)(void *conf, void *priv); void (*exit)(void *priv); int (*send)(handle_t ctx, void *drv_msg); int (*recv)(handle_t ctx, void *drv_msg); int (*get_usage)(void *param); }; 2.  一种算法，一个scheduler? (可以多个woker/driver） wd_xxx_init -> alloc_scheduler 3.  为简化, worker 暂时只增加不减少，初始化的时候，parse条件，一步到位，先不考虑动态减少worker的情况。后面看情况 4.  为简化，暂时不提供动态修改参数的api，诸如threshold. Example: 1.  参数传递 ... --vdev "crypto_aesni_mb0,name=aesni_mb_1" --vdev "crypto_aesni_mb1,name=aesni_mb_2" --vdev "crypto_scheduler,worker=aesni_mb_1,worker=aesni_mb_2" ...  

2 years, 4 months

3
7
0 0

2025

2024

2023

2022

Acc