Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 11 Aug 2020 15:21:31 -0700
From:      Conrad Meyer <cem@freebsd.org>
To:        Konstantin Belousov <kostikbel@gmail.com>, Mark Johnston <markj@freebsd.org>
Cc:        src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>,  svn-src-head <svn-src-head@freebsd.org>
Subject:   Re: svn commit: r364129 - head/sys/vm
Message-ID:  <CAG6CVpWhW4dtv1j4ced0Me2qj6eUvY5j59%2BOcYXPTDOvMno4_Q@mail.gmail.com>
In-Reply-To: <20200811213213.GT2551@kib.kiev.ua>
References:  <202008112037.07BKbjsL056699@repo.freebsd.org> <20200811213213.GT2551@kib.kiev.ua>

next in thread | previous in thread | raw e-mail | index | archive | help
Hi Konstantin, Mark (raised the same question),

On Tue, Aug 11, 2020 at 2:32 PM Konstantin Belousov <kostikbel@gmail.com> wrote:
>
> On Tue, Aug 11, 2020 at 08:37:45PM +0000, Conrad Meyer wrote:
> > Author: cem
> > Date: Tue Aug 11 20:37:45 2020
> > New Revision: 364129
> > URL: https://svnweb.freebsd.org/changeset/base/364129
> >
> > Log:
> >   Add support for multithreading the inactive queue pageout within a domain.
> > ...
> > @@ -2488,7 +2488,7 @@ vm_page_zone_import(void *arg, void **store, int cnt,
> >        * main purpose is to replenish the store of free pages.
> >        */
> >       if (vmd->vmd_severeset || curproc == pageproc ||
> > -         !_vm_domain_allocate(vmd, VM_ALLOC_NORMAL, cnt))
> > +         !_vm_domain_allocate(vmd, VM_ALLOC_SYSTEM, cnt))
>
> Why this change needed ?

The change was inherited from Jeff, along with the rest of it.  I
don't know why he changed it, but it does seem orthogonal to the rest
of the revision.  This part is nonessential as far as I know, and
could be backed out.

Best,
Conrad



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