[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080716103337.GA22931@elte.hu>
Date: Wed, 16 Jul 2008 12:33:37 +0200
From: Ingo Molnar <mingo@...e.hu>
To: James Bottomley <James.Bottomley@...senPartnership.com>
Cc: Andrew Morton <akpm@...ux-foundation.org>,
Linus Torvalds <torvalds@...ux-foundation.org>,
linux-scsi <linux-scsi@...r.kernel.org>,
linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [build fix] Re: [GIT PATCH] SCSI part 1
* Ingo Molnar <mingo@...e.hu> wrote:
> The problem is this aspect of that change:
>
> | @@ -9,6 +9,7 @@
> | #define _SCSI_SCSI_H
> |
> | #include <linux/types.h>
> | +#include <scsi/scsi_cmnd.h>
> |
> | /*
> | * The maximum number of SG segments that we will put inside a
>
> scsi_cmnd.h depends on symbols defined in blkdev.h. The fix is to
> include blkdev.h as well.
that wont work - a better replacement fix is the one below. The problem
is that scsi.h is included even on !CONFIG_BLOCK and then the
BLK_MAX_CDB symbol is meaningless.
Ingo
----------->
commit 7c2bc6236e6c787ab9962cfc6fad04551d9c4057
Author: Ingo Molnar <mingo@...e.hu>
Date: Wed Jul 16 11:56:08 2008 +0200
scsi: fix build error in fs/compat_ioctl.c
-tip testing found that the build broke in fs/compat_ioctl.c:
----------->
In file included from include/scsi/scsi.h:12,
from fs/compat_ioctl.c:72:
include/scsi/scsi_cmnd.h:27:25: warning: "BLK_MAX_CDB" is not defined
include/scsi/scsi_cmnd.h:28:3: error: #error MAX_COMMAND_SIZE can not be bigger than BLK_MAX_CDB
In file included from include/scsi/scsi.h:12,
from fs/compat_ioctl.c:72:
include/scsi/scsi_cmnd.h: In function ‘scsi_bidi_cmnd’:
include/scsi/scsi_cmnd.h:182: error: implicit declaration of function ‘blk_bidi_rq’
include/scsi/scsi_cmnd.h:183: error: dereferencing pointer to incomplete type
include/scsi/scsi_cmnd.h: In function ‘scsi_in’:
include/scsi/scsi_cmnd.h:189: error: dereferencing pointer to incomplete type
<-----------
with this config:
http://redhat.com/~mingo/misc/config-Wed_Jul_16_11_32_32_CEST_2008.bad
I have bisected it down to:
| feac6a07c4a3578bffd6769bb4927e8a7e1f3ffe is first bad commit
| commit feac6a07c4a3578bffd6769bb4927e8a7e1f3ffe
| Author: Martin Petermann <martin@...ux.vnet.ibm.com>
| Date: Wed Jul 2 10:56:35 2008 +0200
|
| [SCSI] zfcp: Move status accessors from zfcp to SCSI include file.
|
| Move the accessor functions for the scsi_cmnd status from zfcp to the
| SCSI include file. Change the interface to the functions to pass the
| scsi_cmnd pointer instead of the status pointer.
|
| Signed-off-by: Martin Petermann <martin@...ux.vnet.ibm.com>
| Signed-off-by: Christof Schmitt <christof.schmitt@...ibm.com>
| Signed-off-by: James Bottomley <James.Bottomley@...senPartnership.com>
The problem is due to this aspect of that change:
| @@ -9,6 +9,7 @@
| #define _SCSI_SCSI_H
|
| #include <linux/types.h>
| +#include <scsi/scsi_cmnd.h>
|
| /*
| * The maximum number of SG segments that we will put inside a
scsi_cmnd.h depends on symbols defined in blkdev.h but those symbols
are not available if !CONFIG_BLOCK.
Only include scsi/scsi_cmnd.h if on CONFIG_BLOCK.
(those methods are not used when CONFIG_BLOCK is off anyway).
Signed-off-by: Ingo Molnar <mingo@...e.hu>
---
include/scsi/scsi.h | 5 ++++-
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/include/scsi/scsi.h b/include/scsi/scsi.h
index 00137a7..b212859 100644
--- a/include/scsi/scsi.h
+++ b/include/scsi/scsi.h
@@ -9,7 +9,10 @@
#define _SCSI_SCSI_H
#include <linux/types.h>
-#include <scsi/scsi_cmnd.h>
+
+#ifdef CONFIG_BLOCK
+# include <scsi/scsi_cmnd.h>
+#endif
/*
* The maximum number of SG segments that we will put inside a
--
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