From ac9a59075e457532373a5e0f45ed13c2b4d218eb Mon Sep 17 00:00:00 2001 From: Martin Radev Date: Thu, 16 Mar 2023 13:05:50 +0200 Subject: [PATCH] gpu: nvgpu: Print flags after all flags are set Without this change, nvgpu would print out some flags as disabled in dmesg but enable them shortly after. This leads to confusion when examining UMD and nvgpu reporting in UMDs. This patch adds code to print out the flags after all flags are set. Bug 4031904 Change-Id: I67b9a4567886fd5e076f7ac3b8f284b52c03d7e4 Signed-off-by: Martin Radev Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvgpu/+/2871606 Reviewed-by: svcacv Reviewed-by: svc-mobile-coverity Reviewed-by: svc-mobile-misra Reviewed-by: svc-mobile-cert Reviewed-by: Alex Waterman Reviewed-by: Vijayakumar Subbu Reviewed-by: Seema Khowala GVS: Gerrit_Virtual_Submit --- drivers/gpu/nvgpu/common/init/nvgpu_init.c | 1 - drivers/gpu/nvgpu/os/linux/module.c | 3 +++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/nvgpu/common/init/nvgpu_init.c b/drivers/gpu/nvgpu/common/init/nvgpu_init.c index 790602637..c6ea2f4c3 100644 --- a/drivers/gpu/nvgpu/common/init/nvgpu_init.c +++ b/drivers/gpu/nvgpu/common/init/nvgpu_init.c @@ -1055,7 +1055,6 @@ int nvgpu_finalize_poweron(struct gk20a *g) } } - nvgpu_print_enabled_flags(g); return err; done: diff --git a/drivers/gpu/nvgpu/os/linux/module.c b/drivers/gpu/nvgpu/os/linux/module.c index 8067ddccf..dd1389388 100644 --- a/drivers/gpu/nvgpu/os/linux/module.c +++ b/drivers/gpu/nvgpu/os/linux/module.c @@ -537,6 +537,9 @@ int gk20a_pm_finalize_poweron(struct device *dev) /* Initialize linux specific flags */ gk20a_init_linux_characteristics(g); + /* Print out the NvGPU flags for debug purposes after all flags are set. */ + nvgpu_print_enabled_flags(g); + err = nvgpu_init_os_linux_ops(l); if (err) goto done;