All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Radim Krčmář" <rkrcmar@redhat.com>
To: Phil White <pwhite@mvista.com>
Cc: kvm@vger.kernel.org
Subject: Re: [PATCH] KVM: Introduce dynamically registered hypercall capability
Date: Thu, 27 Nov 2014 16:31:29 +0100	[thread overview]
Message-ID: <20141127153128.GD8120@potion.brq.redhat.com> (raw)
In-Reply-To: <CAEV8qvsm+6EX8HqqWcEaZ7PG4t5xd_AxLUjm0CYmUWvQKm49+Q@mail.gmail.com>

2014-11-27 05:30-0800, Phil White:
> This introduces a list of entries which associate a function pointer of
> kvm_hc_type to a hypercall number and allows the ability to register and
> unregister entries.  In addition, it also allows the ability to retrieve a
> function pointer of kvm_hc_type for a given hypercall number which is meant
> to be called from the arch-specific section.
> 
> The main intent is to allow modules to register hypercalls which they own
> rather than requiring the addition of a stub of some sort.  It will also
> allow each arch to maintain separate lists of hypercalls rather than having
> to respect changes in include/uapi/linux/kvm_para.h
> 
> Signed-off-by: Phil White <pwhite@mvista.com>
> ---

Apart from other problems,
how are guests going to use these hypercalls?

(If hc_nr is dynamic, a guest doesn't know its number and even if it is
 static, someone could have registered it beforehand => this needs some
 kind of synchronization with host modules.  A hardcoded reservation?)

  parent reply	other threads:[~2014-11-27 15:31 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-27 13:30 [PATCH] KVM: Introduce dynamically registered hypercall capability Phil White
2014-11-27 14:35 ` Jan Kiszka
2014-11-27 15:31 ` Radim Krčmář [this message]
2014-11-29  1:29   ` Phil White
2014-12-01 13:47     ` Radim Krčmář
2014-12-01 23:43       ` Phil White
2014-12-02 16:08         ` Radim Krčmář

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=20141127153128.GD8120@potion.brq.redhat.com \
    --to=rkrcmar@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=pwhite@mvista.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.