On Fri, 15 Feb 2008, Andrew Morton wrote:

> > +#define mmu_rmap_notifier(function, args...)                               
> > \
> > +   do {                                                            \
> > +           struct mmu_rmap_notifier *__mrn;                        \
> > +           struct hlist_node *__n;                                 \
> > +                                                                   \
> > +           rcu_read_lock();                                        \
> > +           hlist_for_each_entry_rcu(__mrn, __n,                    \
> > +                           &mmu_rmap_notifier_list, hlist)         \
> > +                   if (__mrn->ops->function)                       \
> > +                           __mrn->ops->function(__mrn, args);      \
> > +           rcu_read_unlock();                                      \
> > +   } while (0);
> > +
> 
> buggy macro: use locals.

Ok. Same as the non rmap version.

> > +EXPORT_SYMBOL(mmu_rmap_export_page);
> 
> The other patch used EXPORT_SYMBOL_GPL.

Ok will make that consistent.


--
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/

Reply via email to