From: Samuel Thibault <samuel.thibault@ens-lyon.org>
To: Andrew Cooper <andrew.cooper3@citrix.com>
Cc: Tim Deegan <tim@xen.org>,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
Keir Fraser <keir@xen.org>, Jan Beulich <JBeulich@suse.com>,
Xen-devel <xen-devel@lists.xen.org>
Subject: Re: [PATCH] x86/minios: Remove more vestigial remnants of SUPERVISOR_MODE_KERNEL
Date: Thu, 15 Jan 2015 20:31:00 +0100 [thread overview]
Message-ID: <20150115193100.GI2981@type> (raw)
In-Reply-To: <1421348867-2660-1-git-send-email-andrew.cooper3@citrix.com>
Andrew Cooper, le Thu 15 Jan 2015 19:07:47 +0000, a écrit :
> MiniOS has never advertised support for supervisor_mode_kernel, and the
> feature was only ever implemented for 32bit Xen.
>
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
> CC: Keir Fraser <keir@xen.org>
> CC: Jan Beulich <JBeulich@suse.com>
> CC: Tim Deegan <tim@xen.org>
> CC: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
> CC: Samuel Thibault <samuel.thibault@ens-lyon.org>
Acked-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
> ---
> extras/mini-os/arch/x86/x86_64.S | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/extras/mini-os/arch/x86/x86_64.S b/extras/mini-os/arch/x86/x86_64.S
> index df3469e..72921b1 100644
> --- a/extras/mini-os/arch/x86/x86_64.S
> +++ b/extras/mini-os/arch/x86/x86_64.S
> @@ -125,13 +125,10 @@ KERNEL_CS_MASK = 0xfc
> testl $NMI_MASK,2*8(%rsp)
> jnz 2f
>
> - testb $1,(xen_features+XENFEAT_supervisor_mode_kernel)
> - jnz 1f
> -
> /* Direct iret to kernel space. Correct CS and SS. */
> orb $3,1*8(%rsp)
> orb $3,4*8(%rsp)
> -1: iretq
> + iretq
>
> 2: /* Slow iret via hypervisor. */
> andl $~NMI_MASK, 16(%rsp)
> --
> 1.7.10.4
>
--
Samuel
> dvips -o $@ $<
Faut faire gffe de pas te couper avec ton truc, t'as mis des ciseaux ($<)
partout :))
-+- Dom in Guide du linuxien pervers - "J'aime pas les Makefile !" -+-
prev parent reply other threads:[~2015-01-15 19:31 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-15 19:07 [PATCH] x86/minios: Remove more vestigial remnants of SUPERVISOR_MODE_KERNEL Andrew Cooper
2015-01-15 19:31 ` Samuel Thibault [this message]
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=20150115193100.GI2981@type \
--to=samuel.thibault@ens-lyon.org \
--cc=JBeulich@suse.com \
--cc=andrew.cooper3@citrix.com \
--cc=keir@xen.org \
--cc=stefano.stabellini@eu.citrix.com \
--cc=tim@xen.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.