From: "Kuppuswamy, Sathyanarayanan" <sathyanarayanan.kuppuswamy@linux.intel.com>
To: Sean V Kelley <sean.v.kelley@intel.com>,
bhelgaas@google.com, Jonathan.Cameron@huawei.com,
xerces.zhao@gmail.com, rafael.j.wysocki@intel.com,
ashok.raj@intel.com, tony.luck@intel.com,
sathyanarayanan.kuppuswamy@intel.com, qiuxu.zhuo@intel.com
Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v11 14/16] PCI/AER: Add pcie_walk_rcec() to RCEC AER handling
Date: Tue, 17 Nov 2020 22:04:52 -0800 [thread overview]
Message-ID: <9e88e209-8d3a-5b15-5463-350ae661daa3@linux.intel.com> (raw)
In-Reply-To: <20201117191954.1322844-15-sean.v.kelley@intel.com>
On 11/17/20 11:19 AM, Sean V Kelley wrote:
> Root Complex Event Collectors (RCEC) appear as peers to Root Ports and also
> have the AER capability. In addition, actions need to be taken for
> associated RCiEPs. In such cases the RCECs will need to be walked in order
> to find and act upon their respective RCiEPs.
>
> Extend the existing ability to link the RCECs with a walking function
> pcie_walk_rcec(). Add RCEC support to the current AER service driver and
> attach the AER service driver to the RCEC device.
>
Reviewed-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>
> [bhelgaas: kernel doc, whitespace cleanup]
> Co-developed-by: Qiuxu Zhuo <qiuxu.zhuo@intel.com>
> Link: https://lore.kernel.org/r/20201002184735.1229220-13-seanvk.dev@oregontracks.org
> Signed-off-by: Qiuxu Zhuo <qiuxu.zhuo@intel.com>
> Signed-off-by: Sean V Kelley <sean.v.kelley@intel.com>
> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
> Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
> drivers/pci/pci.h | 6 ++++++
> drivers/pci/pcie/aer.c | 29 ++++++++++++++++++++++-------
> drivers/pci/pcie/rcec.c | 37 +++++++++++++++++++++++++++++++++++++
> 3 files changed, 65 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/pci/pci.h b/drivers/pci/pci.h
> index ae2ee4df1cff..e988cc930d0b 100644
> --- a/drivers/pci/pci.h
> +++ b/drivers/pci/pci.h
> @@ -473,10 +473,16 @@ static inline void pci_dpc_init(struct pci_dev *pdev) {}
> void pci_rcec_init(struct pci_dev *dev);
> void pci_rcec_exit(struct pci_dev *dev);
> void pcie_link_rcec(struct pci_dev *rcec);
> +void pcie_walk_rcec(struct pci_dev *rcec,
> + int (*cb)(struct pci_dev *, void *),
> + void *userdata);
> #else
> static inline void pci_rcec_init(struct pci_dev *dev) {}
> static inline void pci_rcec_exit(struct pci_dev *dev) {}
> static inline void pcie_link_rcec(struct pci_dev *rcec) {}
> +static inline void pcie_walk_rcec(struct pci_dev *rcec,
> + int (*cb)(struct pci_dev *, void *),
> + void *userdata) {}
> #endif
>
> #ifdef CONFIG_PCI_ATS
> diff --git a/drivers/pci/pcie/aer.c b/drivers/pci/pcie/aer.c
> index 2869212af8b4..72723a1b67af 100644
> --- a/drivers/pci/pcie/aer.c
> +++ b/drivers/pci/pcie/aer.c
> @@ -300,7 +300,8 @@ int pci_aer_raw_clear_status(struct pci_dev *dev)
> return -EIO;
>
> port_type = pci_pcie_type(dev);
> - if (port_type == PCI_EXP_TYPE_ROOT_PORT) {
> + if (port_type == PCI_EXP_TYPE_ROOT_PORT ||
> + port_type == PCI_EXP_TYPE_RC_EC) {
> pci_read_config_dword(dev, aer + PCI_ERR_ROOT_STATUS, &status);
> pci_write_config_dword(dev, aer + PCI_ERR_ROOT_STATUS, status);
> }
> @@ -595,7 +596,8 @@ static umode_t aer_stats_attrs_are_visible(struct kobject *kobj,
> if ((a == &dev_attr_aer_rootport_total_err_cor.attr ||
> a == &dev_attr_aer_rootport_total_err_fatal.attr ||
> a == &dev_attr_aer_rootport_total_err_nonfatal.attr) &&
> - pci_pcie_type(pdev) != PCI_EXP_TYPE_ROOT_PORT)
> + ((pci_pcie_type(pdev) != PCI_EXP_TYPE_ROOT_PORT) &&
> + (pci_pcie_type(pdev) != PCI_EXP_TYPE_RC_EC)))
> return 0;
>
> return a->mode;
> @@ -916,7 +918,10 @@ static bool find_source_device(struct pci_dev *parent,
> if (result)
> return true;
>
> - pci_walk_bus(parent->subordinate, find_device_iter, e_info);
> + if (pci_pcie_type(parent) == PCI_EXP_TYPE_RC_EC)
> + pcie_walk_rcec(parent, find_device_iter, e_info);
> + else
> + pci_walk_bus(parent->subordinate, find_device_iter, e_info);
>
> if (!e_info->error_dev_num) {
> pci_info(parent, "can't find device of ID%04x\n", e_info->id);
> @@ -1053,6 +1058,7 @@ int aer_get_device_error_info(struct pci_dev *dev, struct aer_err_info *info)
> if (!(info->status & ~info->mask))
> return 0;
> } else if (pci_pcie_type(dev) == PCI_EXP_TYPE_ROOT_PORT ||
> + pci_pcie_type(dev) == PCI_EXP_TYPE_RC_EC ||
> pci_pcie_type(dev) == PCI_EXP_TYPE_DOWNSTREAM ||
> info->severity == AER_NONFATAL) {
>
> @@ -1205,6 +1211,7 @@ static int set_device_error_reporting(struct pci_dev *dev, void *data)
> int type = pci_pcie_type(dev);
>
> if ((type == PCI_EXP_TYPE_ROOT_PORT) ||
> + (type == PCI_EXP_TYPE_RC_EC) ||
> (type == PCI_EXP_TYPE_UPSTREAM) ||
> (type == PCI_EXP_TYPE_DOWNSTREAM)) {
> if (enable)
> @@ -1229,9 +1236,12 @@ static void set_downstream_devices_error_reporting(struct pci_dev *dev,
> {
> set_device_error_reporting(dev, &enable);
>
> - if (!dev->subordinate)
> - return;
> - pci_walk_bus(dev->subordinate, set_device_error_reporting, &enable);
> + if (pci_pcie_type(dev) == PCI_EXP_TYPE_RC_EC)
> + pcie_walk_rcec(dev, set_device_error_reporting, &enable);
> + else if (dev->subordinate)
> + pci_walk_bus(dev->subordinate, set_device_error_reporting,
> + &enable);
> +
> }
>
> /**
> @@ -1329,6 +1339,11 @@ static int aer_probe(struct pcie_device *dev)
> struct device *device = &dev->device;
> struct pci_dev *port = dev->port;
>
> + /* Limit to Root Ports or Root Complex Event Collectors */
> + if ((pci_pcie_type(port) != PCI_EXP_TYPE_RC_EC) &&
> + (pci_pcie_type(port) != PCI_EXP_TYPE_ROOT_PORT))
> + return -ENODEV;
> +
> rpc = devm_kzalloc(device, sizeof(struct aer_rpc), GFP_KERNEL);
> if (!rpc)
> return -ENOMEM;
> @@ -1409,7 +1424,7 @@ static pci_ers_result_t aer_root_reset(struct pci_dev *dev)
>
> static struct pcie_port_service_driver aerdriver = {
> .name = "aer",
> - .port_type = PCI_EXP_TYPE_ROOT_PORT,
> + .port_type = PCIE_ANY_PORT,
> .service = PCIE_PORT_SERVICE_AER,
>
> .probe = aer_probe,
> diff --git a/drivers/pci/pcie/rcec.c b/drivers/pci/pcie/rcec.c
> index cdec277cbd62..2c5c552994e4 100644
> --- a/drivers/pci/pcie/rcec.c
> +++ b/drivers/pci/pcie/rcec.c
> @@ -53,6 +53,18 @@ static int link_rcec_helper(struct pci_dev *dev, void *data)
> return 0;
> }
>
> +static int walk_rcec_helper(struct pci_dev *dev, void *data)
> +{
> + struct walk_rcec_data *rcec_data = data;
> + struct pci_dev *rcec = rcec_data->rcec;
> +
> + if ((pci_pcie_type(dev) == PCI_EXP_TYPE_RC_END) &&
> + rcec_assoc_rciep(rcec, dev))
> + rcec_data->user_callback(dev, rcec_data->user_data);
> +
> + return 0;
> +}
> +
> static void walk_rcec(int (*cb)(struct pci_dev *dev, void *data),
> void *userdata)
> {
> @@ -109,6 +121,31 @@ void pcie_link_rcec(struct pci_dev *rcec)
> walk_rcec(link_rcec_helper, &rcec_data);
> }
>
> +/**
> + * pcie_walk_rcec - Walk RCiEP devices associating with RCEC and call callback.
> + * @rcec: RCEC whose RCiEP devices should be walked
> + * @cb: Callback to be called for each RCiEP device found
> + * @userdata: Arbitrary pointer to be passed to callback
> + *
> + * Walk the given RCEC. Call the callback on each RCiEP found.
> + *
> + * If @cb returns anything other than 0, break out.
> + */
> +void pcie_walk_rcec(struct pci_dev *rcec, int (*cb)(struct pci_dev *, void *),
> + void *userdata)
> +{
> + struct walk_rcec_data rcec_data;
> +
> + if (!rcec->rcec_ea)
> + return;
> +
> + rcec_data.rcec = rcec;
> + rcec_data.user_callback = cb;
> + rcec_data.user_data = userdata;
> +
> + walk_rcec(walk_rcec_helper, &rcec_data);
> +}
> +
> void pci_rcec_init(struct pci_dev *dev)
> {
> struct rcec_ea *rcec_ea;
>
--
Sathyanarayanan Kuppuswamy
Linux Kernel Developer
next prev parent reply other threads:[~2020-11-18 6:05 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-17 19:19 [PATCH v11 00/16] Add RCEC handling to PCI/AER Sean V Kelley
2020-11-17 19:19 ` [PATCH v11 01/16] AER: aer_root_reset() non-native handling Sean V Kelley
2020-11-17 20:04 ` Kuppuswamy, Sathyanarayanan
2020-11-17 19:19 ` [PATCH v11 02/16] PCI/RCEC: Add RCEC class code and extended capability Sean V Kelley
2020-11-17 20:07 ` Kuppuswamy, Sathyanarayanan
2020-11-17 22:39 ` Kelley, Sean V
2020-11-17 19:19 ` [PATCH v11 03/16] PCI/RCEC: Bind RCEC devices to the Root Port driver Sean V Kelley
2020-11-17 20:09 ` Kuppuswamy, Sathyanarayanan
2020-11-17 19:19 ` [PATCH v11 04/16] PCI/RCEC: Cache RCEC capabilities in pci_init_capabilities() Sean V Kelley
2020-11-17 19:19 ` [PATCH v11 05/16] PCI/ERR: Rename reset_link() to reset_subordinates() Sean V Kelley
2020-11-17 20:10 ` Kuppuswamy, Sathyanarayanan
2020-11-17 19:19 ` [PATCH v11 06/16] PCI/ERR: Simplify by using pci_upstream_bridge() Sean V Kelley
2020-11-17 20:11 ` Kuppuswamy, Sathyanarayanan
2020-11-17 19:19 ` [PATCH v11 07/16] PCI/ERR: Simplify by computing pci_pcie_type() once Sean V Kelley
2020-11-17 21:58 ` Kuppuswamy, Sathyanarayanan
2020-11-17 22:47 ` Kelley, Sean V
2020-11-17 19:19 ` [PATCH v11 08/16] PCI/ERR: Use "bridge" for clarity in pcie_do_recovery() Sean V Kelley
2020-11-17 21:59 ` Kuppuswamy, Sathyanarayanan
2020-11-17 19:19 ` [PATCH v11 09/16] PCI/ERR: Avoid negated conditional for clarity Sean V Kelley
2020-11-17 22:00 ` Kuppuswamy, Sathyanarayanan
2020-11-17 19:19 ` [PATCH v11 10/16] PCI/ERR: Add pci_walk_bridge() to pcie_do_recovery() Sean V Kelley
2020-11-17 19:19 ` [PATCH v11 11/16] PCI/ERR: Limit AER resets in pcie_do_recovery() Sean V Kelley
2020-11-17 19:19 ` [PATCH v11 12/16] PCI/RCEC: Add pcie_link_rcec() to associate RCiEPs Sean V Kelley
2020-11-17 19:19 ` [PATCH v11 13/16] PCI/RCEC: Add RCiEP's linked RCEC to AER/ERR Sean V Kelley
2020-11-17 19:19 ` [PATCH v11 14/16] PCI/AER: Add pcie_walk_rcec() to RCEC AER handling Sean V Kelley
2020-11-18 6:04 ` Kuppuswamy, Sathyanarayanan [this message]
2020-11-17 19:19 ` [PATCH v11 15/16] PCI/PME: Add pcie_walk_rcec() to RCEC PME handling Sean V Kelley
2020-11-18 6:03 ` Kuppuswamy, Sathyanarayanan
2020-11-18 16:11 ` Kelley, Sean V
2020-11-17 19:19 ` [PATCH v11 16/16] PCI/AER: Add RCEC AER error injection support Sean V Kelley
2020-11-18 6:02 ` Kuppuswamy, Sathyanarayanan
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=9e88e209-8d3a-5b15-5463-350ae661daa3@linux.intel.com \
--to=sathyanarayanan.kuppuswamy@linux.intel.com \
--cc=Jonathan.Cameron@huawei.com \
--cc=ashok.raj@intel.com \
--cc=bhelgaas@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=qiuxu.zhuo@intel.com \
--cc=rafael.j.wysocki@intel.com \
--cc=sathyanarayanan.kuppuswamy@intel.com \
--cc=sean.v.kelley@intel.com \
--cc=tony.luck@intel.com \
--cc=xerces.zhao@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox