From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759577AbYDCWeB (ORCPT ); Thu, 3 Apr 2008 18:34:01 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756682AbYDCWdw (ORCPT ); Thu, 3 Apr 2008 18:33:52 -0400 Received: from saeurebad.de ([85.214.36.134]:52248 "EHLO saeurebad.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755091AbYDCWdv (ORCPT ); Thu, 3 Apr 2008 18:33:51 -0400 From: Johannes Weiner To: Sam Ravnborg Cc: Heiko Carstens , linux-kernel@vger.kernel.org, linux-mm@kvack.org, mingo@elte.hu, davem@davemloft.net, hskinnemoen@atmel.com, cooloney@kernel.org, starvik@axis.com, dhowells@redhat.com, ysato@users.sourceforge.net, takata@linux-m32r.org, geert@linux-m68k.org, ralf@linux-mips.org, kyle@parisc-linux.org, paulus@samba.org, schwidefsky@de.ibm.com, lethal@linux-sh.org, jdike@addtoit.com, miles@gnu.org, chris@zankel.net, rmk@arm.linux.org.uk, tony.luck@intel.com Subject: Re: [RFC 01/22] Generic show_mem() implementation References: <12071688283927-git-send-email-hannes@saeurebad.de> <1207168839586-git-send-email-hannes@saeurebad.de> <20080403075545.GC4125@osiris.boeblingen.de.ibm.com> <20080403124820.GA30356@uranus.ravnborg.org> <871w5nouwp.fsf@saeurebad.de> <20080403181202.GA32319@uranus.ravnborg.org> Date: Fri, 04 Apr 2008 00:33:42 +0200 In-Reply-To: <20080403181202.GA32319@uranus.ravnborg.org> (Sam Ravnborg's message of "Thu, 3 Apr 2008 20:12:02 +0200") Message-ID: <87prt6muux.fsf@saeurebad.de> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.0.60 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.1.3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Sam Ravnborg writes: > On Thu, Apr 03, 2008 at 04:49:42PM +0200, Johannes Weiner wrote: >> Hi, >> >> Sam Ravnborg writes: >> >> >> e.g. we currently have this in arch/s390/Kconfig: >> >> >> >> config S390 >> >> def_bool y >> >> select HAVE_OPROFILE >> >> select HAVE_KPROBES >> >> select HAVE_KRETPROBES >> >> >> >> just add a select HAVE_GENERIC_SHOWMEM or something like that in the arch >> >> specific patches. >> > Seconded. >> > See Documentation/kbuild/kconfig-language.txt for a few more hints >> > how to do it. >> >> After more thinking about it, wouldn't it be better to have >> HAVE_ARCH_SHOW_MEM in mm/Kconfig and let archs with their own show_mem() >> select it? Because there are far more archs that use the generic >> version than those having their own. > > Positive logic is almost always simpler to grasp. > And the usual way to do this is to let arch's select what they > use. > We do not want to have a situation where in most cases we select > a generic version but in some oddball case we select to have > a local version. I can not follow you. Of course the arch selects what they use. But they should not _all_ have to be flagged with an extra select. So what default-value are you arguing for? Hannes