Re: [PATCH v2] x86/boot/64/clang: Use fixup_pointer() to access '__supported_pte_mask'

From: Alexander Potapenko
Date: Wed May 09 2018 - 05:16:21 EST


On Tue, May 8, 2018 at 11:44 PM Kirill A. Shutemov <
kirill.shutemov@xxxxxxxxxxxxxxx> wrote:

> On Tue, May 08, 2018 at 04:28:29PM +0000, Alexander Potapenko wrote:
> > @@ -196,7 +204,8 @@ unsigned long __head __startup_64(unsigned long
physaddr,
> >
> > pmd_entry = __PAGE_KERNEL_LARGE_EXEC & ~_PAGE_GLOBAL;
> > /* Filter out unsupported __PAGE_KERNEL_* bits: */
> > - pmd_entry &= __supported_pte_mask;
> > + mask_ptr = (pteval_t *)fixup_pointer(&__supported_pte_mask,
physaddr);

> Do we really need the cast here?
Correct, we do not.

> --
> Kirill A. Shutemov



--
Alexander Potapenko
Software Engineer

Google Germany GmbH
Erika-Mann-StraÃe, 33
80636 MÃnchen

GeschÃftsfÃhrer: Paul Manicle, Halimah DeLaine Prado
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg