All of lore.kernel.org
 help / color / mirror / Atom feed
From: Robert Richter <rric@kernel.org>
To: Marc Zyngier <marc.zyngier@arm.com>
Cc: Ganapatrao Kulkarni <ganapatrao.kulkarni@cavium.com>,
	linux-acpi@vger.kernel.org, devel@acpica.org,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, lorenzo.pieralisi@arm.com,
	jason@lakedaemon.net, catalin.marinas@arm.com,
	jnair@caviumnetworks.com, will.deacon@arm.com,
	tglx@linutronix.de
Subject: Re: [Devel] [PATCH v4 2/2] acpi, gicv3-its, numa: Adding numa node mapping for gic-its units
Date: Mon, 26 Jun 2017 19:38:10 +0200	[thread overview]
Message-ID: <20170626173810.GW658@rric.localdomain> (raw)
In-Reply-To: <2d99c999-8ded-ac3d-57a2-fd81dbadadea@arm.com>

On 22.06.17 15:49:25, Marc Zyngier wrote:
> On 22/06/17 07:10, Ganapatrao Kulkarni wrote:
> > Add code to parse SRAT ITS Affinity sub table as defined in ACPI 6.2.
> > Later in per device probe, ITS devices are mapped to numa node using
> > ITS Id to proximity domain mapping.
> > 
> > Signed-off-by: Ganapatrao Kulkarni <ganapatrao.kulkarni@cavium.com>
> > ---
> >  drivers/irqchip/irq-gic-v3-its.c | 75 +++++++++++++++++++++++++++++++++++++++-
> >  1 file changed, 74 insertions(+), 1 deletion(-)
> > 
> > diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c
> > index 45ea1933..1c21e01 100644
> > --- a/drivers/irqchip/irq-gic-v3-its.c
> > +++ b/drivers/irqchip/irq-gic-v3-its.c
> > @@ -1833,6 +1833,77 @@ static int __init its_of_probe(struct device_node *node)
> >  
> >  #define ACPI_GICV3_ITS_MEM_SIZE (SZ_128K)
> >  
> > +#ifdef CONFIG_ACPI_NUMA
> 
> So given that there is a dependency issue between the irqchip and apcica
> trees, I plan on taking this patch with the following change:
> 
> #if defined(CONFIG_ACPI_NUMA) && (ACPI_CA_VERSION >= 0x20170531)

The struct name was changed :/

 s/acpi_srat_its_affinity/acpi_srat_gic_its_affinity/

You need to update this too.

-Robert

> 
> matching what is currently in -next.
> 
> Thanks,
> 
> 	M.
> -- 
> Jazz is not dead. It just smells funny...
> _______________________________________________
> Devel mailing list
> Devel@acpica.org
> https://lists.acpica.org/mailman/listinfo/devel

WARNING: multiple messages have this Message-ID (diff)
From: rric@kernel.org (Robert Richter)
To: linux-arm-kernel@lists.infradead.org
Subject: [Devel] [PATCH v4 2/2] acpi, gicv3-its, numa: Adding numa node mapping for gic-its units
Date: Mon, 26 Jun 2017 19:38:10 +0200	[thread overview]
Message-ID: <20170626173810.GW658@rric.localdomain> (raw)
In-Reply-To: <2d99c999-8ded-ac3d-57a2-fd81dbadadea@arm.com>

On 22.06.17 15:49:25, Marc Zyngier wrote:
> On 22/06/17 07:10, Ganapatrao Kulkarni wrote:
> > Add code to parse SRAT ITS Affinity sub table as defined in ACPI 6.2.
> > Later in per device probe, ITS devices are mapped to numa node using
> > ITS Id to proximity domain mapping.
> > 
> > Signed-off-by: Ganapatrao Kulkarni <ganapatrao.kulkarni@cavium.com>
> > ---
> >  drivers/irqchip/irq-gic-v3-its.c | 75 +++++++++++++++++++++++++++++++++++++++-
> >  1 file changed, 74 insertions(+), 1 deletion(-)
> > 
> > diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c
> > index 45ea1933..1c21e01 100644
> > --- a/drivers/irqchip/irq-gic-v3-its.c
> > +++ b/drivers/irqchip/irq-gic-v3-its.c
> > @@ -1833,6 +1833,77 @@ static int __init its_of_probe(struct device_node *node)
> >  
> >  #define ACPI_GICV3_ITS_MEM_SIZE (SZ_128K)
> >  
> > +#ifdef CONFIG_ACPI_NUMA
> 
> So given that there is a dependency issue between the irqchip and apcica
> trees, I plan on taking this patch with the following change:
> 
> #if defined(CONFIG_ACPI_NUMA) && (ACPI_CA_VERSION >= 0x20170531)

The struct name was changed :/

 s/acpi_srat_its_affinity/acpi_srat_gic_its_affinity/

You need to update this too.

-Robert

> 
> matching what is currently in -next.
> 
> Thanks,
> 
> 	M.
> -- 
> Jazz is not dead. It just smells funny...
> _______________________________________________
> Devel mailing list
> Devel at acpica.org
> https://lists.acpica.org/mailman/listinfo/devel

WARNING: multiple messages have this Message-ID (diff)
From: Robert Richter <rric at kernel.org>
To: devel@acpica.org
Subject: Re: [Devel] [PATCH v4 2/2] acpi, gicv3-its, numa: Adding numa node mapping for gic-its units
Date: Mon, 26 Jun 2017 19:38:10 +0200	[thread overview]
Message-ID: <20170626173810.GW658@rric.localdomain> (raw)
In-Reply-To: 2d99c999-8ded-ac3d-57a2-fd81dbadadea@arm.com

