Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 16 Jun 2016 10:17:19 -0700
From:      Conrad Meyer <cem@freebsd.org>
To:        Konstantin Belousov <kib@freebsd.org>
Cc:        src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org,  svn-src-head@freebsd.org
Subject:   Re: svn commit: r301959 - head/sys/kern
Message-ID:  <CAG6CVpXDDk6uQLrj7dgjP5aNhQoq_uwF1n3eHy83Qq9E0GEeEw@mail.gmail.com>
In-Reply-To: <201606161158.u5GBwkd1042632@repo.freebsd.org>
References:  <201606161158.u5GBwkd1042632@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Jun 16, 2016 at 4:58 AM, Konstantin Belousov <kib@freebsd.org> wrote:
> --- head/sys/kern/kern_thread.c Thu Jun 16 11:55:24 2016        (r301958)
> +++ head/sys/kern/kern_thread.c Thu Jun 16 11:58:46 2016        (r301959)
> @@ -669,11 +669,6 @@ weed_inhib(int mode, struct thread *td2,
>                         wakeup_swapper |= sleepq_abort(td2, EINTR);
>                 break;
>         case SINGLE_BOUNDARY:
> -               if (TD_IS_SUSPENDED(td2) && (td2->td_flags & TDF_BOUNDARY) == 0)
> -                       wakeup_swapper |= thread_unsuspend_one(td2, p, false);
> -               if (TD_ON_SLEEPQ(td2) && (td2->td_flags & TDF_SINTR) != 0)
> -                       wakeup_swapper |= sleepq_abort(td2, ERESTART);
> -               break;

style(9) nit: /* FALLTHROUGH */

Best,
Conrad

>         case SINGLE_NO_EXIT:
>                 if (TD_IS_SUSPENDED(td2) && (td2->td_flags & TDF_BOUNDARY) == 0)
>                         wakeup_swapper |= thread_unsuspend_one(td2, p, false);
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAG6CVpXDDk6uQLrj7dgjP5aNhQoq_uwF1n3eHy83Qq9E0GEeEw>