On 20.03.2023 16:16, Roger Pau Monné wrote:
> @@ -244,12 +242,18 @@ static void vioapic_write_redirent(
> }
> else
> {
> + int ret;
> +
> unmasked = ent.fields.mask;
> /* Remote IRR and Delivery Status are read-only. */
> ent.bits = ((ent.bits >> 32) << 32) | val;
> ent.fields.delivery_status = 0;
> ent.fields.remote_irr = pent->fields.remote_irr;
> unmasked = unmasked && !ent.fields.mask;
> + ret = mp_register_gsi(gsi, ent.fields.trig_mode,
> ent.fields.polarity);
> + if ( ret && ret != -EEXIST )
> + gprintk(XENLOG_WARNING, "vioapic: error registering GSI %u:
> %d\n",
> + gsi, ret);
> }
I assume this is only meant to be experimental, as I'm missing confinement
to Dom0 here. I also question this when the mask bit as set, as in that
case neither the trigger mode bit nor the polarity one can be relied upon.
At which point it would look to me as if it was necessary for Dom0 to use
a hypercall instead (which naturally would then be PHYSDEVOP_setup_gsi).
Jan
