From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: R: Re: [PATCH] KVM: arm/arm64: Allow usercopy to vcpu->arch.ctxt and arm64 debug Date: Mon, 23 Oct 2017 17:06:14 -0400 (EDT) Message-ID: <1024881182.14288556.1508792774084.JavaMail.zimbra@redhat.com> References: <20171020232525.7387-1-pbonzini@redhat.com> <20171021184545.2497-1-christoffer.dall@linaro.org> <20171022074418.GA3805@cbox> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: Christoffer Dall , Christoffer Dall , kvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org, KVM , kernel-hardening@lists.openwall.com, Radim =?utf-8?B?S3LEjW3DocWZ?= , Marc Zyngier To: Kees Cook Return-path: Received: from mx1.redhat.com ([209.132.183.28]:36633 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750835AbdJWVGP (ORCPT ); Mon, 23 Oct 2017 17:06:15 -0400 In-Reply-To: Sender: kvm-owner@vger.kernel.org List-ID: ----- Kees Cook ha scritto: > On Mon, Oct 23, 2017 at 7:14 AM, Paolo Bonzini wrote: > > On 22/10/2017 09:44, Christoffer Dall wrote: > >> However, I think it's much clearer if I > >> rewrite these to use get_user() and put_user(). v2 incoming. > > > > I'd actually prefer if you all do a trivial conversion to > > kvm_init_usercopy to begin with. In fact, we could just change the > > default from "0, 0" to "0, sizeof (kvm_arch_vcpu)" in kvm_init. Any > > other change can be applied after the patches are merged to Linus's > > tree, especially with KVM Forum and the merge window both coming soon. > > > > I'll send a v2 myself later this week. > > Okay, which patches would you like me to carry in the usercopy > whitelisting tree for the coming merge window? v2 of mine, which shall come in the next couple of days. Paolo > > -Kees > > -- > Kees Cook > Pixel Security