From: Christian Franke <Christian.Franke@t-online.de>
To: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: [PATCH] Ensure boot CD can be accessed
Date: Mon, 03 Mar 2008 21:46:36 +0100 [thread overview]
Message-ID: <47CC63AC.5060204@t-online.de> (raw)
In-Reply-To: <ca0f59980803020042l6eb6a346i88c38a7a958fdb0f@mail.gmail.com>
Bean wrote:
> On Sat, Feb 16, 2008 at 5:13 AM, Christian Franke
> <...> wrote:
>
>> ...
>> Further testing outdated this patch. The boot CD appeared under the
>> following drive numbers:
>>
>> 0x82 (old Toshiba Laptop with unknown BIOS)
>> 0x9f (PC with Phoenix-Award BIOS, VMware)
>> 0xe0 (VirtualBox)
>> 0xef (PC with AMI BIOS, VirtualPC)
>>
>> Current CVS would only work for 0xe0, with the previous patch it works
>> for all >= 0x9f.
>>
>> This new patch also handles the case that the CD appears under the
>> legacy controller drive number (0x82 = secondary master). It is accessed
>> as "hd2" in this case.
>>
>> Christian
>>
>> ...
>>
>
> I'm thinking, is there a more generic way to test for cd drive ? also,
> if the cd drive can appear as (hd2), maybe we can use (hdx) for other
> cases as well, such as (hd96), etc. The name might be a little
> strange, but we can get the bios drive number quite easily, which can
> be useful sometimes (some boot loader need to know the boot drive
> number).
>
>
Good point.
Besides the heuristics "(boot drive >= 0x9f) => (likely a boot CD)",
there is apparently no clean method to distinguish CD from HDD on the
legacy BIOS call level.
Is the special handling for CD in grub_biosdisk_rw() necessary, or can
it be generalized to work with both CD and HDD? In the latter case, no
actual CD detection is necessary.
So is would be probably better to remove the (cdN) device names and
accept funny names like e.g. (hd111) for boot CD 0xef. It also would be
easier to track down problems when the boot device number is no longer
hidden behind (cd0).
BTW: None of the BIOSes I tested provided access to a second (non-boot)
CD drive.
Christian
next prev parent reply other threads:[~2008-03-03 20:46 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-10 12:23 [PATCH] Ensure boot CD can be accessed Christian Franke
2008-02-10 12:37 ` Bean
2008-02-10 12:47 ` Christian Franke
2008-02-10 12:56 ` Bean
2008-02-10 14:42 ` Christian Franke
2008-02-15 21:13 ` Christian Franke
2008-03-02 8:42 ` Bean
2008-03-03 20:46 ` Christian Franke [this message]
2008-03-05 7:05 ` Bean
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=47CC63AC.5060204@t-online.de \
--to=christian.franke@t-online.de \
--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.