[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/7] hurd: Simplify _hurd_critical_section_lock a bit
From: |
Samuel Thibault |
Subject: |
Re: [PATCH 1/7] hurd: Simplify _hurd_critical_section_lock a bit |
Date: |
Sat, 29 Apr 2023 16:50:38 +0200 |
User-agent: |
NeoMutt/20170609 (1.8.3) |
Applied, thanks!
Sergey Bugaev, le sam. 29 avril 2023 16:13:48 +0300, a ecrit:
> This block of code was doing exactly what _hurd_self_sigstate does; so
> just call that and let it do its job.
>
> Signed-off-by: Sergey Bugaev <bugaevc@gmail.com>
> ---
> hurd/hurd/signal.h | 14 +-------------
> 1 file changed, 1 insertion(+), 13 deletions(-)
>
> diff --git a/hurd/hurd/signal.h b/hurd/hurd/signal.h
> index 662e955e..302ca25e 100644
> --- a/hurd/hurd/signal.h
> +++ b/hurd/hurd/signal.h
> @@ -218,19 +218,7 @@ _hurd_critical_section_lock (void)
> return NULL;
> #endif
>
> - ss = THREAD_GETMEM (THREAD_SELF, _hurd_sigstate);
> - if (ss == NULL)
> - {
> - thread_t self = __mach_thread_self ();
> -
> - /* The thread variable is unset; this must be the first time we've
> - asked for it. In this case, the critical section flag cannot
> - possible already be set. Look up our sigstate structure the slow
> - way. */
> - ss = _hurd_thread_sigstate (self);
> - THREAD_SETMEM (THREAD_SELF, _hurd_sigstate, ss);
> - __mach_port_deallocate (__mach_task_self (), self);
> - }
> + ss = _hurd_self_sigstate ();
>
> if (! __spin_try_lock (&ss->critical_section_lock))
> /* We are already in a critical section, so do nothing. */
> --
> 2.40.1
>
--
Samuel
---
Pour une évaluation indépendante, transparente et rigoureuse !
Je soutiens la Commission d'Évaluation de l'Inria.
- [PATCH 3/7] hurd: Mark various conditions as unlikely, (continued)
- [PATCH 3/7] hurd: Mark various conditions as unlikely, Sergey Bugaev, 2023/04/29
- [PATCH 6/7] hurd: Respect existing FD_CLOEXEC in S_msg_set_fd, Sergey Bugaev, 2023/04/29
- [PATCH 4/7] hurd: Make _exit work during early boot-up, Sergey Bugaev, 2023/04/29
- [PATCH 5/7] hurd: Don't leak the auth port in msg* RPCs, Sergey Bugaev, 2023/04/29
- [PATCH 7/7] hurd: Fix FS_RETRY_MAGICAL "machtype" handling, Sergey Bugaev, 2023/04/29
- Re: [PATCH 1/7] hurd: Simplify _hurd_critical_section_lock a bit,
Samuel Thibault <=