From: Pavel Roskin <proski@gnu.org>
To: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: contradiction in boot/i386/pc/boot.S
Date: Tue, 08 Jan 2008 18:52:36 -0500 [thread overview]
Message-ID: <1199836356.21357.23.camel@dv> (raw)
In-Reply-To: <20080108233718.GB10662@thorin>
On Wed, 2008-01-09 at 00:37 +0100, Robert Millan wrote:
> On Tue, Jan 08, 2008 at 05:57:16PM -0500, Pavel Roskin wrote:
> >
> > We can save 4 more bytes if we do "orb $0x80, %dl" unconditionally for
> > the hard drives. However, I'm not sure if there are BIOSes that pass 1
> > in %dl and how critical it would be if it becomes 0x81.
>
> I think the assumption is that if the BIOS passes 0x01, etc, it's
> garbage and should be ignored completely.
I understand the assumption.
> > That's
> > something I'll rather not do without seeing the original bug reports.
>
> Same here. Maybe Okuji will know..
When I tried to actually change the code, it turned out that
grub-mkimage doesn't change the boot sector, so grub-mkimage needs to be
adjusted, and maybe some other tools.
Alternatively, the logic should not be reverted, i.e. the adjustments
should be done for the hard drive installs. This means that grub-setup
should write valid x86 code over NOPs. It gets too complicated to
bother without having a very good incentive to save those 2 bytes.
--
Regards,
Pavel Roskin
next prev parent reply other threads:[~2008-01-08 23:52 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-08 10:50 contradiction in boot/i386/pc/boot.S Robert Millan
2008-01-08 18:13 ` Pavel Roskin
2008-01-08 18:24 ` Pavel Roskin
2008-01-08 20:24 ` Robert Millan
2008-01-08 22:57 ` Pavel Roskin
2008-01-08 23:37 ` Robert Millan
2008-01-08 23:48 ` Vincent Pelletier
2008-01-08 23:51 ` Vincent Pelletier
2008-01-08 23:52 ` Pavel Roskin [this message]
2008-01-08 23:57 ` Robert Millan
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=1199836356.21357.23.camel@dv \
--to=proski@gnu.org \
--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 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.