All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andi Kleen <andi@firstfloor.org>
To: ebiederm@xmission.com (Eric W. Biederman)
Cc: "Huang, Ying" <ying.huang@intel.com>,
	akpm@linux-foundation.org, Yinghai Lu <yhlu.kernel@gmail.com>,
	Randy Dunlap <randy.dunlap@oracle.com>,
	Chandramouli Narayanan <mouli@linux.intel.com>,
	linux-kernel@vger.kernel.org, "Mao, Bibo" <bibo.mao@intel.com>
Subject: Re: [PATCH 0/5] x86_64 EFI support -v3
Date: 08 Aug 2007 22:41:43 +0200	[thread overview]
Message-ID: <p73y7glg3nc.fsf@bingen.suse.de> (raw)
In-Reply-To: <m1d4xyc6v2.fsf@ebiederm.dsl.xmission.com>

ebiederm@xmission.com (Eric W. Biederman) writes:
> 
> Since there are people actively investigating things like booting
> OpenBSD via kexec things get even worse. Nothing hardly runs
> on ia64 so that issue doesn't come up.

If you want to do a popularity contest I expect there are far more ia64 
linux users than kexec-of-openbsd users.

> As for not using EFI at all.  If we can avoid it/not use it in the
> dump kernel there is very little point in having it in the primary
> kernel.

One interesting area is to use it for saving oops data. But
that has to be simple. I'm not sure complicated context switches
are a good idea here. 

However I agree it probably doesn't make sense to do virtual
mode just for the clock services -- so far we seem to be fine
just talking to the hardware directly.

> So far there don't seem to be any compelling advantages to running
> EFI in virtual address mode and several compelling disadvantages
> included having to change the permissions on the kernels memory
> map to running EFI in virtual mode.

I don't think it's a big issue to have a few less NX bits. Just
the original patch for it was ugly.
 
> Please let's stick to a physical mode trampoline and only revisit
> the topic when users start having problems because of the performance
> hit of going through our trampoline to the EFI runtime services.

So you want to switch to new page tables when calling EFI services
after boot? 

Potential problems:
- Interrupts have to be disabled. Is that ok? 
- When EFI BIOS start crashing how do we set up exception handlers
for this? 

I guess it would get complex long term. Also doesn't really sound 
attractive.

-Andi

      reply	other threads:[~2007-08-08 19:47 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-31  3:12 [PATCH 0/5] x86_64 EFI support -v3 Huang, Ying
2007-07-31  4:16 ` Eric W. Biederman
2007-07-31  8:55   ` Huang, Ying
2007-08-01 17:21     ` Eric W. Biederman
2007-07-31  4:47 ` Eric W. Biederman
2007-08-06  5:40   ` Huang, Ying
2007-08-08 16:45     ` Eric W. Biederman
2007-08-08 20:41       ` Andi Kleen [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=p73y7glg3nc.fsf@bingen.suse.de \
    --to=andi@firstfloor.org \
    --cc=akpm@linux-foundation.org \
    --cc=bibo.mao@intel.com \
    --cc=ebiederm@xmission.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mouli@linux.intel.com \
    --cc=randy.dunlap@oracle.com \
    --cc=yhlu.kernel@gmail.com \
    --cc=ying.huang@intel.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.