Re: [PATCH 0/1] pc-bios: Support List-Directed IPL from ECKD DASD

Re: [PATCH 0/1] pc-bios: Support List-Directed IPL from ECKD DASD

[Jared Rossi]

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

Hi,

Pinging this patch since v2.26.0 of s390-tools adds ECKD DASD list-directed IPL.
This patch set allows QEMU to utilize that new boot pointer format.
Any feedback about what needs to be changed for acceptance is much appreciated.

Thanks,
Jared Rossi

On 1/30/23 4:24 PM, Jared Rossi wrote:

> Add support for List-Directed IPL (LD-IPL) type pointers.
>
> We check for a boot record indicating LD-IPL and use it if it is valid,
> otherwise we use the CCW CDL or LDL format as usual. When a block is accessed
> during IPL the block number is first calculated based on the cylinder, head,
> and record numbers included in the block pointer; if LD-IPL has been initiated
> then each pointer will be interpreted using the new format.
>
> For simplicity, there is no choice presented to forcibly use CCW-IPL if LD-IPL
> is available.  Because both sets of pointers ultimately go to the same
> kernel/initrd, using CCW- or LD-IPL is transparent to the user.
>
> One aspect of the user experience that does change is the availability of the
> interactive boot menu when a loadparm is not specified.  For the existing
> CCW-IPL, when the user does not specify a loadparm they are presented with a
> list of boot options; however, this list is only written in the old style
> pointers.  Therefore, if no loadparm is specified, and LD-IPL is supported, the
> default boot option will be used automatically.
>
> Jared Rossi (1):
>    pc-bios: Add support for List-Directed IPL from ECKD DASD
>
>   pc-bios/s390-ccw/bootmap.c | 157 ++++++++++++++++++++++++++++---------
>   pc-bios/s390-ccw/bootmap.h |  30 ++++++-
>   2 files changed, 148 insertions(+), 39 deletions(-)
>

