linux-mips
[Top] [All Lists]

Re: [PATCH 0/2] KVM/MIPS32: Fixes for Linux 3.10

To: Sanjay Lal <sanjayl@kymasys.com>
Subject: Re: [PATCH 0/2] KVM/MIPS32: Fixes for Linux 3.10
From: David Daney <ddaney.cavm@gmail.com>
Date: Mon, 13 May 2013 14:07:48 -0700
Cc: kvm@vger.kernel.org, linux-mips@linux-mips.org, gleb@redhat.com, mtosatti@redhat.com, ralf@linux-mips.org
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:message-id:date:from:user-agent:mime-version:to:cc :subject:references:in-reply-to:content-type :content-transfer-encoding; bh=xlcaDKOT+vKvai60lkOax1IXhQmSviCZrcHgOmSxXYw=; b=S4zThYEwOG/o6GyAro5fWzvEwazSnyuyZxBZsF/DgZmGdWYNiQOl2Pkfh0hDUX+cau eWZHoNnWJgzo6hNg1+KB0KKnV/J/sjUoE7S37SAIpP/PCxsmLNEA7/LyHgoDlV7Bam3V z61njwMC8r2Iw3+TK/BH+L53kFkUrugWHH92E9393fm3U2wSTnlhqeEnM4DnD0EdZFzb Qic1u/5jQpWNLKs/oYYxUqFySXFeY2JoTjAQpcxtrm2258j2Y+O3u8hATkm69tnzlkWi T+NffQKvjM3HaUUpen+ZUUA4C3NczKsJKzQUCj+i18AWK3r/CU+GaP3ZbrmGChVBDbFi je5g==
In-reply-to: <1368476500-20031-1-git-send-email-sanjayl@kymasys.com>
List-archive: <http://www.linux-mips.org/archives/linux-mips/>
List-help: <mailto:ecartis@linux-mips.org?Subject=help>
List-id: linux-mips <linux-mips.eddie.linux-mips.org>
List-owner: <mailto:ralf@linux-mips.org>
List-post: <mailto:linux-mips@linux-mips.org>
List-software: Ecartis version 1.0.0
List-subscribe: <mailto:ecartis@linux-mips.org?subject=subscribe%20linux-mips>
List-unsubscribe: <mailto:ecartis@linux-mips.org?subject=unsubscribe%20linux-mips>
Original-recipient: rfc822;linux-mips@linux-mips.org
References: <n> <1368476500-20031-1-git-send-email-sanjayl@kymasys.com>
Sender: linux-mips-bounce@linux-mips.org
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130311 Thunderbird/17.0.4
On 05/13/2013 01:21 PM, Sanjay Lal wrote:
The following patch set fixes a couple of issues with KVM/MIPS32 in Linux 3.10.

- As suggested by Gleb, wrap calls to gfn_to_pfn() with srcu_read_lock/unlock().
- kvm_mips_map_page() now returns an error code to it's callers, instead of 
calling panic()
   if it cannot find a mapping for a particular gfn.
- Follow the latest convention and move the kvm.h API file under uapi/...


Sanjay,

Have you looked at:

http://www.linux-mips.org/archives/linux-mips/2013-05/msg00049.html

We should start working toward unifying the KVM interface.

David Daney.


--
Sanjay Lal (2):
   KVM/MIPS32: Move include/asm/kvm.h => include/uapi/asm/kvm.h since it
     is a user visible API.
   KVM/MIPS32: Wrap calls to gfn_to_pfn() with srcu_read_lock/unlock()

  arch/mips/include/asm/kvm.h      | 55 ----------------------------------------
  arch/mips/include/uapi/asm/kvm.h | 55 ++++++++++++++++++++++++++++++++++++++++
  arch/mips/kvm/kvm_tlb.c          | 38 ++++++++++++++++++++-------
  3 files changed, 84 insertions(+), 64 deletions(-)
  delete mode 100644 arch/mips/include/asm/kvm.h
  create mode 100644 arch/mips/include/uapi/asm/kvm.h



<Prev in Thread] Current Thread [Next in Thread>