From owner-freebsd-hackers@FreeBSD.ORG Mon Jul 18 13:54:33 2011 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 4670C106564A for ; Mon, 18 Jul 2011 13:54:33 +0000 (UTC) (envelope-from mdf356@gmail.com) Received: from mail-qw0-f54.google.com (mail-qw0-f54.google.com [209.85.216.54]) by mx1.freebsd.org (Postfix) with ESMTP id 035268FC1C for ; Mon, 18 Jul 2011 13:54:32 +0000 (UTC) Received: by qwc9 with SMTP id 9so2089946qwc.13 for ; Mon, 18 Jul 2011 06:54:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type; bh=P7mDoYXDfth/SVhKDjlCfcVJqvcyd987l6ga1Ms4Bog=; b=e64FJtXCMk4D1+K6ZqHFPxpZJLhFAfNYfUXcXa2IhUuoUIzaZpaw4mh+gFFj0+sx41 NuFLkXIkPpFHwPD9OfiUE9kdEGTdZHoaEiW0fmB83F55MCdkgaxwY/xr1V3zW0eIwVId U1U3Rn/si998FRnY6giA49RMXK6xNb8hlKyhs= MIME-Version: 1.0 Received: by 10.229.7.137 with SMTP id d9mr4742042qcd.251.1310997272286; Mon, 18 Jul 2011 06:54:32 -0700 (PDT) Sender: mdf356@gmail.com Received: by 10.229.11.40 with HTTP; Mon, 18 Jul 2011 06:54:32 -0700 (PDT) In-Reply-To: <4E2427E4.5080201@uffe.org> References: <4E2427E4.5080201@uffe.org> Date: Mon, 18 Jul 2011 06:54:32 -0700 X-Google-Sender-Auth: DthI1KzXSciKBxdftAi4KoYuO70 Message-ID: From: mdf@FreeBSD.org To: Uffe Jakobsen Content-Type: text/plain; charset=ISO-8859-1 Cc: freebsd-hackers@freebsd.org Subject: Re: [PATCH] sysctl cleanup - unifying sysctls: vm.kvm_size and vm.kvm_free X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 18 Jul 2011 13:54:33 -0000 On Mon, Jul 18, 2011 at 5:32 AM, Uffe Jakobsen wrote: > Please consider this patch - it unifies sysctls: vm.kvm_size and > vm.kvm_free. > > Currently these sysctls are only found under i386 and amd64: > > sys/i386/i386/pmap.c > sys/i386/xen/pmap.c > sys/amd64/amd64/pmap.c > > It seems logical (to me) to move them into a generic location suce as > sys/vm/vm_kern.c > > Patch against HEAD (revision 224180) attached. I don't believe that VM_MAX_KERNEL_ADDRESS is relevant to all architectures, which is why these sysctls are in i386/amd64 specific files. Thanks, matthew