diff options
| author | Yifei Wan <ywan@nvidia.com> | 2014-05-27 17:35:38 -0500 |
|---|---|---|
| committer | Mandar Padmawar <mpadmawar@nvidia.com> | 2014-06-24 04:48:01 -0700 |
| commit | 6089b3610a5e400e6e049b73ae50d5d4d92f46bf (patch) | |
| tree | 84b73fe81b47f6bd117d9ff089b488bb2b5caeb2 /arch/arm64 | |
| parent | 3bdda390a84905c8fab9132829d0f217f2efc089 (diff) | |
nvdumper: Null pointer check before reference it.
- Fixed NULL pointer check before reference it.
- Also fixed some type cast warning message.
Bug 1517779
Change-Id: I8584017be83884f45e3f01a6fec60244440469c4
Signed-off-by: Yifei Wan <ywan@nvidia.com>
Reviewed-on: http://git-master/r/415443
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Mark Peters <mpeters@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Mitch Luban <mluban@nvidia.com>
Diffstat (limited to 'arch/arm64')
| -rw-r--r-- | arch/arm64/mach-tegra/nvdumper_regdump.c | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/arch/arm64/mach-tegra/nvdumper_regdump.c b/arch/arm64/mach-tegra/nvdumper_regdump.c index db2351eb454c..23e1ed4a5f59 100644 --- a/arch/arm64/mach-tegra/nvdumper_regdump.c +++ b/arch/arm64/mach-tegra/nvdumper_regdump.c @@ -28,7 +28,7 @@ #define DEBUG_REGDUMP 0 static int max_cpus; -struct nvdumper_cpu_data_t *nvdumper_cpu_data; +static struct nvdumper_cpu_data_t *nvdumper_cpu_data; static dma_addr_t nvdumper_p; void save_aar64_sys_regs(struct aar64_system_regs_t *aar64_sys_regs) @@ -62,7 +62,15 @@ void save_aar64_sys_regs(struct aar64_system_regs_t *aar64_sys_regs) void nvdumper_save_regs(void *data) { int id = smp_processor_id(); - nvdumper_cpu_data[id].current_task = current_thread_info()->task; + + if (!nvdumper_cpu_data) { + pr_info("nvdumper_cpu_data is not initialized!\n"); + return; + } + + if (current_thread_info()) + nvdumper_cpu_data[id].current_task = + current_thread_info()->task; nvdumper_cpu_data[id].is_online = true; __asm__ __volatile__ ( @@ -140,6 +148,8 @@ void nvdumper_save_regs(void *data) ); save_aar64_sys_regs(&nvdumper_cpu_data[id].aar64_sys_regs); + + pr_info("nvdumper: all registers are saved.\n"); } void nvdumper_crash_setup_regs(void) |