From: David Daney <ddaney.cavm@gmail.com>
To: Sanjay Lal <sanjayl@kymasys.com>
Cc: kvm@vger.kernel.org, linux-mips@linux-mips.org, gleb@redhat.com,
mtosatti@redhat.com, ralf@linux-mips.org
Subject: Re: [PATCH 0/2] KVM/MIPS32: Fixes for Linux 3.10
Date: Mon, 13 May 2013 14:07:48 -0700 [thread overview]
Message-ID: <51915624.4010104@gmail.com> (raw)
In-Reply-To: <1368476500-20031-1-git-send-email-sanjayl@kymasys.com>
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
>
next prev parent reply other threads:[~2013-05-13 21:07 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-13 20:21 [PATCH 0/2] KVM/MIPS32: Fixes for Linux 3.10 Sanjay Lal
2013-05-13 20:21 ` [PATCH 1/2] KVM/MIPS32: Move include/asm/kvm.h => include/uapi/asm/kvm.h since it is a user visible API Sanjay Lal
2013-05-13 20:21 ` [PATCH 2/2] KVM/MIPS32: Wrap calls to gfn_to_pfn() with srcu_read_lock/unlock() Sanjay Lal
2013-05-14 9:27 ` Gleb Natapov
2013-05-15 15:54 ` Sanjay Lal
2013-05-15 16:54 ` David Daney
2013-05-15 17:30 ` Gleb Natapov
2013-05-15 18:36 ` Sanjay Lal
2013-05-16 5:53 ` Gleb Natapov
2013-05-13 21:07 ` David Daney [this message]
2013-05-13 22:36 ` [PATCH 0/2] KVM/MIPS32: Fixes for Linux 3.10 Sanjay Lal
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51915624.4010104@gmail.com \
--to=ddaney.cavm@gmail.com \
--cc=gleb@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=linux-mips@linux-mips.org \
--cc=mtosatti@redhat.com \
--cc=ralf@linux-mips.org \
--cc=sanjayl@kymasys.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox