* Rename all derived CONFIG variables @ 2001-03-12 7:03 Keith Owens 2001-03-12 7:25 ` Jeff Garzik ` (3 more replies) 0 siblings, 4 replies; 10+ messages in thread From: Keith Owens @ 2001-03-12 7:03 UTC (permalink / raw) To: linux-kernel; +Cc: elenstev, kbuild-devel In 2.4.2-ac18 there are 130 CONFIG options that are always derived from other options, the user has no control over them. It is useful for the kernel build process to know which variables are derived and which variables the user can control. There are also 6 CONFIG options that are not used anywhere. ftp://ftp.ocs.com.au/pub/2.4.2-ac18-config_derived.gz is a 583,904 byte (unzipped) 114,291 (gzipped) patch which removes the unused variables and renames the 130 derived variables from CONFIG_FOO to CONFIG_FOO_DERIVED. The affected variables are :- Removed. There were a couple more that could have been removed because nothing checks them, but they identify architecture types in .config and are useful for bug reporting. CONFIG_ABSTRACT_CONSOLE CONFIG_ACPI_INTERPRETER CONFIG_CACHE_LINE_SHIFT CONFIG_FB_CONSOLE CONFIG_PCMCIA_SCSICARD CONFIG_PERCPU_IRQ Renamed to _DERIVED. CONFIG_ALPHA_APECS CONFIG_ALPHA_BROKEN_IRQ_MASK CONFIG_ALPHA_CIA CONFIG_ALPHA_EISA CONFIG_ALPHA_EV4 CONFIG_ALPHA_EV5 CONFIG_ALPHA_EV6 CONFIG_ALPHA_EV6 CONFIG_ALPHA_IRONGATE CONFIG_ALPHA_LCA CONFIG_ALPHA_MCPCIA CONFIG_ALPHA_POLARIS CONFIG_ALPHA_PYXIS CONFIG_ALPHA_T2 CONFIG_ALPHA_TSUNAMI CONFIG_ARC32 CONFIG_ARC64 CONFIG_ARCH_ACORN CONFIG_ARCH_EBSA285 CONFIG_ARCH_S390 CONFIG_ARCH_S390X CONFIG_ARC_MEMORY CONFIG_ARM CONFIG_ATM_FORE200E CONFIG_BLK_DEV_COMMERIAL CONFIG_BLK_DEV_HD CONFIG_BLK_DEV_IDEDISK_VENDOR CONFIG_BLK_DEV_IDEDMA CONFIG_BLK_DEV_IDE_MODES CONFIG_BOARD_SCACHE CONFIG_BOOT_ELF32 CONFIG_BOOT_ELF64 CONFIG_BUS_I2C CONFIG_COBALT_27 CONFIG_COBALT_LCD CONFIG_COHERENT_IO CONFIG_CPU_26 CONFIG_CPU_32 CONFIG_CPU_32v3 CONFIG_CPU_32v4 CONFIG_CPU_SH3 CONFIG_CPU_SH4 CONFIG_CRIS_LOW_MAP CONFIG_DMASOUND CONFIG_DMA_NONPCI CONFIG_DMA_NONPCI_DERIVED CONFIG_DUMMY_CONSOLE CONFIG_FBCON_STI CONFIG_FB_APOLLO CONFIG_FB_G364 CONFIG_FB_HP300 CONFIG_FB_MAC CONFIG_FB_Q40 CONFIG_FOOTBRIDGE CONFIG_FOOTBRIDGE_ADDIN CONFIG_FOOTBRIDGE_HOST CONFIG_FUSION_BOOT CONFIG_HAVE_DEC_LOCK CONFIG_IA64 CONFIG_IA64_BRL_EMU CONFIG_IDEDMA_AUTO CONFIG_IP_NF_NAT_FTP CONFIG_IP_NF_NAT_NEEDED CONFIG_ISA CONFIG_ISA_DMA CONFIG_ISDN_CAPI_CAPIFS CONFIG_ITANIUM CONFIG_KBDMOUSE CONFIG_LOCKD CONFIG_LOCKD_V4 CONFIG_M68K_L2_CACHE CONFIG_MACH_SPECIFIC CONFIG_MAC_HID CONFIG_MIPS_JAZZ CONFIG_MSNDCLAS_HAVE_BOOT CONFIG_MSNDPIN_HAVE_BOOT CONFIG_MTD_AMDSTD CONFIG_MTD_DOCPROBE CONFIG_MTD_JEDEC CONFIG_NET_CLS_ROUTE CONFIG_NLS CONFIG_NUBUS CONFIG_PARIDE_PARPORT CONFIG_PCI_BIOS CONFIG_PCI_CONSOLE CONFIG_PCI_DIRECT CONFIG_PCMCIA_CHRDEV CONFIG_PCMCIA_NETCARD CONFIG_PC_KEYB CONFIG_PC_KEYMAP CONFIG_PPC CONFIG_PPC64BRIDGE CONFIG_QL_ISP_A64 CONFIG_RPCMOUSE CONFIG_SA1111 CONFIG_SBUS CONFIG_SBUSCHAR CONFIG_SGI CONFIG_SH_HP600 CONFIG_SH_RTC CONFIG_SMB_NLS CONFIG_SPARC32 CONFIG_SPARC64 CONFIG_SUNRPC CONFIG_SUN_AUXIO CONFIG_SUN_CONSOLE CONFIG_SUN_IO CONFIG_SUN_SERIAL CONFIG_SUPERH CONFIG_TQM8xxL CONFIG_UID16 CONFIG_X86 CONFIG_X86_ALIGNMENT_16 CONFIG_X86_BSWAP CONFIG_X86_CMPXCHG CONFIG_X86_GOOD_APIC CONFIG_X86_INVLPG CONFIG_X86_IO_APIC CONFIG_X86_L1_CACHE_SHIFT CONFIG_X86_LOCAL_APIC CONFIG_X86_PAE CONFIG_X86_PGE CONFIG_X86_POPAD_OK CONFIG_X86_TSC CONFIG_X86_USE_3DNOW CONFIG_X86_USE_PPRO_CHECKSUM CONFIG_X86_USE_STRING_486 CONFIG_X86_VISWS_APIC CONFIG_X86_WP_WORKS_OK CONFIG_ZFT_COMPRESSOR The patch hits these files. Documentation/Configure.help Documentation/kbuild/config-language.txt Documentation/kbuild/makefiles.txt Makefile arch/alpha/Makefile arch/alpha/config.in arch/alpha/defconfig arch/alpha/kernel/Makefile arch/alpha/kernel/alpha_ksyms.c arch/alpha/kernel/irq_alpha.c arch/alpha/kernel/irq_i8259.c arch/alpha/kernel/process.c arch/alpha/kernel/setup.c arch/alpha/lib/Makefile arch/arm/Makefile arch/arm/boot/Makefile arch/arm/boot/compressed/Makefile arch/arm/config.in arch/arm/def-configs/a5k arch/arm/def-configs/assabet arch/arm/def-configs/brutus arch/arm/def-configs/cerf arch/arm/def-configs/clps7500 arch/arm/def-configs/ebsa110 arch/arm/def-configs/empeg arch/arm/def-configs/footbridge arch/arm/def-configs/graphicsclient arch/arm/def-configs/integrator arch/arm/def-configs/lart arch/arm/def-configs/lusl7200 arch/arm/def-configs/neponset arch/arm/def-configs/pangolin arch/arm/def-configs/rpc arch/arm/def-configs/shark arch/arm/def-configs/sherman arch/arm/def-configs/victor arch/arm/defconfig arch/arm/kernel/Makefile arch/arm/kernel/arch.c arch/arm/kernel/armksyms.c arch/arm/kernel/bios32.c arch/arm/kernel/debug-armv.S arch/arm/kernel/dma-footbridge.c arch/arm/kernel/ecard.c arch/arm/kernel/entry-armv.S arch/arm/kernel/fiq.c arch/arm/kernel/irq.c arch/arm/kernel/process.c arch/arm/kernel/semaphore.c arch/arm/kernel/setup.c arch/arm/kernel/signal.c arch/arm/kernel/traps.c arch/arm/lib/Makefile arch/arm/lib/backtrace.S arch/arm/lib/csumpartialcopyuser.S arch/arm/lib/ecard.S arch/arm/lib/io-acorn.S arch/arm/mach-footbridge/Makefile arch/arm/mach-footbridge/arch.c arch/arm/mach-sa1100/hw.c arch/arm/mach-shark/dma.c arch/arm/mm/Makefile arch/arm/mm/fault-common.c arch/arm/mm/init.c arch/arm/mm/mm-footbridge.c arch/arm/nwfpe/Makefile arch/arm/nwfpe/fpmodule.h arch/arm/tools/getconstants.c arch/cris/config.in arch/cris/defconfig arch/cris/drivers/Config.in arch/cris/drivers/axisflashmap.c arch/cris/drivers/ide.c arch/cris/kernel/head.S arch/cris/mm/init.c arch/i386/config.in arch/i386/defconfig arch/i386/kernel/Makefile arch/i386/kernel/apic.c arch/i386/kernel/entry.S arch/i386/kernel/head.S arch/i386/kernel/i386_ksyms.c arch/i386/kernel/i8259.c arch/i386/kernel/irq.c arch/i386/kernel/mpparse.c arch/i386/kernel/pci-irq.c arch/i386/kernel/pci-pc.c arch/i386/kernel/setup.c arch/i386/kernel/time.c arch/i386/kernel/traps.c arch/i386/lib/Makefile arch/i386/lib/checksum.S arch/i386/lib/memcpy.c arch/i386/mm/init.c arch/ia64/config.in arch/ia64/defconfig arch/ia64/kernel/Makefile arch/ia64/kernel/head.S arch/ia64/kernel/irq.c arch/ia64/kernel/ivt.S arch/ia64/kernel/setup.c arch/ia64/kernel/traps.c arch/m68k/amiga/config.c arch/m68k/apollo/config.c arch/m68k/atari/config.c arch/m68k/config.in arch/m68k/defconfig arch/m68k/hp300/config.c arch/m68k/kernel/setup.c arch/m68k/mm/memory.c arch/mips/Makefile arch/mips/config.in arch/mips/defconfig arch/mips/defconfig-cobalt arch/mips/defconfig-decstation arch/mips/defconfig-ip22 arch/mips/defconfig-orion arch/mips/defconfig-rm200 arch/mips/kernel/mips_ksyms.c arch/mips/kernel/setup.c arch/mips/lib/Makefile arch/mips/mm/init.c arch/mips64/Makefile arch/mips64/arc/Makefile arch/mips64/arc/memory.c arch/mips64/config.in arch/mips64/defconfig arch/mips64/defconfig-ip22 arch/mips64/defconfig-ip27 arch/mips64/kernel/head.S arch/mips64/kernel/mips64_ksyms.c arch/parisc/config.common arch/parisc/defconfig arch/parisc/kernel/ccio-dma.c arch/parisc/kernel/parisc_ksyms.c arch/parisc/kernel/setup.c arch/ppc/Makefile arch/ppc/boot/Makefile arch/ppc/chrpboot/Makefile arch/ppc/coffboot/Makefile arch/ppc/config.in arch/ppc/configs/IVMS8_defconfig arch/ppc/configs/SM850_defconfig arch/ppc/configs/SPD823TS_defconfig arch/ppc/configs/TQM823L_defconfig arch/ppc/configs/TQM850L_defconfig arch/ppc/configs/TQM860L_defconfig arch/ppc/configs/apus_defconfig arch/ppc/configs/bseip_defconfig arch/ppc/configs/common_defconfig arch/ppc/configs/est8260_defconfig arch/ppc/configs/ibmchrp_defconfig arch/ppc/configs/mbx_defconfig arch/ppc/configs/oak_defconfig arch/ppc/configs/power3_defconfig arch/ppc/configs/rpxcllf_defconfig arch/ppc/configs/rpxlite_defconfig arch/ppc/configs/walnut_defconfig arch/ppc/defconfig arch/ppc/kernel/Makefile arch/ppc/kernel/chrp_pci.c arch/ppc/kernel/chrp_setup.c arch/ppc/kernel/hashtable.S arch/ppc/kernel/head.S arch/ppc/kernel/misc.S arch/ppc/kernel/pmac_setup.c arch/ppc/kernel/ppc_asm.h arch/ppc/kernel/ppc_ksyms.c arch/ppc/kernel/prom.c arch/ppc/kernel/setup.c arch/ppc/lib/string.S arch/ppc/mm/init.c arch/ppc/xmon/xmon.c arch/s390/config.in arch/s390/defconfig arch/s390/kernel/debug.c arch/s390x/config.in arch/s390x/defconfig arch/s390x/kernel/debug.c arch/sh/Makefile arch/sh/config.in arch/sh/defconfig arch/sh/kernel/Makefile arch/sh/kernel/setup.c arch/sparc/config.in arch/sparc/defconfig arch/sparc/kernel/Makefile arch/sparc/kernel/devices.c arch/sparc/kernel/ebus.c arch/sparc/kernel/ioport.c arch/sparc/kernel/process.c arch/sparc/kernel/setup.c arch/sparc/kernel/sparc_ksyms.c arch/sparc/mm/io-unit.c arch/sparc/mm/iommu.c arch/sparc/mm/srmmu.c arch/sparc/mm/sun4c.c arch/sparc/prom/misc.c arch/sparc64/config.in arch/sparc64/defconfig arch/sparc64/kernel/ebus.c arch/sparc64/kernel/head.S arch/sparc64/kernel/process.c arch/sparc64/kernel/setup.c arch/sparc64/kernel/sparc64_ksyms.c arch/sparc64/mm/init.c arch/sparc64/prom/misc.c drivers/Makefile drivers/acorn/char/Makefile drivers/acorn/char/keyb_arc.c drivers/atm/Config.in drivers/atm/Makefile drivers/atm/atmdev_init.c drivers/block/Config.in drivers/block/Makefile drivers/block/genhd.c drivers/block/ll_rw_blk.c drivers/block/paride/Config.in drivers/block/rd.c drivers/char/Config.in drivers/char/Makefile drivers/char/cyclades.c drivers/char/ftape/Config.in drivers/char/ftape/Makefile drivers/char/ftape/zftape/zftape-init.c drivers/char/joystick/analog.c drivers/char/misc.c drivers/char/pcmcia/Config.in drivers/ide/Config.in drivers/ide/Makefile drivers/ide/aec62xx.c drivers/ide/alim15x3.c drivers/ide/amd7409.c drivers/ide/cmd64x.c drivers/ide/cs5530.c drivers/ide/cy82c693.c drivers/ide/hpt34x.c drivers/ide/hpt366.c drivers/ide/ide-cd.c drivers/ide/ide-disk.c drivers/ide/ide-features.c drivers/ide/ide-floppy.c drivers/ide/ide-pci.c drivers/ide/ide-probe.c drivers/ide/ide-tape.c drivers/ide/ide.c drivers/ide/ide_modes.h drivers/ide/ns87415.c drivers/ide/osb4.c drivers/ide/pdc202xx.c drivers/ide/piix.c drivers/ide/sis5513.c drivers/ide/slc90e66.c drivers/ide/trm290.c drivers/ide/via82cxxx.c drivers/input/keybdev.c drivers/isdn/Config.in drivers/isdn/avmb1/Makefile drivers/isdn/avmb1/capi.c drivers/macintosh/Makefile drivers/macintosh/adb.c drivers/macintosh/mac_hid.c drivers/macintosh/mac_keyb.c drivers/macintosh/via-cuda.c drivers/media/video/Config.in drivers/media/video/Makefile drivers/message/fusion/Config.in drivers/mtd/Config.in drivers/mtd/Makefile drivers/net/Config.in drivers/net/Makefile drivers/net/acenic.c drivers/net/acenic.h drivers/net/arcnet/Config.in drivers/net/de4x5.c drivers/net/pcmcia/Config.in drivers/net/sunhme.c drivers/net/sunhme.h drivers/net/tokenring/Config.in drivers/net/wan/Config.in drivers/parport/Config.in drivers/pci/Makefile drivers/pcmcia/cs.c drivers/pcmcia/i82365.c drivers/pcmcia/rsrc_mgr.c drivers/s390/Config.in drivers/s390/block/xpram.c drivers/s390/idals.c drivers/s390/misc/chandev.c drivers/s390/s390io.c drivers/sbus/Makefile drivers/sbus/audio/Config.in drivers/sbus/sbus.c drivers/scsi/Config.in drivers/scsi/Makefile drivers/scsi/pcmcia/Config.in drivers/scsi/qlogicisp.c drivers/scsi/scsi.h drivers/scsi/sd.c drivers/sound/Config.in drivers/sound/Makefile drivers/sound/dmasound/Config.in drivers/sound/dmasound/Makefile drivers/sound/msnd_pinnacle.c drivers/video/Config.in drivers/video/Makefile drivers/video/aty/atyfb_base.c drivers/video/aty128fb.c drivers/video/cyber2000fb.c drivers/video/fbmem.c drivers/video/font_acorn_8x8.c drivers/video/imsttfb.c drivers/video/matrox/matroxfb_base.c drivers/video/matrox/matroxfb_base.h drivers/video/promcon.c drivers/video/vgacon.c fs/Config.in fs/Makefile fs/lockd/Makefile fs/lockd/svc.c fs/lockd/svclock.c fs/lockd/svcproc.c fs/lockd/svcsubs.c fs/lockd/xdr.c fs/ncpfs/ioctl.c fs/nls/Config.in fs/partitions/Config.in fs/partitions/mac.c fs/proc/array.c fs/proc/proc_misc.c fs/stat.c include/asm-alpha/cache.h include/asm-alpha/core_cia.h include/asm-alpha/io.h include/asm-alpha/mmu_context.h include/asm-alpha/pgalloc.h include/asm-alpha/pgtable.h include/asm-alpha/system.h include/asm-arm/arch-ebsa285/memory.h include/asm-arm/arch-sa1100/hardware.h include/asm-arm/arch-sa1100/irq.h include/asm-arm/arch-sa1100/irqs.h include/asm-arm/hardware/dec21285.h include/asm-arm/proc-fns.h include/asm-cris/page.h include/asm-cris/pgtable.h include/asm-cris/processor.h include/asm-i386/apic.h include/asm-i386/bugs.h include/asm-i386/byteorder.h include/asm-i386/cache.h include/asm-i386/cobalt.h include/asm-i386/fixmap.h include/asm-i386/highmem.h include/asm-i386/io_apic.h include/asm-i386/irq.h include/asm-i386/lithium.h include/asm-i386/page.h include/asm-i386/pgalloc.h include/asm-i386/pgtable.h include/asm-i386/processor.h include/asm-i386/smp.h include/asm-i386/string-486.h include/asm-i386/string.h include/asm-i386/system.h include/asm-i386/timex.h include/asm-i386/uaccess.h include/asm-ia64/delay.h include/asm-mips/arc/types.h include/asm-mips/pci.h include/asm-mips/serial.h include/asm-mips64/arc/types.h include/asm-mips64/bcache.h include/asm-mips64/io.h include/asm-mips64/pci.h include/asm-mips64/sgiarcs.h include/asm-parisc/io.h include/asm-ppc/cache.h include/asm-ppc/mmu.h include/asm-ppc/mpc8xx.h include/asm-ppc/processor.h include/asm-ppc/prom.h include/asm-ppc/tqm8xx.h include/asm-s390/idals.h include/asm-s390/irq.h include/asm-s390x/idals.h include/asm-s390x/irq.h include/asm-sh/io.h include/asm-sh/machvec.h include/linux/blk.h include/linux/highuid.h include/linux/ide.h include/linux/kernel_stat.h include/linux/linkage.h include/linux/lockd/lockd.h include/linux/lockd/nlm.h include/linux/lvm.h include/linux/netfilter_ipv4/ip_conntrack.h include/linux/types.h include/net/dst.h include/net/ip_fib.h include/net/pkt_sched.h include/net/profile.h init/main.c kernel/Makefile kernel/ksyms.c kernel/panic.c kernel/sched.c kernel/softirq.c kernel/sysctl.c lib/Makefile net/Makefile net/ipv4/fib_rules.c net/ipv4/fib_semantics.c net/ipv4/ip_input.c net/ipv4/netfilter/Config.in net/ipv4/netfilter/Makefile net/ipv4/route.c net/ipv6/route.c net/sched/Config.in ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Rename all derived CONFIG variables 2001-03-12 7:03 Rename all derived CONFIG variables Keith Owens @ 2001-03-12 7:25 ` Jeff Garzik 2001-03-12 14:24 ` Alan Cox 2001-03-12 8:26 ` Peter Samuelson ` (2 subsequent siblings) 3 siblings, 1 reply; 10+ messages in thread From: Jeff Garzik @ 2001-03-12 7:25 UTC (permalink / raw) To: Keith Owens; +Cc: linux-kernel, elenstev, kbuild-devel, Alan Cox Keith Owens wrote: > > In 2.4.2-ac18 there are 130 CONFIG options that are always derived from > other options, the user has no control over them. It is useful for the > kernel build process to know which variables are derived and which > variables the user can control. There are also 6 CONFIG options that > are not used anywhere. > > ftp://ftp.ocs.com.au/pub/2.4.2-ac18-config_derived.gz > > is a 583,904 byte (unzipped) 114,291 (gzipped) patch which removes the > unused variables and renames the 130 derived variables from CONFIG_FOO > to CONFIG_FOO_DERIVED. The affected variables are :- Not only do I think that CONFIG_xxx_DERIVED needlessly extends the name of derived vars, but your patch does not belong in a stable series. Derived CONFIG_xxx vars are likely to be referenced in source. Changing those vars in the middle of a stable series pointlessly breaks external source code. I hope vendors don't start applying this patch... Jeff -- Jeff Garzik | "You see, in this world there's two kinds of Building 1024 | people, my friend: Those with loaded guns MandrakeSoft | and those who dig. You dig." --Blondie ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Rename all derived CONFIG variables 2001-03-12 7:25 ` Jeff Garzik @ 2001-03-12 14:24 ` Alan Cox 0 siblings, 0 replies; 10+ messages in thread From: Alan Cox @ 2001-03-12 14:24 UTC (permalink / raw) To: Jeff Garzik; +Cc: Keith Owens, linux-kernel, elenstev, kbuild-devel, Alan Cox > Not only do I think that CONFIG_xxx_DERIVED needlessly extends the name > of derived vars, but your patch does not belong in a stable series. > Derived CONFIG_xxx vars are likely to be referenced in source. Changing > those vars in the middle of a stable series pointlessly breaks external > source code. > > I hope vendors don't start applying this patch... I dont see them doing that. I'm not going to be applying it for -ac either. It would appear that this is a wrong way to fix the problem. The config option checker should be scanning for define_*** options and removing them from its output ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Rename all derived CONFIG variables 2001-03-12 7:03 Rename all derived CONFIG variables Keith Owens 2001-03-12 7:25 ` Jeff Garzik @ 2001-03-12 8:26 ` Peter Samuelson 2001-03-12 8:53 ` [kbuild-devel] " Eric S. Raymond 2001-03-12 11:37 ` Philipp Rumpf 2001-03-12 12:12 ` Hugh Dickins 3 siblings, 1 reply; 10+ messages in thread From: Peter Samuelson @ 2001-03-12 8:26 UTC (permalink / raw) To: Keith Owens; +Cc: linux-kernel, elenstev, kbuild-devel [kaos] > In 2.4.2-ac18 there are 130 CONFIG options that are always derived > from other options, the user has no control over them. I've thought about these before ... but never got around to doing anything about them. I agree they should have a separate namespace. However, I would vote the for namespace CONFIG__* rather than CONFIG_*_DERIVED. As Jeff noted, _DERIVED is quite a mouthful to type, and CONFIG__* seems all-around less intrusive. Peter ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [kbuild-devel] Re: Rename all derived CONFIG variables 2001-03-12 8:26 ` Peter Samuelson @ 2001-03-12 8:53 ` Eric S. Raymond 2001-03-12 9:18 ` Keith Owens 0 siblings, 1 reply; 10+ messages in thread From: Eric S. Raymond @ 2001-03-12 8:53 UTC (permalink / raw) To: Peter Samuelson; +Cc: Keith Owens, linux-kernel, elenstev, kbuild-devel Peter Samuelson <peter@cadcamlab.org>: > > In 2.4.2-ac18 there are 130 CONFIG options that are always derived > > from other options, the user has no control over them. > > I've thought about these before ... but never got around to doing > anything about them. I agree they should have a separate namespace. > > However, I would vote the for namespace CONFIG__* rather than > CONFIG_*_DERIVED. As Jeff noted, _DERIVED is quite a mouthful to type, > and CONFIG__* seems all-around less intrusive. How much point is there to this kind of cleanup in CML1, really? After the fall LinuxWorld meeting I was under the impression that mec and the rest of the build team were planning to support switching to CML2 for the 2.5 series. If that's not true, someone should clue me in *now*, before I go misrepresenting anybody's position at the 2.5 kickoff workshop. But if we're going to push Linus and the kernel crew to switch to CML2, then why invite the political tsuris of trying to get a large patch into 2.4 now? Maybe I'm missing something here, but this doesn't seem necessary to me. -- <a href="http://www.tuxedo.org/~esr/">Eric S. Raymond</a> The IRS has become morally corrupted by the enormous power which we in Congress have unwisely entrusted to it. Too often it acts like a Gestapo preying upon defenseless citizens. -- Senator Edward V. Long ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [kbuild-devel] Re: Rename all derived CONFIG variables 2001-03-12 8:53 ` [kbuild-devel] " Eric S. Raymond @ 2001-03-12 9:18 ` Keith Owens 2001-03-12 14:28 ` Alan Cox 2001-03-14 21:34 ` Oliver Xymoron 0 siblings, 2 replies; 10+ messages in thread From: Keith Owens @ 2001-03-12 9:18 UTC (permalink / raw) To: esr; +Cc: Peter Samuelson, linux-kernel, elenstev, kbuild-devel On Mon, 12 Mar 2001 03:53:07 -0500, "Eric S. Raymond" <esr@thyrsus.com> wrote: >But if we're going to push Linus and the kernel crew to switch to >CML2, then why invite the political tsuris of trying to get a large >patch into 2.4 now? Maybe I'm missing something here, but this doesn't >seem necessary to me. The derived config variables should be in a separate name space, whether config is CML1 or CML2. This patch does it for CML1. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [kbuild-devel] Re: Rename all derived CONFIG variables 2001-03-12 9:18 ` Keith Owens @ 2001-03-12 14:28 ` Alan Cox 2001-03-14 21:34 ` Oliver Xymoron 1 sibling, 0 replies; 10+ messages in thread From: Alan Cox @ 2001-03-12 14:28 UTC (permalink / raw) To: Keith Owens; +Cc: esr, Peter Samuelson, linux-kernel, elenstev, kbuild-devel > The derived config variables should be in a separate name space, > whether config is CML1 or CML2. This patch does it for CML1. Why ? The only tool that needs to seperate them is the config check script and that has enough information to deduce them ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [kbuild-devel] Re: Rename all derived CONFIG variables 2001-03-12 9:18 ` Keith Owens 2001-03-12 14:28 ` Alan Cox @ 2001-03-14 21:34 ` Oliver Xymoron 1 sibling, 0 replies; 10+ messages in thread From: Oliver Xymoron @ 2001-03-14 21:34 UTC (permalink / raw) To: Keith Owens; +Cc: esr, Peter Samuelson, linux-kernel, elenstev, kbuild-devel On Mon, 12 Mar 2001, Keith Owens wrote: > On Mon, 12 Mar 2001 03:53:07 -0500, > "Eric S. Raymond" <esr@thyrsus.com> wrote: > >But if we're going to push Linus and the kernel crew to switch to > >CML2, then why invite the political tsuris of trying to get a large > >patch into 2.4 now? Maybe I'm missing something here, but this doesn't > >seem necessary to me. > > The derived config variables should be in a separate name space, > whether config is CML1 or CML2. This patch does it for CML1. I don't think this makes sense at all. The derivation of the config values is the concern of the configuration system, not the code. Consider something like CONFIG_CPU_HAS_FEATURE_FOO that might currently be derived from CONFIG_CPU_BAR but may in the future be made independent. Or vice-versa. Your proposed name-change means additional maintenance headache and gets you nothing that you couldn't get by simply including whatever script you wrote to deduce the dependencies. Such a script would at least be able to tell you what a variable was derived from. -- "Love the dolphins," she advised him. "Write by W.A.S.T.E.." ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Rename all derived CONFIG variables 2001-03-12 7:03 Rename all derived CONFIG variables Keith Owens 2001-03-12 7:25 ` Jeff Garzik 2001-03-12 8:26 ` Peter Samuelson @ 2001-03-12 11:37 ` Philipp Rumpf 2001-03-12 12:12 ` Hugh Dickins 3 siblings, 0 replies; 10+ messages in thread From: Philipp Rumpf @ 2001-03-12 11:37 UTC (permalink / raw) To: Keith Owens; +Cc: linux-kernel, elenstev, kbuild-devel On Mon, Mar 12, 2001 at 06:03:22PM +1100, Keith Owens wrote: > In 2.4.2-ac18 there are 130 CONFIG options that are always derived from > other options, the user has no control over them. It is useful for the > kernel build process to know which variables are derived and which > variables the user can control. There are also 6 CONFIG options that Yes, it is. However, I don't see a reason to use a different namespace for them. At least some of the options below might become non-derived options again, and having that knowledge spread out over most of the kernel tree doesn't make any sense to me. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Rename all derived CONFIG variables 2001-03-12 7:03 Rename all derived CONFIG variables Keith Owens ` (2 preceding siblings ...) 2001-03-12 11:37 ` Philipp Rumpf @ 2001-03-12 12:12 ` Hugh Dickins 3 siblings, 0 replies; 10+ messages in thread From: Hugh Dickins @ 2001-03-12 12:12 UTC (permalink / raw) To: Keith Owens; +Cc: linux-kernel, elenstev, kbuild-devel On Mon, 12 Mar 2001, Keith Owens wrote: > In 2.4.2-ac18 there are 130 CONFIG options that are always derived from > other options, the user has no control over them. It is useful for the > kernel build process to know which variables are derived and which > variables the user can control. If it's useful for the (future?) kernel build process to know this, shouldn't that config/build process derive this info itself, rather than going by _DERIVED on the end of a CONFIG_ name? And aren't there many CONFIG_ options (I think your "always" implies so) which are derived in some cases (e.g. on some architectures), specified by the user in others? Those don't get to be called _DERIVED though often they are derived. And easy to imagine an option always derived in one release, sometimes specified in the next, always derived in the next... Yes, we can keep on editing the CONFIG_ name in all sources affected, but I just don't think the name should be trying to carry that info. Hugh ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2001-03-14 21:35 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2001-03-12 7:03 Rename all derived CONFIG variables Keith Owens 2001-03-12 7:25 ` Jeff Garzik 2001-03-12 14:24 ` Alan Cox 2001-03-12 8:26 ` Peter Samuelson 2001-03-12 8:53 ` [kbuild-devel] " Eric S. Raymond 2001-03-12 9:18 ` Keith Owens 2001-03-12 14:28 ` Alan Cox 2001-03-14 21:34 ` Oliver Xymoron 2001-03-12 11:37 ` Philipp Rumpf 2001-03-12 12:12 ` Hugh Dickins
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).