From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756961AbYHMX53 (ORCPT ); Wed, 13 Aug 2008 19:57:29 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753211AbYHMX5V (ORCPT ); Wed, 13 Aug 2008 19:57:21 -0400 Received: from bombadil.infradead.org ([18.85.46.34]:44095 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752573AbYHMX5U (ORCPT ); Wed, 13 Aug 2008 19:57:20 -0400 Date: Wed, 13 Aug 2008 16:54:25 -0700 From: Greg KH To: Sam Ravnborg Cc: Andrew Morton , jbaron@redhat.com, randy.dunlap@oracle.com, linux-kernel@vger.kernel.org, joe@perches.com, nick@nick-andrew.net Subject: Re: [PATCH 1/7] dynamic debug v2 - infrastructure Message-ID: <20080813235425.GD32154@kroah.com> References: <20080808193851.24210bc4.randy.dunlap@oracle.com> <20080811173616.GC6103@redhat.com> <20080811223309.GD9810@kroah.com> <20080812194817.GD6056@redhat.com> <20080812200908.GA444@kroah.com> <20080812204619.GE6056@redhat.com> <20080813010804.GA17446@kroah.com> <20080812181653.32d6f373.akpm@linux-foundation.org> <20080813033859.GA9443@kroah.com> <20080813200047.GA2053@uranus.ravnborg.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20080813200047.GA2053@uranus.ravnborg.org> User-Agent: Mutt/1.5.16 (2007-06-09) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Aug 13, 2008 at 10:00:47PM +0200, Sam Ravnborg wrote: > On Tue, Aug 12, 2008 at 08:38:59PM -0700, Greg KH wrote: > > On Tue, Aug 12, 2008 at 06:16:53PM -0700, Andrew Morton wrote: > > > On Tue, 12 Aug 2008 18:08:04 -0700 > > > Greg KH wrote: > > > > > > > On Tue, Aug 12, 2008 at 04:46:19PM -0400, Jason Baron wrote: > > > > > On Tue, Aug 12, 2008 at 01:09:08PM -0700, Greg KH wrote: > > > > > > So close, can I have a good changelog comment with the patch so people > > > > > > know what it is when they look in the logs? > > > > > > > > > > > > Care to resend it with that? > > > > > > > > > > > > > > > > Base infrastructure to enable per-module debug messages. > > > > > > > > Ah, so close... > > > > > > > > With this patch, I get the following build error: > > > > > > > > CC [M] drivers/usb/gadget/u_ether.o > > > > drivers/usb/gadget/u_ether.c: In function ___gether_setup___: > > > > drivers/usb/gadget/u_ether.c:787: error: ___KBUILD_MODNAME___ undeclared (first use in this function) > > > > drivers/usb/gadget/u_ether.c:787: error: (Each undeclared identifier is reported only once > > > > drivers/usb/gadget/u_ether.c:787: error: for each function it appears in.) > > > > drivers/usb/gadget/u_ether.c:787: error: unknown field ___Usage___ specified in initializer > > > > drivers/usb/gadget/u_ether.c:787: error: expected expression before ___.___ token > > > > drivers/usb/gadget/u_ether.c:787: error: initializer element is not constant > > > > drivers/usb/gadget/u_ether.c:787: error: (near initialization for ___descriptor.hash___) > > > > drivers/usb/gadget/u_ether.c:787: error: ___Usage___ undeclared (first use in this function) > > > > drivers/usb/gadget/u_ether.c:787: error: invalid operands to binary << (have ___long long int___ and ___char *___) > > > > drivers/usb/gadget/u_ether.c:787: error: expected ___)___ before ___:___ token > > > > drivers/usb/gadget/u_ether.c:787: error: invalid operands to binary & (have ___long long int___ and ___char *___) > > > > drivers/usb/gadget/u_ether.c:787: error: invalid operands to binary << (have ___long long int___ and ___char *___) > > > > drivers/usb/gadget/u_ether.c:787: error: expected ___)___ before ___:___ token > > > > drivers/usb/gadget/u_ether.c:787: error: invalid operands to binary & (have ___long long int___ and ___char *___) > > > > drivers/usb/gadget/u_ether.c:787: error: expected ___)___ before ___:___ token > > > > drivers/usb/gadget/u_ether.c:787: warning: passing argument 4 of _____dynamic_dbg_enabled_helper___ makes integer from pointer without a cast > > > > drivers/usb/gadget/u_ether.c:787: error: expected ___)___ before ___KBUILD_MODNAME___ > > > > > > (lol at gcc crap) > > > > > > > drivers/usb/gadget/u_ether.c:787: warning: too few arguments for format > > > > > > argh, not that again. We had this with a patch from Jesse a while > > > back, which I fixed thusly: > > > > > > > > Yeah, I remember that too, but in this case, KBUILD_MODNAME is being > > used in a macro, not an inline function, the exact opposite. > > > > Sam, any ideas here? I'll bounce the original patch at you if you > > missed it last time. > > I have not looked into the particular error but what happens is > that u_ether.o is used by more than one module so kbuild does > not know what value to assign the KBUILD_MODNAME and therefore > decide not to assign KBUILD_MODNAME at all. Ick. Hm, how do I fix this? Build u_ether.c into two different .o files, depending on which module it is being included into? Or do: #include "u_ether.c" in the source code that was linking it in? Actually, as ugly as that last one is, it probably would solve this issue, and be the same end-result, right? thanks, greg k-h