Date: Fri, 26 Jan 2018 19:07:21 +0000 From: Alan Cox <gnomes@...rguk.ukuu.org.uk> To: "Jason A. Donenfeld" <Jason@...c4.com> Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>, LKML <linux-kernel@...r.kernel.org>, kernel-hardening@...ts.openwall.com Subject: Re: Re: [PATCH v2] cpu: do not leak vulnerabilities to unprivileged users On Fri, 26 Jan 2018 18:47:28 +0100 "Jason A. Donenfeld" <Jason@...c4.com> wrote: > On Fri, Jan 26, 2018 at 5:43 PM, Alan Cox <gnomes@...rguk.ukuu.org.uk> wrote: > > a) The info is already trivially accessible via /proc/cpuinfo > > No, /proc/cpuinfo shows if the CPU itself has these bugs, but doesn't > show whether or not the kernel has gone to lengths to mitigate these > bugs. It tells you immediately what microcode, what hardware properties. A few user space instructions later and you know the rest. > Right, so without this, an attacker has to measure. The purpose of > this patchset is to require the attacker to perform an additional > measurement. That seems worthwhile, especially if measurements are or > would ever become non-trivial to make. You mean 'I'm magically going to make it secure because the attacker won't be smart enough to paste in a function from gitub' ? I don't buy it. Most attack tools are automated, they will contain the needed code. > > b) Some JIT and other environments need to know > > Shouldn't JITs do the best they can with the environment they're in? > And for that, isn't /proc/cpuinfo enough? No The JIT needs to know whether the mitigations are present because it needs to JIT very different code if it is responsible for generating things like lfence and and/mask references or can assume the CPU is covering some or part of it. Likewise you can imagine other code using those files in order to figure out how to self patch / which library to load for performance critical stuff. Alan
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.