From: Farhan Ali <alifm@linux.vnet.ibm.com>
To: Thomas Huth <thuth@redhat.com>,
Christian Borntraeger <borntraeger@de.ibm.com>
Cc: qemu-devel@nongnu.org, Cornelia Huck <cornelia.huck@de.ibm.com>,
Jens Freimann <jfrei@linux.vnet.ibm.com>,
Viktor Mihajlovski <mihajlov@linux.vnet.ibm.com>,
Alexander Graf <agraf@suse.de>
Subject: Re: [Qemu-devel] [PATCH 1/5] elf-loader: Allow late loading of elf
Date: Fri, 24 Feb 2017 09:21:58 -0500 [thread overview]
Message-ID: <35df55f3-03f3-c370-05e7-ea83a1b21f99@linux.vnet.ibm.com> (raw)
In-Reply-To: <aade1638-9dd8-98e7-bf40-f12ab9322da6@redhat.com>
On 02/24/2017 05:44 AM, Thomas Huth wrote:
> On 21.02.2017 11:23, Christian Borntraeger wrote:
>> On 02/20/2017 04:33 PM, Thomas Huth wrote:
>>> On 20.02.2017 15:19, Cornelia Huck wrote:
>>>> From: Farhan Ali <alifm@linux.vnet.ibm.com>
>>>>
>>>> The current QEMU ROM infrastructure rejects late loading of ROMs.
>>>> And ELFs are currently loaded as ROM, this prevents delayed loading
>>>> of ELFs. So when loading ELF, allow the user to specify if ELF should
>>>> be loaded as ROM or not.
>>>>
>>>> If an ELF is not loaded as ROM, then they are not restored on a
>>>> guest reboot/reset and so its upto the user to handle the reloading.
>>>
>>> Could you maybe also explain here why you need such a delayed ELF
>>> loading? Why can't you load the s390-netboot.img at the same time as
>>> s390-ccw.img?
>>
>> Please read the cover letter for some details how to build such a netrom.
>
> Sure, understood, but I still did not see an explanation why this can't
> be loaded as "ROM", too / why it needs to be loaded "delayed"? Does the
> image data need to be writable in memory? Or is the information not
> available yet at that point in time, whether the user wants to do a
> network boot or not? Don't get me wrong, I'm basically fine with this
> patch, I'm just missing some explanation *why* you have to do it this way.
>
>> Long term we certainly want to have a look at implementing something in
>> the ccw bios, but this (tcp stack, virtio net, etc) will take some (a lot?)
>> more time.
>
> Just an idea: There's a complete TFTP-boot-over-virtio-net stack in
> SLOF, written by IBM ... maybe you could use that for s390x, too?
>
>> This patch (1) has another advantage.
>> Right now we load the ccw bios all the time, even for -kernel xxx boot,
>> to allow the guest user to use chreipl to reboot from a disk. With this
>> in place we can rework our ccw bios loader to load the ccw bios lazily
>> as well.
>
> But this "RAM"-loader here has still the problem that the images are not
> restored during reboot/reset, right? ... so before you're going to
> implement that as well, let me ask another question: Did you already
> have a look at the generic loader device? I think that might already
> provide what you're trying to implement here, so your problem could
> maybe be solved with some few lines of creating an instance of that
> device instead (see
> https://lists.gnu.org/archive/html/qemu-devel/2017-01/msg05366.html for
> an example how to use that device).
>
> Thomas
>
Hi Thomas,
I did think of the generic loader device, but doesn't the generic loader
device also load ELF's as ROMs? This would again bring us back to the
problem of ROMs cannot be loaded after initializations of the ROM. So if
a user wants to switch boot device from disk to network device we still
wouldn't be able to load the ROM?
Thanks
Farhan
next prev parent reply other threads:[~2017-02-24 14:22 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-20 14:19 [Qemu-devel] [PATCH 0/5] s390x: network boot Cornelia Huck
2017-02-20 14:19 ` [Qemu-devel] [PATCH 1/5] elf-loader: Allow late loading of elf Cornelia Huck
2017-02-20 15:33 ` Thomas Huth
2017-02-21 10:13 ` Cornelia Huck
2017-02-21 10:23 ` Christian Borntraeger
2017-02-24 10:44 ` Thomas Huth
2017-02-24 11:09 ` Christian Borntraeger
2017-02-24 11:13 ` Thomas Huth
2017-02-24 14:21 ` Farhan Ali [this message]
2017-02-24 18:23 ` Thomas Huth
2017-02-20 14:19 ` [Qemu-devel] [PATCH 2/5] s390x/ipl: Extend S390IPLState to support network boot Cornelia Huck
2017-02-20 14:19 ` [Qemu-devel] [PATCH 3/5] s390x/ipl: Load network boot image Cornelia Huck
2017-02-20 15:28 ` Thomas Huth
2017-02-22 15:01 ` Farhan Ali
2017-02-24 10:11 ` Thomas Huth
2017-02-24 11:15 ` Christian Borntraeger
2017-02-24 14:24 ` Farhan Ali
2017-02-20 14:19 ` [Qemu-devel] [PATCH 4/5] pc-bios/s390-ccw: Use the ccw bios to start the network boot Cornelia Huck
2017-02-20 14:19 ` [Qemu-devel] [PATCH 5/5] pc-bios/s390-ccw.img: rebuild image Cornelia Huck
2017-02-20 15:43 ` [Qemu-devel] [PATCH 0/5] s390x: network boot Thomas Huth
2017-02-20 16:01 ` Alexander Graf
2017-02-21 13:35 ` Viktor Mihajlovski
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=35df55f3-03f3-c370-05e7-ea83a1b21f99@linux.vnet.ibm.com \
--to=alifm@linux.vnet.ibm.com \
--cc=agraf@suse.de \
--cc=borntraeger@de.ibm.com \
--cc=cornelia.huck@de.ibm.com \
--cc=jfrei@linux.vnet.ibm.com \
--cc=mihajlov@linux.vnet.ibm.com \
--cc=qemu-devel@nongnu.org \
--cc=thuth@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).