From: Jonathan Cameron <Jonathan.Cameron@Huawei.com>
To: Shuai Xue <xueshuai@linux.alibaba.com>
Cc: <linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>, <will@kernel.org>,
<mark.rutland@arm.com>, <baolin.wang@linux.alibaba.com>,
<yaohongbo@linux.alibaba.com>, <nengchen@linux.alibaba.com>,
<zhuo.song@linux.alibaba.com>
Subject: Re: [RESEND PATCH v2 1/3] docs: perf: Add description for Alibaba's T-Head PMU driver
Date: Tue, 19 Jul 2022 13:35:09 +0100 [thread overview]
Message-ID: <20220719133509.00006ee9@Huawei.com> (raw)
In-Reply-To: <20220715151310.90091-2-xueshuai@linux.alibaba.com>
On Fri, 15 Jul 2022 23:13:08 +0800
Shuai Xue <xueshuai@linux.alibaba.com> wrote:
> Alibaba's T-Head SoC implements uncore PMU for performance and functional
> debugging to facilitate system maintenance. Document it to provide guidance
> on how to use it.
>
> Signed-off-by: Shuai Xue <xueshuai@linux.alibaba.com>
I'm far from an expert on this, but looks good to me.
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
> .../admin-guide/perf/alibaba_pmu.rst | 100 ++++++++++++++++++
> Documentation/admin-guide/perf/index.rst | 1 +
> 2 files changed, 101 insertions(+)
> create mode 100644 Documentation/admin-guide/perf/alibaba_pmu.rst
>
> diff --git a/Documentation/admin-guide/perf/alibaba_pmu.rst b/Documentation/admin-guide/perf/alibaba_pmu.rst
> new file mode 100644
> index 000000000000..11de998bb480
> --- /dev/null
> +++ b/Documentation/admin-guide/perf/alibaba_pmu.rst
> @@ -0,0 +1,100 @@
> +=============================================================
> +Alibaba's T-Head SoC Uncore Performance Monitoring Unit (PMU)
> +=============================================================
> +
> +The Yitian 710, custom-built by Alibaba Group's chip development business,
> +T-Head, implements uncore PMU for performance and functional debugging to
> +facilitate system maintenance.
> +
> +DDR Sub-System Driveway (DRW) PMU Driver
> +=========================================
> +
> +Yitian 710 employs eight DDR5/4 channels, four on each die. Each DDR5 channel
> +is independent of others to service system memory requests. And one DDR5
> +channel is split into two independent sub-channels. The DDR Sub-System Driveway
> +implements separate PMUs for each sub-channel to monitor various performance
> +metrics.
> +
> +The Driveway PMU devices are named as ali_drw_<sys_base_addr> with perf.
> +For example, ali_drw_21000 and ali_drw_21080 are two PMU devices for two
> +sub-channels of the same channel in die 0. And the PMU device of die 1 is
> +prefixed with ali_drw_400XXXXX, e.g. ali_drw_40021000.
> +
> +Each sub-channel has 36 PMU counters in total, which is classified into
> +four groups:
> +
> +- Group 0: PMU Cycle Counter. This group has one pair of counters
> + pmu_cycle_cnt_low and pmu_cycle_cnt_high, that is used as the cycle count
> + based on DDRC core clock.
> +
> +- Group 1: PMU Bandwidth Counters. This group has 8 counters that are used
> + to count the total access number of either the eight bank groups in a
> + selected rank, or four ranks separately in the first 4 counters. The base
> + transfer unit is 64B.
> +
> +- Group 2: PMU Retry Counters. This group has 10 counters, that intend to
> + count the total retry number of each type of uncorrectable error.
> +
> +- Group 3: PMU Common Counters. This group has 16 counters, that are used
> + to count the common events.
> +
> +For now, the Driveway PMU driver only uses counters in group 0 and group 3.
> +
> +The DDR Controller (DDRCTL) and DDR PHY combine to create a complete solution
> +for connecting an SoC application bus to DDR memory devices. The DDRCTL
> +receives transactions Host Interface (HIF) which is custom-defined by Synopsys.
> +These transactions are queued internally and scheduled for access while
> +satisfying the SDRAM protocol timing requirements, transaction priorities, and
> +dependencies between the transactions. The DDRCTL in turn issues commands on
> +the DDR PHY Interface (DFI) to the PHY module, which launches and captures data
> +to and from the SDRAM. The driveway PMUs have hardware logic to gather
> +statistics and performance logging signals on HIF, DFI, etc.
> +
> +By counting the READ, WRITE and RMW commands sent to the DDRC through the HIF
> +interface, we could calculate the bandwidth. Example usage of counting memory
> +data bandwidth::
> +
> + perf stat \
> + -e ali_drw_21000/hif_wr/ \
> + -e ali_drw_21000/hif_rd/ \
> + -e ali_drw_21000/hif_rmw/ \
> + -e ali_drw_21000/cycle/ \
> + -e ali_drw_21080/hif_wr/ \
> + -e ali_drw_21080/hif_rd/ \
> + -e ali_drw_21080/hif_rmw/ \
> + -e ali_drw_21080/cycle/ \
> + -e ali_drw_23000/hif_wr/ \
> + -e ali_drw_23000/hif_rd/ \
> + -e ali_drw_23000/hif_rmw/ \
> + -e ali_drw_23000/cycle/ \
> + -e ali_drw_23080/hif_wr/ \
> + -e ali_drw_23080/hif_rd/ \
> + -e ali_drw_23080/hif_rmw/ \
> + -e ali_drw_23080/cycle/ \
> + -e ali_drw_25000/hif_wr/ \
> + -e ali_drw_25000/hif_rd/ \
> + -e ali_drw_25000/hif_rmw/ \
> + -e ali_drw_25000/cycle/ \
> + -e ali_drw_25080/hif_wr/ \
> + -e ali_drw_25080/hif_rd/ \
> + -e ali_drw_25080/hif_rmw/ \
> + -e ali_drw_25080/cycle/ \
> + -e ali_drw_27000/hif_wr/ \
> + -e ali_drw_27000/hif_rd/ \
> + -e ali_drw_27000/hif_rmw/ \
> + -e ali_drw_27000/cycle/ \
> + -e ali_drw_27080/hif_wr/ \
> + -e ali_drw_27080/hif_rd/ \
> + -e ali_drw_27080/hif_rmw/ \
> + -e ali_drw_27080/cycle/ -- sleep 10
> +
> +The average DRAM bandwidth can be calculated as follows:
> +
> +- Read Bandwidth = perf_hif_rd * DDRC_WIDTH * DDRC_Freq / DDRC_Cycle
> +- Write Bandwidth = (perf_hif_wr + perf_hif_rmw) * DDRC_WIDTH * DDRC_Freq / DDRC_Cycle
> +
> +Here, DDRC_WIDTH = 64 bytes.
> +
> +The current driver does not support sampling. So "perf record" is
> +unsupported. Also attach to a task is unsupported as the events are all
> +uncore.
> diff --git a/Documentation/admin-guide/perf/index.rst b/Documentation/admin-guide/perf/index.rst
> index 69b23f087c05..823db08863db 100644
> --- a/Documentation/admin-guide/perf/index.rst
> +++ b/Documentation/admin-guide/perf/index.rst
> @@ -17,3 +17,4 @@ Performance monitor support
> xgene-pmu
> arm_dsu_pmu
> thunderx2-pmu
> + alibaba_pmu
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: Jonathan Cameron <Jonathan.Cameron@Huawei.com>
To: Shuai Xue <xueshuai@linux.alibaba.com>
Cc: <linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>, <will@kernel.org>,
<mark.rutland@arm.com>, <baolin.wang@linux.alibaba.com>,
<yaohongbo@linux.alibaba.com>, <nengchen@linux.alibaba.com>,
<zhuo.song@linux.alibaba.com>
Subject: Re: [RESEND PATCH v2 1/3] docs: perf: Add description for Alibaba's T-Head PMU driver
Date: Tue, 19 Jul 2022 13:35:09 +0100 [thread overview]
Message-ID: <20220719133509.00006ee9@Huawei.com> (raw)
In-Reply-To: <20220715151310.90091-2-xueshuai@linux.alibaba.com>
On Fri, 15 Jul 2022 23:13:08 +0800
Shuai Xue <xueshuai@linux.alibaba.com> wrote:
> Alibaba's T-Head SoC implements uncore PMU for performance and functional
> debugging to facilitate system maintenance. Document it to provide guidance
> on how to use it.
>
> Signed-off-by: Shuai Xue <xueshuai@linux.alibaba.com>
I'm far from an expert on this, but looks good to me.
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
> .../admin-guide/perf/alibaba_pmu.rst | 100 ++++++++++++++++++
> Documentation/admin-guide/perf/index.rst | 1 +
> 2 files changed, 101 insertions(+)
> create mode 100644 Documentation/admin-guide/perf/alibaba_pmu.rst
>
> diff --git a/Documentation/admin-guide/perf/alibaba_pmu.rst b/Documentation/admin-guide/perf/alibaba_pmu.rst
> new file mode 100644
> index 000000000000..11de998bb480
> --- /dev/null
> +++ b/Documentation/admin-guide/perf/alibaba_pmu.rst
> @@ -0,0 +1,100 @@
> +=============================================================
> +Alibaba's T-Head SoC Uncore Performance Monitoring Unit (PMU)
> +=============================================================
> +
> +The Yitian 710, custom-built by Alibaba Group's chip development business,
> +T-Head, implements uncore PMU for performance and functional debugging to
> +facilitate system maintenance.
> +
> +DDR Sub-System Driveway (DRW) PMU Driver
> +=========================================
> +
> +Yitian 710 employs eight DDR5/4 channels, four on each die. Each DDR5 channel
> +is independent of others to service system memory requests. And one DDR5
> +channel is split into two independent sub-channels. The DDR Sub-System Driveway
> +implements separate PMUs for each sub-channel to monitor various performance
> +metrics.
> +
> +The Driveway PMU devices are named as ali_drw_<sys_base_addr> with perf.
> +For example, ali_drw_21000 and ali_drw_21080 are two PMU devices for two
> +sub-channels of the same channel in die 0. And the PMU device of die 1 is
> +prefixed with ali_drw_400XXXXX, e.g. ali_drw_40021000.
> +
> +Each sub-channel has 36 PMU counters in total, which is classified into
> +four groups:
> +
> +- Group 0: PMU Cycle Counter. This group has one pair of counters
> + pmu_cycle_cnt_low and pmu_cycle_cnt_high, that is used as the cycle count
> + based on DDRC core clock.
> +
> +- Group 1: PMU Bandwidth Counters. This group has 8 counters that are used
> + to count the total access number of either the eight bank groups in a
> + selected rank, or four ranks separately in the first 4 counters. The base
> + transfer unit is 64B.
> +
> +- Group 2: PMU Retry Counters. This group has 10 counters, that intend to
> + count the total retry number of each type of uncorrectable error.
> +
> +- Group 3: PMU Common Counters. This group has 16 counters, that are used
> + to count the common events.
> +
> +For now, the Driveway PMU driver only uses counters in group 0 and group 3.
> +
> +The DDR Controller (DDRCTL) and DDR PHY combine to create a complete solution
> +for connecting an SoC application bus to DDR memory devices. The DDRCTL
> +receives transactions Host Interface (HIF) which is custom-defined by Synopsys.
> +These transactions are queued internally and scheduled for access while
> +satisfying the SDRAM protocol timing requirements, transaction priorities, and
> +dependencies between the transactions. The DDRCTL in turn issues commands on
> +the DDR PHY Interface (DFI) to the PHY module, which launches and captures data
> +to and from the SDRAM. The driveway PMUs have hardware logic to gather
> +statistics and performance logging signals on HIF, DFI, etc.
> +
> +By counting the READ, WRITE and RMW commands sent to the DDRC through the HIF
> +interface, we could calculate the bandwidth. Example usage of counting memory
> +data bandwidth::
> +
> + perf stat \
> + -e ali_drw_21000/hif_wr/ \
> + -e ali_drw_21000/hif_rd/ \
> + -e ali_drw_21000/hif_rmw/ \
> + -e ali_drw_21000/cycle/ \
> + -e ali_drw_21080/hif_wr/ \
> + -e ali_drw_21080/hif_rd/ \
> + -e ali_drw_21080/hif_rmw/ \
> + -e ali_drw_21080/cycle/ \
> + -e ali_drw_23000/hif_wr/ \
> + -e ali_drw_23000/hif_rd/ \
> + -e ali_drw_23000/hif_rmw/ \
> + -e ali_drw_23000/cycle/ \
> + -e ali_drw_23080/hif_wr/ \
> + -e ali_drw_23080/hif_rd/ \
> + -e ali_drw_23080/hif_rmw/ \
> + -e ali_drw_23080/cycle/ \
> + -e ali_drw_25000/hif_wr/ \
> + -e ali_drw_25000/hif_rd/ \
> + -e ali_drw_25000/hif_rmw/ \
> + -e ali_drw_25000/cycle/ \
> + -e ali_drw_25080/hif_wr/ \
> + -e ali_drw_25080/hif_rd/ \
> + -e ali_drw_25080/hif_rmw/ \
> + -e ali_drw_25080/cycle/ \
> + -e ali_drw_27000/hif_wr/ \
> + -e ali_drw_27000/hif_rd/ \
> + -e ali_drw_27000/hif_rmw/ \
> + -e ali_drw_27000/cycle/ \
> + -e ali_drw_27080/hif_wr/ \
> + -e ali_drw_27080/hif_rd/ \
> + -e ali_drw_27080/hif_rmw/ \
> + -e ali_drw_27080/cycle/ -- sleep 10
> +
> +The average DRAM bandwidth can be calculated as follows:
> +
> +- Read Bandwidth = perf_hif_rd * DDRC_WIDTH * DDRC_Freq / DDRC_Cycle
> +- Write Bandwidth = (perf_hif_wr + perf_hif_rmw) * DDRC_WIDTH * DDRC_Freq / DDRC_Cycle
> +
> +Here, DDRC_WIDTH = 64 bytes.
> +
> +The current driver does not support sampling. So "perf record" is
> +unsupported. Also attach to a task is unsupported as the events are all
> +uncore.
> diff --git a/Documentation/admin-guide/perf/index.rst b/Documentation/admin-guide/perf/index.rst
> index 69b23f087c05..823db08863db 100644
> --- a/Documentation/admin-guide/perf/index.rst
> +++ b/Documentation/admin-guide/perf/index.rst
> @@ -17,3 +17,4 @@ Performance monitor support
> xgene-pmu
> arm_dsu_pmu
> thunderx2-pmu
> + alibaba_pmu
next prev parent reply other threads:[~2022-07-19 12:36 UTC|newest]
Thread overview: 82+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-17 11:18 [PATCH v1 0/3] drivers/perf: add DDR Sub-System Driveway PMU driver for Yitian 710 SoC Shuai Xue
2022-06-17 11:18 ` Shuai Xue
2022-06-17 11:18 ` [PATCH v1 1/3] docs: perf: Add description for Alibaba's T-Head PMU driver Shuai Xue
2022-06-17 11:18 ` Shuai Xue
2022-06-20 11:49 ` Jonathan Cameron
2022-06-20 11:49 ` Jonathan Cameron
2022-06-23 8:34 ` Shuai Xue
2022-06-23 8:34 ` Shuai Xue
2022-07-04 3:53 ` kernel test robot
2022-07-04 3:53 ` kernel test robot
2022-06-17 11:18 ` [PATCH v1 2/3] drivers/perf: add DDR Sub-System Driveway PMU driver for Yitian 710 SoC Shuai Xue
2022-06-17 11:18 ` Shuai Xue
2022-06-17 19:59 ` kernel test robot
2022-06-17 19:59 ` kernel test robot
2022-06-20 13:50 ` Jonathan Cameron
2022-06-20 13:50 ` Jonathan Cameron
2022-06-24 7:04 ` Shuai Xue
2022-06-24 7:04 ` Shuai Xue
2022-06-17 11:18 ` [PATCH v1 3/3] MAINTAINERS: add maintainers for Alibaba' T-Head PMU driver Shuai Xue
2022-06-17 11:18 ` Shuai Xue
2022-06-28 3:16 ` [PATCH v2 0/3] drivers/perf: add DDR Sub-System Driveway PMU driver for Yitian 710 SoC Shuai Xue
2022-06-28 3:16 ` Shuai Xue
2022-06-28 3:16 ` [PATCH v2 1/3] docs: perf: Add description for Alibaba's T-Head PMU driver Shuai Xue
2022-06-28 3:16 ` Shuai Xue
2022-06-28 3:16 ` [PATCH v2 2/3] drivers/perf: add DDR Sub-System Driveway PMU driver for Yitian 710 SoC Shuai Xue
2022-06-28 3:16 ` Shuai Xue
2022-06-28 3:16 ` [PATCH v2 3/3] MAINTAINERS: add maintainers for Alibaba' T-Head PMU driver Shuai Xue
2022-06-28 3:16 ` Shuai Xue
2022-07-15 15:13 ` [RESEND PATCH v2 0/3] drivers/perf: add DDR Sub-System Driveway PMU driver for Yitian 710 SoC Shuai Xue
2022-07-15 15:13 ` Shuai Xue
2022-07-15 15:13 ` [RESEND PATCH v2 1/3] docs: perf: Add description for Alibaba's T-Head PMU driver Shuai Xue
2022-07-15 15:13 ` Shuai Xue
2022-07-19 12:35 ` Jonathan Cameron [this message]
2022-07-19 12:35 ` Jonathan Cameron
2022-07-20 1:41 ` Shuai Xue
2022-07-20 1:41 ` Shuai Xue
2022-07-15 15:13 ` [RESEND PATCH v2 2/3] drivers/perf: add DDR Sub-System Driveway PMU driver for Yitian 710 SoC Shuai Xue
2022-07-15 15:13 ` Shuai Xue
2022-07-15 19:05 ` Randy Dunlap
2022-07-15 19:05 ` Randy Dunlap
2022-07-17 13:02 ` Shuai Xue
2022-07-17 13:02 ` Shuai Xue
2022-07-19 13:19 ` Jonathan Cameron
2022-07-19 13:19 ` Jonathan Cameron
2022-07-20 6:11 ` Shuai Xue
2022-07-20 6:11 ` Shuai Xue
2022-07-15 15:13 ` [RESEND PATCH v2 3/3] MAINTAINERS: add maintainers for Alibaba' T-Head PMU driver Shuai Xue
2022-07-15 15:13 ` Shuai Xue
2022-07-20 6:58 ` [PATCH v3 0/3] drivers/perf: add DDR Sub-System Driveway PMU driver for Yitian 710 SoC Shuai Xue
2022-07-20 6:58 ` Shuai Xue
2022-08-05 9:07 ` Shuai Xue
2022-08-05 9:07 ` Shuai Xue
2022-08-17 8:15 ` Baolin Wang
2022-08-17 8:15 ` Baolin Wang
2022-08-17 8:19 ` Shuai Xue
2022-08-17 8:19 ` Shuai Xue
2022-07-20 6:58 ` [PATCH v3 1/3] docs: perf: Add description for Alibaba's T-Head PMU driver Shuai Xue
2022-07-20 6:58 ` Shuai Xue
2022-07-20 6:58 ` [PATCH v3 2/3] drivers/perf: add DDR Sub-System Driveway PMU driver for Yitian 710 SoC Shuai Xue
2022-07-20 6:58 ` Shuai Xue
2022-07-20 6:58 ` [PATCH v3 3/3] MAINTAINERS: add maintainers for Alibaba' T-Head PMU driver Shuai Xue
2022-07-20 6:58 ` Shuai Xue
2022-08-18 3:18 ` [PATCH v4 0/3] drivers/perf: add DDR Sub-System Driveway PMU driver for Yitian 710 SoC Shuai Xue
2022-08-18 3:18 ` Shuai Xue
2022-09-06 4:58 ` Shuai Xue
2022-09-06 4:58 ` Shuai Xue
2022-09-22 20:33 ` Will Deacon
2022-09-22 20:33 ` Will Deacon
2022-08-18 3:18 ` [PATCH v4 1/3] docs: perf: Add description for Alibaba's T-Head PMU driver Shuai Xue
2022-08-18 3:18 ` Shuai Xue
2022-08-18 3:18 ` [PATCH v4 2/3] drivers/perf: add DDR Sub-System Driveway PMU driver for Yitian 710 SoC Shuai Xue
2022-08-18 3:18 ` Shuai Xue
2022-08-18 3:18 ` [PATCH v4 3/3] MAINTAINERS: add maintainers for Alibaba' T-Head PMU driver Shuai Xue
2022-08-18 3:18 ` Shuai Xue
2022-09-14 2:23 ` [RESEND PATCH v4 0/3] drivers/perf: add DDR Sub-System Driveway PMU driver for Yitian 710 SoC Shuai Xue
2022-09-14 2:23 ` Shuai Xue
2022-09-14 2:23 ` [RESEND PATCH v4 1/3] docs: perf: Add description for Alibaba's T-Head PMU driver Shuai Xue
2022-09-14 2:23 ` Shuai Xue
2022-09-14 2:23 ` [RESEND PATCH v4 2/3] drivers/perf: add DDR Sub-System Driveway PMU driver for Yitian 710 SoC Shuai Xue
2022-09-14 2:23 ` Shuai Xue
2022-09-14 2:23 ` [RESEND PATCH v4 3/3] MAINTAINERS: add maintainers for Alibaba' T-Head PMU driver Shuai Xue
2022-09-14 2:23 ` Shuai Xue
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=20220719133509.00006ee9@Huawei.com \
--to=jonathan.cameron@huawei.com \
--cc=baolin.wang@linux.alibaba.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=nengchen@linux.alibaba.com \
--cc=will@kernel.org \
--cc=xueshuai@linux.alibaba.com \
--cc=yaohongbo@linux.alibaba.com \
--cc=zhuo.song@linux.alibaba.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.