linux-kernel - Re: [patch 02/19] perfmon2 minimal v2: base code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 23 Jun 2008 12:00:53 +1000
From:	Stephen Rothwell <sfr@...b.auug.org.au>
To:	eranian@...glemail.com
Cc:	linux-kernel@...r.kernel.org
Subject: Re: [patch 02/19] perfmon2 minimal v2:  base code

Hi Stephane,

On Tue, 17 Jun 2008 15:02:10 -0700 (PDT) eranian@...glemail.com wrote:
>
> +++ o/perfmon/Makefile	2008-06-17 08:33:23.000000000 +0200
> @@ -0,0 +1,7 @@
> +#
> +# Copyright (c) 2005-2006 Hewlett-Packard Development Company, L.P.
> +# Contributed by Stephane Eranian <eranian@...il.com>
> +#
> +obj-$(CONFIG_PERFMON) = perfmon_ctx.o \
> +			perfmon_file.o perfmon_attach.o 	\
> +			perfmon_res.o perfmon_init.o

> +++ o/Makefile	2008-06-17 08:34:19.000000000 +0200
> @@ -607,7 +607,7 @@
>  
>  
>  ifeq ($(KBUILD_EXTMOD),)
> -core-y		+= kernel/ mm/ fs/ ipc/ security/ crypto/ block/
> +core-y		+= kernel/ mm/ fs/ ipc/ security/ crypto/ block/ perfmon/

Is there some reason that this is not

core-$(CONFIG_PERFMON)	+= perfmon/

and then you could just use "obj-y" in perfmon/Makefile

Or are there going to be things in perfmon/ that need building if
CONFIG_PERFMON is not set?

-- 
Cheers,
Stephen Rothwell                    sfr@...b.auug.org.au
http://www.canb.auug.org.au/~sfr/

Content of type "application/pgp-signature" skipped