net/hns3: check setting VF PCI bus return value
authorHongbo Zheng <zhenghongbo3@huawei.com>
Thu, 29 Oct 2020 12:51:56 +0000 (20:51 +0800)
committerFerruh Yigit <ferruh.yigit@intel.com>
Tue, 3 Nov 2020 22:35:06 +0000 (23:35 +0100)
Currently hns3vf_reinit_dev only judge whether the return value of
setting PCI bus function is not 0, while it will return a negative
value when execute failed.

Fixes: 243651cb6c8c ("net/hns3: check PCI config space reads")
Cc: stable@dpdk.org
Signed-off-by: Hongbo Zheng <zhenghongbo3@huawei.com>
Signed-off-by: Lijun Ou <oulijun@huawei.com>
drivers/net/hns3/hns3_ethdev_vf.c

index 9fb7941..05a9341 100644 (file)
@@ -2619,7 +2619,7 @@ hns3vf_reinit_dev(struct hns3_adapter *hns)
        if (hw->reset.level == HNS3_VF_FULL_RESET) {
                rte_intr_disable(&pci_dev->intr_handle);
                ret = hns3vf_set_bus_master(pci_dev, true);
-               if (ret) {
+               if (ret < 0) {
                        hns3_err(hw, "failed to set pci bus, ret = %d", ret);
                        return ret;
                }