All of lore.kernel.org
 help / color / mirror / Atom feed
From: Axel Kellermann <axel.kellermann@gmx.de>
To: grub-devel@gnu.org
Cc: Phillip Susi <psusi@cfl.rr.com>
Subject: Re: [PATCH] Skip partitions during OS probing
Date: Mon, 18 Jul 2011 20:06:05 +0200	[thread overview]
Message-ID: <4E24760D.9060400@gmx.de> (raw)
In-Reply-To: <4E1B5B56.50609@cfl.rr.com>

On 07/11/2011 10:21 PM, Phillip Susi wrote:
> Fixing the underlying problem of detecting and ignoring the Win7 system
> partition would render the patch moot.  If it does really still cover a
> use-case to be able to manually specify partitions to ignore, it needs
> to be done with UUIDs or something else more stable.
Just to tie up that loose end, I had a look at the way update-grub
determines the presence of a bootable Win7 partition. Turns out it
utilizes the os-prober package, which basically uses a four step process
to check for a Win7 system:

1. check for presence of file /bootmgr
2. check for presence of directory /boot/
3. check for presence of file /boot/bcd
4. grep OS version string from file /boot/bcd

If all four steps succeed, the partition is classified as bootable and
update-grub writes a boot meny entry.

The Win7 system partition on my machine passes all the checks os-prober
does, but actually booting from that partition fails with an error
message. To solve this, os-prober would have to include some kind of
sanity check whether the bootloader configuration on the examined
partition is set up correctly and really boots the system. I'm not sure
if that is feasible and/or possible at all, and one could also argue
that it's up to the user to fix the faulty boot setup on the partition
and not blame grub/os-prober for detecting a messed up bootloader. :)

Thoughts?

BTW, I still think the patch covers a use-case and its functionality
could be handy.


Thanks,

Axel


      reply	other threads:[~2011-07-18 18:06 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-09 21:43 [PATCH] Skip partitions during OS probing Axel Kellermann
2011-07-11 15:13 ` Phillip Susi
2011-07-11 15:55   ` Axel Kellermann
2011-07-11 18:25     ` Lennart Sorensen
2011-07-11 20:21       ` Axel Kellermann
2011-07-11 20:21     ` Phillip Susi
2011-07-18 18:06       ` Axel Kellermann [this message]

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=4E24760D.9060400@gmx.de \
    --to=axel.kellermann@gmx.de \
    --cc=grub-devel@gnu.org \
    --cc=psusi@cfl.rr.com \
    /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.