linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: David Woodhouse <dwmw2@infradead.org>
To: Grant Likely <grant.likely@secretlab.ca>
Cc: bbrv@genesi-usa.com, Paul Mackerras <paulus@samba.org>,
	Linux PPC DEV <linuxppc-dev@ozlabs.org>
Subject: Re: [PATCH] Probe Efika platform before CHRP.
Date: Sun, 07 Jan 2007 07:13:09 +0800	[thread overview]
Message-ID: <1168125190.24110.67.camel@shinybook.infradead.org> (raw)
In-Reply-To: <528646bc0701061423o270df3dfj9d27d5572840ec79@mail.gmail.com>

On Sat, 2007-01-06 at 15:23 -0700, Grant Likely wrote:
> 2. Dwmw2 says that reading the HD has an off-by-one showstopper bug
> which prevents Linux from reading the partition table (and that it is
> the same bug that was on the pegasos, but is now fixed).

It's an off-by-one in partition table numbering. When we ask for
"hd:0" (the whole disk) we get "hd:1", and so on. It screws yaboot's
partition table scan, because when it asks for the first sector of the
disk it's given the first sector of the first partition.

Something very similar to this already got fixed on Pegasos after I
reported it -- and I didn't think it was _particularly_ far-fetched for
me to expect the Efika firmware to be adapted to the "definitions, whims
and determinations" of the CHRP specification too.

> I have not confirmed this issues as I don't have a working HD on my
> Efika.  However, dwmw2 has stated that he cannot support the Efika in
> Fedora until this issue is resolved. 

"Cannot" is a strong word when you're talking about software -- and is
almost always untrue. I think what I said was "can't sensibly", which
makes a lot more sense.

I've got a locally-hacked yaboot which works around it enough to boot my
system, but I'm not going to get far if I suggest that we ship a
different copy of the boot loader just for Efika "because it's speshul".
Especially when the bug was known so long ago.

I think the workaround on Pegasos was to use Amiga partitions, and to
hack (perhaps even unintentionally) the Amiga partition code in yaboot
to compensate for the brokenness. So the Amiga partition code probably
won't work on CHRP-compatible machines, while the rest of the partition
code won't work on Efika/oldPegasos. I baulked at the idea of continuing
this hack by using Amiga partitions on Efika too, and preferred to put
the thing back on the shelf and wait for it to be fixed -- especially as
I'm away for the rest of the month anyway.

-- 
dwmw2

  reply	other threads:[~2007-01-06 23:12 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-19  6:23 Bug fixes for 2.6.20 Paul Mackerras
2006-12-19  9:54 ` [PATCH] Probe Efika platform before CHRP David Woodhouse
2006-12-19 11:39   ` Benjamin Herrenschmidt
2006-12-19 11:55     ` Sylvain Munaut
2006-12-19 14:29       ` David Woodhouse
2006-12-19 15:46         ` Grant Likely
2006-12-19 19:58         ` Benjamin Herrenschmidt
2006-12-19 22:33           ` Sylvain Munaut
2007-01-02 21:29           ` David Woodhouse
2007-01-02 21:37             ` Benjamin Herrenschmidt
2007-01-02 22:04               ` David Woodhouse
2007-01-02 23:04                 ` Grant Likely
     [not found]                   ` <45A01416.6080401@genesi-usa.com>
2007-01-06 22:23                     ` Grant Likely
2007-01-06 23:13                       ` David Woodhouse [this message]
2007-01-06 23:37                         ` Grant Likely
2007-01-07  2:55                       ` Sylvain Munaut
2007-01-07  9:11                         ` Raquel Velasco and Bill Buck
2007-03-31 13:15                           ` David Woodhouse
2007-03-31 13:18                             ` Raquel Velasco and Bill Buck
2007-03-31 13:21                               ` David Woodhouse
2007-05-09 11:42                               ` Olaf Hering
2007-05-09 11:45                                 ` Raquel Velasco and Bill Buck
2007-01-07 20:09                       ` Matt Sealey
2007-01-07 20:24                         ` Segher Boessenkool
2007-01-07 22:10                           ` Matt Sealey
2007-01-07 22:35                             ` Benjamin Herrenschmidt
2007-01-07 22:35                             ` Sylvain Munaut
2007-01-07 23:04                               ` Segher Boessenkool
2007-01-07 23:07                               ` Matt Sealey
2007-01-07 23:38                                 ` Sylvain Munaut
2007-01-08  0:37                                 ` Benjamin Herrenschmidt
2007-01-07 22:32                           ` Matt Sealey
2007-01-07 22:39                           ` Matt Sealey
2007-01-07 23:27                             ` Segher Boessenkool
2007-01-07 21:16                         ` Benjamin Herrenschmidt
2007-01-08  2:17                         ` David Woodhouse
2007-01-08  9:18                           ` Matt Sealey
2007-01-08 12:52                             ` Sylvain Munaut
2007-01-08 13:16                               ` Sylvain Munaut
2007-01-08 17:25                                 ` Matt Sealey
2007-01-08 15:44                               ` Sylvain Munaut
2007-01-08 17:25                                 ` Matt Sealey
2007-01-08 17:24                               ` Matt Sealey
2007-01-08 17:59                                 ` Sylvain Munaut
2006-12-19 19:02 ` [PATCH] powerpc export rtas_set_slot_reset() Linas Vepstas
2006-12-19 22:08   ` Brian King
2006-12-19 19:06 ` [PATCH] powerpc fixup error message Linas Vepstas
2006-12-19 20:00 ` [PATCH] powerpc initialize pci device channel state Linas Vepstas

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=1168125190.24110.67.camel@shinybook.infradead.org \
    --to=dwmw2@infradead.org \
    --cc=bbrv@genesi-usa.com \
    --cc=grant.likely@secretlab.ca \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=paulus@samba.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).