[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <200707281446.l6SEkGOD003581@mbox.iij4u.or.jp>
Date: Sat, 28 Jul 2007 23:45:43 +0900
From: FUJITA Tomonori <tomof@....org>
To: James.Bottomley@...elEye.com, nix.or.die@...glemail.com
Cc: linux-kernel@...r.kernel.org, fujita.tomonori@....ntt.co.jp,
linux-scsi@...r.kernel.org, tomof@....org
Subject: Re: BSG: BLK_DEV_BSG=y , BLOCK=n compile error
From: Gabriel C <nix.or.die@...glemail.com>
Subject: BSG: BLK_DEV_BSG=y , BLOCK=n compile error
Date: Sat, 28 Jul 2007 14:54:02 +0200
> Hi,
>
> BSG does not compile without BLOCK set :
Thanks, this has already been addressed.
http://marc.info/?l=linux-kernel&m=118534836402440&w=2
James, could you add the patch to scsi-rc-fixes? Thanks.
---
From: Paul Mundt <lethal@...ux-sh.org>
Subject: [PATCH] bsg: Fix build for CONFIG_BLOCK=n
BLK_DEV_BSG was added outside of the if BLOCK check, which allows it to
be enabled when CONFIG_BLOCK=n. This leads to many screenlengths of
errors, starting with a parse error on the request_queue_t definition.
Obviously this wasn't intended for CONFIG_BLOCK=n usage, so just move the
option back in to the block.
Caught with a randconfig on sh.
Signed-off-by: Paul Mundt <lethal@...ux-sh.org>
Acked-by: Jens Axboe <jens.axboe@...cle.com>
Acked-by: FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>
---
block/Kconfig | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/block/Kconfig b/block/Kconfig
index ca2ef4e..2484e0e 100644
--- a/block/Kconfig
+++ b/block/Kconfig
@@ -49,8 +49,6 @@ config LSF
If unsure, say Y.
-endif # BLOCK
-
config BLK_DEV_BSG
bool "Block layer SG support v4 (EXPERIMENTAL)"
depends on EXPERIMENTAL
@@ -64,4 +62,6 @@ config BLK_DEV_BSG
protocols (e.g. Task Management Functions and SMP in Serial
Attached SCSI).
+endif # BLOCK
+
source block/Kconfig.iosched
--
1.5.2.4
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists