All of lore.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: kvm-ppc@vger.kernel.org
Subject: Re: re-writing on powerpc
Date: Mon, 13 Dec 2010 08:45:30 +0000	[thread overview]
Message-ID: <4D05DD2A.1070808@redhat.com> (raw)
In-Reply-To: <FD1A71B048BD854792EE753E0E9F27DE80A7@az33exm20>

On 12/13/2010 10:42 AM, Alexander Graf wrote:
> On 13.12.2010, at 09:35, Avi Kivity wrote:
>
> >  On 12/13/2010 06:45 AM, Yoder Stuart-B08248 wrote:
> >>  Avi/Hollis,
> >>
> >>  Exchanged some emails with Alex on the topic of rewriting on
> >>  powerpc KVM-- the current approach taken by Alex's PV patch is
> >>  to have a guest Linux paravirt  itself, by re-writing certain
> >>  instructions.
> >>
> >>  The downside to this approach (guest side patching) is that every OS
> >>  to be run on KVM has to be modified or dynamically patched.
> >>
> >>  What were the reasons for not going down the path of doing the
> >>  re-writing in the hypervisor?  (Alex couldn't remember the
> >>  specifics).    What about doing it from Qemu?
> >>
> >
> >  Rewriting is dangerous if the guest is unaware of it.  As soon as it is made aware of it, it might as well actually do it in the best way that suits it.
>
> Yeah, let me rephrase my exact memory on this:
>
> If the HV just rewrites instructions in the guest, it behaves different from real hw which is bad. It could potentially break checksumming inside the guest.
>
> If, however, the guest sends a hypercall to the HV saying "please patch me" or there's a flag on creation time to enable patching, I have a hard time finding a reason to do it inside the guest context.
>
> Back when I implemented this, we did however have discussions on exactly that distinction between patching in host or guest space and for some reason I remember that you and Hollis figured that guest patching is superior. I just really can't remember why and couldn't find traces of this in my inbox either :).

The interface is a lot simpler.  The guest decides what to patch and 
where to jump.  A "please patch me" flag needs a ton of documentation on 
what patch means and what the constraints on the guest environment are.

-- 
error compiling committee.c: too many arguments to function


  parent reply	other threads:[~2010-12-13  8:45 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-13  4:45 re-writing on powerpc Yoder Stuart-B08248
2010-12-13  8:35 ` Avi Kivity
2010-12-13  8:42 ` Alexander Graf
2010-12-13  8:45 ` Avi Kivity [this message]
2010-12-13 17:12 ` Hollis Blanchard
2010-12-13 17:15 ` Avi Kivity
2010-12-13 17:17 ` Hollis Blanchard
2010-12-13 19:03 ` Scott Wood
2010-12-13 23:54 ` Alexander Graf
2010-12-14  0:18 ` Scott Wood
2010-12-14  0:24 ` Alexander Graf
2010-12-14  8:40 ` Avi Kivity
2010-12-14  8:42 ` Avi Kivity
2010-12-14  8:48 ` Avi Kivity
2010-12-14  9:08 ` Alexander Graf
2010-12-14 15:45 ` Yoder Stuart-B08248
2010-12-14 15:48 ` Avi Kivity
2010-12-14 16:55 ` Scott Wood
2010-12-14 17:48 ` Alexander Graf
2010-12-14 17:53 ` Hollis Blanchard
2010-12-14 18:37 ` Scott Wood
2010-12-14 18:41 ` Scott Wood
2010-12-14 20:04 ` Scott Wood
2010-12-14 23:00 ` Alexander Graf
2010-12-14 23:17 ` Scott Wood
2010-12-14 23:29 ` Alexander Graf
2010-12-15  0:00 ` Scott Wood
2010-12-15  0:13 ` Alexander Graf
2010-12-15  0:57 ` Andreas Färber
2010-12-15  9:48 ` Avi Kivity
2010-12-15 11:16 ` Sethi Varun-B16395
2010-12-15 11:18 ` Avi Kivity
2010-12-15 11:32 ` Sethi Varun-B16395
2010-12-15 12:25 ` Avi Kivity
2010-12-17 21:59 ` Benjamin Herrenschmidt
2010-12-17 22:00 ` Benjamin Herrenschmidt

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=4D05DD2A.1070808@redhat.com \
    --to=avi@redhat.com \
    --cc=kvm-ppc@vger.kernel.org \
    /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.