diff mbox series

platform/x86/amd/hsmp: Fix iomem handling

Message ID 20231019135122.21774-1-W_Armin@gmx.de
State New
Headers
Series platform/x86/amd/hsmp: Fix iomem handling |

Commit Message

Armin Wolf Oct. 19, 2023, 1:51 p.m. UTC 
  Since the metrics table is marked as io memory,
use memcpy_fromio() when copying its contents.

Compile-tested only.

Fixes: 5150542b8ec5 ("platform/x86/amd/hsmp: add support for metrics tbl")
Signed-off-by: Armin Wolf <W_Armin@gmx.de>
---
 drivers/platform/x86/amd/hsmp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.39.2

Comments

Ilpo Järvinen Oct. 25, 2023, 9:49 a.m. UTC | #1 
On Thu, 19 Oct 2023 15:51:22 +0200, Armin Wolf wrote:

> Since the metrics table is marked as io memory,
> use memcpy_fromio() when copying its contents.
> 
> Compile-tested only.
> 
> 


Thank you for your contribution, it has been applied to my local
review-ilpo branch. Note it will show up in the public
platform-drivers-x86/review-ilpo branch only once I've pushed my
local branch there, which might take a while.

Once I've run some tests on the review-ilpo branch the patches
there will be added to the platform-drivers-x86/for-next branch
and eventually will be included in the pdx86 pull-request to
Linus for the next merge-window.

The list of commits applied:
[1/1] platform/x86/amd/hsmp: Fix iomem handling
      commit: 02a258ac3ebf8a530d340ec24a694c13ac9d25c0

--
 i.
diff mbox series

Patch

diff --git a/drivers/platform/x86/amd/hsmp.c b/drivers/platform/x86/amd/hsmp.c
index b056a5c8654a..b55d80e29139 100644
--- a/drivers/platform/x86/amd/hsmp.c
+++ b/drivers/platform/x86/amd/hsmp.c
@@ -362,7 +362,7 @@  static ssize_t hsmp_metric_tbl_read(struct file *filp, struct kobject *kobj,
 	ret = hsmp_send_message(&msg);
 	if (ret)
 		return ret;
-	memcpy(buf, sock->metric_tbl_addr, bin_attr->size);
+	memcpy_fromio(buf, sock->metric_tbl_addr, bin_attr->size);

 	return bin_attr->size;
 }