Date: Wed, 16 Jul 2008 08:57:03 -0600
From: Matthew Wilcox <matthew@wil.cx>
To: Ingo Molnar <mingo@elte.hu>
Cc: James Bottomley <James.Bottomley@HansenPartnership.com>,
       Andrew Morton <akpm@linux-foundation.org>,
       Linus Torvalds <torvalds@linux-foundation.org>,
       linux-scsi <linux-scsi@vger.kernel.org>,
       linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: [build fix] Re: [GIT PATCH] SCSI part 1
Message-ID: <20080716145703.GF25255@parisc-linux.org>
References: <1216138543.3312.60.camel@localhost.localdomain> <20080716101634.GA8494@elte.hu> <20080716103337.GA22931@elte.hu> <20080716131543.GA3673@elte.hu> <1216216320.3230.1.camel@localhost.localdomain> <20080716141805.GB22631@elte.hu> <20080716144106.GD25255@parisc-linux.org> <20080716144602.GB4369@elte.hu>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20080716144602.GB4369@elte.hu>
User-Agent: Mutt/1.5.13 (2006-08-11)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1536
Lines: 31

On Wed, Jul 16, 2008 at 04:46:02PM +0200, Ingo Molnar wrote:
> yeah, that's OK - but why is scsi.h #ifdef-ed? For example we can 
> include blkdev.h without #ifdef CONFIG_BLOCK.

Sure.  The important bit comes when deciding which bits need to be
available when CONFIG_BLOCK isn't set.  Obviously, there is no SCSI
without block (while sg is a character device, not a block device,
it depends on the block infrastructure to the extent that you couldn't
use it without CONFIG_BLOCK).

While it's not impossible that someone could want the SCSI protocol
and opcodes, status codes, etc. for a different character device, that
seems unlikely.  The four opcodes defined in the header seem to be
pretty SCSI-specific and not useful to use on non-scsi devices.
SCSI_IOCTL_GET_PCI looks like it might possibly be useful on more than
just SCSI, but we have better ways (ie sysfs) of determining the same
information in a more general way.

So I think it's fair to put an #ifdef CONFIG_BLOCK right after the
_SCSI_SCSI_H define and close it right at the end of the file.

-- 
Intel are signing my paycheques ... these opinions are still mine
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours.  We can't possibly take such
a retrograde step."
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/