mirror of https://github.com/xemu-project/xemu.git
virtio-scsi: Report missed events
When an event is reported but no buffers are present in the event vq, we can set a flag and report a dummy event as soon as one is added. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
parent
b6866fee58
commit
64f64855d0
|
@ -141,6 +141,7 @@ typedef struct {
|
||||||
uint32_t sense_size;
|
uint32_t sense_size;
|
||||||
uint32_t cdb_size;
|
uint32_t cdb_size;
|
||||||
int resetting;
|
int resetting;
|
||||||
|
bool events_dropped;
|
||||||
VirtQueue *ctrl_vq;
|
VirtQueue *ctrl_vq;
|
||||||
VirtQueue *event_vq;
|
VirtQueue *event_vq;
|
||||||
VirtQueue *cmd_vqs[0];
|
VirtQueue *cmd_vqs[0];
|
||||||
|
@ -416,10 +417,6 @@ static void virtio_scsi_handle_ctrl(VirtIODevice *vdev, VirtQueue *vq)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
static void virtio_scsi_handle_event(VirtIODevice *vdev, VirtQueue *vq)
|
|
||||||
{
|
|
||||||
}
|
|
||||||
|
|
||||||
static void virtio_scsi_command_complete(SCSIRequest *r, uint32_t status,
|
static void virtio_scsi_command_complete(SCSIRequest *r, uint32_t status,
|
||||||
size_t resid)
|
size_t resid)
|
||||||
{
|
{
|
||||||
|
@ -594,13 +591,22 @@ static void virtio_scsi_push_event(VirtIOSCSI *s, SCSIDevice *dev,
|
||||||
{
|
{
|
||||||
VirtIOSCSIReq *req = virtio_scsi_pop_req(s, s->event_vq);
|
VirtIOSCSIReq *req = virtio_scsi_pop_req(s, s->event_vq);
|
||||||
VirtIOSCSIEvent *evt;
|
VirtIOSCSIEvent *evt;
|
||||||
|
|
||||||
if (req) {
|
|
||||||
int in_size;
|
int in_size;
|
||||||
|
|
||||||
|
if (!req) {
|
||||||
|
s->events_dropped = true;
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
if (req->elem.out_num || req->elem.in_num != 1) {
|
if (req->elem.out_num || req->elem.in_num != 1) {
|
||||||
virtio_scsi_bad_req();
|
virtio_scsi_bad_req();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (s->events_dropped) {
|
||||||
|
event |= VIRTIO_SCSI_T_EVENTS_MISSED;
|
||||||
|
s->events_dropped = false;
|
||||||
|
}
|
||||||
|
|
||||||
in_size = req->elem.in_sg[0].iov_len;
|
in_size = req->elem.in_sg[0].iov_len;
|
||||||
if (in_size < sizeof(VirtIOSCSIEvent)) {
|
if (in_size < sizeof(VirtIOSCSIEvent)) {
|
||||||
virtio_scsi_bad_req();
|
virtio_scsi_bad_req();
|
||||||
|
@ -610,6 +616,9 @@ static void virtio_scsi_push_event(VirtIOSCSI *s, SCSIDevice *dev,
|
||||||
memset(evt, 0, sizeof(VirtIOSCSIEvent));
|
memset(evt, 0, sizeof(VirtIOSCSIEvent));
|
||||||
evt->event = event;
|
evt->event = event;
|
||||||
evt->reason = reason;
|
evt->reason = reason;
|
||||||
|
if (!dev) {
|
||||||
|
assert(event == VIRTIO_SCSI_T_NO_EVENT);
|
||||||
|
} else {
|
||||||
evt->lun[0] = 1;
|
evt->lun[0] = 1;
|
||||||
evt->lun[1] = dev->id;
|
evt->lun[1] = dev->id;
|
||||||
|
|
||||||
|
@ -618,8 +627,17 @@ static void virtio_scsi_push_event(VirtIOSCSI *s, SCSIDevice *dev,
|
||||||
evt->lun[2] = (dev->lun >> 8) | 0x40;
|
evt->lun[2] = (dev->lun >> 8) | 0x40;
|
||||||
}
|
}
|
||||||
evt->lun[3] = dev->lun & 0xFF;
|
evt->lun[3] = dev->lun & 0xFF;
|
||||||
|
}
|
||||||
virtio_scsi_complete_req(req);
|
virtio_scsi_complete_req(req);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static void virtio_scsi_handle_event(VirtIODevice *vdev, VirtQueue *vq)
|
||||||
|
{
|
||||||
|
VirtIOSCSI *s = (VirtIOSCSI *)vdev;
|
||||||
|
|
||||||
|
if (s->events_dropped) {
|
||||||
|
virtio_scsi_push_event(s, NULL, VIRTIO_SCSI_T_NO_EVENT, 0);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
static void virtio_scsi_hotplug(SCSIBus *bus, SCSIDevice *dev)
|
static void virtio_scsi_hotplug(SCSIBus *bus, SCSIDevice *dev)
|
||||||
|
|
Loading…
Reference in New Issue