On Wednesday 01 February 2006 23:01, Pekka Enberg wrote:
> > > +
> > > +static inline void suspend_initialise_module_lists(void) {
> > > + INIT_LIST_HEAD(&suspend_filters);
> > > + INIT_LIST_HEAD(&suspend_writers);
> > > + INIT_LIST_HEAD(&suspend_modules);
> > > +}
> >
> > I couldn't find a user for this. I would imagine there's only one,
> > though, and this should be inlined there?
On Thu, 2 Feb 2006, Nigel Cunningham wrote:
> I forgot to mention re this - yes, there's just one caller, in another set of
> patches I'll send later (this was just the first set!). Having the function
> to be inlined in this .h so that it's with other module specific code, and
> then used in the caller once it has been #included, isn't that the right way
> to do things?
Sorry, I can't parse the above :-). My point was that this is
probably called in a .c file so move the function in that file and
introduce it whenever you introduce the caller.
Pekka
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
