grub-devel.gnu.org archive mirror
 help / color / mirror / Atom feed
From: Ross Lagerwall <rosslagerwall@gmail.com>
To: Andrei Borzenkov <arvidjaar@gmail.com>
Cc: grub-devel@gnu.org
Subject: Re: [PATCH] core/partmap: Add El Torito boot catalog parsing
Date: Mon, 8 Jun 2015 18:31:06 +0100	[thread overview]
Message-ID: <20150608173106.GA9456@hobo.lan> (raw)
In-Reply-To: <20150608195144.69ed708c@opensuse.site>

[-- Attachment #1: Type: text/plain, Size: 1474 bytes --]

On Mon, Jun 08, 2015 at 07:51:44PM +0300, Andrei Borzenkov wrote:
> В Sun,  7 Jun 2015 11:24:46 +0100
> Ross Lagerwall <rosslagerwall@gmail.com> пишет:
> 
> > Add a module, part_eltorito, to allow parsing of the El Torito boot
> > catalog into partitions. This follows the El Torito Bootable CD-ROM
> > Format Specification Version 1.0 and the UEFI Specification 2.5.
> > In cases where the specification is unclear, the code follows the UEFI
> > reference implementation.
> > 
> > This is useful when booting CDs in UEFI mode. Before, GRUB would not be
> > able to use the embedded ESP from which it was executed, so it would
> > have a root and prefix set to the top level of the CD. This could result
> > in subtle configuration bugs, because the same ISO booted from a USB
> > disk (using isohybrid) would have its root and prefix set to the
> > embedded ESP because it can find it through the MBR.
> 
> You can already access ESP on hybrid image using e.g. (cd0,msdos1); why
> is it not enough?

This is about booting a CD in UEFI mode; it may not be a hybrid disc in
which case it doesn't have an MBR.

Also, in the case of a hybrid image booted using a CD, the mismatch between
the 2048 byte sectors of the CD and the 512 byte sector values used in the
MBR means that GRUB is not able to access the correctly ESP anyway.

For these reasons, I think it is useful to parse the El Torito boot
catalog.

Regards,
-- 
Ross Lagerwall

[-- Attachment #2: Type: application/pgp-signature, Size: 213 bytes --]

  reply	other threads:[~2015-06-08 17:31 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-07 10:24 [PATCH] core/partmap: Add El Torito boot catalog parsing Ross Lagerwall
2015-06-08 16:51 ` Andrei Borzenkov
2015-06-08 17:31   ` Ross Lagerwall [this message]
2015-06-08 17:50     ` Andrei Borzenkov
2015-06-08 18:25       ` Ross Lagerwall
2015-06-08 19:01         ` Andrei Borzenkov
2015-06-20 10:16           ` Ross Lagerwall
2015-06-20 10:19             ` Vladimir 'phcoder' Serbinenko
2015-06-20 13:52               ` Thomas Schmitt
2015-06-21 11:28             ` Andrei Borzenkov
2016-02-12 18:31             ` Vladimir 'φ-coder/phcoder' Serbinenko
2015-06-08 19:31         ` Thomas Schmitt

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=20150608173106.GA9456@hobo.lan \
    --to=rosslagerwall@gmail.com \
    --cc=arvidjaar@gmail.com \
    --cc=grub-devel@gnu.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).