From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp02.citrix.com ([66.165.176.63]:50809 "EHLO SMTP02.CITRIX.COM" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752316AbbGaJlM (ORCPT ); Fri, 31 Jul 2015 05:41:12 -0400 Message-ID: <55BB42B3.1040103@citrix.com> Date: Fri, 31 Jul 2015 10:41:07 +0100 From: David Vrabel MIME-Version: 1.0 To: Andy Lutomirski , X86 ML , "Borislav Petkov" , "linux-kernel@vger.kernel.org" CC: "security@kernel.org" , Peter Zijlstra , Andrew Cooper , "Steven Rostedt" , xen-devel , , Jan Beulich , Sasha Levin , Boris Ostrovsky Subject: Re: [Xen-devel] [PATCH v6 1/4] x86/xen: Probe target addresses in set_aliased_prot before the hypercall References: <0b0e55b995cda11e7829f140b833ef932fcabe3a.1438291540.git.luto@kernel.org> In-Reply-To: <0b0e55b995cda11e7829f140b833ef932fcabe3a.1438291540.git.luto@kernel.org> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: stable-owner@vger.kernel.org List-ID: On 30/07/15 22:31, Andy Lutomirski wrote: > The update_va_mapping hypercall can fail if the VA isn't present in > the guest's page tables. Under certain loads, this can result in an > OOPS when the target address is in unpopulated vmap space. > > While we're at it, add comments to help explain what's going on. > > This isn't a great long-term fix. This code should probably be > changed to use something like set_memory_ro. Reviewed-by: David Vrabel Thanks. David