Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 1 Dec 2013 12:56:02 -0800
From:      Adrian Chadd <adrian@freebsd.org>
To:        Gleb Smirnoff <glebius@freebsd.org>
Cc:        svn-src-projects@freebsd.org, "src-committers@freebsd.org" <src-committers@freebsd.org>
Subject:   Re: svn commit: r258813 - projects/sendfile/sys/kern
Message-ID:  <CAJ-VmokgxoCA-aHa6Y1RdEr=3_dTwFPCQ=QXSG2bbmMp3tgsBA@mail.gmail.com>
In-Reply-To: <201312012032.rB1KWHTR096175@svn.freebsd.org>
References:  <201312012032.rB1KWHTR096175@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Is this required for -HEAD right now?


-a

On 1 December 2013 12:32, Gleb Smirnoff <glebius@freebsd.org> wrote:
> Author: glebius
> Date: Sun Dec  1 20:32:16 2013
> New Revision: 258813
> URL: http://svnweb.freebsd.org/changeset/base/258813
>
> Log:
>   More correct code that handles sf wait logic.
>
>   Submitted by: kib
>
> Modified:
>   projects/sendfile/sys/kern/uipc_syscalls.c
>
> Modified: projects/sendfile/sys/kern/uipc_syscalls.c
> ==============================================================================
> --- projects/sendfile/sys/kern/uipc_syscalls.c  Sun Dec  1 20:03:00 2013        (r258812)
> +++ projects/sendfile/sys/kern/uipc_syscalls.c  Sun Dec  1 20:32:16 2013        (r258813)
> @@ -2564,9 +2564,7 @@ retry_space:
>                          * threads might exhaust the buffers and then
>                          * deadlock.
>                          */
> -                       if (m != NULL)
> -                               sfwait = SFB_NOWAIT;
> -                       sf = sf_buf_alloc(pg, sfwait);
> +                       sf = sf_buf_alloc(pg, m != NULL : SFB_NOWAIT ? sfwait);
>                         if (sf == NULL) {
>                                 SFSTAT_INC(sf_allocfail);
>                                 vm_page_lock(pg);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-VmokgxoCA-aHa6Y1RdEr=3_dTwFPCQ=QXSG2bbmMp3tgsBA>