<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Wed, Nov 11, 2026 at 11:30 AM Luca Vizzarro <<a href="mailto:Luca.Vizzarro@arm.com">Luca.Vizzarro@arm.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hey Koushik,<br>
<br>
It looks like not all of my comments were addressed, e.g. Artifact.<br></blockquote><div><br></div><div>Hi Luca, as I mentioned in the previous email all vhost socket files are handled by testpmd, ensuring that they are created and deleted properly. I don't think the use of Artifact will be required here, as it will interfere with testpmd's handling of the files.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +from pydantic import BaseModel, ConfigDict<br>
> +<br>
>   from api.params import (<br>
>       Params,<br>
>       Switch,<br>
> @@ -618,3 +620,15 @@ class TestPmdParams(EalParams):<br>
>       )<br>
>       record_core_cycles: Switch = None<br>
>       record_burst_status: Switch = None<br>
> +<br>
> +<br>
> +class PerfSuiteConfig(BaseModel):<br>
<br>
I wouldn't call the TestParameter a PerfSuiteConfig. Also I am not sure <br>
this is the right place for this? I think it should just live next to <br>
BaseConfig in api.test_suite. The idea is that PerfSuiteConfig would <br>
just be the test suite config model, as the name suggests. Instead of <br>
replicating the model across test suites just to set defaults. These can <br>
be a constant and then set during test suite setup as it's already done <br>
with other test suites. We only extend the config if we have extra <br>
special fields which are unique for said test suite. anything that is <br>
common across all of them should be kept in a common config.<br>
<br>
These changes can be their own commit before the test suites. Updating <br>
other test suites using the same pattern would also be a welcome change.<br>
<br></blockquote><div><br></div><div>I'm a bit confused by this, do you mean to say that the PerfSuiteConfig is something that should already hold the default values, but can be overridden by the specific test parameters in tests_config.yaml? So this would mean that performance test suites would only need to import this config for usage so the config declaration in the test suites wouldn't be necessary. </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +    """Generic performance test suite parameters."""<br>
> +<br>
> +    model_config = ConfigDict(strict=True, extra="forbid", frozen=False, from_attributes=True)<br>
> +    frame_size: int<br>
> +    num_descriptors: int<br>
> +    expected_mpps: float<br>
> +    performance_delta: float | None = None<br>
> +    measured_mpps: float | None = None<br>
> +    passing: float | None = None<br>
<br>
Still lacking documentation for the fields.<br></blockquote><div><br></div><div>Will add documentation in my next version, thanks! </div></div></div>