All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>
To: The development of GNU GRUB <grub-devel@gnu.org>
Subject: Re: passing options to grub in xen,openfirmware and efi
Date: Thu, 07 May 2015 16:45:08 +0200	[thread overview]
Message-ID: <554B7A74.7010404@gmail.com> (raw)
In-Reply-To: <20150429164921.GA9393@aepfle.de>

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


> Sigh..
> There are likely ways to find out where grub was loaded from even on a
> PC, and use that as initial root=. In a PV guest no such thing exists
> because itself grub is the firmware.
grub-install already takes this into account.

> In OF I can imaging that it might
> be useful to point grub right away to some other device than listed in
> /chosen/bootpath.
>
We don't support loading modules from another version that what was
compiled with kernel. And if you move different parts of GRUB install
after grub-install, then you have only yourself to blame. Think of any
other program: if you start moving its files around it will not work.

>>> And grub2 does not grab the cmdline provided via extra=. I think that
>>> providing root=xen/xvdb is the right way to control grub inside the
>>> domU.
>>> In anycase, what the OF does today in its init code is valid and should
>>> stay.
>> Mixing up namespaces is certainly not valid. This will lead to both intended
>> misuses like changing variables that shouldn't be and unintentional when e.g.
>> root=/dev/xvda2 meant for Linux will sneak into grub breaking stuff
> 
> Since the kernel= is grub and the stuff in cmdline is obviously meant
> for that very kernel (grub), it can have no meaning for an OS that
> possibly loaded later. It does not even know about that string.
> 
The GRUB which is used for kernel= may be programmed to use /vmlinuz and
pass root= to it. merging unrelated namespaces is always
> Olaf
> 
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/grub-devel
> 



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

  reply	other threads:[~2015-05-07 14:45 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-23  8:03 passing options to grub in xen,openfirmware and efi Olaf Hering
2015-04-23  9:35 ` Andrei Borzenkov
2015-04-23  9:43   ` Olaf Hering
2015-04-23  9:54     ` Andrei Borzenkov
2015-04-23 11:25       ` Olaf Hering
2015-04-23 11:40         ` Andrei Borzenkov
2015-04-23 12:17           ` Olaf Hering
2015-04-29 15:02   ` Vladimir 'φ-coder/phcoder' Serbinenko
2015-04-29 15:10     ` Olaf Hering
2015-04-29 15:21       ` Vladimir 'phcoder' Serbinenko
2015-04-29 15:35         ` Olaf Hering
2015-04-29 15:41           ` Vladimir 'phcoder' Serbinenko
2015-04-29 15:48             ` Olaf Hering
2015-04-29 16:22               ` Vladimir 'phcoder' Serbinenko
2015-04-29 16:28                 ` Olaf Hering
2015-04-29 16:37                   ` Vladimir 'phcoder' Serbinenko
2015-04-29 16:49                     ` Olaf Hering
2015-05-07 14:45                       ` Vladimir 'φ-coder/phcoder' Serbinenko [this message]
2015-05-07 14:54                         ` Olaf Hering
2015-05-07 14:59                           ` Vladimir 'φ-coder/phcoder' Serbinenko
2015-05-07 15:03                             ` Andrei Borzenkov
2015-05-07 15:21                               ` Vladimir 'φ-coder/phcoder' Serbinenko
2015-05-08 10:09                                 ` Olaf Hering
2015-05-08 10:57                                   ` Vladimir 'phcoder' Serbinenko
2015-05-08 11:06                                     ` Andrei Borzenkov
2015-05-08 11:17                                       ` Olaf Hering
2015-05-08 11:15                                     ` Olaf Hering
2015-05-08 11:33                                       ` Andrei Borzenkov
2015-05-08 11:46                                         ` Olaf Hering
2015-05-08 12:57                                           ` Vladimir 'phcoder' Serbinenko
2015-05-11  9:36                                         ` Olaf Hering
2015-05-11 10:22                                           ` Andrei Borzenkov
2015-05-11 10:43                                             ` Olaf Hering
2015-05-11 10:46                                               ` Vladimir 'phcoder' Serbinenko
2015-05-11 11:04                                                 ` Olaf Hering
2015-05-11 11:22                                                   ` Vladimir 'phcoder' Serbinenko
2015-05-11 14:10                                 ` Olaf Hering
2015-05-08 10:44                               ` Michael Chang
2015-04-23 17:05 ` Curt Brune

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=554B7A74.7010404@gmail.com \
    --to=phcoder@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 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.