pds_vdpa: set features order

Fix up the order that the device and negotiated features
are checked to get a more reliable difference when things
get changed.

Signed-off-by: Shannon Nelson <[email protected]>
Message-Id: <20231110221802[email protected]>
Signed-off-by: Michael S. Tsirkin <[email protected]>
Acked-by: Jason Wang <[email protected]>

diff --git a/drivers/vdpa/pds/vdpa_dev.c b/drivers/vdpa/pds/vdpa_dev.c
index 9fc89c82d1f015c69e6cd8c2656c05932db59c08..25c0fe5ec3d5dfacdb53fa31a709851adb118942 100644 (file)
--- a/drivers/vdpa/pds/vdpa_dev.c
+++ b/drivers/vdpa/pds/vdpa_dev.c
@@ -318,9 +318,8 @@ static int pds_vdpa_set_driver_features(struct vdpa_device *vdpa_dev, u64 featur
                return -EOPNOTSUPP;
        }
 
                return -EOPNOTSUPP;
        }
 

-       pdsv->negotiated_features = nego_features;
-

        driver_features = pds_vdpa_get_driver_features(vdpa_dev);
        driver_features = pds_vdpa_get_driver_features(vdpa_dev);

+       pdsv->negotiated_features = nego_features;

        dev_dbg(dev, "%s: %#llx => %#llx\n",
                __func__, driver_features, nego_features);
 
        dev_dbg(dev, "%s: %#llx => %#llx\n",
                __func__, driver_features, nego_features);