On 24.04.2024 05:34, Henry Wang wrote:
> --- a/xen/common/dt-overlay.c
> +++ b/xen/common/dt-overlay.c
> @@ -381,9 +381,14 @@ static int remove_node_resources(struct dt_device_node
> *device_node)
> {
> if ( dt_device_is_protected(device_node) )
> {
> + write_lock(&dt_host_lock);
> rc = iommu_remove_dt_device(device_node);
Any particular reason you add two call sites to the unlock function,
instead of putting it here?
Jan
> if ( rc < 0 )
> + {
> + write_unlock(&dt_host_lock);
> return rc;
> + }
> + write_unlock(&dt_host_lock);
> }
> }
>
