pds_vdpa: set features order
authorShannon Nelson <shannon.nelson@amd.com>
Fri, 10 Nov 2023 22:18:02 +0000 (14:18 -0800)
committerMichael S. Tsirkin <mst@redhat.com>
Fri, 1 Dec 2023 14:55:01 +0000 (09:55 -0500)
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 <shannon.nelson@amd.com>
Message-Id: <20231110221802.46841-4-shannon.nelson@amd.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Acked-by: Jason Wang <jasowang@redhat.com>
drivers/vdpa/pds/vdpa_dev.c

index 9fc89c82d1f015c69e6cd8c2656c05932db59c08..25c0fe5ec3d5dfacdb53fa31a709851adb118942 100644 (file)
@@ -318,9 +318,8 @@ static int pds_vdpa_set_driver_features(struct vdpa_device *vdpa_dev, u64 featur
                return -EOPNOTSUPP;
        }
 
-       pdsv->negotiated_features = nego_features;
-
        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);