From owner-svn-src-all@FreeBSD.ORG Sat Jan 12 16:56:24 2013 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id 760E6373; Sat, 12 Jan 2013 16:56:24 +0000 (UTC) (envelope-from adrian.chadd@gmail.com) Received: from mail-wi0-x229.google.com (wi-in-x0229.1e100.net [IPv6:2a00:1450:400c:c05::229]) by mx1.freebsd.org (Postfix) with ESMTP id 2DE9BFF3; Sat, 12 Jan 2013 16:56:23 +0000 (UTC) Received: by mail-wi0-f169.google.com with SMTP id hq12so482973wib.0 for ; Sat, 12 Jan 2013 08:56:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type; bh=q+q0YjXkji2n4a1qWatdaZ3BA1azA7I4bjuw/XOLA0Q=; b=YN/nf2FrMlS3kfOrCtgYj3RNT2gFqD4lt49zdD9Xv0rzMii+vb447u1UM9UzrzHjSm V5F0fzw1wG+2uQwCsrzeutwENKDnnrj39Pn9F4DlcSK0H4sKs9xS4VE4TSmCUcoitWeZ 7Q+9s/YXWGZjle4WOYrEwOTzUdiwUSogw/2EI29EXSlrUxZ2SO86bOu6CZHvd/YusIG+ KEovTPBeNnaR3tZT1SnoXPDph8PhcKN92IolsTiqGXMcQTiw+X7BX54F89Bsqd65ZPmF aDjRDLBeLU4+5E5rd9Cfz1RJCzmeB0aHP4hVIBOwI5twnYeT7KZyxBjFxB4RMVMPtgrl lIyQ== MIME-Version: 1.0 Received: by 10.180.88.40 with SMTP id bd8mr4537826wib.33.1358009782041; Sat, 12 Jan 2013 08:56:22 -0800 (PST) Sender: adrian.chadd@gmail.com Received: by 10.217.57.9 with HTTP; Sat, 12 Jan 2013 08:56:21 -0800 (PST) In-Reply-To: References: <201211272119.qARLJxXV061083@svn.freebsd.org> <50C1BC90.90106@freebsd.org> <50C25A27.4060007@bluezbox.com> <50C26331.6030504@freebsd.org> <50C26AE9.4020600@bluezbox.com> <50C3A3D3.9000804@freebsd.org> <50C3AF72.4010902@rice.edu> <330405A1-312A-45A5-BB86-4969478D8BBD@bluezbox.com> <50D03E83.8060908@rice.edu> <50DD081E.8000409@bluezbox.com> <50EB1841.5030006@bluezbox.com> <50EB22D2.6090103@rice.edu> <50EB415F.8020405@freebsd.org> <50F04FE5.7010406@rice.edu> Date: Sat, 12 Jan 2013 08:56:21 -0800 X-Google-Sender-Auth: 2CBnz6-loTgqGKTy30-VMnQ25Jo Message-ID: Subject: Re: svn commit: r243631 - in head/sys: kern sys From: Adrian Chadd To: "Jayachandran C." Content-Type: text/plain; charset=ISO-8859-1 Cc: src-committers@freebsd.org, Andre Oppermann , Alan Cox , svn-src-all@freebsd.org, Oleksandr Tymoshenko , freebsd-arch@freebsd.org, svn-src-head@freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 12 Jan 2013 16:56:24 -0000 Hi, I think this outlines a larger scale problem here, which is that way, way too many things are relying on maxfiles here and it wasn't properly reviewed or thought out before it made it into the tree. So, can we either: * review _all_ the places maxfiles is being used, and finally fix those; * .. or revert this work until said review and fixup is done? This is the kind of thing that we should just not mess up at this point.. Thanks, Adrian On 12 January 2013 07:46, Jayachandran C. wrote: > On Fri, Jan 11, 2013 at 11:16 PM, Alan Cox wrote: >> On 01/11/2013 05:38, Jayachandran C. wrote: > [...] >>> I see an issue with commit on MIPS XLP platform as well. >>> >>> With 16 GB physical memory, the ncallout is calculated to be 538881 >>> (since it is based on maxfiles - which is now based on the physical >>> memory). Due to this, the callwheel allocation per cpu is 16MB >>> (callwheelsize is 1MB). And on a 32 CPU machine, the total allocation >>> for callouts comes to 32*16MB = 512MB. >>> >>> I have worked around this issue for now by increasing VM_KMEM_SIZE_MAX >>> (which is 200MB now) - but I think a better fix is needed for this. >>> >> >> MIPS should use a definition for VM_KMEM_SIZE_MAX that scales with the >> kernel address space size, like amd64, i386, and sparc64, and not a >> fixed number. I think that the following should work for both 32- and >> 64-bit processors: >> >> Index: mips/include/vmparam.h >> =================================================================== >> --- mips/include/vmparam.h (revision 245229) >> +++ mips/include/vmparam.h (working copy) >> @@ -130,10 +130,11 @@ >> #endif >> >> /* >> - * Ceiling on amount of kmem_map kva space. >> + * Ceiling on the amount of kmem_map KVA space: 40% of the entire KVA >> space. >> */ >> #ifndef VM_KMEM_SIZE_MAX >> -#define VM_KMEM_SIZE_MAX (200 * 1024 * 1024) >> +#define VM_KMEM_SIZE_MAX ((VM_MAX_KERNEL_ADDRESS - \ >> + VM_MIN_KERNEL_ADDRESS + 1) * 2 / 5) >> #endif >> >> /* initial pagein size of beginning of executable file */ > > This fix is needed, can you please check it in? I have tested it for > 32 and 64 bit. > > But the second part of the problem - allocating 512MB out of 16GB at > boot-time for callouts - might need a fix as well. > > Thanks, > JC.