From owner-svn-src-head@FreeBSD.ORG Wed Oct 31 18:50:18 2012 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 32587ACE; Wed, 31 Oct 2012 18:50:18 +0000 (UTC) (envelope-from adrian.chadd@gmail.com) Received: from mail-pb0-f54.google.com (mail-pb0-f54.google.com [209.85.160.54]) by mx1.freebsd.org (Postfix) with ESMTP id E193E8FC15; Wed, 31 Oct 2012 18:50:17 +0000 (UTC) Received: by mail-pb0-f54.google.com with SMTP id rp8so1275339pbb.13 for ; Wed, 31 Oct 2012 11:50:17 -0700 (PDT) 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=E7dGQrKCD+VptTo+YT4kemVmw3TvGALCZop9azqXUA8=; b=ErCV4jgavewes0nFg7PuKPb/6oObLbYMzCMsT+z5OgdNmy3bfuJr+Sj+T6x/MGdFDa KFhr8qNlu6ssTibU+w6MH0pqiWVZ/Owg/+ayof7x9pm0iz1UerQtSkFi65n0OBqbx8LK /8uGOw7ynGZH0V++kD6Ko7fIFOMA+mB4Eq4RzMu+nOfAf8kvCq8wsJWl/RpIM2yw8gmu Wskh3umQ801QITcjsH9dkZAw8VQRkHuRSiRw4//digsKa36H6YhoswKk110xmDc+AbR1 l8iOdaJMEfo1GECNdEMn0m884jrWGXXzSEK3zYQfRBbmqq/3Yrm1jn/b+SM2rmpqMvek Jd4g== MIME-Version: 1.0 Received: by 10.68.137.41 with SMTP id qf9mr44981185pbb.103.1351709417366; Wed, 31 Oct 2012 11:50:17 -0700 (PDT) Sender: adrian.chadd@gmail.com Received: by 10.68.124.130 with HTTP; Wed, 31 Oct 2012 11:50:17 -0700 (PDT) In-Reply-To: References: <201210311807.q9VI7IcX000993@svn.freebsd.org> <1351707964.1120.97.camel@revolution.hippie.lan> Date: Wed, 31 Oct 2012 11:50:17 -0700 X-Google-Sender-Auth: 0OQYUSoNmcQEDu6ptPaJVmDQVqY Message-ID: Subject: Re: svn commit: r242402 - in head/sys: kern vm From: Adrian Chadd To: attilio@freebsd.org Content-Type: text/plain; charset=ISO-8859-1 Cc: svn-src-head@freebsd.org, Ian Lepore , svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 31 Oct 2012 18:50:18 -0000 On 31 October 2012 11:33, Attilio Rao wrote: >> Doesn't this padding to cache line size only help x86 processors in an >> SMP kernel? I was expecting to see some #ifdef SMP so that we don't pay >> a big price for no gain in small-memory ARM systems and such. But maybe >> I'm misunderstanding the reason for the padding. > > I didn't want to do this because this would be meaning that SMP option > may become a completely killer for modules/kernel ABI compatibility. Right, but you didn't make it configurable for us embedded peeps who still care about memory usage. > Also, if you look at the modified list of locks I don't think they > should be too much, I hardly believe ARM UP is going to hurt that much > from loosing some padding in tdq structures or callout. There's a few million more embedded MIPS boards out there with 16mb/32mb of RAM than target PCs for FreeBSD. Would you mind making the padding part configurable and just default it to "do the padding" ? That way for the atheros MIPS builds I can turn it off and save on the memory overhead. Thanks, Adrian