Re: [Qemu-devel] [PATCH v8 00/10] Convert msix_init() to error

[Cao jin <caoj.fnst@cn.fujitsu.com>]

On 01/13/2017 04:22 PM, Markus Armbruster wrote:
> Cao jin <caoj.fnst@cn.fujitsu.com> writes:
> 
>> Only a tiny modification in patch "megasas: remove unnecessary
>> megasas_use_msix()" to fix a megasas issue.
> 
> Please have a look at Michael's review in
> Message-ID: <20170112163519-mutt-send-email-mst@kernel.org>
> 

Oh...thanks for reminding, I missed that mail...

>> v8 changelog:
>> 1. reorder: place the "megasas: remove unnecessary megasas_use_msix()"
>>    as the last one. and fix the bug in it, detailed description in it,
>>    also removed the R-b of it.
>> 2. Add the Acked-by from Marcel for first 9 patches; add the R-b from Markus
>>    to "hcd-xhci: check & correct param before using it".
>>
>> Test:
>> 1. make check ok
>> 2. command line test for all affected device, make sure their realization
>>    is ok.
>> 3. detailed test for megasas, hcd-xhci, vmxnet3.
>>    megasas: install a linux distro is ok
>>    ./qemu-system-x86_64 --enable-kvm -m 1024
>>    -device megasas,id=scsi0,bus=pci.0
>>    -drive file=/xx/scsi-disk.img,if=none,id=drive-scsi0
>>    -device scsi-disk,bus=scsi0.0,channel=0,scsi-id=4,lun=0,drive=drive-scsi0,id=scsi0-4
>>    -cdrom /xx/Fedora-Server-DVD-x86_64-23.iso --monitor stdio
>>
>>    hcd-xhci: partition the usbstick.img, mkfs, write to file, is ok
>>    ./qemu-system-x86_64 -M q35 -m 1024 --enable-kvm
>>    -drive if=none,id=usbstick,file=/xx/usbstick.img
>>    -device nec-usb-xhci,id=usb,p2=8,p3=8,bus=pcie.0
>>    -device usb-storage,bus=usb.0,drive=usbstick --monitor stdio
>>    /xx/FedoraWorkStatsion23-x86_64.img
>>
>>    vmxnet3: ping another destination belongs to host's network is ok.
>>    But no migration test, because I don't have a spare machine for now.
>>    ./qemu-system-x86_64 -M q35 -m 1024 --enable-kvm
>>    -netdev tap,id=mynet0 -device vmxnet3,netdev=mynet0
>>    --monitor stdio /home/pino/vm/FedoraWorkStatsion23-x86_64.img
> 
> You could migrate to a new QEMU on the same machine.
> 
> Or, if you don't want to deal with two instances of QEMU running at the
> same time, migrate to file:
> 
>     (qemu) migrate "exec:cat >zzz"
> 
> Start the target with -incoming "exec:cat zzz".  Better than nothing.
> 

Thanks for the info, I will try

-- 
Sincerely,
Cao jin