From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: [PATCH] zcache/debug: compiler failure on PPC64. Date: Thu, 7 Mar 2013 09:31:58 +0800 Message-ID: <20130307013158.GC2910@kroah.com> References: <1362615216-12293-1-git-send-email-konrad.wilk@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1362615216-12293-1-git-send-email-konrad.wilk@oracle.com> Sender: linux-kernel-owner@vger.kernel.org To: Konrad Rzeszutek Wilk Cc: linux-kernel@vger.kernel.org, linux-next@vger.kernel.org, sfr@canb.auug.org.au List-Id: linux-next.vger.kernel.org On Wed, Mar 06, 2013 at 07:13:36PM -0500, Konrad Rzeszutek Wilk wrote: > On PPC64 we get this: > In file included from drivers/staging/zcache/debug.c:2: > drivers/staging/zcache/debug.h: In function 'dec_zcache_obj_count': > drivers/staging/zcache/debug.h:16: error: implicit declaration of function 'BUG_ON' > > This simple patch adds the appropiate header file to finish > the compile. > > Reported-by: Stephen Rothwell > Signed-off-by: Konrad Rzeszutek Wilk > --- > drivers/staging/zcache/debug.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/staging/zcache/debug.h b/drivers/staging/zcache/debug.h > index eef67db..0d9c6b4 100644 > --- a/drivers/staging/zcache/debug.h > +++ b/drivers/staging/zcache/debug.h > @@ -1,5 +1,6 @@ > #ifdef CONFIG_ZCACHE_DEBUG > > +#include > /* we try to keep these statistics SMP-consistent */ Put the #include files at the top of the file. Also, you didn't really test this on linux-next as you can't select this option at the moment, I think you also want to fix that at the same time, unless you really don't want anyone to ever select this option :) Try it again? thanks, greg k-h