On 8/2/2024 9:26 AM, Sami Mujawar wrote:
Hi Shrikanth,
It appears I had missed out passing the correct virtio-scsi request
size when enqueuing the request.
I have now fixed that in the pull request at
https://github.com/tianocore/edk2/pull/6038.
Can you try the patch in the pull request at and let me know if it
fixes the issue, please?
Yes, this pull request fixes the issue. Thank you.
Reported-by: Aithal Srikanth <[email protected]>
Tested-by: Aithal Srikanth <[email protected]>
Regards,
Sami Mujawar
*From: *Sami Mujawar <[email protected]>
*Date: *Thursday 1 August 2024 at 16:53
*To: *"[email protected]" <[email protected]>,
"[email protected]" <[email protected]>, Ard Biesheuvel
<[email protected]>, Jiewen Yao <[email protected]>, Gerd
Hoffmann <[email protected]>
*Subject: *Re: [edk2-devel] Patch OvmfPkg: Use heap memory for
virtio-scsi request causing regression
Hi Srikanth,
Thanks for reporting this issue. It is strange that the virtio-blk
driver is working with a similar change.
I am looking into why this may be failing.
Regards,
Sami Mujawar
*From: *<[email protected]> on behalf of "Aithal, Srikanth via
groups.io" <[email protected]>
*Reply to: *"[email protected]" <[email protected]>,
"[email protected]" <[email protected]>
*Date: *Thursday 1 August 2024 at 12:28
*To: *"[email protected]" <[email protected]>, Sami Mujawar
<[email protected]>, Ard Biesheuvel <[email protected]>,
Jiewen Yao <[email protected]>, Gerd Hoffmann <[email protected]>
*Subject: *[edk2-devel] Patch OvmfPkg: Use heap memory for virtio-scsi
request causing regression
*Resent from: *<[email protected]>
*Resent date: *Thursday 1 August 2024 at 12:28
Hello all,
After the patches [1fc55a39, b342070c] OvmfPkg: Use heap memory for
virtio-blk request/OvmfPkg: Use heap memory for virtio-scsi request
got merged, this regression is spotted.
AMD SVM/SEV/SEVES guest boot fails with OVMF packages [Ia3264/X64]
built from today's upstream edk2 repository[Head: 85fad99].
Guest boot with OVMF.fd or only with OVMF_CODE.fd gets stuck at below
point:
/Divisor = 1
BaudRate/Actual (115200/115200) = 100%
PciSioSerial: Create SIO child serial device - Device Error
PlatformBootManagerAfterConsole
Found Mass Storage device: PciRoot(0x0)/Pci(0x3,0x0)
InstallProtocolInterface: FA920010-6785-4941-B6EC-498C579F160A 3DEB82A0
InstallProtocolInterface: 143B7632-B81B-4CB7-ABD3-B625A5B9BFFE 3DE10AA0
InstallProtocolInterface: 0167CCC4-D0F7-4F21-A3EF-9E64B7CDCE8B 3DE5D4A0
qemu-system-x86_64: wrong size for virtio-scsi headers <==========
gets hung/
If both OVMF code and vars file are provided the issue is not hit.
Qemu command line to recreate the issue:
/qemu-system-x86_64 \
-machine q35,kernel_irqchip=split \
-name guest=vm,debug-threads=on \
-drive if=pflash,format=raw,unit=0,file=OVMF_CODE.fd,readonly \
-cpu host \
-m 1024 \
-smp 1,cores=1,threads=1,dies=1,sockets=1 \
-kernel bzImage \
-append "root=/dev/sda rw console=ttyS0 net.ifnames=0 biosdevname=0
movable_node swiotlb=65536 " \
-drive id=disk0,file=22.04-server.qcow2,format=qcow2,if=none \
-device virtio-scsi-pci,id=scsi0,disable-legacy=on,iommu_platform=true \
-device scsi-hd,drive=disk0 \
--enable-kvm \
--nographic/
Thanks,
Srikanth Aithal
IMPORTANT NOTICE: The contents of this email and any attachments are
confidential and may also be privileged. If you are not the intended
recipient, please notify the sender immediately and do not disclose
the contents to any other person, use it for any purpose, or store or
copy the information in any medium. Thank you.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120204): https://edk2.groups.io/g/devel/message/120204
Mute This Topic: https://groups.io/mt/107663187/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