[-- Attachment #1: Type: text/plain, Size: 1525 bytes --]

On 22.06.17 15:49:25, Marc Zyngier wrote:
> On 22/06/17 07:10, Ganapatrao Kulkarni wrote:
> > Add code to parse SRAT ITS Affinity sub table as defined in ACPI 6.2.
> > Later in per device probe, ITS devices are mapped to numa node using
> > ITS Id to proximity domain mapping.
> > 
> > Signed-off-by: Ganapatrao Kulkarni <ganapatrao.kulkarni(a)cavium.com>
> > ---
> >  drivers/irqchip/irq-gic-v3-its.c | 75 +++++++++++++++++++++++++++++++++++++++-
> >  1 file changed, 74 insertions(+), 1 deletion(-)
> > 
> > diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c
> > index 45ea1933..1c21e01 100644
> > --- a/drivers/irqchip/irq-gic-v3-its.c
> > +++ b/drivers/irqchip/irq-gic-v3-its.c
> > @@ -1833,6 +1833,77 @@ static int __init its_of_probe(struct device_node *node)
> >  
> >  #define ACPI_GICV3_ITS_MEM_SIZE (SZ_128K)
> >  
> > +#ifdef CONFIG_ACPI_NUMA
> 
> So given that there is a dependency issue between the irqchip and apcica
> trees, I plan on taking this patch with the following change:
> 
> #if defined(CONFIG_ACPI_NUMA) && (ACPI_CA_VERSION >= 0x20170531)

The struct name was changed :/

 s/acpi_srat_its_affinity/acpi_srat_gic_its_affinity/

You need to update this too.

-Robert

> 
> matching what is currently in -next.
> 
> Thanks,
> 
> 	M.
> -- 
> Jazz is not dead. It just smells funny...
> _______________________________________________
> Devel mailing list
> Devel(a)acpica.org
> https://lists.acpica.org/mailman/listinfo/devel

  reply	other threads:[~2017-06-26 17:38 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-22  6:10 [PATCH v4 0/2] acpi, gicv3-its, numa: Adding numa node mapping for ITS Ganapatrao Kulkarni
2017-06-22  6:10 ` [Devel] " Ganapatrao Kulkarni
2017-06-22  6:10 ` Ganapatrao Kulkarni
2017-06-22  6:10 ` [PATCH v4 1/2] ACPICA: ACPI 6.2: Add support for new SRAT subtable Ganapatrao Kulkarni
2017-06-22  6:10   ` [Devel] " Ganapatrao Kulkarni
2017-06-22  6:10   ` Ganapatrao Kulkarni
2017-06-22 13:43   ` Lorenzo Pieralisi
2017-06-22 13:43     ` [Devel] " Lorenzo Pieralisi
2017-06-22 13:43     ` Lorenzo Pieralisi
2017-06-22 14:13     ` Moore, Robert
2017-06-22 14:13       ` [Devel] " Moore, Robert
2017-06-22 14:13       ` Moore, Robert
2017-06-22 14:13       ` Moore, Robert
2017-06-22 14:27       ` Rafael J. Wysocki
2017-06-22 14:27         ` [Devel] " Rafael J. Wysocki
2017-06-22 14:27         ` Rafael J. Wysocki
2017-06-22 14:27         ` Rafael J. Wysocki
2017-06-22 16:21         ` Ganapatrao Kulkarni
2017-06-22 16:21           ` [Devel] " Ganapatrao Kulkarni
2017-06-22 16:21           ` Ganapatrao Kulkarni
2017-06-22 16:21           ` Ganapatrao Kulkarni
2017-06-22 21:06         ` Lorenzo Pieralisi
2017-06-22 21:06           ` [Devel] " Lorenzo Pieralisi
2017-06-22 21:06           ` Lorenzo Pieralisi
2017-06-22 21:06           ` Lorenzo Pieralisi
2017-06-22  6:10 ` [PATCH v4 2/2] acpi, gicv3-its, numa: Adding numa node mapping for gic-its units Ganapatrao Kulkarni
2017-06-22  6:10   ` [Devel] " Ganapatrao Kulkarni
2017-06-22  6:10   ` Ganapatrao Kulkarni
2017-06-22 13:46   ` Lorenzo Pieralisi
2017-06-22 13:46     ` [Devel] " Lorenzo Pieralisi
2017-06-22 13:46     ` Lorenzo Pieralisi
2017-06-22 14:49   ` Marc Zyngier
2017-06-22 14:49     ` [Devel] " Marc Zyngier
2017-06-22 14:49     ` Marc Zyngier
2017-06-26 17:38     ` Robert Richter [this message]
2017-06-26 17:38       ` [Devel] " Robert Richter
2017-06-26 17:38       ` Robert Richter
2017-06-26 17:43       ` Marc Zyngier
2017-06-26 17:43         ` Marc Zyngier
2017-06-26 17:43         ` Marc Zyngier
2017-06-26 17:45         ` Robert Richter
2017-06-26 17:45           ` Robert Richter
2017-06-26 17:45           ` Robert Richter

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170626173810.GW658@rric.localdomain \
    --to=rric@kernel.org \
    --cc=catalin.marinas@arm.com \
    --cc=devel@acpica.org \
    --cc=ganapatrao.kulkarni@cavium.com \
    --cc=jason@lakedaemon.net \
    --cc=jnair@caviumnetworks.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lorenzo.pieralisi@arm.com \
    --cc=marc.zyngier@arm.com \
    --cc=tglx@linutronix.de \
    --cc=will.deacon@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.