mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Christian Dietrich <qy03fugy@stud.informatik.uni-erlangen.de>
To: "Mauro Carvalho Chehab" <mchehab@infradead.org>,
	"Patrick Boettcher" <pboettcher@kernellabs.com>,
	"Olivier Grenie" <Olivier.Grenie@dibcom.fr>,
	"Márton Németh" <nm127@freemail.hu>, "Tejun Heo" <tj@kernel.org>,
	"Muralidharan Karicheri" <mkaricheri@gmail.com>,
	"Vaibhav Hiremath" <hvaibhav@ti.com>,
	"Julia Lawall" <julia@diku.dk>,
	"Jonathan Corbet" <corbet@lwn.net>,
	linux-media@vger.kernel.org, linux-kernel@vger.kernel.org
Cc: vamos-dev@i4.informatik.uni-erlangen.de
Subject: [PATCH 0/4] Removing dead code
Date: Wed, 14 Jul 2010 15:21:06 +0200	[thread overview]
Message-ID: <cover.1279111369.git.qy03fugy@stud.informatik.uni-erlangen.de> (raw)

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 drivers/media sourcetree for
config Items not defined in Kconfig and found 4 such cases. Sourcecode
blocks depending on these Items are not reachable from a vanilla
kernel -- dead code. I've seen such dead blocks made on purpose
e.g. while integrating new features into the kernel but generally
they're just useless.

        There were also CONFIG_ options set within a source file. This
created code blocks which were always selected and compiled.

        Each of the patches in this patchset removes on such dead
config Item, I'd be glad if you consider applying them. I've been
doing deeper analysis of such issues before and can do so again but
I'm not so sure they were fastly usefull.

        I build the patches against a vanilla kernel 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 (4):
  drivers/media/dvb: Remove dead Configs
  drivers/media/dvb: Remove undead configs
  drivers/media/video: Remove dead CONFIG_FB_OMAP2_FORCE_AUTO_UPDATE
  drivers/media/video: Remove dead CONFIG_OLPC_X0_1

 drivers/media/dvb/frontends/dib0090.c |  126 ---------------------------------
 drivers/media/video/omap/omap_vout.c  |    8 --
 drivers/media/video/ov7670.c          |   37 ----------
 3 files changed, 0 insertions(+), 171 deletions(-)


             reply	other threads:[~2010-07-14 13:21 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-14 13:21 Christian Dietrich [this message]
2010-07-14 13:21 ` [PATCH 1/4] drivers/media/dvb: Remove dead Configs Christian Dietrich
2010-07-14 13:21 ` [PATCH 2/4] drivers/media/dvb: Remove undead configs Christian Dietrich
2010-07-14 13:21 ` [PATCH 3/4] drivers/media/video: Remove dead CONFIG_FB_OMAP2_FORCE_AUTO_UPDATE Christian Dietrich
2010-07-14 13:21 ` [PATCH 4/4] drivers/media/video: Remove dead CONFIG_OLPC_X0_1 Christian Dietrich
2010-07-19 22:39   ` Jonathan Corbet
2010-08-10  9:11 [PATCH 0/4] Removing dead code Christian Dietrich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1279111369.git.qy03fugy@stud.informatik.uni-erlangen.de \
    --to=qy03fugy@stud.informatik.uni-erlangen.de \
    --cc=Olivier.Grenie@dibcom.fr \
    --cc=corbet@lwn.net \
    --cc=hvaibhav@ti.com \
    --cc=julia@diku.dk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@infradead.org \
    --cc=mkaricheri@gmail.com \
    --cc=nm127@freemail.hu \
    --cc=pboettcher@kernellabs.com \
    --cc=tj@kernel.org \
    --cc=vamos-dev@i4.informatik.uni-erlangen.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®