lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20101206155625.GC23214@kroah.com>
Date:	Mon, 6 Dec 2010 07:56:25 -0800
From:	Greg KH <greg@...ah.com>
To:	Namhyung Kim <namhyung@...il.com>
Cc:	Greg KH <gregkh@...e.de>, linux-usb@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] USB: whci-hcd: fix compiler warning

On Mon, Dec 06, 2010 at 10:05:30PM +0900, Namhyung Kim wrote:
> 2010-12-01 (수), 09:36 -0800, Greg KH:
> > On Thu, Dec 02, 2010 at 01:49:11AM +0900, Namhyung Kim wrote:
> > > 2010-12-01 (수), 08:10 -0800, Greg KH:
> > > > On Wed, Dec 01, 2010 at 01:28:41PM +0900, Namhyung Kim wrote:
> > > > > Annotate whci_hcd_id_table as '__used' to fix following warning:
> > > > > 
> > > > >   CC      drivers/usb/host/whci/hcd.o
> > > > > drivers/usb/host/whci/hcd.c:359: warning: ‘whci_hcd_id_table’ defined but not used
> > > > 
> > > > Are you seeing this warning when you build the module into the kernel,
> > > > or as a module?
> > > > 
> > > > thanks,
> > > > 
> > > > greg k-h
> > > 
> > > I saw it during allyesconfig build but not on allmodconfig.
> > > Maybe we can wrap it up in #ifdef CONFIG_MODULES ?
> > 
> > No, don't do that, your __used is the correct thing here, I just wanted
> > to make sure why you were seeing this warning.
> > 
> > thanks,
> > 
> > greg k-h
> 
> Hi, Greg
> 
> It seems this one is not applied in your tree yet. Do you need a resend?

Yes please, I must have missed it, thinking it was part of your other
series.  Sorry.

greg k-h
--
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

Powered by Openwall GNU/*/Linux Powered by OpenVZ