* Christoph Hellwig <[EMAIL PROTECTED]> wrote:

> On Mon, Dec 10, 2007 at 12:55:33PM -0800, Jeremy Fitzhardinge wrote:
> > +#define set_pte_atomic(pteptr, pteval)             set_pte(pteptr,pteval)
> > +#define set_pte_present(mm,addr,ptep,pteval)       
> > set_pte_at(mm,addr,ptep,pteval)
> >  
> > +#define pte_clear(mm,addr,xp)                              \
> > +   do {                                            \
> > +           set_pte_at(mm, addr, xp, __pte(0));     \
> > +   } while (0)
> > +
> >  #undef pmd_clear
> > +#define pmd_clear(xp)                              \
> > +   do {                                    \
> > +           set_pmd(xp, __pmd(0));          \
> > +   } while (0)
> 
> What about turning these into inline functions to get better type 
> checking?

yes, please. That will also trigger and resolve any still present 
include file dependency mess. Macros are often used to hack around 
unclean include file layouts.

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