xen: events: turn irq_info constructors into initialiser functions
Signed-off-by: Ian Campbell <ian.campbell@citrix.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
This commit is contained in:
parent
cb60d11427
commit
9158c3588a
|
@ -118,46 +118,76 @@ static struct irq_chip xen_dynamic_chip;
|
|||
static struct irq_chip xen_percpu_chip;
|
||||
static struct irq_chip xen_pirq_chip;
|
||||
|
||||
/* Constructor for packed IRQ information. */
|
||||
static struct irq_info mk_unbound_info(void)
|
||||
{
|
||||
return (struct irq_info) { .type = IRQT_UNBOUND };
|
||||
}
|
||||
|
||||
static struct irq_info mk_evtchn_info(unsigned short evtchn)
|
||||
{
|
||||
return (struct irq_info) { .type = IRQT_EVTCHN, .evtchn = evtchn,
|
||||
.cpu = 0 };
|
||||
}
|
||||
|
||||
static struct irq_info mk_ipi_info(unsigned short evtchn, enum ipi_vector ipi)
|
||||
{
|
||||
return (struct irq_info) { .type = IRQT_IPI, .evtchn = evtchn,
|
||||
.cpu = 0, .u.ipi = ipi };
|
||||
}
|
||||
|
||||
static struct irq_info mk_virq_info(unsigned short evtchn, unsigned short virq)
|
||||
{
|
||||
return (struct irq_info) { .type = IRQT_VIRQ, .evtchn = evtchn,
|
||||
.cpu = 0, .u.virq = virq };
|
||||
}
|
||||
|
||||
static struct irq_info mk_pirq_info(unsigned short evtchn, unsigned short pirq,
|
||||
unsigned short gsi, unsigned short vector)
|
||||
{
|
||||
return (struct irq_info) { .type = IRQT_PIRQ, .evtchn = evtchn,
|
||||
.cpu = 0,
|
||||
.u.pirq = { .pirq = pirq, .gsi = gsi, .vector = vector } };
|
||||
}
|
||||
|
||||
/*
|
||||
* Accessors for packed IRQ information.
|
||||
*/
|
||||
/* Get info for IRQ */
|
||||
static struct irq_info *info_for_irq(unsigned irq)
|
||||
{
|
||||
return &irq_info[irq];
|
||||
}
|
||||
|
||||
/* Constructors for packed IRQ information. */
|
||||
static void xen_irq_info_common_init(struct irq_info *info,
|
||||
enum xen_irq_type type,
|
||||
unsigned short evtchn,
|
||||
unsigned short cpu)
|
||||
{
|
||||
|
||||
BUG_ON(info->type != IRQT_UNBOUND && info->type != type);
|
||||
|
||||
info->type = type;
|
||||
info->evtchn = evtchn;
|
||||
info->cpu = cpu;
|
||||
}
|
||||
|
||||
static void xen_irq_info_evtchn_init(unsigned irq,
|
||||
unsigned short evtchn)
|
||||
{
|
||||
struct irq_info *info = info_for_irq(irq);
|
||||
|
||||
xen_irq_info_common_init(info, IRQT_EVTCHN, evtchn, 0);
|
||||
}
|
||||
|
||||
static void xen_irq_info_ipi_init(unsigned irq,
|
||||
unsigned short evtchn,
|
||||
enum ipi_vector ipi)
|
||||
{
|
||||
struct irq_info *info = info_for_irq(irq);
|
||||
|
||||
xen_irq_info_common_init(info, IRQT_IPI, evtchn, 0);
|
||||
|
||||
info->u.ipi = ipi;
|
||||
}
|
||||
|
||||
static void xen_irq_info_virq_init(unsigned irq,
|
||||
unsigned short evtchn,
|
||||
unsigned short virq)
|
||||
{
|
||||
struct irq_info *info = info_for_irq(irq);
|
||||
|
||||
xen_irq_info_common_init(info, IRQT_VIRQ, evtchn, 0);
|
||||
|
||||
info->u.virq = virq;
|
||||
}
|
||||
|
||||
static void xen_irq_info_pirq_init(unsigned irq,
|
||||
unsigned short evtchn,
|
||||
unsigned short pirq,
|
||||
unsigned short gsi,
|
||||
unsigned short vector,
|
||||
unsigned char flags)
|
||||
{
|
||||
struct irq_info *info = info_for_irq(irq);
|
||||
|
||||
xen_irq_info_common_init(info, IRQT_PIRQ, evtchn, 0);
|
||||
|
||||
info->u.pirq.pirq = pirq;
|
||||
info->u.pirq.gsi = gsi;
|
||||
info->u.pirq.vector = vector;
|
||||
info->u.pirq.flags = flags;
|
||||
}
|
||||
|
||||
/*
|
||||
* Accessors for packed IRQ information.
|
||||
*/
|
||||
static unsigned int evtchn_from_irq(unsigned irq)
|
||||
{
|
||||
if (unlikely(WARN(irq < 0 || irq >= nr_irqs, "Invalid irq %d!\n", irq)))
|
||||
|
@ -414,6 +444,8 @@ static int xen_allocate_irq_gsi(unsigned gsi)
|
|||
|
||||
static void xen_free_irq(unsigned irq)
|
||||
{
|
||||
irq_info[irq].type = IRQT_UNBOUND;
|
||||
|
||||
/* Legacy IRQ descriptors are managed by the arch. */
|
||||
if (irq < NR_IRQS_LEGACY)
|
||||
return;
|
||||
|
@ -610,8 +642,8 @@ int xen_bind_pirq_gsi_to_irq(unsigned gsi,
|
|||
goto out;
|
||||
}
|
||||
|
||||
irq_info[irq] = mk_pirq_info(0, pirq, gsi, irq_op.vector);
|
||||
irq_info[irq].u.pirq.flags |= shareable ? PIRQ_SHAREABLE : 0;
|
||||
xen_irq_info_pirq_init(irq, 0, pirq, gsi, irq_op.vector,
|
||||
shareable ? PIRQ_SHAREABLE : 0);
|
||||
pirq_to_irq[pirq] = irq;
|
||||
|
||||
out:
|
||||
|
@ -649,7 +681,7 @@ int xen_bind_pirq_msi_to_irq(struct pci_dev *dev, struct msi_desc *msidesc,
|
|||
set_irq_chip_and_handler_name(irq, &xen_pirq_chip,
|
||||
handle_level_irq, name);
|
||||
|
||||
irq_info[irq] = mk_pirq_info(0, pirq, 0, vector);
|
||||
xen_irq_info_pirq_init(irq, 0, pirq, 0, vector, 0);
|
||||
pirq_to_irq[pirq] = irq;
|
||||
ret = set_irq_msi(irq, msidesc);
|
||||
if (ret < 0)
|
||||
|
@ -688,8 +720,6 @@ int xen_destroy_irq(int irq)
|
|||
}
|
||||
pirq_to_irq[info->u.pirq.pirq] = -1;
|
||||
|
||||
irq_info[irq] = mk_unbound_info();
|
||||
|
||||
xen_free_irq(irq);
|
||||
|
||||
out:
|
||||
|
@ -717,7 +747,7 @@ int bind_evtchn_to_irq(unsigned int evtchn)
|
|||
handle_fasteoi_irq, "event");
|
||||
|
||||
evtchn_to_irq[evtchn] = irq;
|
||||
irq_info[irq] = mk_evtchn_info(evtchn);
|
||||
xen_irq_info_evtchn_init(irq, evtchn);
|
||||
}
|
||||
|
||||
spin_unlock(&irq_mapping_update_lock);
|
||||
|
@ -750,7 +780,7 @@ static int bind_ipi_to_irq(unsigned int ipi, unsigned int cpu)
|
|||
evtchn = bind_ipi.port;
|
||||
|
||||
evtchn_to_irq[evtchn] = irq;
|
||||
irq_info[irq] = mk_ipi_info(evtchn, ipi);
|
||||
xen_irq_info_ipi_init(irq, evtchn, ipi);
|
||||
per_cpu(ipi_to_irq, cpu)[ipi] = irq;
|
||||
|
||||
bind_evtchn_to_cpu(evtchn, cpu);
|
||||
|
@ -785,7 +815,7 @@ int bind_virq_to_irq(unsigned int virq, unsigned int cpu)
|
|||
evtchn = bind_virq.port;
|
||||
|
||||
evtchn_to_irq[evtchn] = irq;
|
||||
irq_info[irq] = mk_virq_info(evtchn, virq);
|
||||
xen_irq_info_virq_init(irq, evtchn, virq);
|
||||
|
||||
per_cpu(virq_to_irq, cpu)[virq] = irq;
|
||||
|
||||
|
@ -828,11 +858,9 @@ static void unbind_from_irq(unsigned int irq)
|
|||
evtchn_to_irq[evtchn] = -1;
|
||||
}
|
||||
|
||||
if (irq_info[irq].type != IRQT_UNBOUND) {
|
||||
irq_info[irq] = mk_unbound_info();
|
||||
BUG_ON(irq_info[irq].type == IRQT_UNBOUND);
|
||||
|
||||
xen_free_irq(irq);
|
||||
}
|
||||
xen_free_irq(irq);
|
||||
|
||||
spin_unlock(&irq_mapping_update_lock);
|
||||
}
|
||||
|
@ -1093,7 +1121,7 @@ void rebind_evtchn_irq(int evtchn, int irq)
|
|||
BUG_ON(info->type == IRQT_UNBOUND);
|
||||
|
||||
evtchn_to_irq[evtchn] = irq;
|
||||
irq_info[irq] = mk_evtchn_info(evtchn);
|
||||
xen_irq_info_evtchn_init(irq, evtchn);
|
||||
|
||||
spin_unlock(&irq_mapping_update_lock);
|
||||
|
||||
|
@ -1229,7 +1257,7 @@ static void restore_pirqs(void)
|
|||
if (rc) {
|
||||
printk(KERN_WARNING "xen map irq failed gsi=%d irq=%d pirq=%d rc=%d\n",
|
||||
gsi, irq, pirq, rc);
|
||||
irq_info[irq] = mk_unbound_info();
|
||||
xen_free_irq(irq);
|
||||
pirq_to_irq[pirq] = -1;
|
||||
continue;
|
||||
}
|
||||
|
@ -1261,7 +1289,7 @@ static void restore_cpu_virqs(unsigned int cpu)
|
|||
|
||||
/* Record the new mapping. */
|
||||
evtchn_to_irq[evtchn] = irq;
|
||||
irq_info[irq] = mk_virq_info(evtchn, virq);
|
||||
xen_irq_info_virq_init(irq, evtchn, virq);
|
||||
bind_evtchn_to_cpu(evtchn, cpu);
|
||||
}
|
||||
}
|
||||
|
@ -1286,7 +1314,7 @@ static void restore_cpu_ipis(unsigned int cpu)
|
|||
|
||||
/* Record the new mapping. */
|
||||
evtchn_to_irq[evtchn] = irq;
|
||||
irq_info[irq] = mk_ipi_info(evtchn, ipi);
|
||||
xen_irq_info_ipi_init(irq, evtchn, ipi);
|
||||
bind_evtchn_to_cpu(evtchn, cpu);
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue