All of lore.kernel.org
 help / color / mirror / Atom feed
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




  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.