qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v3 00/16] s390: vfio-ccw dasd ipl support


From: Christian Borntraeger
Subject: Re: [Qemu-devel] [PATCH v3 00/16] s390: vfio-ccw dasd ipl support
Date: Mon, 4 Mar 2019 18:53:05 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1


On 04.03.2019 17:24, Cornelia Huck wrote:
> On Fri,  1 Mar 2019 13:59:20 -0500
> "Jason J. Herne" <address@hidden> wrote:
> 
>> This is to support booting from vfio-ccw dasd devices. We basically implement
>> the real hardware ipl procedure. This allows for booting Linux guests on
>> vfio-ccw devices.
>>
>> vfio-ccw's channel program prefetch algorithm complicates ipl because most 
>> ipl
>> channel programs dynamically modify themselves. Details on the ipl process 
>> and
>> how we worked around this issue can be found in docs/devel/s390-dasd-ipl.txt.
>>
>> Note to Connie: In the v2 review you mentioned some issues with the error
>> handling code involving printing of the eckd dasd sense data. I've addressed
>> those concerns in a new patch just to make those changes clearly stand out 
>> from
>> the rest of the code. I can squash them into patches 10, 11, and 15 if you
>> prefer that to having it as a separate patch.
> 
> Whatever makes sense... but I'll let the s390 bios maintainers decide.

Both variants are fine with me.
> 
> Speaking of which: Please do keep Thomas on cc:...
> 
> [This is probably a bit late for 4.0, as softfreeze is next Tuesday and
> pull requests for master need to be on-list by then... speaking of pull
> requests: I'd prefer to queue this via a pull request from the bios
> maintainers rather than picking the patches myself, if possible.]


there are still some minor discussions on the patch set, but I (or Thomas) can
certainly pick up the next version and send a pull request. 
Thomas do you want to do it or shall I take care about it?




reply via email to

[Prev in Thread] Current Thread [Next in Thread]