mirror of https://github.com/xemu-project/xemu.git
virtio: get_features() can fail
Signed-off-by: Jason Wang <jasowang@redhat.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Acked-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
parent
27462695cd
commit
9d5b731dd2
|
@ -21,7 +21,8 @@
|
|||
#include "virtio-9p-coth.h"
|
||||
#include "hw/virtio/virtio-access.h"
|
||||
|
||||
static uint64_t virtio_9p_get_features(VirtIODevice *vdev, uint64_t features)
|
||||
static uint64_t virtio_9p_get_features(VirtIODevice *vdev, uint64_t features,
|
||||
Error **errp)
|
||||
{
|
||||
virtio_add_feature(&features, VIRTIO_9P_MOUNT_TAG);
|
||||
return features;
|
||||
|
|
|
@ -722,7 +722,8 @@ static void virtio_blk_set_config(VirtIODevice *vdev, const uint8_t *config)
|
|||
aio_context_release(blk_get_aio_context(s->blk));
|
||||
}
|
||||
|
||||
static uint64_t virtio_blk_get_features(VirtIODevice *vdev, uint64_t features)
|
||||
static uint64_t virtio_blk_get_features(VirtIODevice *vdev, uint64_t features,
|
||||
Error **errp)
|
||||
{
|
||||
VirtIOBlock *s = VIRTIO_BLK(vdev);
|
||||
|
||||
|
|
|
@ -500,7 +500,8 @@ static void handle_input(VirtIODevice *vdev, VirtQueue *vq)
|
|||
}
|
||||
}
|
||||
|
||||
static uint64_t get_features(VirtIODevice *vdev, uint64_t features)
|
||||
static uint64_t get_features(VirtIODevice *vdev, uint64_t features,
|
||||
Error **errp)
|
||||
{
|
||||
VirtIOSerial *vser;
|
||||
|
||||
|
|
|
@ -89,7 +89,8 @@ static void virtio_gpu_set_config(VirtIODevice *vdev, const uint8_t *config)
|
|||
}
|
||||
}
|
||||
|
||||
static uint64_t virtio_gpu_get_features(VirtIODevice *vdev, uint64_t features)
|
||||
static uint64_t virtio_gpu_get_features(VirtIODevice *vdev, uint64_t features,
|
||||
Error **errp)
|
||||
{
|
||||
return features;
|
||||
}
|
||||
|
|
|
@ -166,7 +166,8 @@ static void virtio_input_set_config(VirtIODevice *vdev,
|
|||
virtio_notify_config(vdev);
|
||||
}
|
||||
|
||||
static uint64_t virtio_input_get_features(VirtIODevice *vdev, uint64_t f)
|
||||
static uint64_t virtio_input_get_features(VirtIODevice *vdev, uint64_t f,
|
||||
Error **errp)
|
||||
{
|
||||
return f;
|
||||
}
|
||||
|
|
|
@ -446,7 +446,8 @@ static void virtio_net_set_queues(VirtIONet *n)
|
|||
|
||||
static void virtio_net_set_multiqueue(VirtIONet *n, int multiqueue);
|
||||
|
||||
static uint64_t virtio_net_get_features(VirtIODevice *vdev, uint64_t features)
|
||||
static uint64_t virtio_net_get_features(VirtIODevice *vdev, uint64_t features,
|
||||
Error **errp)
|
||||
{
|
||||
VirtIONet *n = VIRTIO_NET(vdev);
|
||||
NetClientState *nc = qemu_get_queue(n->nic);
|
||||
|
|
|
@ -153,7 +153,8 @@ static void vhost_scsi_stop(VHostSCSI *s)
|
|||
}
|
||||
|
||||
static uint64_t vhost_scsi_get_features(VirtIODevice *vdev,
|
||||
uint64_t features)
|
||||
uint64_t features,
|
||||
Error **errp)
|
||||
{
|
||||
VHostSCSI *s = VHOST_SCSI(vdev);
|
||||
|
||||
|
|
|
@ -629,7 +629,8 @@ static void virtio_scsi_set_config(VirtIODevice *vdev,
|
|||
}
|
||||
|
||||
static uint64_t virtio_scsi_get_features(VirtIODevice *vdev,
|
||||
uint64_t requested_features)
|
||||
uint64_t requested_features,
|
||||
Error **errp)
|
||||
{
|
||||
VirtIOSCSI *s = VIRTIO_SCSI(vdev);
|
||||
|
||||
|
|
|
@ -310,7 +310,8 @@ static void virtio_balloon_set_config(VirtIODevice *vdev,
|
|||
trace_virtio_balloon_set_config(dev->actual, oldactual);
|
||||
}
|
||||
|
||||
static uint64_t virtio_balloon_get_features(VirtIODevice *vdev, uint64_t f)
|
||||
static uint64_t virtio_balloon_get_features(VirtIODevice *vdev, uint64_t f,
|
||||
Error **errp)
|
||||
{
|
||||
VirtIOBalloon *dev = VIRTIO_BALLOON(vdev);
|
||||
f |= dev->host_features;
|
||||
|
|
|
@ -54,7 +54,8 @@ void virtio_bus_device_plugged(VirtIODevice *vdev, Error **errp)
|
|||
|
||||
/* Get the features of the plugged device. */
|
||||
assert(vdc->get_features != NULL);
|
||||
vdev->host_features = vdc->get_features(vdev, vdev->host_features);
|
||||
vdev->host_features = vdc->get_features(vdev, vdev->host_features,
|
||||
errp);
|
||||
}
|
||||
|
||||
/* Reset the virtio_bus */
|
||||
|
|
|
@ -104,7 +104,7 @@ static void handle_input(VirtIODevice *vdev, VirtQueue *vq)
|
|||
virtio_rng_process(vrng);
|
||||
}
|
||||
|
||||
static uint64_t get_features(VirtIODevice *vdev, uint64_t f)
|
||||
static uint64_t get_features(VirtIODevice *vdev, uint64_t f, Error **errp)
|
||||
{
|
||||
return f;
|
||||
}
|
||||
|
|
|
@ -101,7 +101,9 @@ typedef struct VirtioDeviceClass {
|
|||
/* This is what a VirtioDevice must implement */
|
||||
DeviceRealize realize;
|
||||
DeviceUnrealize unrealize;
|
||||
uint64_t (*get_features)(VirtIODevice *vdev, uint64_t requested_features);
|
||||
uint64_t (*get_features)(VirtIODevice *vdev,
|
||||
uint64_t requested_features,
|
||||
Error **errp);
|
||||
uint64_t (*bad_features)(VirtIODevice *vdev);
|
||||
void (*set_features)(VirtIODevice *vdev, uint64_t val);
|
||||
int (*validate_features)(VirtIODevice *vdev);
|
||||
|
|
Loading…
Reference in New Issue