From: Yazen Ghannam <yazen.ghannam@amd.com>
To: John Allen <john.allen@amd.com>,
bp@alien8.de, linux-edac@vger.kernel.org, tony.luck@intel.com
Cc: yazen.ghannam@amd.com, linux-kernel@vger.kernel.org,
avadhut.naik@amd.com, muralidhara.mk@amd.com
Subject: Re: [PATCH 1/4] RAS: ATL: Read DRAM hole base early
Date: Mon, 18 Mar 2024 11:06:02 -0400 [thread overview]
Message-ID: <10919f69-4008-4156-88e4-3c72e3775562@amd.com> (raw)
In-Reply-To: <20240314163527.63321-2-john.allen@amd.com>
On 3/14/24 12:35, John Allen wrote:
Please use "RAS/AMD/ATL:" for the $SUBJECT prefix.
> Read DRAM hole base when constructing the address map as the value will
> not change during translation.
More specifically, the value will not change during run time.
>
> Signed-off-by: John Allen <john.allen@amd.com>
> ---
> drivers/ras/amd/atl/core.c | 15 ++-------------
> drivers/ras/amd/atl/internal.h | 2 ++
> drivers/ras/amd/atl/system.c | 21 +++++++++++++++++++++
> 3 files changed, 25 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/ras/amd/atl/core.c b/drivers/ras/amd/atl/core.c
> index 6dc4e06305f7..c1710d233adb 100644
> --- a/drivers/ras/amd/atl/core.c
> +++ b/drivers/ras/amd/atl/core.c
> @@ -51,22 +51,11 @@ static bool legacy_hole_en(struct addr_ctx *ctx)
>
> static int add_legacy_hole(struct addr_ctx *ctx)
> {
> - u32 dram_hole_base;
> - u8 func = 0;
> -
> if (!legacy_hole_en(ctx))
> return 0;
>
> - if (df_cfg.rev >= DF4)
> - func = 7;
> -
> - if (df_indirect_read_broadcast(ctx->node_id, func, 0x104, &dram_hole_base))
> - return -EINVAL;
> -
> - dram_hole_base &= DF_DRAM_HOLE_BASE_MASK;
> -
> - if (ctx->ret_addr >= dram_hole_base)
> - ctx->ret_addr += (BIT_ULL(32) - dram_hole_base);
> + if (ctx->addr >= df_cfg.dram_hole_base)
> + ctx->addr += (BIT_ULL(32) - df_cfg.dram_hole_base);
There is a compilation error here. Please make sure to build each patch
individually.
Thanks,
Yazen
next prev parent reply other threads:[~2024-03-18 15:06 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-14 16:35 [PATCH 0/4] RAS: ATL: DF 4.5 NP2 Denormalization John Allen
2024-03-14 16:35 ` [PATCH 1/4] RAS: ATL: Read DRAM hole base early John Allen
2024-03-18 15:06 ` Yazen Ghannam [this message]
2024-03-14 16:35 ` [PATCH 2/4] RAS: ATL: Expand helpers for adding and removing base and hole John Allen
2024-03-18 15:28 ` Yazen Ghannam
2024-03-25 19:27 ` John Allen
2024-03-26 14:13 ` Yazen Ghannam
2024-03-14 16:35 ` [PATCH 3/4] RAS: ATL: Add map_bits_valid to header John Allen
2024-03-18 15:46 ` Yazen Ghannam
2024-03-25 19:57 ` John Allen
2024-03-26 13:47 ` Yazen Ghannam
2024-03-14 16:35 ` [PATCH 4/4] RAS: ATL: Implement DF 4.5 NP2 denormalization John Allen
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=10919f69-4008-4156-88e4-3c72e3775562@amd.com \
--to=yazen.ghannam@amd.com \
--cc=avadhut.naik@amd.com \
--cc=bp@alien8.de \
--cc=john.allen@amd.com \
--cc=linux-edac@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=muralidhara.mk@amd.com \
--cc=tony.luck@intel.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;
as well as URLs for NNTP newsgroup(s).