public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andres Salomon <dilinger@queued.net>
To: Mitch Bradley <wmb@firmworks.com>
Cc: Yinghai Lu <yhlu.kernel@gmail.com>,
	"H. Peter Anvin" <hpa@zytor.com>,
	"Eric W. Biederman" <ebiederm@xmission.com>,
	Ingo Molnar <mingo@elte.hu>,
	Andrew Morton <akpm@linux-foundation.org>,
	Joseph Fannin <jfannin@gmail.com>,
	linux-kernel@vger.kernel.org, jordan.crouse@amd.com
Subject: Re: [PATCH 1/2] OLPC: Add support for calling into Open Firmware
Date: Mon, 21 Apr 2008 10:24:17 -0400	[thread overview]
Message-ID: <20080421102417.6de71391@ephemeral> (raw)
In-Reply-To: <480C1286.3040307@firmworks.com>

On Sun, 20 Apr 2008 18:05:26 -1000
Mitch Bradley <wmb@firmworks.com> wrote:

> 
> 
> Yinghai Lu wrote:
> > On Sun, Apr 20, 2008 at 8:09 PM, Mitch Bradley <wmb@firmworks.com> wrote:
> >   
> >>
> >>  Yinghai Lu wrote:
> >>
> >>     
> >>> how about changing to ofw_32.c?
> >>>
> >>> YH
> >>>
> >>>
> >>>       
> >>  Is your suggestion to change the filename from "ofw.c" to "ofw_32.c"?  That
> >> seems like a good idea to me.
> >>     
> >
> > Yes.
> >
> > BTW,  why olpc need OFW runtime service?
> > why not just put the info in in ram with some signiture, so
> > kernel/util just need to loot at the table if needed?
> >   
> 
> In SPARC land, at least on SunOS and Solaris, it was very convenient for 
> debugging to interrupt the OS with Stop-A and use OFW to inspect the 
> system state.  That was especially handy for live crash analysis.  Dumps 
> are useful as far as they go, but they often fail to capture detailed 
> I/O device state.
> 
> I was hoping to do that on x86 too.  So far we (OLPC) haven't 
> implemented a sysrq hook to enter OFW, but I haven't given up hope yet.  
> It doesn't cost much to leave OFW around, but once you decide to eject 
> it, you can't easily get it back.
> 

I'm not actually convinced that we *do* want to keep OFW resident in memory,
especially given the memory tricks we need to play.  I also don't actually
like the OFW interface that we.  The debugging aspect of it was a
compelling argument up until a week ago (when kernel debuggers started
finally finding their way into the kernel).

However, until we clean up the promfs stuff, there's no chance of getting
an OFW device tree upstream.


> Apple made the early decision to eject OFW and just keep a device tree 
> table.  That decision was probably due to several factors, including the 
> rather lame state of Apple's first OFW implementation and the complexity 
> of their OS startup process at the time (which included "trampolining" 
> to a 68000 emulator to run their legacy code).  Once they went down that 
> path, the die was cast, and the PowerPC community got used to the "OFW 
> ends up as just a table" idea.
> 
> >   


-- 
Need a kernel or Debian developer?  Contact me, I'm looking for contracts.

  parent reply	other threads:[~2008-04-21 14:21 UTC|newest]

