All of lore.kernel.org
 help / color / mirror / Atom feed
From: Cornelia Huck <cohuck@redhat.com>
To: Janosch Frank <frankja@linux.ibm.com>
Cc: borntraeger@de.ibm.com, qemu-s390x@nongnu.org,
	qemu-devel@nongnu.org, david@redhat.com
Subject: Re: [PATCH v10 05/16] s390x: protvirt: Inhibit balloon when switching to protected mode
Date: Wed, 18 Mar 2020 18:27:53 +0100	[thread overview]
Message-ID: <20200318182753.4c48d8cb.cohuck@redhat.com> (raw)
In-Reply-To: <20200318143047.2335-6-frankja@linux.ibm.com>

On Wed, 18 Mar 2020 10:30:36 -0400
Janosch Frank <frankja@linux.ibm.com> wrote:

> Ballooning in protected VMs can only be done when the guest shares the
> pages it gives to the host. If pages are not shared, the integrity
> checks will fail once those pages have been altered and are given back
> to the guest.
> 
> As we currently do not yet have a solution for this we will continue
> like this:
> 
> 1. We block ballooning now in QEMU (with this patch).
> 
> 2. Later we will provide a change to virtio that removes the blocker
> and adds VIRTIO_F_IOMMU_PLATFORM automatically by QEMU when doing the
> protvirt switch. This is OK, as the balloon driver in Linux (the only
> supported guest) will refuse to work with the IOMMU_PLATFORM feature
> bit set.
> 
> 3. Later, we can fix the guest balloon driver to accept the IOMMU
> feature bit and correctly exercise sharing and unsharing of balloon
> pages.
> 
> Signed-off-by: Janosch Frank <frankja@linux.ibm.com>
> Reviewed-by: David Hildenbrand <david@redhat.com>
> Reviewed-by: Christian Borntraeger <borntraeger@de.ibm.com>
> Reviewed-by: Claudio Imbrenda <imbrenda@linux.ibm.com>
> ---
>  hw/s390x/s390-virtio-ccw.c | 11 +++++++++++
>  1 file changed, 11 insertions(+)

Reviewed-by: Cornelia Huck <cohuck@redhat.com>



  reply	other threads:[~2020-03-18 17:28 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-18 14:30 [PATCH v10 00/16] s390x: Protected Virtualization support Janosch Frank
2020-03-18 14:30 ` [PATCH v10 01/16] s390x: Move diagnose 308 subcodes and rcs into ipl.h Janosch Frank
2020-03-18 16:42   ` Cornelia Huck
2020-03-18 20:29   ` Christian Borntraeger
2020-03-19 12:42   ` Claudio Imbrenda
2020-03-18 14:30 ` [PATCH v10 02/16] Sync pv Janosch Frank
2020-03-18 14:30 ` [PATCH v10 03/16] s390x: protvirt: Support unpack facility Janosch Frank
2020-03-19 11:08   ` Cornelia Huck
2020-03-19 11:55     ` Janosch Frank
2020-03-19 11:57       ` Christian Borntraeger
2020-03-19 11:57         ` Janosch Frank
2020-03-18 14:30 ` [PATCH v10 04/16] s390x: protvirt: Add migration blocker Janosch Frank
2020-03-18 14:30 ` [PATCH v10 05/16] s390x: protvirt: Inhibit balloon when switching to protected mode Janosch Frank
2020-03-18 17:27   ` Cornelia Huck [this message]
2020-03-18 14:30 ` [PATCH v10 06/16] s390x: protvirt: KVM intercept changes Janosch Frank
2020-03-18 14:30 ` [PATCH v10 07/16] s390x: Add SIDA memory ops Janosch Frank
2020-03-18 14:30 ` [PATCH v10 08/16] s390x: protvirt: Move STSI data over SIDAD Janosch Frank
2020-03-18 17:57   ` Cornelia Huck
2020-03-18 14:30 ` [PATCH v10 09/16] s390x: protvirt: SCLP interpretation Janosch Frank
2020-03-19 12:09   ` Cornelia Huck
2020-03-19 12:46     ` Janosch Frank
2020-03-19 12:50       ` Cornelia Huck
2020-03-18 14:30 ` [PATCH v10 10/16] s390x: protvirt: Set guest IPL PSW Janosch Frank
2020-03-18 18:00   ` Cornelia Huck
2020-03-19  8:19     ` Janosch Frank
2020-03-18 14:30 ` [PATCH v10 11/16] s390x: protvirt: Move diag 308 data over SIDA Janosch Frank
2020-03-19 12:12   ` Cornelia Huck
2020-03-18 14:30 ` [PATCH v10 12/16] s390x: protvirt: Disable address checks for PV guest IO emulation Janosch Frank
2020-03-18 14:30 ` [PATCH v10 13/16] s390x: protvirt: Move IO control structures over SIDA Janosch Frank
2020-03-19 12:23   ` Cornelia Huck
2020-03-19 12:50     ` Janosch Frank
2020-03-18 14:30 ` [PATCH v10 14/16] s390x: protvirt: Handle SIGP store status correctly Janosch Frank
2020-03-18 14:30 ` [PATCH v10 15/16] docs: system: Add protvirt docs Janosch Frank
2020-03-18 14:53   ` Cornelia Huck
2020-03-19  9:04     ` Janosch Frank
2020-03-19  9:59       ` Cornelia Huck
2020-03-18 16:50   ` Christian Borntraeger
2020-03-18 14:30 ` [PATCH v10 16/16] s390x: Add unpack facility feature to GA1 Janosch Frank
2020-03-18 16:43   ` Cornelia Huck

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=20200318182753.4c48d8cb.cohuck@redhat.com \
    --to=cohuck@redhat.com \
    --cc=borntraeger@de.ibm.com \
    --cc=david@redhat.com \
    --cc=frankja@linux.ibm.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-s390x@nongnu.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.