Re: [PATCH 3/7] hurd: Mark various conditions as unlikely

[Samuel Thibault <samuel.thibault@gnu.org>]

Applied, thanks!

Sergey Bugaev, le sam. 29 avril 2023 16:13:50 +0300, a ecrit:
> Signed-off-by: Sergey Bugaev <bugaevc@gmail.com>
> ---
>  hurd/hurd/fd.h     | 10 +++++-----
>  hurd/hurd/signal.h |  4 ++--
>  2 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/hurd/hurd/fd.h b/hurd/hurd/fd.h
> index 241797bf..824b7dbb 100644
> --- a/hurd/hurd/fd.h
> +++ b/hurd/hurd/fd.h
> @@ -73,18 +73,18 @@ _hurd_fd_get (int fd)
>  
>    HURD_CRITICAL_BEGIN;
>    __mutex_lock (&_hurd_dtable_lock);
> -  if (fd < 0 || fd >= _hurd_dtablesize)
> +  if (__glibc_unlikely (fd < 0 || fd >= _hurd_dtablesize))
>      descriptor = NULL;
>    else
>      {
>        struct hurd_fd *cell = _hurd_dtable[fd];
> -      if (cell == NULL)
> +      if (__glibc_unlikely (cell == NULL))
>  	/* No descriptor allocated at this index.  */
>  	descriptor = NULL;
>        else
>  	{
>  	  __spin_lock (&cell->port.lock);
> -	  if (cell->port.port == MACH_PORT_NULL)
> +	  if (__glibc_unlikely (cell->port.port == MACH_PORT_NULL))
>  	    /* The descriptor at this index has no port in it.
>  	       This happens if it existed before but was closed.  */
>  	    descriptor = NULL;
> @@ -107,7 +107,7 @@ _hurd_fd_get (int fd)
>  
>  #define	HURD_FD_USE(fd, expr)						      \
>    ({ struct hurd_fd *descriptor = _hurd_fd_get (fd);			      \
> -     descriptor == NULL ? EBADF : (expr); })
> +     __glibc_unlikely (descriptor == NULL) ? EBADF : (expr); })
>  
>  /* Evaluate EXPR with the variable `port' bound to the port to FD, and
>     `ctty' bound to the ctty port.  */
> @@ -125,7 +125,7 @@ _hurd_fd_get (int fd)
>       io_t port, ctty;							      \
>       void *crit = _hurd_critical_section_lock ();			      \
>       __spin_lock (&__d->port.lock);					      \
> -     if (__d->port.port == MACH_PORT_NULL)				      \
> +     if (__glibc_unlikely (__d->port.port == MACH_PORT_NULL))		      \
>         {								      \
>  	 __spin_unlock (&__d->port.lock);				      \
>  	 _hurd_critical_section_unlock (crit);				      \
> diff --git a/hurd/hurd/signal.h b/hurd/hurd/signal.h
> index 302ca25e..8a547fd0 100644
> --- a/hurd/hurd/signal.h
> +++ b/hurd/hurd/signal.h
> @@ -249,9 +249,9 @@ _hurd_critical_section_unlock (void *our_lock)
>        sigset_t pending;
>        _hurd_sigstate_lock (ss);
>        __spin_unlock (&ss->critical_section_lock);
> -      pending = _hurd_sigstate_pending(ss) & ~ss->blocked;
> +      pending = _hurd_sigstate_pending (ss) & ~ss->blocked;
>        _hurd_sigstate_unlock (ss);
> -      if (! __sigisemptyset (&pending))
> +      if (__glibc_unlikely (!__sigisemptyset (&pending)))
>  	/* There are unblocked signals pending, which weren't
>  	   delivered because we were in the critical section.
>  	   Tell the signal thread to deliver them now.  */
> -- 
> 2.40.1
> 

-- 
Samuel
---
Pour une évaluation indépendante, transparente et rigoureuse !
Je soutiens la Commission d'Évaluation de l'Inria.