Re: [PATCH 8/14] FRV: GP-REL data support

[Christoph Hellwig <hch@infradead.org>]

On Tue, Nov 02, 2004 at 04:34:01PM +0000, David Howells wrote:
> 
> > > +#ifdef CONFIG_FRV
> > > +#define __jiffy_data  __attribute__((section(".data")))
> > > +#else
> > > +#define __jiffy_data
> > > +#endif
> > 
> > please avoid per-arch ifdefs in common code, this needs to go into some asm/
> > header, or and __ARCH_HAVE_FOO ifdef.
> 
> You're advocating using an ifdef? I thought you hated the things...

You have an ifdef already.  Using the right symbol at least avoids this
into turning to

#if define(foo) || defined(bar) || (defined(baz) && !defined(foobar))

> I want to avoid changing every other arch.
> 
> > Anyway, would doing this unconditionally cause any problems?
> 
> Not as far as I know.

So go for that.

> > > +#ifndef __ASSEMBLY__
> > > +extern const char linux_banner[];
> > > +#endif
> > 
> > totally wrong place.  this is not about linkage at all.
> 
> Actually, it's almost entirely about linkage:-) But in this case, you may be
> right.

So how is the linux banner about linkage?

> Anyone any suggestions as to the right place for this? linux/kernel.h
> perhaps?

Maybe.