Messages in this thread |  | | | Date | Tue, 11 Dec 2007 09:54:23 +0100 | | From | Ingo Molnar <> | | Subject | Re: [PATCH 5/4] x86: fix up asm-x86/pgtable*.h formatting |
* Christoph Hellwig <hch@infradead.org> 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
|  |