On Mon,  4 Jan 2021 14:24:01 +0100
Laszlo Ersek <[email protected]> wrote:

> virtio-fs qualifies as a bootable device minimally under OVMF, but
> currently the necessary "bootindex" property is missing (fw_cfg kernel
> boot notwithstanding).
> 
> Add the property. For completeness, add it to the CCW device as well;
> other virtio-ccw devices seem to have "bootindex" properties too.
> 
> Example OpenFirmware device path for the "vhost-user-fs-pci" device in the
> "bootorder" fw_cfg file:
> 
>   /pci@i0cf8/pci-bridge@1,6/pci1af4,105a@0/filesystem@0
> 
> Cc: "Dr. David Alan Gilbert" <[email protected]>
> Cc: "Michael S. Tsirkin" <[email protected]>
> Cc: Christian Borntraeger <[email protected]>
> Cc: Cornelia Huck <[email protected]>
> Cc: Halil Pasic <[email protected]>
> Cc: Stefan Hajnoczi <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Laszlo Ersek <[email protected]>
> ---
>  include/hw/virtio/vhost-user-fs.h |  1 +
>  hw/s390x/vhost-user-fs-ccw.c      |  2 ++
>  hw/virtio/vhost-user-fs-pci.c     |  2 ++
>  hw/virtio/vhost-user-fs.c         | 10 ++++++++++
>  4 files changed, 15 insertions(+)
> 

I agree that you should just drop the ccw hunk for now. The rest of the
patch looks good to me.


Reply via email to