[PATCH 00/14] Removing dead code

[PATCH 00/14] Removing dead code

[Christian Dietrich]

Hi all!
       
As part of the VAMOS[0] research project at the University of Erlangen
we are looking at multiple integrity errors in linux' configuration
system.

I've been running a check on the whole sourcetree for
code block which are undead. This means they can't be deselected and
are always in the kernel if the enviroment of the block is
selected. They have often the form of

#ifdef ABC
....
#ifdef ABC
fooooo
#endif
#endif

Here the ifdefs of the inner block can be removed without any harm. At
this point we just checked the source code (for the undead), but we
want to do a crosscheck with the Kconfig for undead code blocks in the
future.

I build the patches against a vanilla kernel (v2.6.36-rc3) in order to
try if the kernel compiles with this patches.

Please keep me informed of this patch getting confirmed / merged so we
can keep track of it.

Regards

        Christian Dietrich

[0] http://vamos1.informatik.uni-erlangen.de/

Christian Dietrich (14):
  arch/arm: Removing undead ifdef __ASSEMBLY__
  arch/h8300: Removing dead ifdef __H8300_TLB_H__
  arch/parisc: Removing undead ifdef CONFIG_PA20
  arch/{s390,powerpc}: Removing undead ifdef __KERNEL__
  arch/x86: Removing undead ifdef ACPI/X86_IO_ACPI
  drivers/net: Removing undead ifdef CHELSIO_T1_1G
  drivers/scsi: Removing undead ifdef __ISAPNP__
  drivers/scsi: Removing undead ifdef CONFIG_PCI
  drivers/scsi: Removing undead ifdef REAL_DMA
  drivers/video: Removing undead ifdef ATAFB_FALCON
  drivers/video: Removing undead ifdef CONFIG_FB_MATROX_G
  include/linux: Removing undead ifdef __KERNEL__
  kernel/: Removing undead ifdef CONFIG_SMP
  kernel/: Removing undead ifdef CONFIG_DEBUG_LOCK_ALLOC

 arch/arm/mach-ixp23xx/include/mach/platform.h |    3 ---
 arch/h8300/include/asm/tlb.h                  |   13 -------------
 arch/parisc/kernel/unaligned.c                |    3 ---
 arch/powerpc/include/asm/processor.h          |    2 --
 arch/powerpc/include/asm/vdso_datapage.h      |    2 --
 arch/s390/include/asm/processor.h             |    4 ----
 arch/x86/kernel/early-quirks.c                |    2 --
 drivers/net/chelsio/subr.c                    |    2 --
 drivers/scsi/aha152x.c                        |    2 --
 drivers/scsi/aic7xxx_old.c                    |    2 --
 drivers/scsi/atari_NCR5380.c                  |    6 ------
 drivers/video/atafb.c                         |    2 --
 drivers/video/matrox/matroxfb_DAC1064.c       |    5 +----
 include/linux/socket.h                        |    6 +-----
 kernel/sched.c                                |    4 +---
 kernel/srcu.c                                 |    2 --
 16 files changed, 3 insertions(+), 57 deletions(-)

[PATCH 01/14] arch/arm: Removing undead ifdef __ASSEMBLY__

[Christian Dietrich]

The __ASSEMBLY__ ifdef isn't necessary at this point, because it is
checked in an outer ifdef level already and has no effect here.

Signed-off-by: Christian Dietrich <qy03fugy@stud.informatik.uni-erlangen.de>
---
 arch/arm/mach-ixp23xx/include/mach/platform.h |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/arch/arm/mach-ixp23xx/include/mach/platform.h b/arch/arm/mach-ixp23xx/include/mach/platform.h
index db9d941..9f05ea4 100644
--- a/arch/arm/mach-ixp23xx/include/mach/platform.h
+++ b/arch/arm/mach-ixp23xx/include/mach/platform.h
@@ -43,7 +43,6 @@ extern struct sys_timer ixp23xx_timer;
 
 #define IXP23XX_UART_XTAL		14745600
 
-#ifndef __ASSEMBLY__
 /*
  * Is system memory on the XSI or CPP bus?
  */
@@ -51,7 +50,5 @@ static inline unsigned ixp23xx_cpp_boot(void)
 {
 	return (*IXP23XX_EXP_CFG0 & IXP23XX_EXP_CFG0_XSI_NOT_PRES);
 }
-#endif
-
 
 #endif
-- 
1.7.0.4