Date: Wed, 13 Jan 2016 11:31:03 +0000 From: Catalin Marinas <catalin.marinas@....com> To: Kees Cook <keescook@...omium.org> Cc: David Brown <david.brown@...aro.org>, "kernel-hardening@...ts.openwall.com" <kernel-hardening@...ts.openwall.com>, Russell King - ARM Linux <linux@....linux.org.uk> Subject: Re: Self Introduction + rmk (actually cc'ing him this time) On Tue, Jan 12, 2016 at 11:31:50AM -0800, Kees Cook wrote: > On Mon, Jan 11, 2016 at 10:33 AM, David Brown <david.brown@...aro.org> wrote: > > On Thu, Dec 10, 2015 at 03:52:16PM -0800, Kees Cook wrote: > > > >>> I haven't done any further improvements to them, nor have I received any > >>> feedback. I'll rebase them against latest kernel if anyone else is > >>> willing to test. I had a plan to run some benchmarks and see how > >>> performance is affected (including the CPU_SW_DOMAIN_PAN) before pushing > >>> again for upstreaming but I haven't had the time. > >> > >> > >> David, getting back to something that might good to get your help > >> with: would you be able to test Catalin's LPAE TTBR0 PAN series on > >> real hardware? (Are you familiar with the LKDTM tests for this?) > > > > > > Sorry for the delay in getting back to you. Been both moving and > > taking vacation. > > > > I'd like to test this. I'm just trying to see if I can track down > > some hardware that'll boot LPAE. > > Awesome! Thanks for the update. > > Catalin, did you end up figuring out if your TTBR0 stuff was correct? > You'd mentioned you needed to check something about the > implementation? Unfortunately, I checked with the ARM architecture folk. While the trick is probably fine on existing hardware, the architecture allows caching of the TTBCR bits (or their effect) in the TLB. Therefore changing the TTBCR.EPD0 (or A1) to disable TTBR0 page table walks is not guaranteed to have an effect until the TLBs are invalidated. CPU implementations are allowed to rely on this, so we can't safely use it in Linux. The minor side effect is that the PAN may not always work. We could invalidate the TLBs at every PAN change and take a significant performance hit. However, the more serious side effect is that TLB conflicts may happen, leading to aborts (though unlikely on existing hardware but we can't guarantee it for the future). The latter can't be avoided (only reduce) by more TLB invalidation (the proper fix would be MMU disabling). In conclusion, I'm NAK'ing my own patch ;). The alternative is to make TTBR0 point to swapper_pg_dir where there are no user mappings. The difficulty is that such patch won't fit nicely onto the existing uaccess_* macros that rmk added. It also doesn't work well with switch_mm(), we would have to defer the TTBR0 setting until returning to user space (cpu_switch_mm() should no longer be called in check_and_switch_context() as it automatically enables privileged access to user). I haven't had time to look into re-writing the patch. Hopefully sometime in February, unless someone else volunteers to take over. Question for Russell: do we have any guarantees that lowmem doesn't cross a 32-bit boundary? This would simplify the code a bit by preserving the top 32-bit of TTBR0 when changing. But looking at the sanity_check_meminfo(), I couldn't see anything that would guarantee this. Even the virt_to_phys() macro does a carry add for the top 32-bit. -- Catalin
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.