[-- Attachment #2: Type: text/html, Size: 1973 bytes --]

Re: [PATCH 0/1] pc-bios: Support List-Directed IPL from ECKD DASD

[Thomas Huth]

On 20/02/2023 16.13, Jared Rossi wrote:
> Hi,
> 
> Pinging this patch since v2.26.0 of s390-tools adds ECKD DASD list-directed IPL.
> This patch set allows QEMU to utilize that new boot pointer format.
> Any feedback about what needs to be changed for acceptance is much appreciated.

Where did you send your patches to? I can't find it in neither my qemu-s390x 
nor my qemu-devel mailing list folders.

  Thomas


> 
> On 1/30/23 4:24 PM, Jared Rossi wrote:
> 
>> Add support for List-Directed IPL (LD-IPL) type pointers.
>>
>> We check for a boot record indicating LD-IPL and use it if it is valid,
>> otherwise we use the CCW CDL or LDL format as usual. When a block is accessed
>> during IPL the block number is first calculated based on the cylinder, head,
>> and record numbers included in the block pointer; if LD-IPL has been initiated
>> then each pointer will be interpreted using the new format.
>>
>> For simplicity, there is no choice presented to forcibly use CCW-IPL if LD-IPL
>> is available.  Because both sets of pointers ultimately go to the same
>> kernel/initrd, using CCW- or LD-IPL is transparent to the user.
>>
>> One aspect of the user experience that does change is the availability of the
>> interactive boot menu when a loadparm is not specified.  For the existing
>> CCW-IPL, when the user does not specify a loadparm they are presented with a
>> list of boot options; however, this list is only written in the old style
>> pointers.  Therefore, if no loadparm is specified, and LD-IPL is supported, the
>> default boot option will be used automatically.
>>
>> Jared Rossi (1):
>>    pc-bios: Add support for List-Directed IPL from ECKD DASD
>>
>>   pc-bios/s390-ccw/bootmap.c | 157 ++++++++++++++++++++++++++++---------
>>   pc-bios/s390-ccw/bootmap.h |  30 ++++++-
>>   2 files changed, 148 insertions(+), 39 deletions(-)
>>

Re: [PATCH 0/1] pc-bios: Support List-Directed IPL from ECKD DASD

[Jared Rossi]

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

On 2/20/23 10:45 AM, Thomas Huth wrote:
> On 20/02/2023 16.13, Jared Rossi wrote:
>> Hi,
>>
>> Pinging this patch since v2.26.0 of s390-tools adds ECKD DASD 
>> list-directed IPL.
>> This patch set allows QEMU to utilize that new boot pointer format.
>> Any feedback about what needs to be changed for acceptance is much 
>> appreciated.
>
> Where did you send your patches to? I can't find it in neither my 
> qemu-s390x nor my qemu-devel mailing list folders.
>
>  Thomas

Hi Thomas,

I tried to send them toqemu-s390x@nongnu.org  andqemu-devel@nongnu.org  lists with some additional people (including you)
in CC.  The initial mail was on January 30th.  I can resend them if there was a problem with the delivery.

Regards,
Jared Rossi

>
>
>>
>> On 1/30/23 4:24 PM, Jared Rossi wrote:
>>
>>> Add support for List-Directed IPL (LD-IPL) type pointers.
>>>
>>> We check for a boot record indicating LD-IPL and use it if it is valid,
>>> otherwise we use the CCW CDL or LDL format as usual. When a block is 
>>> accessed
>>> during IPL the block number is first calculated based on the 
>>> cylinder, head,
>>> and record numbers included in the block pointer; if LD-IPL has been 
>>> initiated
>>> then each pointer will be interpreted using the new format.
>>>
>>> For simplicity, there is no choice presented to forcibly use CCW-IPL 
>>> if LD-IPL
>>> is available.  Because both sets of pointers ultimately go to the same
>>> kernel/initrd, using CCW- or LD-IPL is transparent to the user.
>>>
>>> One aspect of the user experience that does change is the 
>>> availability of the
>>> interactive boot menu when a loadparm is not specified.  For the 
>>> existing
>>> CCW-IPL, when the user does not specify a loadparm they are 
>>> presented with a
>>> list of boot options; however, this list is only written in the old 
>>> style
>>> pointers.  Therefore, if no loadparm is specified, and LD-IPL is 
>>> supported, the
>>> default boot option will be used automatically.
>>>
>>> Jared Rossi (1):
>>>    pc-bios: Add support for List-Directed IPL from ECKD DASD
>>>
>>>   pc-bios/s390-ccw/bootmap.c | 157 
>>> ++++++++++++++++++++++++++++---------
>>>   pc-bios/s390-ccw/bootmap.h |  30 ++++++-
>>>   2 files changed, 148 insertions(+), 39 deletions(-)
>>>
>

[-- Attachment #2: Type: text/html, Size: 3796 bytes --]

Re: [PATCH 0/1] pc-bios: Support List-Directed IPL from ECKD DASD

[Thomas Huth]

On 20/02/2023 17.21, Jared Rossi wrote:
> On 2/20/23 10:45 AM, Thomas Huth wrote:
>> On 20/02/2023 16.13, Jared Rossi wrote:
>>> Hi,
>>>
>>> Pinging this patch since v2.26.0 of s390-tools adds ECKD DASD 
>>> list-directed IPL.
>>> This patch set allows QEMU to utilize that new boot pointer format.
>>> Any feedback about what needs to be changed for acceptance is much 
>>> appreciated.
>>
>> Where did you send your patches to? I can't find it in neither my 
>> qemu-s390x nor my qemu-devel mailing list folders.
>>
>>  Thomas
> 
> Hi Thomas,
> 
> I tried to send them toqemu-s390x@nongnu.org  andqemu-devel@nongnu.org  lists with some additional people (including you)
> in CC.  The initial mail was on January 30th.  I can resend them if there was a problem with the delivery.

I also don't see your mail in the usual archives, e.g. not in:

  https://lore.kernel.org/qemu-devel/?q=Support+List-Directed+IPL+from+ECKD+DASD

and also not in:

  https://lists.gnu.org/archive/html/qemu-devel/2023-01/threads.html

It seems like your patches never left your mail server. Thus please resend!

  Thanks,
   Thomas