From owner-freebsd-arch@FreeBSD.ORG  Tue Sep  4 17:10:44 2012
Return-Path: <owner-freebsd-arch@FreeBSD.ORG>
Delivered-To: freebsd-arch@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 16B7E1065670;
	Tue,  4 Sep 2012 17:10:44 +0000 (UTC) (envelope-from avg@FreeBSD.org)
Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140])
	by mx1.freebsd.org (Postfix) with ESMTP id BA3728FC18;
	Tue,  4 Sep 2012 17:10:42 +0000 (UTC)
Received: from odyssey.starpoint.kiev.ua (alpha-e.starpoint.kiev.ua
	[212.40.38.101])
	by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id UAA07436;
	Tue, 04 Sep 2012 20:10:41 +0300 (EEST)
	(envelope-from avg@FreeBSD.org)
Message-ID: <50463610.6070805@FreeBSD.org>
Date: Tue, 04 Sep 2012 20:10:40 +0300
From: Andriy Gapon <avg@FreeBSD.org>
User-Agent: Mozilla/5.0 (X11; FreeBSD amd64;
	rv:15.0) Gecko/20120830 Thunderbird/15.0
MIME-Version: 1.0
To: Andrey Zonov <zont@FreeBSD.org>,
	Edward Tomasz Napierala <trasz@FreeBSD.org>
References: <503DD433.2030108@FreeBSD.org>
	<201208290906.q7T96C9j032802@gw.catspoiler.org>
	<20120829092318.GW33100@deviant.kiev.zoral.com.ua>
	<503F2D24.8050103@FreeBSD.org> <503F476E.1010505@FreeBSD.org>
In-Reply-To: <503F476E.1010505@FreeBSD.org>
X-Enigmail-Version: 1.4.3
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Cc: freebsd-arch@FreeBSD.org
Subject: Re: [patch] unprivileged mlock(2)
X-BeenThere: freebsd-arch@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Discussion related to FreeBSD architecture <freebsd-arch.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-arch>,
	<mailto:freebsd-arch-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-arch>
List-Post: <mailto:freebsd-arch@freebsd.org>
List-Help: <mailto:freebsd-arch-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-arch>,
	<mailto:freebsd-arch-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 04 Sep 2012 17:10:44 -0000

on 30/08/2012 13:58 Andrey Zonov said the following:
> [1] http://people.freebsd.org/~zont/vm_mmap.c.patch

This patch is correct.

> [2] http://people.freebsd.org/~zont/racct.patch

This patch looks correct.

And it also makes me wonder why kern/kern_racct.c is marked as standard while
all(?) uses of racct API are placed under RACCT option.
Ditto for kern_rctl.c/RCTL.
I think that excluding these file if the options are not used would help to catch
cases where the API is used unconditionally and it would also help to reduce
kernel sizes a tiny bit too.

-- 
Andriy Gapon