[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/5] hurd: Simplify _S_catch_exception_raise
From: |
Samuel Thibault |
Subject: |
Re: [PATCH 4/5] hurd: Simplify _S_catch_exception_raise |
Date: |
Tue, 18 Apr 2023 01:24:15 +0200 |
User-agent: |
NeoMutt/20170609 (1.8.3) |
Applied, thanks!
Sergey Bugaev, le ven. 14 avril 2023 22:36:59 +0300, a ecrit:
> _hurd_thread_sigstate () already handles finding an existing sigstate
> before allocating a new one, so just use that. Bonus: this will only
> lock the _hurd_siglock once.
>
> Signed-off-by: Sergey Bugaev <bugaevc@gmail.com>
> ---
> hurd/catch-exc.c | 8 +-------
> 1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/hurd/catch-exc.c b/hurd/catch-exc.c
> index 5ee2233a..d375bf67 100644
> --- a/hurd/catch-exc.c
> +++ b/hurd/catch-exc.c
> @@ -58,13 +58,7 @@ _S_catch_exception_raise (mach_port_t port,
> _hurd_exception2signal (&d, &signo);
>
> /* Find the sigstate structure for the faulting thread. */
> - __mutex_lock (&_hurd_siglock);
> - for (ss = _hurd_sigstates; ss != NULL; ss = ss->next)
> - if (ss->thread == thread)
> - break;
> - __mutex_unlock (&_hurd_siglock);
> - if (ss == NULL)
> - ss = _hurd_thread_sigstate (thread); /* Allocate a fresh one. */
> + ss = _hurd_thread_sigstate (thread);
>
> if (__spin_lock_locked (&ss->lock))
> {
> --
> 2.39.2
>
--
Samuel
---
Pour une évaluation indépendante, transparente et rigoureuse !
Je soutiens la Commission d'Évaluation de l'Inria.
- [PATCH 1/5] hurd: Fix restoring reply port in sigreturn, Sergey Bugaev, 2023/04/14
- [PATCH 3/5] hurd: Implement sigreturn for x86_64, Sergey Bugaev, 2023/04/14
- [PATCH 4/5] hurd: Simplify _S_catch_exception_raise, Sergey Bugaev, 2023/04/14
- Re: [PATCH 4/5] hurd: Simplify _S_catch_exception_raise,
Samuel Thibault <=
- [PATCH 5/5] hurd: Avoid leaking task & thread ports, Sergey Bugaev, 2023/04/14
- Re: [PATCH 1/5] hurd: Fix restoring reply port in sigreturn, Samuel Thibault, 2023/04/17