On 8/26/25 09:50, Mario Limonciello wrote:
On 8/26/2025 11:03 AM, Lizhi Hou wrote:
drivers/accel/amdxdna/aie2_pci.c:794:13: sparse: sparse: incorrect
type in assignment (different address spaces)
Fixes: c8cea4371e5e ("accel/amdxdna: Add a function to walk hardware
contexts")
Reported-by: kernel test robot <l...@intel.com>
Closes:
https://lore.kernel.org/oe-kbuild-all/202508230855.0b9effl6-...@intel.com/
Signed-off-by: Lizhi Hou <lizhi....@amd.com>
---
drivers/accel/amdxdna/aie2_pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/accel/amdxdna/aie2_pci.c
b/drivers/accel/amdxdna/aie2_pci.c
index 16ac0cab4f44..2885a3c17e89 100644
--- a/drivers/accel/amdxdna/aie2_pci.c
+++ b/drivers/accel/amdxdna/aie2_pci.c
@@ -785,7 +785,7 @@ static int aie2_get_clock_metadata(struct
amdxdna_client *client,
static int aie2_hwctx_status_cb(struct amdxdna_hwctx *hwctx, void
*arg)
{
- struct amdxdna_drm_query_hwctx __user *buf, *tmp __free(kfree) =
NULL;
+ struct amdxdna_drm_query_hwctx *tmp __free(kfree) = NULL, __user
*buf;
IMO it would be better to just split these into two separate
declarations so that we don't have to worry about nuance of order of
operations resolving the type.
Sure.
Thanks,
Lizhi
struct amdxdna_drm_get_info *get_info_args = arg;
if (get_info_args->buffer_size < sizeof(*tmp))
