@@ -295,10 +295,17 @@ enum idxd_interrupt_type {
IDXD_IRQ_IMS,
};
+struct aux_mdev_ops {
+ void (*notify_error)(struct idxd_wq *wq);
+};
+
struct idxd_mdev_aux_drv {
struct auxiliary_driver auxiliary_drv;
+ const struct aux_mdev_ops ops;
};
+#define to_mdev_aux_drv(_aux_drv) container_of(_aux_drv, struct idxd_mdev_aux_drv, auxiliary_drv)
+
static inline int idxd_get_wq_portal_offset(enum idxd_portal_prot prot,
enum idxd_interrupt_type irq_type)
{
@@ -118,6 +118,8 @@ static int process_misc_interrupts(struct idxd_device *idxd, u32 cause)
u32 val = 0;
int i;
bool err = false;
+ struct auxiliary_driver *auxdrv = to_auxiliary_drv(idxd->mdev_auxdev->dev.driver);
+ struct idxd_mdev_aux_drv *mdevdrv = to_mdev_aux_drv(auxdrv);
if (cause & IDXD_INTC_ERR) {
spin_lock_bh(&idxd->dev_lock);
@@ -132,6 +134,8 @@ static int process_misc_interrupts(struct idxd_device *idxd, u32 cause)
if (wq->type == IDXD_WQT_USER)
wake_up_interruptible(&wq->idxd_cdev.err_queue);
+ else if (wq->type == IDXD_WQT_MDEV)
+ mdevdrv->ops.notify_error(wq);
} else {
int i;
@@ -140,6 +144,8 @@ static int process_misc_interrupts(struct idxd_device *idxd, u32 cause)
if (wq->type == IDXD_WQT_USER)
wake_up_interruptible(&wq->idxd_cdev.err_queue);
+ else if (wq->type == IDXD_WQT_MDEV)
+ mdevdrv->ops.notify_error(wq);
}
}
@@ -1266,12 +1266,17 @@ static const struct auxiliary_device_id idxd_mdev_auxbus_id_table[] = {
};
MODULE_DEVICE_TABLE(auxiliary, idxd_mdev_auxbus_id_table);
+static const struct aux_mdev_ops aux_mdev_ops = {
+ .notify_error = idxd_wq_vidxd_send_errors,
+};
+
static struct idxd_mdev_aux_drv idxd_mdev_aux_drv = {
.auxiliary_drv = {
.id_table = idxd_mdev_auxbus_id_table,
.probe = idxd_mdev_aux_probe,
.remove = idxd_mdev_aux_remove,
},
+ .ops = aux_mdev_ops,
};
static int idxd_mdev_auxdev_drv_register(struct idxd_mdev_aux_drv *drv)
@@ -980,3 +980,35 @@ void vidxd_do_command(struct vdcm_idxd *vidxd, u32 val)
break;
}
}
+
+static void vidxd_send_errors(struct vdcm_idxd *vidxd)
+{
+ struct idxd_device *idxd = vidxd->idxd;
+ u8 *bar0 = vidxd->bar0;
+ union sw_err_reg *swerr = (union sw_err_reg *)(bar0 + IDXD_SWERR_OFFSET);
+ union genctrl_reg *genctrl = (union genctrl_reg *)(bar0 + IDXD_GENCTRL_OFFSET);
+ u32 *intcause = (u32 *)(bar0 + IDXD_INTCAUSE_OFFSET);
+ int i;
+
+ if (swerr->valid) {
+ if (!swerr->overflow)
+ swerr->overflow = 1;
+ return;
+ }
+
+ lockdep_assert_held(&idxd->dev_lock);
+ for (i = 0; i < 4; i++)
+ swerr->bits[i] = idxd->sw_err.bits[i];
+
+ *intcause |= IDXD_INTC_ERR;
+ if (genctrl->softerr_int_en)
+ vidxd_send_interrupt(&vidxd->irq_entries[0]);
+}
+
+void idxd_wq_vidxd_send_errors(struct idxd_wq *wq)
+{
+ struct vdcm_idxd *vidxd;
+
+ list_for_each_entry(vidxd, &wq->vdcm_list, list)
+ vidxd_send_errors(vidxd);
+}
@@ -23,5 +23,6 @@ int vidxd_send_interrupt(struct ims_irq_entry *iie);
int vidxd_setup_ims_entries(struct vdcm_idxd *vidxd);
void vidxd_free_ims_entries(struct vdcm_idxd *vidxd);
void vidxd_do_command(struct vdcm_idxd *vidxd, u32 val);
+void idxd_wq_vidxd_send_errors(struct idxd_wq *wq);
#endif
When a device error occurs, the mediated device need to be notified in order to notify the guest of device error. Add support to notify the specific mdev when an error is wq specific and broadcast errors to all mdev when it's a generic device error. Signed-off-by: Dave Jiang <dave.jiang@intel.com> --- drivers/dma/idxd/idxd.h | 7 +++++++ drivers/dma/idxd/irq.c | 6 ++++++ drivers/vfio/mdev/idxd/mdev.c | 5 +++++ drivers/vfio/mdev/idxd/vdev.c | 32 ++++++++++++++++++++++++++++++++ drivers/vfio/mdev/idxd/vdev.h | 1 + 5 files changed, 51 insertions(+)