All of lore.kernel.org
 help / color / mirror / Atom feed
From: Cornelia Huck <cohuck@redhat.com>
To: Christian Borntraeger <borntraeger@de.ibm.com>
Cc: Thomas Huth <thuth@redhat.com>,
	Janosch Frank <frankja@linux.ibm.com>,
	David Hildenbrand <david@redhat.com>,
	qemu-devel <qemu-devel@nongnu.org>,
	Halil Pasic <pasic@linux.ibm.com>,
	qemu-s390x <qemu-s390x@nongnu.org>
Subject: Re: [PATCH 1/1] s390x/protvirt: allow to IPL secure execution guests with -no-reboot
Date: Thu, 23 Jul 2020 17:05:15 +0200	[thread overview]
Message-ID: <20200723170515.4e2ee21e.cohuck@redhat.com> (raw)
In-Reply-To: <d2d0d040-3fa9-df7b-eb05-8a5906212486@de.ibm.com>

On Tue, 21 Jul 2020 14:29:29 +0200
Christian Borntraeger <borntraeger@de.ibm.com> wrote:

> On 21.07.20 14:25, Janosch Frank wrote:
> > On 7/21/20 12:32 PM, Christian Borntraeger wrote:  
> >> Right now -no-reboot does prevent secure execution guests from running.  
> > 
> > s/-no-reboot/--no-reboot/  
> 
> Actually qemu --help gives the parameters with just one "-"
> 
> 
> Not sure about secure vs protected. Whatever Conny prefers.

The doc seems to talk about "protected virtualization", "protected
mode", and "secure guests". What about (slight rewording):

"s390x/protvirt: allow to IPL secure guests with -no-reboot

Right now, -no-reboot prevents secure guests from running. This is
correct from an implementation point of view, as we have modeled the
transition from non-secure to secure as a program directed IPL. From a
user perspective, this is not the behavior of least surprise.

We should implement the IPL into protected mode similar to the functions
that we use for kdump/kexec. In other words, we do not stop here when
-no-reboot is specified on the command line. Like function 0 or function
1, function 10 is not a classic reboot. For example, it can only be called
once. Before calling it a second time, a real reboot/reset must happen
in-between. So function code 10 is more or less a state transition
reset, but not a "standard" reset or reboot."

I think this is still appropriate for hard freeze.



  reply	other threads:[~2020-07-23 15:06 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-21 10:32 [PATCH 1/1] s390x/protvirt: allow to IPL secure execution guests with -no-reboot Christian Borntraeger
2020-07-21 10:49 ` Viktor Mihajlovski
2020-07-21 10:55 ` David Hildenbrand
2020-07-21 12:25 ` Janosch Frank
2020-07-21 12:29   ` Christian Borntraeger
2020-07-23 15:05     ` Cornelia Huck [this message]
2020-07-23 15:52       ` Christian Borntraeger
2020-07-23 17:15 ` 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=20200723170515.4e2ee21e.cohuck@redhat.com \
    --to=cohuck@redhat.com \
    --cc=borntraeger@de.ibm.com \
    --cc=david@redhat.com \
    --cc=frankja@linux.ibm.com \
    --cc=pasic@linux.ibm.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-s390x@nongnu.org \
    --cc=thuth@redhat.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.