On 01/11/21 17:22, Cornelia Huck wrote: > 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. >
Thank you as well -- with the ccw hunk gone, I guess the CC list will be shorter in v2, too :) Dave, Stefan, Michael -- can you please comment? For the files to remain in v2, "get_maintainer.pl" gives me: "Dr. David Alan Gilbert" <[email protected]> (supporter:virtiofs) "Michael S. Tsirkin" <[email protected]> (supporter:vhost) "Michael S. Tsirkin" <[email protected]> (supporter:virtio) Stefan Hajnoczi <[email protected]> (supporter:virtiofs) Thanks Laszlo
