Subject: Re: [PATCH 1/2] Make setjmp/longjmp code generic

From: Michael Ellerman <[email-masked]>
Date: Fri, 18 Jan 2008 08:49:22 +1100

On Thu, 2008-01-17 at 06:45 -0600, Josh Boyer wrote:
> On Thu, 17 Jan 2008 16:36:49 +1100
> Michael Neuling <[email-masked]> wrote:
> > In message <[email-masked]> you wrote:
> > > Hi,
> > > 
> > > On Thu, Jan 17, 2008 at 03:45:42PM +1100, Michael Neuling wrote:
> > > 
> > > > Index: linux-2.6-ozlabs/include/asm-powerpc/setjmp.h
> > > > ===================================================================
> > > > --- /dev/null
> > > > +++ linux-2.6-ozlabs/include/asm-powerpc/setjmp.h
> > > > @@ -0,0 +1,18 @@
> > > > +/*
> > > > + * Copyright (C) 2007 Michael Neuling
> >                      ^^^^^^
> > 
> > > > + *
> > > > + *      This program is free software; you can redistribute it and/or
> > > > + *      modify it under the terms of the GNU General Public License
> > > > + *      as published by the Free Software Foundation; either version
> > > > + *      2 of the License, or (at your option) any later version.
> > > > + *
> > > > + */
> > > > +#ifndef _ASM_POWERPC_SETJMP_H
> > > > +#define _ASM_POWERPC_SETJMP_H
> > > > +
> > > > +#define JMP_BUF_LEN    23
> > > > +
> > > > +extern long setjmp(long *);
> > > > +extern void longjmp(long *, long);
> > > > +
> > > > +#endif /* _ASM_POWERPC_SETJMP_H */
> > > 
> > > Should the above be inside #ifdef __KERNEL__?
> > 
> > Yep... _and_ it's 2008 now!
> > 
> > I'll update.
> 
> While you're off updating it, remove the (C).  It's bogus and shouldn't
> be there.

And it should say IBM corp too.

Oh and my car needs a wash when you're free ..

cheers

-- 
Michael Ellerman
OzLabs, IBM Australia Development Lab

wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person