Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Hannes Reinecke
On 11/09/2016 02:15 AM, Christoph Hellwig wrote: Only calculate the affinity for the main I/O vectors, and skip the pre or post vectors specified by struct irq_affinity. Also remove the irq_affinity cpumask argument that has never been used. If we ever need it in the future we can pass it

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Hannes Reinecke
On 11/09/2016 02:15 AM, Christoph Hellwig wrote: Only calculate the affinity for the main I/O vectors, and skip the pre or post vectors specified by struct irq_affinity. Also remove the irq_affinity cpumask argument that has never been used. If we ever need it in the future we can pass it

[PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Christoph Hellwig
Only calculate the affinity for the main I/O vectors, and skip the pre or post vectors specified by struct irq_affinity. Also remove the irq_affinity cpumask argument that has never been used. If we ever need it in the future we can pass it through struct irq_affinity. Signed-off-by: Christoph

[PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Christoph Hellwig
Only calculate the affinity for the main I/O vectors, and skip the pre or post vectors specified by struct irq_affinity. Also remove the irq_affinity cpumask argument that has never been used. If we ever need it in the future we can pass it through struct irq_affinity. Signed-off-by: Christoph

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Bjorn Helgaas
On Mon, Nov 07, 2016 at 10:47:38AM -0800, Christoph Hellwig wrote: > From: Christogh Hellwig > > Only calculate the affinity for the main I/O vectors, and skip the > pre or post vectors specified by struct irq_affinity. > > Also remove the irq_affinity cpumask argument that has

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Bjorn Helgaas
On Mon, Nov 07, 2016 at 10:47:38AM -0800, Christoph Hellwig wrote: > From: Christogh Hellwig > > Only calculate the affinity for the main I/O vectors, and skip the > pre or post vectors specified by struct irq_affinity. > > Also remove the irq_affinity cpumask argument that has never been used.

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Christoph Hellwig
On Tue, Nov 08, 2016 at 05:27:52PM +0100, Thomas Gleixner wrote: > On Tue, 8 Nov 2016, Christoph Hellwig wrote: > > > On Tue, Nov 08, 2016 at 03:59:16PM +0100, Hannes Reinecke wrote: > > > > > > Which you don't in this patch: > > > > True. We will always in the end, but the split isn't right,

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Christoph Hellwig
On Tue, Nov 08, 2016 at 05:27:52PM +0100, Thomas Gleixner wrote: > On Tue, 8 Nov 2016, Christoph Hellwig wrote: > > > On Tue, Nov 08, 2016 at 03:59:16PM +0100, Hannes Reinecke wrote: > > > > > > Which you don't in this patch: > > > > True. We will always in the end, but the split isn't right,

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Thomas Gleixner
On Tue, 8 Nov 2016, Christoph Hellwig wrote: > On Tue, Nov 08, 2016 at 03:59:16PM +0100, Hannes Reinecke wrote: > > > > Which you don't in this patch: > > True. We will always in the end, but the split isn't right, we'll > need to pass the non-NULL argument starting in this patch. No, in the

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Thomas Gleixner
On Tue, 8 Nov 2016, Christoph Hellwig wrote: > On Tue, Nov 08, 2016 at 03:59:16PM +0100, Hannes Reinecke wrote: > > > > Which you don't in this patch: > > True. We will always in the end, but the split isn't right, we'll > need to pass the non-NULL argument starting in this patch. No, in the

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Hannes Reinecke
On 11/08/2016 03:55 PM, Christoph Hellwig wrote: [please trim the f***king context in your replies, thanks..] On Tue, Nov 08, 2016 at 09:15:27AM +0100, Hannes Reinecke wrote: +irq_create_affinity_masks(int nvecs, const struct irq_affinity *affd) { - int n, nodes, vecs_per_node,

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Hannes Reinecke
On 11/08/2016 03:55 PM, Christoph Hellwig wrote: [please trim the f***king context in your replies, thanks..] On Tue, Nov 08, 2016 at 09:15:27AM +0100, Hannes Reinecke wrote: +irq_create_affinity_masks(int nvecs, const struct irq_affinity *affd) { - int n, nodes, vecs_per_node,

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Christoph Hellwig
On Tue, Nov 08, 2016 at 03:59:16PM +0100, Hannes Reinecke wrote: > > Which you don't in this patch: True. We will always in the end, but the split isn't right, we'll need to pass the non-NULL argument starting in this patch.

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Christoph Hellwig
On Tue, Nov 08, 2016 at 03:59:16PM +0100, Hannes Reinecke wrote: > > Which you don't in this patch: True. We will always in the end, but the split isn't right, we'll need to pass the non-NULL argument starting in this patch.

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Christoph Hellwig
[please trim the f***king context in your replies, thanks..] On Tue, Nov 08, 2016 at 09:15:27AM +0100, Hannes Reinecke wrote: >> +irq_create_affinity_masks(int nvecs, const struct irq_affinity *affd) >> { >> -int n, nodes, vecs_per_node, cpus_per_vec, extra_vecs, curvec = 0; >> +int n,

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Christoph Hellwig
[please trim the f***king context in your replies, thanks..] On Tue, Nov 08, 2016 at 09:15:27AM +0100, Hannes Reinecke wrote: >> +irq_create_affinity_masks(int nvecs, const struct irq_affinity *affd) >> { >> -int n, nodes, vecs_per_node, cpus_per_vec, extra_vecs, curvec = 0; >> +int n,

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Hannes Reinecke
On 11/07/2016 07:47 PM, Christoph Hellwig wrote: From: Christogh Hellwig Only calculate the affinity for the main I/O vectors, and skip the pre or post vectors specified by struct irq_affinity. Also remove the irq_affinity cpumask argument that has never been used. If we ever

Re: [PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-08 Thread Hannes Reinecke
On 11/07/2016 07:47 PM, Christoph Hellwig wrote: From: Christogh Hellwig Only calculate the affinity for the main I/O vectors, and skip the pre or post vectors specified by struct irq_affinity. Also remove the irq_affinity cpumask argument that has never been used. If we ever need it in the

[PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-07 Thread Christoph Hellwig
From: Christogh Hellwig Only calculate the affinity for the main I/O vectors, and skip the pre or post vectors specified by struct irq_affinity. Also remove the irq_affinity cpumask argument that has never been used. If we ever need it in the future we can pass it through struct

[PATCH 3/7] genirq/affinity: Handle pre/post vectors in irq_create_affinity_masks()

2016-11-07 Thread Christoph Hellwig
From: Christogh Hellwig Only calculate the affinity for the main I/O vectors, and skip the pre or post vectors specified by struct irq_affinity. Also remove the irq_affinity cpumask argument that has never been used. If we ever need it in the future we can pass it through struct irq_affinity.