Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759616AbYHDScg (ORCPT ); Mon, 4 Aug 2008 14:32:36 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1757738AbYHDSbu (ORCPT ); Mon, 4 Aug 2008 14:31:50 -0400 Received: from e36.co.us.ibm.com ([32.97.110.154]:35463 "EHLO e36.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759315AbYHDSbp (ORCPT ); Mon, 4 Aug 2008 14:31:45 -0400 From: Dave Hansen To: Avi Kivity Cc: kvm-devel , Anthony Liguori , linux-kernel@vger.kernel.org, Dave Hansen Subject: [PATCH 3/4] reduce stack usage in kvm_arch_vcpu_ioctl() Date: Mon, 4 Aug 2008 11:31:36 -0700 Message-Id: <1217874696-28459-1-git-send-email-dave@linux.vnet.ibm.com> X-Mailer: git-send-email 1.6.0.rc1.34.g0fe8c1 In-Reply-To: <1217874693-28398-1-git-send-email-dave@linux.vnet.ibm.com> References: <1217874693-28398-1-git-send-email-dave@linux.vnet.ibm.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1855 Lines: 67 This time it is kvm_arch_vcpu_ioctl(). Use dynamic allocations to reduce its stack usage. Signed-off-by: Dave Hansen --- arch/x86/kvm/x86.c | 20 +++++++++++++------- 1 files changed, 13 insertions(+), 7 deletions(-) diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c index 9d77da1..3bbd123 100644 --- a/arch/x86/kvm/x86.c +++ b/arch/x86/kvm/x86.c @@ -1303,13 +1303,16 @@ long kvm_arch_vcpu_ioctl(struct file *filp, struct kvm_vcpu *vcpu = filp->private_data; void __user *argp = (void __user *)arg; int r; + struct kvm_lapic_state *lapic = NULL; switch (ioctl) { case KVM_GET_LAPIC: { - struct kvm_lapic_state lapic; + lapic = kzalloc(sizeof(struct kvm_lapic_state), GFP_KERNEL); - memset(&lapic, 0, sizeof lapic); - r = kvm_vcpu_ioctl_get_lapic(vcpu, &lapic); + r = -ENOMEM; + if (!lapic) + goto out; + r = kvm_vcpu_ioctl_get_lapic(vcpu, lapic); if (r) goto out; r = -EFAULT; @@ -1319,12 +1322,14 @@ long kvm_arch_vcpu_ioctl(struct file *filp, break; } case KVM_SET_LAPIC: { - struct kvm_lapic_state lapic; - + lapic = kmalloc(sizeof(struct kvm_lapic_state), GFP_KERNEL); + r = -ENOMEM; + if (!lapic) + goto out; r = -EFAULT; - if (copy_from_user(&lapic, argp, sizeof lapic)) + if (copy_from_user(lapic, argp, sizeof(struct kvm_lapic_state))) goto out; - r = kvm_vcpu_ioctl_set_lapic(vcpu, &lapic);; + r = kvm_vcpu_ioctl_set_lapic(vcpu, lapic); if (r) goto out; r = 0; @@ -1422,6 +1427,7 @@ long kvm_arch_vcpu_ioctl(struct file *filp, r = -EINVAL; } out: + kfree(lapic); return r; } -- 1.5.4.3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/