[dts] Any comments for those patches?

[dts] Any comments for those patches?

[Phil Yang]

[-- Attachment #1: Type: text/plain, Size: 208 bytes --]

Hi Marvin,

Any comment for those two patches are welcomed.  Thanks
http://dpdk.org/ml/archives/dts/2018-May/004061.html
http://dpdk.org/ml/archives/dts/2018-June/004064.html

Best Regards,
Phil Yang

[-- Attachment #2: Type: text/html, Size: 2480 bytes --]

Re: [dts] Any comments for those patches?

[Liu, Yong]

[-- Attachment #1: Type: text/plain, Size: 769 bytes --]

Phil,
Sorry for late response.
I remember that VF daemon has fixed environment setup based on our requirement. Anyway I think it will be fine to merge your patch in if  work with vfio-pci. Also add original test plan owner since may need update the test plan.
Fix in second patch look like has been covered by another merged patch.

Thanks,
Marvin

From: Phil.Yang@arm.com [mailto:Phil.Yang@arm.com]
Sent: Friday, June 15, 2018 4:57 PM
To: Liu, Yong <yong.liu@intel.com>
Cc: dts@dpdk.org; nd <nd@arm.com>
Subject: Any comments for those patches?

Hi Marvin,

Any comment for those two patches are welcomed.  Thanks
http://dpdk.org/ml/archives/dts/2018-May/004061.html
http://dpdk.org/ml/archives/dts/2018-June/004064.html

Best Regards,
Phil Yang

[-- Attachment #2: Type: text/html, Size: 4084 bytes --]

Re: [dts] Any comments for those patches?

[Phil Yang]

[-- Attachment #1: Type: text/plain, Size: 1449 bytes --]

Thank you very much.

Copy that.
I noticed that ba805a3 has already merged and it solved the same defect as I encountered. I didn't notice that when I upstream the patch. Mine should be abandoned.

PCI-stub is not supported on latest QEMU for qemu-system-aarch64. So basically all the VF cases need add vfio-pci config support.

Thanks,
Phil Yang

From: Liu, Yong <yong.liu@intel.com>
Sent: Friday, June 15, 2018 5:13 PM
To: Phil Yang <Phil.Yang@arm.com>; Lin, Xueqin <xueqin.lin@intel.com>
Cc: dts@dpdk.org; nd <nd@arm.com>
Subject: RE: Any comments for those patches?

Phil,
Sorry for late response.
I remember that VF daemon has fixed environment setup based on our requirement. Anyway I think it will be fine to merge your patch in if  work with vfio-pci. Also add original test plan owner since may need update the test plan.
Fix in second patch look like has been covered by another merged patch.

Thanks,
Marvin

From: Phil.Yang@arm.com<mailto:Phil.Yang@arm.com> [mailto:Phil.Yang@arm.com]
Sent: Friday, June 15, 2018 4:57 PM
To: Liu, Yong <yong.liu@intel.com<mailto:yong.liu@intel.com>>
Cc: dts@dpdk.org<mailto:dts@dpdk.org>; nd <nd@arm.com<mailto:nd@arm.com>>
Subject: Any comments for those patches?

Hi Marvin,

Any comment for those two patches are welcomed.  Thanks
http://dpdk.org/ml/archives/dts/2018-May/004061.html
http://dpdk.org/ml/archives/dts/2018-June/004064.html

Best Regards,
Phil Yang

[-- Attachment #2: Type: text/html, Size: 6159 bytes --]