Re: [PATCH 3/8] genirq/affinity: factor out a irq_affinity_set helper

2017-06-17 Thread Thomas Gleixner
On Sat, 3 Jun 2017, Christoph Hellwig wrote: > + > +bool irq_affinity_set(int irq, struct irq_desc *desc, const cpumask_t *mask) > +{ > + struct irq_data *data = irq_desc_get_irq_data(desc); > + struct irq_chip *chip = irq_data_get_irq_chip(data); > + bool ret = false; > + > + if (!

Re: [PATCH 3/8] genirq/affinity: factor out a irq_affinity_set helper

2017-06-16 Thread Thomas Gleixner
On Fri, 16 Jun 2017, Thomas Gleixner wrote: > On Sat, 3 Jun 2017, Christoph Hellwig wrote: > > +bool irq_affinity_set(int irq, struct irq_desc *desc, const cpumask_t > > *mask) > > +{ > > + struct irq_data *data = irq_desc_get_irq_data(desc); > > + struct irq_chip *chip = irq_data_get_irq_chip

Re: [PATCH 3/8] genirq/affinity: factor out a irq_affinity_set helper

2017-06-16 Thread Thomas Gleixner
On Sat, 3 Jun 2017, Christoph Hellwig wrote: > +bool irq_affinity_set(int irq, struct irq_desc *desc, const cpumask_t *mask) > +{ > + struct irq_data *data = irq_desc_get_irq_data(desc); > + struct irq_chip *chip = irq_data_get_irq_chip(data); > + bool ret = false; > + > + if (!irq_

Re: [PATCH 3/8] genirq/affinity: factor out a irq_affinity_set helper

2017-06-16 Thread Thomas Gleixner
On Sat, 3 Jun 2017, Christoph Hellwig wrote: > + > +bool irq_affinity_set(int irq, struct irq_desc *desc, const cpumask_t *mask) This should be named irq_affinity_force() because it circumvents the 'move in irq context' mechanism. I'll do that myself. No need to resend. Thanks, tglx

Re: [PATCH 3/8] genirq/affinity: factor out a irq_affinity_set helper

2017-06-04 Thread Sagi Grimberg
Looks good to me, Reviewed-by: Sagi Grimberg