public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "John Stoffel" <john@stoffel.org>
To: Greg KH <greg@kroah.com>
Cc: Christoph Hellwig <hch@infradead.org>,
	David Howells <dhowells@redhat.com>,
	linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org,
	zippel@linux-m68k.org
Subject: Re: [PATCH 17/17] BLOCK: Make it possible to disable the block layer [try #2]
Date: Tue, 29 Aug 2006 17:08:46 -0400	[thread overview]
Message-ID: <17652.44254.620358.974993@stoffel.org> (raw)
In-Reply-To: <20060829195845.GA13357@kroah.com>

>>>>> "Greg" == Greg KH <greg@kroah.com> writes:

Greg> On Tue, Aug 29, 2006 at 01:25:01PM +0100, Christoph Hellwig wrote:
>> On Tue, Aug 29, 2006 at 01:23:18PM +0100, David Howells wrote:
>> > Christoph Hellwig <hch@infradead.org> wrote:
>> > 
>> > > Same as above.  USB_STORAGE already selects scsi so it shouldn't need
>> > > to depend on block.
>> > 
>> > Ah, you've got it the wrong way round.
>> > 
>> > Because USB_STORAGE _selects_ SCSI rather than depending on it, even if SCSI
>> > is disabled, USB_STORAGE can be enabled, and that turns on CONFIG_SCSI, even
>> > if not all of its dependencies are available.
>> > 
>> > Run "make allyesconfig" and then try to turn off CONFIG_SCSI without this...
>> 
>> Eeek.  The easy fix is to change USB_STORAGE to depend on SCSI (*), but in
>> addition to that we should probably fix Kconfig aswell to adhere to
>> such constraints.

Greg> No, the reason this was switched around like this (it used to be the
Greg> other way), was that people constantly complained about not being able
Greg> to select the usb-storage driver in their configurations.

Maybe the better solution is to remove SCSI as an option, and to just
offer SCSI drivers and USB-STORAGE and other SCSI core using drivers
instead.  Then the SCSI core gets pulled in automatically.  It's not
like people care about the SCSI core, just the drivers which depend on
it.

John

  reply	other threads:[~2006-08-29 21:09 UTC|newest]

Thread overview: 75+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-24 21:32 [PATCH 01/17] BLOCK: Move functions out of buffer code [try #2] David Howells
2006-08-24 21:32 ` [PATCH 02/17] BLOCK: Remove duplicate declaration of exit_io_context() " David Howells
2006-08-25 14:08   ` Christoph Hellwig
2006-08-24 21:32 ` [PATCH 03/17] BLOCK: Stop fallback_migrate_page() from using page_has_buffers() " David Howells
2006-08-25 14:10   ` Christoph Hellwig
2006-08-25 16:11     ` David Howells
2006-08-24 21:33 ` [PATCH 04/17] BLOCK: Separate the bounce buffering code from the highmem code " David Howells
2006-08-25 14:12   ` Christoph Hellwig
2006-08-24 21:33 ` [PATCH 05/17] BLOCK: Don't call block_sync_page() from AFS " David Howells
2006-08-25 14:13   ` Christoph Hellwig
2006-08-25 16:12     ` David Howells
2006-08-24 21:33 ` [PATCH 06/17] BLOCK: Move bdev_cache_init() declaration to headerfile " David Howells
2006-08-25 14:13   ` Christoph Hellwig
2006-08-24 21:33 ` [PATCH 07/17] BLOCK: Remove dependence on existence of blockdev_superblock " David Howells
2006-08-25 14:14   ` Christoph Hellwig
2006-08-24 21:33 ` [PATCH 08/17] BLOCK: Dissociate generic_writepages() from mpage stuff " David Howells
2006-08-25 14:16   ` Christoph Hellwig
2006-08-24 21:33 ` [PATCH 09/17] BLOCK: Move __invalidate_device() to block_dev.c " David Howells
2006-08-25 14:16   ` Christoph Hellwig
2006-08-24 21:33 ` [PATCH 10/17] BLOCK: Move the loop device ioctl compat stuff to the loop driver " David Howells
     [not found]   ` <200608250023.13204.arnd@arndb.de>
2006-08-25  9:27     ` David Howells
     [not found]       ` <200608251217.24543.arnd@arndb.de>
2006-08-25 11:01         ` David Howells
2006-08-24 21:33 ` [PATCH 11/17] BLOCK: Move common FS-specific ioctls to linux/fs.h " David Howells
     [not found]   ` <200608250023.20903.arnd@arndb.de>
2006-08-25  8:38     ` David Howells
2006-08-25  8:44       ` Arnd Bergmann
2006-08-24 21:33 ` [PATCH 12/17] BLOCK: Move the ReiserFS device ioctl compat stuff to the ReiserFS driver " David Howells
2006-08-24 21:33 ` [PATCH 13/17] BLOCK: Move the Ext2 device ioctl compat stuff to the Ext2 " David Howells
2006-08-24 21:33 ` [PATCH 14/17] BLOCK: Move the Ext3 device ioctl compat stuff to the Ext3 " David Howells
2006-08-24 21:33 ` [PATCH 15/17] BLOCK: Stop CIFS from using EXT2 ioctl numbers directly " David Howells
2006-08-24 21:41   ` Trond Myklebust
2006-08-25  8:12     ` David Howells
2006-08-24 21:33 ` [PATCH 16/17] BLOCK: Move the msdos device ioctl compat stuff to the msdos driver " David Howells
2006-08-24 21:33 ` [PATCH 17/17] BLOCK: Make it possible to disable the block layer " David Howells
2006-08-25 14:27   ` Christoph Hellwig
2006-08-25 14:52     ` Alexey Dobriyan
2006-08-25 16:23     ` David Howells
2006-08-25 18:46       ` David Howells
2006-08-29 11:51       ` Christoph Hellwig
2006-08-29 12:21         ` Stefan Richter
2006-08-29 12:23         ` David Howells
2006-08-29 12:25           ` Christoph Hellwig
2006-08-29 13:50             ` Stefan Richter
2006-08-29 14:13               ` Stefan Richter
2006-08-30  1:12                 ` Roman Zippel
2006-08-30 18:33                   ` Stefan Richter
2006-08-30 21:43                     ` Adrian Bunk
2006-08-30 22:41                       ` Roman Zippel
2006-08-30 23:38                         ` Adrian Bunk
2006-08-30 23:58                           ` Roman Zippel
2006-08-31  8:01                             ` Stefan Richter
2006-09-01  0:15                         ` David Woodhouse
2006-09-01  0:48                           ` Randy.Dunlap
2006-09-01  1:27                             ` David Woodhouse
2006-09-01  1:47                               ` Adrian Bunk
2006-09-01 13:44                               ` Jörn Engel
2006-09-01 15:31                                 ` Stefan Richter
2006-09-01 16:19                                   ` Jörn Engel
2006-09-01 16:34                                     ` Adrian Bunk
2006-09-01 17:51                                       ` Stefan Richter
2006-09-01 18:14                                         ` Sam Ravnborg
2006-08-30 22:50                       ` Stefan Richter
2006-08-30 23:12                         ` Adrian Bunk
2006-08-30  1:11               ` Roman Zippel
2006-08-30 11:29                 ` Stefan Richter
2006-08-29 19:58             ` Greg KH
2006-08-29 21:08               ` John Stoffel [this message]
2006-08-31  3:01                 ` Matthew Wilcox
2006-08-31  3:04                   ` Shaya Potter
2006-08-31  8:53                     ` Stefan Richter
2006-08-31 12:32                       ` Shaya Potter
2006-08-31 13:16                         ` Stefan Richter
2006-08-31 13:27                           ` Shaya Potter
2006-08-31 10:13                 ` David Howells
2006-08-25 14:08 ` [PATCH 01/17] BLOCK: Move functions out of buffer code " Christoph Hellwig
     [not found] <6Paqj-6iH-23@gated-at.bofh.it>
     [not found] ` <6NKRa-8sj-9@gated-at.bofh.it>
     [not found]   ` <6Nv5K-8dh-9@gated-at.bofh.it>
     [not found]     ` <6Nvfu-8t8-17@gated-at.bofh.it>
     [not found]       ` <6NMJp-4wS-25@gated-at.bofh.it>
     [not found]         ` <6PaTj-7vF-17@gated-at.bofh.it>
     [not found]           ` <6PaTj-7vF-15@gated-at.bofh.it>
     [not found]             ` <6Pciq-362-13@gated-at.bofh.it>
     [not found]               ` <6PmUm-1Y7-9@gated-at.bofh.it>
     [not found]                 ` <6PwAr-6jv-5@gated-at.bofh.it>
2006-08-30 13:23                   ` [PATCH 17/17] BLOCK: Make it possible to disable the block layer " Bodo Eggert

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=17652.44254.620358.974993@stoffel.org \
    --to=john@stoffel.org \
    --cc=dhowells@redhat.com \
    --cc=greg@kroah.com \
    --cc=hch@infradead.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=zippel@linux-m68k.org \
    /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