* Commit "arm: convert drivers to scsi proc_ops" breaks linux-next
@ 2012-03-27 2:51 Paul Gortmaker
2012-03-27 8:51 ` James Bottomley
0 siblings, 1 reply; 4+ messages in thread
From: Paul Gortmaker @ 2012-03-27 2:51 UTC (permalink / raw)
To: adobriyan; +Cc: JBottomley, linux-next, linux-scsi
Hi Alexey,
This error showed up in linux-next today:
CC drivers/scsi/arm/powertec.o
drivers/scsi/arm/powertec.c: In function 'powertecscsi_proc_show':
drivers/scsi/arm/powertec.c:199: error: dereferencing pointer to incomplete type
drivers/scsi/arm/powertec.c:202: error: implicit declaration of function 'seq_printf'
drivers/scsi/arm/powertec.c: In function 'powertecscsi_proc_open':
drivers/scsi/arm/powertec.c:214: error: implicit declaration of function 'single_open'
drivers/scsi/arm/powertec.c: At top level:
drivers/scsi/arm/powertec.c:250: error: 'seq_read' undeclared here (not in a function)
drivers/scsi/arm/powertec.c:251: error: 'seq_lseek' undeclared here (not in a function)
drivers/scsi/arm/powertec.c:252: error: 'single_release' undeclared here (not in a function)
and a bisect leads to this:
--------------
commit bb6c20445c1bdd1c49ac5e76801211116694e3b9
Author: Alexey Dobriyan <adobriyan@gmail.com>
Date: Wed Feb 22 22:46:23 2012 +0300
[SCSI] arm: convert drivers to scsi proc_ops
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
--------------
I'm not sure why "new" commits are only showing up in linux-next with
just days left in the merge window, but can you have a look ASAP?
THanks,
Paul.
----------
git bisect start
# bad: [88b5d261d52f81806577e939c1744395b9f4edf9] Add linux-next specific files for 20120326
git bisect bad 88b5d261d52f81806577e939c1744395b9f4edf9
# good: [c16fa4f2ad19908a47c63d8fa436a1178438c7e7] Linux 3.3
git bisect good c16fa4f2ad19908a47c63d8fa436a1178438c7e7
# good: [f37ab0fba271e43edab0e3ae9fe644fcda455402] Merge branch 'hwmon-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jdelvare/staging
git bisect good f37ab0fba271e43edab0e3ae9fe644fcda455402
# bad: [03797c27f6c50cd92e396cc31271bd73224239e0] Merge remote-tracking branch 'sound/for-next'
git bisect bad 03797c27f6c50cd92e396cc31271bd73224239e0
# bad: [9f9aa50fe5d555b356bfc34f135bc66d611fe573] Merge remote-tracking branch 'scsi/master'
git bisect bad 9f9aa50fe5d555b356bfc34f135bc66d611fe573
# good: [0cd69e8184c4242ca15494829d675606183e8c78] Merge remote-tracking branch 'arm/for-next'
git bisect good 0cd69e8184c4242ca15494829d675606183e8c78
# good: [8e52333b697339063e338b80560dec7e4396ce42] Merge remote-tracking branch 'fuse/for-next'
git bisect good 8e52333b697339063e338b80560dec7e4396ce42
# good: [21ea2bf9d8a875d79cbf1d70625a4a3cacfa9a64] Merge branch 'kbuild/kconfig' into kbuild/for-next
git bisect good 21ea2bf9d8a875d79cbf1d70625a4a3cacfa9a64
# good: [cf53892ca8c08d2ca7a553258d204c588d37ac87] Merge remote-tracking branch 'infiniband/for-next'
git bisect good cf53892ca8c08d2ca7a553258d204c588d37ac87
# bad: [7ef79ae61a1f5dbfcfda4b4fb01434692e283e3b] [SCSI] vmw_pvscsi: Try setting host->max_id as suggested by the device.
git bisect bad 7ef79ae61a1f5dbfcfda4b4fb01434692e283e3b
# bad: [104c4fe25dc9bde823ba4591e910a77071b98ab5] [SCSI] remove scsi_host_template::proc_info
git bisect bad 104c4fe25dc9bde823ba4591e910a77071b98ab5
# good: [2f97c74b58d2c2b3e0fd67ea1188f73b17f22c93] [SCSI] aha1740: convert to scsi proc_ops
git bisect good 2f97c74b58d2c2b3e0fd67ea1188f73b17f22c93
# good: [6852cc964c8a068d2a9ebe8b9dc922f6c0e29feb] [SCSI] usbstorage: convert to scsi proc_ops
git bisect good 6852cc964c8a068d2a9ebe8b9dc922f6c0e29feb
# good: [d11ea639a11b70ba8f88676af8df329481a701db] [SCSI] eata: convert to scsi proc_ops
git bisect good d11ea639a11b70ba8f88676af8df329481a701db
# good: [64d7ba67031f12b44a99978a2b108ed7d2f074dc] [SCSI] NCR5380: convert to scsi proc_ops
git bisect good 64d7ba67031f12b44a99978a2b108ed7d2f074dc
# bad: [d089e2cb5dffc1160b19a4e48d68cc583cf249ba] [SCSI] wd33c93: convert to scsi proc_ops
git bisect bad d089e2cb5dffc1160b19a4e48d68cc583cf249ba
# bad: [bb6c20445c1bdd1c49ac5e76801211116694e3b9] [SCSI] arm: convert drivers to scsi proc_ops
git bisect bad bb6c20445c1bdd1c49ac5e76801211116694e3b9
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Commit "arm: convert drivers to scsi proc_ops" breaks linux-next
2012-03-27 2:51 Commit "arm: convert drivers to scsi proc_ops" breaks linux-next Paul Gortmaker
@ 2012-03-27 8:51 ` James Bottomley
2012-03-27 14:49 ` Stephen Rothwell
0 siblings, 1 reply; 4+ messages in thread
From: James Bottomley @ 2012-03-27 8:51 UTC (permalink / raw)
To: Paul Gortmaker; +Cc: adobriyan, linux-next, linux-scsi
On Mon, 2012-03-26 at 22:51 -0400, Paul Gortmaker wrote:
> Hi Alexey,
>
> This error showed up in linux-next today:
>
> CC drivers/scsi/arm/powertec.o
> drivers/scsi/arm/powertec.c: In function 'powertecscsi_proc_show':
> drivers/scsi/arm/powertec.c:199: error: dereferencing pointer to incomplete type
> drivers/scsi/arm/powertec.c:202: error: implicit declaration of function 'seq_printf'
> drivers/scsi/arm/powertec.c: In function 'powertecscsi_proc_open':
> drivers/scsi/arm/powertec.c:214: error: implicit declaration of function 'single_open'
> drivers/scsi/arm/powertec.c: At top level:
> drivers/scsi/arm/powertec.c:250: error: 'seq_read' undeclared here (not in a function)
> drivers/scsi/arm/powertec.c:251: error: 'seq_lseek' undeclared here (not in a function)
> drivers/scsi/arm/powertec.c:252: error: 'single_release' undeclared here (not in a function)
>
> and a bisect leads to this:
>
> --------------
> commit bb6c20445c1bdd1c49ac5e76801211116694e3b9
> Author: Alexey Dobriyan <adobriyan@gmail.com>
> Date: Wed Feb 22 22:46:23 2012 +0300
>
> [SCSI] arm: convert drivers to scsi proc_ops
>
> Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
> Signed-off-by: James Bottomley <JBottomley@Parallels.com>
> --------------
Could you dig a little deeper on this one? seq_printf is defined in
include/linux/seq_file.h which is included by this commit. I can't see
why the function prototype should end up undefined.
Could the include guards for this file be getting set erroneously
somehow?
James
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Commit "arm: convert drivers to scsi proc_ops" breaks linux-next
2012-03-27 8:51 ` James Bottomley
@ 2012-03-27 14:49 ` Stephen Rothwell
2012-03-27 20:05 ` [PATCH] scsi: arm: fix compilation Alexey Dobriyan
0 siblings, 1 reply; 4+ messages in thread
From: Stephen Rothwell @ 2012-03-27 14:49 UTC (permalink / raw)
To: James Bottomley; +Cc: Paul Gortmaker, adobriyan, linux-next, linux-scsi
[-- Attachment #1: Type: text/plain, Size: 1910 bytes --]
Hi James,
On Tue, 27 Mar 2012 08:51:25 +0000 James Bottomley <jbottomley@parallels.com> wrote:
>
> On Mon, 2012-03-26 at 22:51 -0400, Paul Gortmaker wrote:
> >
> > This error showed up in linux-next today:
> >
> > CC drivers/scsi/arm/powertec.o
> > drivers/scsi/arm/powertec.c: In function 'powertecscsi_proc_show':
> > drivers/scsi/arm/powertec.c:199: error: dereferencing pointer to incomplete type
> > drivers/scsi/arm/powertec.c:202: error: implicit declaration of function 'seq_printf'
> > drivers/scsi/arm/powertec.c: In function 'powertecscsi_proc_open':
> > drivers/scsi/arm/powertec.c:214: error: implicit declaration of function 'single_open'
> > drivers/scsi/arm/powertec.c: At top level:
> > drivers/scsi/arm/powertec.c:250: error: 'seq_read' undeclared here (not in a function)
> > drivers/scsi/arm/powertec.c:251: error: 'seq_lseek' undeclared here (not in a function)
> > drivers/scsi/arm/powertec.c:252: error: 'single_release' undeclared here (not in a function)
> >
> > and a bisect leads to this:
> >
> > --------------
> > commit bb6c20445c1bdd1c49ac5e76801211116694e3b9
> > Author: Alexey Dobriyan <adobriyan@gmail.com>
> > Date: Wed Feb 22 22:46:23 2012 +0300
> >
> > [SCSI] arm: convert drivers to scsi proc_ops
> >
> > Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
> > Signed-off-by: James Bottomley <JBottomley@Parallels.com>
> > --------------
>
> Could you dig a little deeper on this one? seq_printf is defined in
> include/linux/seq_file.h which is included by this commit. I can't see
> why the function prototype should end up undefined.
>
> Could the include guards for this file be getting set erroneously
> somehow?
This patch added seq_file.h to drivers/scsi/arm/acornscsi.c, but not to
drivers/scsi/arm/powertec.c .
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] scsi: arm: fix compilation
2012-03-27 14:49 ` Stephen Rothwell
@ 2012-03-27 20:05 ` Alexey Dobriyan
0 siblings, 0 replies; 4+ messages in thread
From: Alexey Dobriyan @ 2012-03-27 20:05 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: James Bottomley, Paul Gortmaker, linux-next, linux-scsi
Add seq_file.h for seq_file stuff.
Broken by commit bb6c20445c1bdd1c49ac5e76801211116694e3b9
"[SCSI] arm: convert drivers to scsi proc_ops"
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
---
drivers/scsi/arm/arxescsi.c | 1 +
drivers/scsi/arm/cumana_2.c | 1 +
drivers/scsi/arm/eesox.c | 1 +
drivers/scsi/arm/fas216.c | 1 +
drivers/scsi/arm/powertec.c | 1 +
5 files changed, 5 insertions(+)
--- a/drivers/scsi/arm/arxescsi.c
+++ b/drivers/scsi/arm/arxescsi.c
@@ -25,6 +25,7 @@
#include <linux/ioport.h>
#include <linux/proc_fs.h>
#include <linux/unistd.h>
+#include <linux/seq_file.h>
#include <linux/stat.h>
#include <linux/delay.h>
#include <linux/init.h>
--- a/drivers/scsi/arm/cumana_2.c
+++ b/drivers/scsi/arm/cumana_2.c
@@ -19,6 +19,7 @@
#include <linux/module.h>
#include <linux/blkdev.h>
#include <linux/kernel.h>
+#include <linux/seq_file.h>
#include <linux/string.h>
#include <linux/ioport.h>
#include <linux/proc_fs.h>
--- a/drivers/scsi/arm/eesox.c
+++ b/drivers/scsi/arm/eesox.c
@@ -25,6 +25,7 @@
#include <linux/module.h>
#include <linux/blkdev.h>
#include <linux/kernel.h>
+#include <linux/seq_file.h>
#include <linux/string.h>
#include <linux/ioport.h>
#include <linux/proc_fs.h>
--- a/drivers/scsi/arm/fas216.c
+++ b/drivers/scsi/arm/fas216.c
@@ -40,6 +40,7 @@
#include <linux/string.h>
#include <linux/ioport.h>
#include <linux/proc_fs.h>
+#include <linux/seq_file.h>
#include <linux/delay.h>
#include <linux/bitops.h>
#include <linux/init.h>
--- a/drivers/scsi/arm/powertec.c
+++ b/drivers/scsi/arm/powertec.c
@@ -10,6 +10,7 @@
#include <linux/module.h>
#include <linux/blkdev.h>
#include <linux/kernel.h>
+#include <linux/seq_file.h>
#include <linux/string.h>
#include <linux/ioport.h>
#include <linux/proc_fs.h>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-03-27 20:05 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-03-27 2:51 Commit "arm: convert drivers to scsi proc_ops" breaks linux-next Paul Gortmaker
2012-03-27 8:51 ` James Bottomley
2012-03-27 14:49 ` Stephen Rothwell
2012-03-27 20:05 ` [PATCH] scsi: arm: fix compilation Alexey Dobriyan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).