svn commit: r325530 - in head/sys: amd64/amd64 arm64/arm64 fs/tmpfs kern mips/mips powerpc/aim powerpc/powerpc sparc64/sparc64 vm x86/iommu

[Emmanuel Vadot]

On Thu, 9 Nov 2017 01:25:28 +0200
Konstantin Belousov <kostikbel at gmail.com> wrote:

> On Wed, Nov 08, 2017 at 11:41:15PM +0100, Emmanuel Vadot wrote:
> > 
> >  Hello Jeff,
> > 
> > On Wed, 8 Nov 2017 02:39:37 +0000 (UTC)
> > Jeff Roberson <jeff at FreeBSD.org> wrote:
> > 
> > > Author: jeff
> > > Date: Wed Nov  8 02:39:37 2017
> > > New Revision: 325530
> > > URL: https://svnweb.freebsd.org/changeset/base/325530
> > > 
> > > Log:
> > >   Replace manyinstances of VM_WAIT with blocking page allocation flags
> > >   similar to the kernel memory allocator.
> > >   
> > >   This simplifies NUMA allocation because the domain will be known at wait
> > >   time and races between failure and sleeping are eliminated.  This also
> > >   reduces boilerplate code and simplifies callers.
> > >   
> > >   A wait primitive is supplied for uma zones for similar reasons.  This
> > >   eliminates some non-specific VM_WAIT calls in favor of more explicit
> > >   sleeps that may be satisfied without new pages.
> > >   
> > >   Reviewed by:	alc, kib, markj
> > >   Tested by:	pho
> > >   Sponsored by:	Netflix, Dell/EMC Isilon
> > > 
> > 
> >  This break armv7 (atleast) kernel :
> > 
> > KDB: debugger backends:
> > ddb KDB: current backend:
> > ddb Copyright (c) 1992-2017 The FreeBSD
> > Project. Copyright (c) 1979, 1980, 1983, 1986, 1988, 1989, 1991, 1992,
> > 1993, 1994 The Regents of the University of California. All rights
> > reserved. FreeBSD is a registered trademark of The FreeBSD
> > Foundation. FreeBSD 12.0-CURRENT #51 3c355d849c9(HEAD): Wed Nov  8
> > 23:24:16 CET 2017
> > elbarto at knuckles.blih.net:/usr/home/elbarto/Work/freebsd/obj/usr/home/elbarto/Work/freebsd/freebsd.git/arm.armv7/sys/GENERIC
> > arm FreeBSD clang version 5.0.0 (tags/RELEASE_500/final 312559) (based
> > on LLVM 5.0.0svn) WARNING: WITNESS option enabled, expect reduced
> > performance. panic: inconsistent object(0)/req(8) cpuid =
> > 0 time =
> 
> Should be fixed by r325566.

 Confirmed, thank you.

-- 
Emmanuel Vadot <manu at bidouilliste.com> <manu at freebsd.org>