31/07/2020 19:23, Ajit Khaparde:
> From: Somnath Kotur <[email protected]>
>
> Set ulp_ctx explicitly to NULL in ulp_ctx_deinit() so that representor
> init is aborted if parent ulp context is not initialized.
> Also check for the same before creation of port default rules.
> Additional checks in VF rep dev ops for proper parent dev initialization
Looks like it should be separate patche.
I don't know what is "ulp", but if it an acronym,
it should be in capital letters.
> Fixes: 322bd6e70272 ("net/bnxt: add port representor infrastructure")
> Fixes: 313ac35ac701 ("net/bnxt: support ULP session manager init")
Not clear it is a fix.
Looks like adding more checks.
> Reviewed-by: Venkat Duvvuru <[email protected]>
Reviewed-by should be below Signed-off-by.
> Signed-off-by: Somnath Kotur <[email protected]>