* linux-next: build failure after merge of the usb tree
@ 2013-04-18 6:05 Stephen Rothwell
2013-04-18 16:06 ` Joe Perches
2013-04-18 21:17 ` [PATCH usb-next] usb: storage: Fix link error Joe Perches
0 siblings, 2 replies; 4+ messages in thread
From: Stephen Rothwell @ 2013-04-18 6:05 UTC (permalink / raw)
To: Greg KH; +Cc: linux-next, linux-kernel, Joe Perches
[-- Attachment #1: Type: text/plain, Size: 1253 bytes --]
Hi Greg,
After merging the usb tree, today's linux-next build (x86_64 allmodconfig)
failed like this:
ERROR: "usb_stor_dbg" [drivers/usb/storage/ums-usbat.ko] undefined!
ERROR: "usb_stor_dbg" [drivers/usb/storage/ums-sddr55.ko] undefined!
ERROR: "usb_stor_dbg" [drivers/usb/storage/ums-sddr09.ko] undefined!
ERROR: "usb_stor_dbg" [drivers/usb/storage/ums-realtek.ko] undefined!
ERROR: "usb_stor_dbg" [drivers/usb/storage/ums-karma.ko] undefined!
ERROR: "usb_stor_dbg" [drivers/usb/storage/ums-jumpshot.ko] undefined!
ERROR: "usb_stor_dbg" [drivers/usb/storage/ums-isd200.ko] undefined!
ERROR: "usb_stor_dbg" [drivers/usb/storage/ums-freecom.ko] undefined!
ERROR: "usb_stor_dbg" [drivers/usb/storage/ums-eneub6250.ko] undefined!
ERROR: "usb_stor_dbg" [drivers/usb/storage/ums-datafab.ko] undefined!
ERROR: "usb_stor_dbg" [drivers/usb/storage/ums-cypress.ko] undefined!
ERROR: "usb_stor_dbg" [drivers/usb/storage/ums-alauda.ko] undefined!
Caused by commit 75b9130e8af6 ("usb: storage: Add usb_stor_dbg, reduce
object size"). Please build test such infrastructure patches with and
without MODULES ...
I have used the usb tree from next-20130417 for today.
--
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
* Re: linux-next: build failure after merge of the usb tree
2013-04-18 6:05 linux-next: build failure after merge of the usb tree Stephen Rothwell
@ 2013-04-18 16:06 ` Joe Perches
2013-04-18 21:17 ` [PATCH usb-next] usb: storage: Fix link error Joe Perches
1 sibling, 0 replies; 4+ messages in thread
From: Joe Perches @ 2013-04-18 16:06 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: Greg KH, linux-next, linux-kernel
On Thu, 2013-04-18 at 16:05 +1000, Stephen Rothwell wrote:
> After merging the usb tree, today's linux-next build (x86_64 allmodconfig)
> failed like this:
>
> ERROR: "usb_stor_dbg" [drivers/usb/storage/ums-usbat.ko] undefined!
[...]
> Caused by commit 75b9130e8af6 ("usb: storage: Add usb_stor_dbg, reduce
> object size"). Please build test such infrastructure patches with and
> without MODULES ...
>
> I have used the usb tree from next-20130417 for today.
Sorry 'bout that.
I did compilation allyesconfig and allmodconfig with
and without CONFIG_USB_STORAGE_VERBOSE and made
.ko's for that directory only but not a full link.
Looks like it needs EXPORT_SYMBOL_GPL(usb_stor_dbg) in
debug.c. It takes me quite a while to do a complete
allmodconfig build, but I'll check.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH usb-next] usb: storage: Fix link error
2013-04-18 6:05 linux-next: build failure after merge of the usb tree Stephen Rothwell
2013-04-18 16:06 ` Joe Perches
@ 2013-04-18 21:17 ` Joe Perches
2013-04-19 2:02 ` Greg KH
1 sibling, 1 reply; 4+ messages in thread
From: Joe Perches @ 2013-04-18 21:17 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Greg KH, linux-next, linux-kernel, Matthew Dharm, linux-usb, usb-storage
Fix allmodconfig link error introduced by commit 75b9130e8a
("usb: storage: Add usb_stor_dbg, reduce object size")
Export the symbol usb_stor_dbg.
Add export.h
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/usb/storage/debug.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/usb/storage/debug.c b/drivers/usb/storage/debug.c
index b428129..29fe08f 100644
--- a/drivers/usb/storage/debug.c
+++ b/drivers/usb/storage/debug.c
@@ -43,6 +43,7 @@
*/
#include <linux/cdrom.h>
+#include <linux/export.h>
#include <scsi/scsi.h>
#include <scsi/scsi_cmnd.h>
#include <scsi/scsi_dbg.h>
@@ -193,3 +194,4 @@ int usb_stor_dbg(const char *fmt, ...)
return r;
}
+EXPORT_SYMBOL_GPL(usb_stor_dbg);
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH usb-next] usb: storage: Fix link error
2013-04-18 21:17 ` [PATCH usb-next] usb: storage: Fix link error Joe Perches
@ 2013-04-19 2:02 ` Greg KH
0 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2013-04-19 2:02 UTC (permalink / raw)
To: Joe Perches
Cc: Stephen Rothwell, linux-next, linux-kernel, Matthew Dharm,
linux-usb, usb-storage
On Thu, Apr 18, 2013 at 02:17:14PM -0700, Joe Perches wrote:
> Fix allmodconfig link error introduced by commit 75b9130e8a
> ("usb: storage: Add usb_stor_dbg, reduce object size")
>
> Export the symbol usb_stor_dbg.
> Add export.h
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> drivers/usb/storage/debug.c | 2 ++
> 1 file changed, 2 insertions(+)
You forgot the "Reported-by:" line :(
I'll go add it now, thanks for the quick fix-up patch.
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-04-19 2:02 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-04-18 6:05 linux-next: build failure after merge of the usb tree Stephen Rothwell
2013-04-18 16:06 ` Joe Perches
2013-04-18 21:17 ` [PATCH usb-next] usb: storage: Fix link error Joe Perches
2013-04-19 2:02 ` Greg KH
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).