From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753370Ab0GUPTw (ORCPT ); Wed, 21 Jul 2010 11:19:52 -0400 Received: from cantor.suse.de ([195.135.220.2]:60360 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750941Ab0GUPTu (ORCPT ); Wed, 21 Jul 2010 11:19:50 -0400 Subject: Re: CONFIG_SCSI_G_NCR5380_MEM dead? From: James Bottomley To: Christian Dietrich Cc: linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, vamos-dev@i4.informatik.uni-erlangen.de In-Reply-To: <86pqyhhtut.fsf@peer.zerties.org> References: <86pqyhhtut.fsf@peer.zerties.org> Content-Type: text/plain; charset="UTF-8" Date: Wed, 21 Jul 2010 08:19:43 -0700 Message-ID: <1279725583.20250.29.camel@mulgrave.site> Mime-Version: 1.0 X-Mailer: Evolution 2.28.2 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2010-07-21 at 15:01 +0200, Christian Dietrich wrote: > 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/scsi sourcetree for > config Items not defined in Kconfig and found such a case. 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. > > I encountered, that the CONFIG_SCSI_G_NCR5380_MEM symbol in > drivers/scsi/g_NCR5380.c isn't defined anywhere. So this codeblocks > seems to be unreachable. But there is a define in g_NCR5380_mmio.c for > SCSI_G_NCR5380_MEM with an include of g_NCR5380.c. So i think this is a > typo-bug and it should be a CONFIG_ define. The code wasn't changed since > linux is in git, so probably no one has used the mmio module, cause > there can't be a difference to the normal pio module. If you think this > ifdefs should be removed, i can send a patch. Actually, I'd rather not remove any code like this. We have no documentation for the board, so that code represents the only knowledge we have about how to program it. Although, in theory, git will remember, in practice no-one will know where to look for it if they want to try resurrecting the MMIO code. James