Re: [PATCH 21/70] x86/boot/compressed/64: Add function to map a page unencrypted

Joerg Roedel
2020-03-20T22:12:17Z

On Fri, Mar 20, 2020 at 02:02:13PM -0700, Dave Hansen wrote:
> It *never* flushes global pages. For a generic function like this, that
> seems pretty dangerous because the PTEs it goes after could quite easily
> be Global. It's also not _obviously_ correct if PCIDs are in play
> (which I don't think they are on AMD).
>
> A flush_tlb_global() is probably more appropriate. Better yet, is there
> a reason not to use flush_tlb_kernel_range()? I don't think it's
> necessary to whack the entire TLB for one PTE set.

This code runs before the actual kernel image is decompressed, so there
is no PCID and no global pages (I think CR4.PGE is still 0). So a
cr3-write is enough to flush the TLB. Also the TLB-flush helpers of the
running kernel are not available here.

Regards,

Joerg