qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Thomas Huth <thuth@redhat.com>
To: Viktor VM Mihajlovski <mihajlov@linux.vnet.ibm.com>,
	Christian Borntraeger <borntraeger@de.ibm.com>,
	qemu-s390x@nongnu.org
Cc: Collin Walling <walling@linux.ibm.com>,
	Cornelia Huck <cohuck@redhat.com>,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [qemu-s390x] [PATCH v1 for-2.13 3/4] pc-bios/s390-ccw/net: Add support for pxelinux-style config files
Date: Thu, 19 Apr 2018 18:55:34 +0200	[thread overview]
Message-ID: <475ea222-cc03-16e4-0f50-f89cf2bfae6f@redhat.com> (raw)
In-Reply-To: <2d59a139-3bdd-4b4e-882c-5cf845488eda@linux.vnet.ibm.com>

On 19.04.2018 14:40, Viktor VM Mihajlovski wrote:
> On 19.04.2018 10:17, Thomas Huth wrote:
>> On 19.04.2018 09:41, Viktor VM Mihajlovski wrote:
>>> On 18.04.2018 14:31, Thomas Huth wrote:
>>>> Since it is quite cumbersome to manually create a combined kernel with
>>>> initrd image for network booting, we now support loading via pxelinux
>>>> configuration files, too. In these files, the kernel, initrd and command
>>>> line parameters can be specified seperately, and the firmware then takes
>>>> care of glueing everything together in memory after the files have been
>>>> downloaded.
> [...]
>>>> +static int net_try_direct_tftp_load(filename_ip_t *fn_ip)
>>>> +{
>>>> +    int rc;
>>>> +    void *baseaddr = (void *)0x2000;  /* Load right after the low-core */
>>>> +
>>>> +    rc = tftp_load(fn_ip, baseaddr, KERNEL_MAX_SIZE - (long)baseaddr);
>>>> +
>>>> +    if (rc > 0 && rc < sizeof(cfgbuf) - 1) {
>>>> +        /* Check whether it is a configuration file instead of a kernel */
>>> That's interesting because treating the bootfile as pxe-ish config is
>>> what DPM does. Which means that with this change the processor
>>> architecture type 0x1f (Basic) will turn into a superset of 0x20
>>> (Extended).
>>
>> Is there any reference available what "basic" and "extended" exactly
>> mean? I just know that there are these two values registered by you at
>> the IANA:
>>
>> https://www.iana.org/assignments/dhcpv6-parameters/dhcpv6-parameters.xhtml#processor-architecture
>>
>> ... but I haven't seen a description of the differences of these two
>> values yet.
>>
> Yep, the IANA stuff is pretty terse for all architectures. In a
> nutshell: 0x1f is standard DHCP-based boot (bootfile is a single
> executable binary) and 0x20 serves a pxe-style config file instead.

Can 0x20 also still handle binary files, or only pxe-style config files?

> In fact, "processor architecture identifier" is a misnomer, as it
> describes a pair of processor architecture and firmware.

Yeah, I know ... it's even worse nowadays, it's processor + firmware +
TCP/IP protocol (HTTP vs. TFTP) ... we should have properly untangled
that when doing RFC 5970, but it was already hard enough to get that
accepted in the current shape, so we missed to clean up the mess from
RFC 4578 / PXE :-(

> Since the QEMU firmware has two significant extensions now (insfile and
> direct config file loading), it might be worthwhile to add a new
> identifier value (say s390 QEMU PC-BIOS or similar) announcing the new
> capabilities, so that a boot server admin can setup her system accordingly.

Not sure ... too many processory-architecture-type entries might also
rather be too confusing for the users? ... so if 0x20 can also handle
binary files like 0x1f, I'd maybe simply go with 0x20 and mainly promote
pxelinux config files in the documentation?

 Thomas

  reply	other threads:[~2018-04-19 16:55 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-18 12:31 [Qemu-devel] [PATCH v1 for-2.13 0/4] pc-bios/s390-ccw: Network boot improvements Thomas Huth
2018-04-18 12:31 ` [Qemu-devel] [PATCH v1 for-2.13 1/4] pc-bios/s390-ccw/net: Split up net_load() into init, load and uninit parts Thomas Huth
2018-04-18 18:11   ` Farhan Ali
2018-04-19  5:20     ` [Qemu-devel] [qemu-s390x] " Thomas Huth
2018-04-18 12:31 ` [Qemu-devel] [PATCH v1 for-2.13 2/4] pc-bios/s390-ccw/net: Stop virtio-net device before jumping into the OS Thomas Huth
2018-04-19 15:49   ` Christian Borntraeger
2018-04-20  6:31     ` Thomas Huth
2018-04-20  7:25       ` Christian Borntraeger
2018-04-18 12:31 ` [Qemu-devel] [PATCH v1 for-2.13 3/4] pc-bios/s390-ccw/net: Add support for pxelinux-style config files Thomas Huth
2018-04-19  7:41   ` Viktor VM Mihajlovski
2018-04-19  8:17     ` Thomas Huth
2018-04-19 12:40       ` Viktor VM Mihajlovski
2018-04-19 16:55         ` Thomas Huth [this message]
2018-04-20  6:53           ` [Qemu-devel] [qemu-s390x] " Viktor VM Mihajlovski
2018-04-20  7:36             ` Thomas Huth
2018-04-20  7:54               ` Viktor VM Mihajlovski
2018-04-20  8:40                 ` Thomas Huth
2018-04-20 12:11                   ` Viktor VM Mihajlovski
2018-04-18 12:31 ` [Qemu-devel] [PATCH v1 for-2.13 4/4] pc-bios/s390-ccw/net: Add support for .INS " Thomas Huth
2018-04-19  8:02   ` Viktor VM Mihajlovski
2018-04-19  8:20     ` Thomas Huth
2018-04-18 18:21 ` [Qemu-devel] [PATCH v1 for-2.13 0/4] pc-bios/s390-ccw: Network boot improvements Farhan Ali
2018-04-19  5:27   ` [Qemu-devel] [qemu-s390x] " Thomas Huth
2018-04-19 12:03     ` Farhan Ali

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=475ea222-cc03-16e4-0f50-f89cf2bfae6f@redhat.com \
    --to=thuth@redhat.com \
    --cc=borntraeger@de.ibm.com \
    --cc=cohuck@redhat.com \
    --cc=mihajlov@linux.vnet.ibm.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-s390x@nongnu.org \
    --cc=walling@linux.ibm.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).