From: Bjorn Helgaas <helgaas@kernel.org>
To: Tyrel Datwyler <tyreld@linux.ibm.com>
Cc: mpe@ellerman.id.au, benh@kernel.crashing.org,
linuxppc-dev@lists.ozlabs.org, tlfalcon@linux.ibm.com,
nathanl@linux.ibm.com, linux-pci@vger.kernel.org
Subject: Re: [PATCH v2 0/9] Fixes and Enablement of ibm,drc-info property
Date: Mon, 11 Nov 2019 13:49:28 -0600 [thread overview]
Message-ID: <20191111194928.GA38342@google.com> (raw)
In-Reply-To: <1573449697-5448-1-git-send-email-tyreld@linux.ibm.com>
On Sun, Nov 10, 2019 at 11:21:27PM -0600, Tyrel Datwyler wrote:
> There was a previous effort to add support for the PAPR
> architected ibm,drc-info property. This property provides a more
> memory compact representation of a parition's Dynamic Reconfig
> Connectors (DRC). These can otherwise be thought of as currently
> partitioned, or available but yet to be partitioned system resources
> such as cpus, memory, and physical/logical IOA devices.
>
> The initial implementation proved buggy and was fully turned off by
> disabling the bit in the appropriate CAS support vector. We now have
> PowerVM firmware in the field that supports this new property, and
> further to support partitions with 24TB+ of possible memory this
> property is required to perform platform migration.
>
> This series fixs the short comings of the previous submission
> in the areas of general implementation, cpu hotplug, and IOA hotplug.
>
> v2 changelog:
> Cover Letter: fixed up spelling errors (mpe, tfalcon)
> Patch 3: added comment regarding indexing of drc values (tfalcon)
> split drc-index and drc-info logic into multiple
> functions for collecting cpu drc's for dlpar (mpe)
> Patch 7: fix up a couple more sparse warnings (mpe)
>
> Tyrel Datwyler (9):
> powerpc/pseries: Fix bad drc_index_start value parsing of drc-info
> entry
> powerpc/pseries: Fix drc-info mappings of logical cpus to drc-index
> powerpc/pseries: Add cpu DLPAR support for drc-info property
> PCI: rpaphp: Fix up pointer to first drc-info entry
> PCI: rpaphp: Don't rely on firmware feature to imply drc-info support
> PCI: rpaphp: Add drc-info support for hotplug slot registration
> PCI: rpaphp: Annotate and correctly byte swap DRC properties
> PCI: rpaphp: Correctly match ibm,my-drc-index to drc-name when using
> drc-info
> powerpc/pseries: Enable support for ibm,drc-info property
>
> arch/powerpc/kernel/prom_init.c | 2 +-
> arch/powerpc/platforms/pseries/hotplug-cpu.c | 127 +++++++++++++++++++++---
> arch/powerpc/platforms/pseries/of_helpers.c | 8 +-
> arch/powerpc/platforms/pseries/pseries_energy.c | 23 ++---
> drivers/pci/hotplug/rpaphp_core.c | 127 +++++++++++++++++-------
For the drivers/pci/* parts:
Acked-by: Bjorn Helgaas <bhelgaas@google.com>
I assume they will be merged along with the rest of the series via
powerpc.
> 5 files changed, 216 insertions(+), 71 deletions(-)
>
> --
> 2.7.4
>
WARNING: multiple messages have this Message-ID (diff)
From: Bjorn Helgaas <helgaas@kernel.org>
To: Tyrel Datwyler <tyreld@linux.ibm.com>
Cc: nathanl@linux.ibm.com, tlfalcon@linux.ibm.com,
linux-pci@vger.kernel.org, linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH v2 0/9] Fixes and Enablement of ibm,drc-info property
Date: Mon, 11 Nov 2019 13:49:28 -0600 [thread overview]
Message-ID: <20191111194928.GA38342@google.com> (raw)
In-Reply-To: <1573449697-5448-1-git-send-email-tyreld@linux.ibm.com>
On Sun, Nov 10, 2019 at 11:21:27PM -0600, Tyrel Datwyler wrote:
> There was a previous effort to add support for the PAPR
> architected ibm,drc-info property. This property provides a more
> memory compact representation of a parition's Dynamic Reconfig
> Connectors (DRC). These can otherwise be thought of as currently
> partitioned, or available but yet to be partitioned system resources
> such as cpus, memory, and physical/logical IOA devices.
>
> The initial implementation proved buggy and was fully turned off by
> disabling the bit in the appropriate CAS support vector. We now have
> PowerVM firmware in the field that supports this new property, and
> further to support partitions with 24TB+ of possible memory this
> property is required to perform platform migration.
>
> This series fixs the short comings of the previous submission
> in the areas of general implementation, cpu hotplug, and IOA hotplug.
>
> v2 changelog:
> Cover Letter: fixed up spelling errors (mpe, tfalcon)
> Patch 3: added comment regarding indexing of drc values (tfalcon)
> split drc-index and drc-info logic into multiple
> functions for collecting cpu drc's for dlpar (mpe)
> Patch 7: fix up a couple more sparse warnings (mpe)
>
> Tyrel Datwyler (9):
> powerpc/pseries: Fix bad drc_index_start value parsing of drc-info
> entry
> powerpc/pseries: Fix drc-info mappings of logical cpus to drc-index
> powerpc/pseries: Add cpu DLPAR support for drc-info property
> PCI: rpaphp: Fix up pointer to first drc-info entry
> PCI: rpaphp: Don't rely on firmware feature to imply drc-info support
> PCI: rpaphp: Add drc-info support for hotplug slot registration
> PCI: rpaphp: Annotate and correctly byte swap DRC properties
> PCI: rpaphp: Correctly match ibm,my-drc-index to drc-name when using
> drc-info
> powerpc/pseries: Enable support for ibm,drc-info property
>
> arch/powerpc/kernel/prom_init.c | 2 +-
> arch/powerpc/platforms/pseries/hotplug-cpu.c | 127 +++++++++++++++++++++---
> arch/powerpc/platforms/pseries/of_helpers.c | 8 +-
> arch/powerpc/platforms/pseries/pseries_energy.c | 23 ++---
> drivers/pci/hotplug/rpaphp_core.c | 127 +++++++++++++++++-------
For the drivers/pci/* parts:
Acked-by: Bjorn Helgaas <bhelgaas@google.com>
I assume they will be merged along with the rest of the series via
powerpc.
> 5 files changed, 216 insertions(+), 71 deletions(-)
>
> --
> 2.7.4
>
next prev parent reply other threads:[~2019-11-11 19:49 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-11-11 5:21 [PATCH v2 0/9] Fixes and Enablement of ibm,drc-info property Tyrel Datwyler
2019-11-11 5:21 ` Tyrel Datwyler
2019-11-11 5:21 ` [PATCH v2 1/9] powerpc/pseries: Fix bad drc_index_start value parsing of drc-info entry Tyrel Datwyler
2019-11-11 5:21 ` Tyrel Datwyler
2019-11-14 9:08 ` Michael Ellerman
2019-11-14 9:08 ` Michael Ellerman
2019-11-11 5:21 ` [PATCH v2 2/9] powerpc/pseries: Fix drc-info mappings of logical cpus to drc-index Tyrel Datwyler
2019-11-11 5:21 ` Tyrel Datwyler
2019-11-11 5:21 ` [PATCH v2 3/9] powerpc/pseries: Add cpu DLPAR support for drc-info property Tyrel Datwyler
2019-11-11 5:21 ` Tyrel Datwyler
2019-11-11 5:21 ` [PATCH v2 4/9] PCI: rpaphp: Fix up pointer to first drc-info entry Tyrel Datwyler
2019-11-11 5:21 ` Tyrel Datwyler
2019-11-11 5:21 ` [PATCH v2 5/9] PCI: rpaphp: Don't rely on firmware feature to imply drc-info support Tyrel Datwyler
2019-11-11 5:21 ` Tyrel Datwyler
2019-11-11 5:21 ` [PATCH v2 6/9] PCI: rpaphp: Add drc-info support for hotplug slot registration Tyrel Datwyler
2019-11-11 5:21 ` Tyrel Datwyler
2019-11-11 5:21 ` [PATCH v2 7/9] PCI: rpaphp: annotate and correctly byte swap DRC properties Tyrel Datwyler
2019-11-11 5:21 ` Tyrel Datwyler
2019-11-11 5:21 ` [PATCH v2 7/9] PCI: rpaphp: Annotate " Tyrel Datwyler
2019-11-11 5:21 ` Tyrel Datwyler
2019-11-11 5:21 ` [PATCH v2 8/9] PCI: rpaphp: Correctly match ibm,my-drc-index to drc-name when using drc-info Tyrel Datwyler
2019-11-11 5:21 ` [PATCH v2 8/9] PCI: rpaphp: Correctly match ibm, my-drc-index " Tyrel Datwyler
2019-11-11 5:21 ` [PATCH v2 9/9] powerpc/pseries: Enable support for ibm,drc-info property Tyrel Datwyler
2019-11-11 5:21 ` [PATCH v2 9/9] powerpc/pseries: Enable support for ibm, drc-info property Tyrel Datwyler
2019-11-11 19:49 ` Bjorn Helgaas [this message]
2019-11-11 19:49 ` [PATCH v2 0/9] Fixes and Enablement of ibm,drc-info property Bjorn Helgaas
2019-11-12 10:50 ` Michael Ellerman
2019-11-12 10:50 ` Michael Ellerman
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=20191111194928.GA38342@google.com \
--to=helgaas@kernel.org \
--cc=benh@kernel.crashing.org \
--cc=linux-pci@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=mpe@ellerman.id.au \
--cc=nathanl@linux.ibm.com \
--cc=tlfalcon@linux.ibm.com \
--cc=tyreld@linux.ibm.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.