Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1667626 > unrolled thread

Re: [PATCH 5/8] genirq/affinity: update CPU affinity for CPU hotplug events

Started byThomas Gleixner <tglx@linutronix.de>
First post2017-06-16 12:40 +0200
Last post2017-06-16 12:40 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH 5/8] genirq/affinity: update CPU affinity for CPU hotplug  events Thomas Gleixner <tglx@linutronix.de> - 2017-06-16 12:40 +0200

#1667626 — Re: [PATCH 5/8] genirq/affinity: update CPU affinity for CPU hotplug events

FromThomas Gleixner <tglx@linutronix.de>
Date2017-06-16 12:40 +0200
SubjectRe: [PATCH 5/8] genirq/affinity: update CPU affinity for CPU hotplug events
Message-ID<tSWPw-7fv-5@gated-at.bofh.it>
On Sat, 3 Jun 2017, Christoph Hellwig wrote:
> +static void irq_affinity_online_irq(unsigned int irq, struct irq_desc *desc,
> +				    unsigned int cpu)
> +{
> +	const struct cpumask *affinity;
> +	struct irq_data *data;
> +	struct irq_chip *chip;
> +	unsigned long flags;
> +	cpumask_var_t mask;
> +
> +	if (!desc)
> +		return;
> +	if (!zalloc_cpumask_var(&mask, GFP_KERNEL))
> +		return;

That's silly. Why want you to do that for each irq descriptor? That's
outright silly. Either you allocated it in irq_affinity_[on|off]line_cpu()
once or just make it a static cpumask. Lemme fix that up.

Thanks,

	tglx

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web