<snipped>

> +#define MAX_DRIVER_NAME              64
> +#define MAX_INPUT_FILE_NAME  64

<snipped>

> +comp_perf_options_check(struct comp_test_data *test_data) {
> +     if (strcmp(test_data->driver_name, "") == 0) {

Should we use strncmp for boundary?

> +             RTE_LOG(ERR, USER1, "Driver name has to be set\n");
> +             return -1;
> +     }
> +
> +     if (strcmp(test_data->input_file, "") == 0) {

Should we use strncmp for boundary?

> +             RTE_LOG(ERR, USER1, "Input file name has to be set\n");
> +             return -1;
> +     }
> +

<snipped>

> +     test_data = rte_zmalloc_socket(NULL, sizeof(struct comp_test_data),
> +                                     0, rte_socket_id());

Suggestion: if the PMD is socket specific, do we need to alloc socket specific 
too? (assuming HW or SW socket specific PMD can be used)

<snipped>

Reply via email to