All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alper Nebi Yasak <alpernebiyasak@gmail.com>
To: Simon Glass <sjg@chromium.org>
Cc: u-boot@lists.denx.de, Michal Simek <michal.simek@amd.com>,
	Tom Rini <trini@konsulko.com>, Bin Meng <bmeng@tinylab.org>,
	Tuomas Tynkkynen <tuomas.tynkkynen@iki.fi>,
	Bharat Gooty <bharat.gooty@broadcom.com>,
	Stefan Roese <sr@denx.de>, Qu Wenruo <wqu@suse.com>,
	Rayagonda Kokatanur <rayagonda.kokatanur@broadcom.com>,
	Bin Meng <bmeng.cn@gmail.com>,
	Ilias Apalodimas <ilias.apalodimas@linaro.org>,
	Neil Armstrong <neil.armstrong@linaro.org>
Subject: Re: [PATCH v2 4/4] doc: qemu: arm: Add a section on booting Linux distros
Date: Sun, 27 Aug 2023 18:49:41 +0300	[thread overview]
Message-ID: <8add4068-7b05-4f32-8eb7-31366fb396b9@gmail.com> (raw)
In-Reply-To: <CAPnjgZ2d1mJp28UP2+zKC6+2pED_=YdovA_jaNJz2r+C=-jufg@mail.gmail.com>

On 2023-08-15 01:43 +03:00, Simon Glass wrote:
> Hi Alper,
> 
> On Mon, 14 Aug 2023 at 11:40, Alper Nebi Yasak <alpernebiyasak@gmail.com> wrote:
>> I actually want to put the root.img device first so that the VM can boot
>> into the installed system when it reboots, but U-Boot can't find the
>> bootflow on the second drive. I tried e.g. `bootdev list -p; bootflow
>> scan -lab`, but it seems to only ever search the first virtio-blk.
>> However, `eficonfig; bootefi bootmgr` makes it boot somehow.
> 
> Yes, this is annoying.
> 
> The problem is that the scanning is getting so complicated. The
> boot_targets var lists things which can either be a uclass, or a
> uclass with a device number. The currently implementation sees
> "virtio" and moves on to the next thing in boot_targets once it has
> processed one virtio device. That is obviously not correct, but...
> 
> Would it be possible to just drop the 'boot_targets' var? That is what
> is stuffing it up, but we don't actually need it now. The defaults end
> up doing the same thing, apart (perhaps) from the strangeness of
> virtio which can be both a network and a blk device.
> 
> I believe it is possible to do the right thing, but I'll need to
> create a better test mechanism to handle all the cases.

I think some kind of a "priority score" could help -- iterate over
boot_targets first just to generate bootdev priorities, then carry them
over as bootflows are discovered (adjusted for bootmeth order), then use
those scores as the order to boot / to show a sorted menu / to test?

  reply	other threads:[~2023-08-27 15:49 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-08-14 17:39 [PATCH v2 0/4] arm: qemu: Enable Bochs, console buffering, USB keyboard Alper Nebi Yasak
2023-08-14 17:39 ` [PATCH v2 1/4] arm: qemu: Enable Bochs video support Alper Nebi Yasak
2023-08-14 17:39 ` [PATCH v2 2/4] arm: qemu: Enable PRE_CONSOLE_BUFFER Alper Nebi Yasak
2023-08-14 17:39 ` [PATCH v2 3/4] arm: qemu: Enable usb keyboard as an input device Alper Nebi Yasak
2023-08-14 17:39 ` [PATCH v2 4/4] doc: qemu: arm: Add a section on booting Linux distros Alper Nebi Yasak
2023-08-14 22:43   ` Simon Glass
2023-08-27 15:49     ` Alper Nebi Yasak [this message]
2023-08-28 17:54       ` Simon Glass
2023-08-31  9:57         ` Alper Nebi Yasak
2023-08-31 19:01           ` Simon Glass
2023-08-30  7:33       ` Ilias Apalodimas
2023-08-31  9:25         ` Alper Nebi Yasak
2023-09-01  8:08           ` Ilias Apalodimas
2023-08-24 21:44 ` [PATCH v2 0/4] arm: qemu: Enable Bochs, console buffering, USB keyboard Tom Rini

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=8add4068-7b05-4f32-8eb7-31366fb396b9@gmail.com \
    --to=alpernebiyasak@gmail.com \
    --cc=bharat.gooty@broadcom.com \
    --cc=bmeng.cn@gmail.com \
    --cc=bmeng@tinylab.org \
    --cc=ilias.apalodimas@linaro.org \
    --cc=michal.simek@amd.com \
    --cc=neil.armstrong@linaro.org \
    --cc=rayagonda.kokatanur@broadcom.com \
    --cc=sjg@chromium.org \
    --cc=sr@denx.de \
    --cc=trini@konsulko.com \
    --cc=tuomas.tynkkynen@iki.fi \
    --cc=u-boot@lists.denx.de \
    --cc=wqu@suse.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.