Re: [PATCH] ifcvf: move IRQ request/free to status change handlers

2020-05-11 Thread Jason Wang
On 2020/5/12 上午11:38, Jason Wang wrote:   static int ifcvf_start_datapath(void *private)   {   struct ifcvf_hw *vf = ifcvf_private_to_vf(private); @@ -118,9 +172,12 @@ static void ifcvf_vdpa_set_status(struct vdpa_device *vdpa_dev, u8 status)   {   struct ifcvf_adapter *adapter;

Re: [PATCH] ifcvf: move IRQ request/free to status change handlers

2020-05-11 Thread Jason Wang
On 2020/5/11 下午6:18, Francesco Lavra wrote: On 5/11/20 11:26 AM, Jason Wang wrote: On 2020/5/11 下午3:19, Zhu Lingshan wrote: This commit move IRQ request and free operations from probe() to VIRTIO status change handler to comply with VIRTIO spec. VIRTIO spec 1.1, section 2.1.2 Device Require

Re: [PATCH] ifcvf: move IRQ request/free to status change handlers

2020-05-11 Thread Jason Wang
On 2020/5/11 下午6:11, Zhu, Lingshan wrote: On 5/11/2020 5:26 PM, Jason Wang wrote: On 2020/5/11 下午3:19, Zhu Lingshan wrote: This commit move IRQ request and free operations from probe() to VIRTIO status change handler to comply with VIRTIO spec. VIRTIO spec 1.1, section 2.1.2 Device Requir

Re: [PATCH] ifcvf: move IRQ request/free to status change handlers

2020-05-11 Thread Francesco Lavra
On 5/11/20 11:26 AM, Jason Wang wrote: On 2020/5/11 下午3:19, Zhu Lingshan wrote: This commit move IRQ request and free operations from probe() to VIRTIO status change handler to comply with VIRTIO spec. VIRTIO spec 1.1, section 2.1.2 Device Requirements: Device Status Field The device MUST NOT

Re: [PATCH] ifcvf: move IRQ request/free to status change handlers

2020-05-11 Thread Jason Wang
On 2020/5/11 下午3:19, Zhu Lingshan wrote: This commit move IRQ request and free operations from probe() to VIRTIO status change handler to comply with VIRTIO spec. VIRTIO spec 1.1, section 2.1.2 Device Requirements: Device Status Field The device MUST NOT consume buffers or send any used buffer

[PATCH] ifcvf: move IRQ request/free to status change handlers

2020-05-11 Thread Zhu Lingshan
This commit move IRQ request and free operations from probe() to VIRTIO status change handler to comply with VIRTIO spec. VIRTIO spec 1.1, section 2.1.2 Device Requirements: Device Status Field The device MUST NOT consume buffers or send any used buffer notifications to the driver before DRIVER_OK