Thread overview: 109+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-18  8:47 2.6.25-mm1 Andrew Morton
2008-04-18 11:26 ` [PATCH] 2.6.25-mm1 - Build Failure with PWRficient onchip memory controller driver Kamalesh Babulal
2008-04-18 13:02 ` StackProtector Oopses - Re: 2.6.25-mm1 Reuben Farrelly
2008-04-18 13:36   ` Ingo Molnar
2008-04-18 13:51     ` Arjan van de Ven
2008-04-18 14:41       ` Reuben Farrelly
2008-04-18 14:49     ` Reuben Farrelly
2008-04-21 15:06       ` Ingo Molnar
2008-04-22  1:48         ` Arjan van de Ven
2008-04-22  2:04           ` Valdis.Kletnieks
2008-04-22  8:34           ` Ingo Molnar
2008-04-22 14:29             ` Arjan van de Ven
2008-04-18 16:40 ` 2.6.25-mm1 (build error: driver core) Randy Dunlap
2008-04-18 16:56   ` Greg KH
2008-04-18 18:38     ` Dan Williams
2008-04-18 16:45 ` 2.6.25-mm1 (build error: trace selftest) Randy Dunlap
2008-04-18 20:14 ` 2.6.25-mm1 Valdis.Kletnieks
2008-04-18 23:09 ` 2.6.25-mm1: orphaned files after build Alexey Dobriyan
2008-04-19  2:13 ` 2.6.25-mm1 Joseph Fannin
2008-04-19  3:02   ` 2.6.25-mm1 Andrew Morton
2008-04-19  4:14     ` 2.6.25-mm1 Dmitry Torokhov
2008-04-19  4:29       ` 2.6.25-mm1 Andrew Morton
2008-04-19  6:33         ` 2.6.25-mm1 Joseph Fannin
2008-04-21 11:07         ` 2.6.25-mm1 Takashi Iwai
2008-04-21 17:44           ` 2.6.25-mm1 (snd-pcsp causes driver conflict) Stas Sergeev
2008-04-22 10:09             ` Takashi Iwai
2008-04-22 17:54               ` Stas Sergeev
2008-04-23  8:55                 ` Takashi Iwai
2008-04-23 14:14                   ` Takashi Iwai
2008-04-21 19:45           ` 2.6.25-mm1 Stas Sergeev
2008-04-21 14:06         ` 2.6.25-mm1 Takashi Iwai
2008-04-21 17:55           ` 2.6.25-mm1 (snd-pcsp doesn't like DEBUG_PAGEALLOC) Stas Sergeev
2008-04-22 10:13             ` Takashi Iwai
2008-04-22 14:01               ` Dmitry Torokhov
2008-04-22 16:42                 ` Stas Sergeev
2008-04-22 18:31               ` Stas Sergeev
2008-04-23  8:49                 ` Takashi Iwai
2008-04-23 14:18                   ` Takashi Iwai
2008-04-23 20:02                   ` Stas Sergeev
2008-04-24  9:40                     ` Takashi Iwai
2008-04-25  3:51                       ` Stas Sergeev
2008-04-25  6:28                         ` Takashi Iwai
2008-04-25 16:45                           ` Stas Sergeev
2008-04-25 16:51                             ` Takashi Iwai
2008-04-25 17:25                               ` Stas Sergeev
2008-05-02 16:44                               ` Takashi Iwai
2008-05-02 16:57                                 ` Stas Sergeev
2008-05-06 10:20                                   ` Takashi Iwai
2008-05-06 16:51                                     ` Stas Sergeev
2008-04-25 18:09                             ` Dmitry Torokhov
2008-04-25 18:31                               ` Stas Sergeev
2008-04-25 18:37                                 ` Dmitry Torokhov
2008-04-19  2:25 ` 2.6.25-mm1 Joseph Fannin
2008-04-19  3:08   ` 2.6.25-mm1 Andrew Morton
2008-04-19  3:10 ` 2.6.25-mm1 Joseph Fannin
2008-04-19  3:29   ` 2.6.25-mm1 Andrew Morton
2008-04-19 13:25     ` 2.6.25-mm1 Andres Salomon
2008-04-19 17:38       ` 2.6.25-mm1 Andrew Morton
2008-04-19 17:50         ` 2.6.25-mm1 Andres Salomon
2008-04-21 14:56           ` 2.6.25-mm1 Jordan Crouse
2008-04-21 15:05             ` 2.6.25-mm1 Andres Salomon
2008-04-21 15:12               ` 2.6.25-mm1 Jordan Crouse
2008-04-19 17:39       ` [PATCH 1/2] OLPC: Add support for calling into Open Firmware Andres Salomon
2008-04-20 10:34         ` Yinghai Lu
2008-04-20 12:07           ` H. Peter Anvin
2008-04-20 17:59             ` Andres Salomon
2008-04-20 18:42               ` Mitch Bradley
2008-04-20 19:12                 ` H. Peter Anvin
2008-04-21  3:39                   ` Mitch Bradley
2008-04-21  4:54                     ` Yinghai Lu
2008-04-21  8:22                       ` Mitch Bradley
2008-04-21 11:36                     ` H. Peter Anvin
2008-04-21 13:09                       ` H. Peter Anvin
2008-04-21 13:13                       ` H. Peter Anvin
2008-04-21 13:19                       ` H. Peter Anvin
2008-04-21 15:05                     ` Jordan Crouse
2008-04-21 14:58                       ` H. Peter Anvin
2008-04-20 19:13               ` [PATCH 1/2] " H. Peter Anvin
2008-04-21  3:09           ` Mitch Bradley
2008-04-21  3:15             ` Yinghai Lu
2008-04-21  4:05               ` Mitch Bradley
2008-04-21  4:26                 ` David Miller
2008-04-21  4:50                 ` Yinghai Lu
2008-04-21  8:03                   ` Mitch Bradley
2008-04-21 14:24                 ` Andres Salomon [this message]
2008-04-21 15:54                   ` David Woodhouse
2008-04-21 16:57                     ` H. Peter Anvin
2008-04-21 18:54                       ` David Woodhouse
2008-04-21 17:03                     ` Andres Salomon
2008-04-21 19:18                       ` David Woodhouse
2008-04-21 19:46                         ` Andres Salomon
2008-04-21 20:25                           ` David Woodhouse
2008-04-21 21:02                             ` [PATCH] OLPC: only check for OFW signature on VSA-less Geodes Andres Salomon
2008-04-21 21:17                               ` Jordan Crouse
2008-04-21 21:17                               ` [PATCH] " David Woodhouse
2008-04-29  3:06                               ` Andrew Morton
2008-04-29  5:32                                 ` [PATCH] x86: GEODE: cache results from geode_has_vsa2() and uninline Andres Salomon
2008-04-29 20:35                                   ` Andrew Morton
2008-04-29 20:57                                     ` Andres Salomon
2008-04-19 17:39       ` [PATCH 2/2] OLPC: drop pre-OpenFirmware workarounds Andres Salomon
2008-04-19 18:21     ` 2.6.25-mm1 Arjan van de Ven
2008-04-20 11:29 ` internal compiler error: SIGSEGV [Was: 2.6.25-mm1] Jiri Slaby
2008-04-21  8:31 ` Jiri Slaby
2008-04-21  9:06   ` Al Viro
2008-04-21  9:37     ` fault in __d_lookup " Jiri Slaby
2008-04-21  9:45       ` Al Viro
2008-04-21  9:59         ` Jiri Slaby
2008-04-21 13:42           ` Rafael J. Wysocki
2008-04-21 17:23           ` Matthew Wilcox

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=20080421102417.6de71391@ephemeral \
    --to=dilinger@queued.net \
    --cc=akpm@linux-foundation.org \
    --cc=ebiederm@xmission.com \
    --cc=hpa@zytor.com \
    --cc=jfannin@gmail.com \
    --cc=jordan.crouse@amd.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=wmb@firmworks.com \
    --cc=yhlu.kernel@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox