Re: [PATCH v8 1/5] soc: qcom: pdr: protect locator_addr with the main mutex

On Thu, 6 Jun 2024 at 01:48, Chris Lew wrote: > > Hi Dmitry, > > On 5/11/2024 2:56 PM, Dmitry Baryshkov wrote: > ... > > @@ -76,12 +76,12 @@ static int pdr_locator_new_server(struct qmi_handle > > *qmi, > > locator_hdl); > > struct pdr_service *pd

Re: [PATCH v8 1/5] soc: qcom: pdr: protect locator_addr with the main mutex

Hi Dmitry, On 5/11/2024 2:56 PM, Dmitry Baryshkov wrote: ... @@ -76,12 +76,12 @@ static int pdr_locator_new_server(struct qmi_handle *qmi, locator_hdl); struct pdr_service *pds; + mutex_lock(&pdr->lock); /* Create a local client p

[PATCH v8 1/5] soc: qcom: pdr: protect locator_addr with the main mutex

If the service locator server is restarted fast enough, the PDR can rewrite locator_addr fields concurrently. Protect them by placing modification of those fields under the main pdr->lock. Fixes: fbe639b44a82 ("soc: qcom: Introduce Protection Domain Restart helpers") Tested-by: Neil Armstrong # o