Re: [RFC v2 1/7] sections.h: add sections header to collect all section info

From: Luis R. Rodriguez
Date: Fri Feb 19 2016 - 15:06:42 EST


On Fri, Feb 19, 2016 at 08:23:29AM -0800, Greg KH wrote:
> On Fri, Feb 19, 2016 at 05:45:53AM -0800, Luis R. Rodriguez wrote:
> > Linux makes extensive use of custom ELF header sections,
> > documentation for these are well scatterred. Unify this
> > documentation in a central place.
>
> Minor questions:
>
> > + <para>
> > + You should have received a copy of the GNU General Public
> > + License along with this documentation; if not, write to the Free
> > + Software Foundation, Inc., 59 Temple Place, Suite 330, Boston,
> > + MA 02111-1307 USA
> > + </para>
>
> Unless you are willing to personally track the office moves of the FSF
> for the next 20+ years, just drop this paragraph please :)

Sure.

> > +++ b/include/linux/sections.h
> > @@ -0,0 +1,224 @@
> > +#ifndef _LINUX_SECTIONS_H
> > +#define _LINUX_SECTIONS_H
> > +/*
> > + * Linux ELF sections
> > + *
> > + * Copyright (C) 2016 Luis R. Rodriguez <mcgrof@xxxxxxxxxx>
> > + *
> > + * 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.
>
> Do you really mean "any later version"?

Yes.

> > + * Due to this file being licensed under the GPL there is controversy over
> > + * whether this permits you to write a module that #includes this file
> > + * without placing your module under the GPL. Please consult a lawyer for
> > + * advice before doing this.
>
> Interesting wording,

This was copied from security.h.

> I don't think there's any "controversy" here, so I'd drop that word if at all
> possible.

Change it for something else? Any preference?

Luis