From: Matt Fleming <matt@codeblueprint.co.uk>
To: Mark Rutland <mark.rutland@arm.com>
Cc: Juergen Gross <jgross@suse.com>,
sstabellini@kernel.org,
Ard Biesheuvel <ard.biesheuvel@linaro.org>,
Daniel Kiper <daniel.kiper@oracle.com>,
linux-kernel@vger.kernel.org, xen-devel@lists.xen.org,
Julien Grall <julien.grall@arm.com>,
linux-efi@vger.kernel.org,
Boris Ostrovsky <boris.ostrovsky@oracle.com>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] arm64: xen: Implement EFI reset_system callback
Date: Tue, 18 Apr 2017 14:46:50 +0100 [thread overview]
Message-ID: <20170418134650.GL24360__45251.4527285986$1492523266$gmane$org@codeblueprint.co.uk> (raw)
In-Reply-To: <20170406155453.GA3966@leverpostej>
On Thu, 06 Apr, at 04:55:11PM, Mark Rutland wrote:
>
> Please, let's keep the Xen knowledge constrained to the Xen EFI wrapper,
> rather than spreading it further.
>
> IMO, given reset_system is a *mandatory* function, the Xen wrapper
> should provide an implementation.
>
> I don't see why you can't implement a wrapper that calls the usual Xen
> poweroff/reset functions.
I realise I'm making a sweeping generalisation, but adding
EFI_PARAVIRT is almost always the wrong thing to do.
I'd much prefer to see Mark's idea implemented.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
next prev parent reply other threads:[~2017-04-18 13:46 UTC|newest]
Thread overview: 84+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-05 18:14 [PATCH] arm64: xen: Implement EFI reset_system callback Julien Grall
2017-04-05 18:14 ` Julien Grall
2017-04-05 19:49 ` Boris Ostrovsky
2017-04-05 19:49 ` Boris Ostrovsky
2017-04-05 19:49 ` Boris Ostrovsky
2017-04-06 6:23 ` Juergen Gross
2017-04-06 6:23 ` Juergen Gross
2017-04-06 6:23 ` Juergen Gross
2017-04-06 8:32 ` Julien Grall
2017-04-06 8:32 ` Julien Grall
2017-04-06 8:32 ` Julien Grall
2017-04-06 8:37 ` Juergen Gross
2017-04-06 8:37 ` Juergen Gross
2017-04-06 8:37 ` Juergen Gross
2017-04-06 14:27 ` Daniel Kiper
2017-04-06 14:27 ` Daniel Kiper
2017-04-06 14:32 ` Julien Grall
2017-04-06 14:32 ` Julien Grall
2017-04-06 14:37 ` Boris Ostrovsky
2017-04-06 14:37 ` Boris Ostrovsky
2017-04-06 14:37 ` Boris Ostrovsky
2017-04-06 14:32 ` Julien Grall
2017-04-06 14:38 ` Juergen Gross
2017-04-06 14:38 ` Juergen Gross
2017-04-06 15:20 ` Daniel Kiper
2017-04-06 15:20 ` Daniel Kiper
2017-04-06 15:39 ` Julien Grall
2017-04-06 15:39 ` Julien Grall
2017-04-06 15:55 ` Mark Rutland
2017-04-06 15:55 ` Mark Rutland
2017-04-06 15:55 ` Mark Rutland
2017-04-06 15:55 ` Mark Rutland
2017-04-18 13:46 ` Matt Fleming [this message]
2017-04-18 13:46 ` Matt Fleming
2017-04-18 13:46 ` Matt Fleming
2017-04-18 13:46 ` Matt Fleming
2017-04-19 19:29 ` Daniel Kiper
2017-04-19 19:29 ` Daniel Kiper
2017-04-19 19:29 ` Daniel Kiper
2017-04-19 19:29 ` Daniel Kiper
2017-04-19 19:37 ` Matt Fleming
2017-04-19 19:37 ` Matt Fleming
2017-04-19 19:37 ` Matt Fleming
2017-04-19 19:43 ` Daniel Kiper
2017-04-19 19:43 ` Daniel Kiper
2017-04-19 19:43 ` Daniel Kiper
2017-04-19 19:43 ` Daniel Kiper
2017-04-19 19:37 ` Matt Fleming
2017-04-06 16:06 ` Daniel Kiper
2017-04-06 16:06 ` Daniel Kiper
2017-04-06 16:06 ` Daniel Kiper
2017-04-06 16:22 ` Juergen Gross
2017-04-06 16:22 ` Juergen Gross
2017-04-06 16:22 ` Juergen Gross
2017-04-06 16:43 ` Daniel Kiper
2017-04-06 16:43 ` Daniel Kiper
2017-04-06 16:43 ` Daniel Kiper
2017-04-06 17:39 ` Juergen Gross
2017-04-06 17:39 ` Juergen Gross
2017-04-18 18:37 ` Stefano Stabellini
2017-04-18 18:37 ` Stefano Stabellini
2017-04-18 18:37 ` Stefano Stabellini
2017-04-18 18:43 ` Juergen Gross
2017-04-18 18:43 ` Juergen Gross
2017-04-18 18:43 ` Juergen Gross
2017-04-18 18:46 ` Stefano Stabellini
2017-04-18 18:46 ` Stefano Stabellini
2017-04-18 18:51 ` Juergen Gross
2017-04-18 18:51 ` Juergen Gross
2017-04-20 18:09 ` Julien Grall
2017-04-20 18:09 ` Julien Grall
2017-04-20 18:09 ` Julien Grall
2017-04-20 18:09 ` Julien Grall
2017-04-18 18:51 ` Juergen Gross
2017-04-18 18:46 ` Stefano Stabellini
2017-04-06 17:39 ` Juergen Gross
2017-04-06 15:39 ` Julien Grall
2017-04-06 15:20 ` Daniel Kiper
2017-04-06 14:38 ` Juergen Gross
2017-04-06 14:27 ` Daniel Kiper
2017-04-05 21:26 ` Stefano Stabellini
2017-04-05 21:26 ` Stefano Stabellini
2017-04-05 21:26 ` Stefano Stabellini
-- strict thread matches above, loose matches on Subject: below --
2017-04-05 18:14 Julien Grall
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='20170418134650.GL24360__45251.4527285986$1492523266$gmane$org@codeblueprint.co.uk' \
--to=matt@codeblueprint.co.uk \
--cc=ard.biesheuvel@linaro.org \
--cc=boris.ostrovsky@oracle.com \
--cc=daniel.kiper@oracle.com \
--cc=jgross@suse.com \
--cc=julien.grall@arm.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-efi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=sstabellini@kernel.org \
--cc=xen-devel@lists.xen.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.