Re: linux-next: build failure after merge of the cifs tree

[Jeff Layton <jlayton@redhat.com>]

On Tue, 25 Oct 2011 09:51:37 -0500
Steve French <smfrench@gmail.com> wrote:

> Am fixing it now.  Jeff missed a stub for the non-freezer case.
> 
> diff --git a/include/linux/freezer.h b/include/linux/freezer.h
> index a155561..cd2d55b 100644
> --- a/include/linux/freezer.h
> +++ b/include/linux/freezer.h
> @@ -202,6 +202,8 @@ static inline void set_freezable_with_signal(void) {}
> 
>  #define wait_event_freezable_timeout(wq, condition, timeout)		\
>  		wait_event_interruptible_timeout(wq, condition, timeout)
> +#define #define wait_event_freezekillable(wq, condition)		\
> +		wait_event_killable(wq, condition)
> 
> 

Yep, silly oversight on my part -- sorry about that. That's exactly
what I was planning to do, so you can add my acked-by on that patch...

Thanks,
Jeff


> On Tue, Oct 25, 2011 at 8:59 AM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> > Hi all,
> >
> > After merging the cifs tree, today's linux-next build (powerpc
> > iseries_defconfig) failed like this:
> >
> > fs/cifs/transport.c: In function 'wait_for_response':
> > fs/cifs/transport.c:328: error: implicit declaration of function 'wait_event_freezekillable'
> >
> > Caused by commit f06ac72e9291 ("cifs, freezer: add
> > wait_event_freezekillable and have cifs use it").  In this config,
> > CONFIG_FREEZER is not set.
> > --
> > Cheers,
> > Stephen Rothwell                    sfr@canb.auug.org.au
> > http://www.canb.auug.org.au/~sfr/
> >
> 
> 
> 


-- 
Jeff Layton <jlayton@redhat.com>