<div dir="ltr"><div dir="ltr">Hi dean, <br><br>Thank you for the review. The current implementation does draw values from tests_config.yaml file, and the config class is how those values are drawn from the config into the test suite.<br>I realize this may be confusing as I had failed to update the tests_config.example in this version. This will be updated with my v2, thank you for pointing this out.<br><br></div><div class="gmail_quote gmail_quote_container"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
Would it be worth creating a separate cryptodev config file in the<br>
configurations directory, or adding a cryptodev section to the<br>
tests_config.yaml file (similar to the single_core_forward_perf<br>
suite)? I feel like this would be more consistent with the current<br>
design of DTS, although if there's a good reason for the config class<br>
being in the suite then disregard this.<br>
<br>
Reviewed-by: Dean Marx <<a href="mailto:dmarx@iol.unh.edu" target="_blank">dmarx@iol.unh.edu</a>><br>
</blockquote></div></div>