From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takuya Yoshikawa Subject: Re: [RFC PATCH 3/3] KVM: MMU: Optimize guest page table walk Date: Fri, 29 Apr 2011 14:38:08 +0900 Message-ID: <20110429143808.29c51c6a.takuya.yoshikawa@gmail.com> References: <20110419033220.e527bcae.takuya.yoshikawa@gmail.com> <20110419033814.3cc7ab5e.takuya.yoshikawa@gmail.com> <4DAEA123.3020403@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Avi Kivity , mtosatti@redhat.com, kvm@vger.kernel.org, yoshikawa.takuya@oss.ntt.co.jp To: Andi Kleen Return-path: Received: from mail-pw0-f46.google.com ([209.85.160.46]:44025 "EHLO mail-pw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751309Ab1D2FiN (ORCPT ); Fri, 29 Apr 2011 01:38:13 -0400 Received: by pwi15 with SMTP id 15so1635601pwi.19 for ; Thu, 28 Apr 2011 22:38:13 -0700 (PDT) In-Reply-To: Sender: kvm-owner@vger.kernel.org List-ID: On Thu, 28 Apr 2011 19:46:00 -0700 Andi Kleen wrote: > Avi Kivity writes: > > > > Good optimization. copy_from_user() really isn't optimized for short > > buffers, I expect much of the improvement comes from that. > > Actually it is equivalent to get_user for the lenghts supported by > get_user, assuming you pass in a constant length. You probably do not. > > -Andi Weird, I actually measured some changes even after dropping other parts than get_user() usage. Only I can guess for that reason is the reduction of some function calls by inlining some functions. Takuya