[AMD Official Use Only - AMD Internal Distribution Only] Hi Tim -----Original Message----- From: Huang, Tim <Tim.Huang@xxxxxxx> Sent: Monday, May 13, 2024 12:23 PM To: Zhang, Jesse(Jie) <Jesse.Zhang@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Koenig, Christian <Christian.Koenig@xxxxxxx> Subject: RE: [PATCH 02/22] drm/amdgpu: the warning dereferencing obj for nbio_v7_4 [AMD Official Use Only - AMD Internal Distribution Only] Hi Jesse, > -----Original Message----- > From: Zhang, Jesse(Jie) <Jesse.Zhang@xxxxxxx> > Sent: Monday, May 13, 2024 10:18 AM > To: Zhang, Jesse(Jie) <Jesse.Zhang@xxxxxxx>; > amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Koenig, Christian > <Christian.Koenig@xxxxxxx>; Huang, Tim <Tim.Huang@xxxxxxx> > Subject: RE: [PATCH 02/22] drm/amdgpu: the warning dereferencing obj > for > nbio_v7_4 > > [AMD Official Use Only - AMD Internal Distribution Only] > > Ping ... > > -----Original Message----- > From: Jesse Zhang <jesse.zhang@xxxxxxx> > Sent: Friday, May 10, 2024 10:50 AM > To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Koenig, Christian > <Christian.Koenig@xxxxxxx>; Huang, Tim <Tim.Huang@xxxxxxx>; Zhang, > Jesse(Jie) <Jesse.Zhang@xxxxxxx>; Zhang, Jesse(Jie) > <Jesse.Zhang@xxxxxxx> > Subject: [PATCH 02/22] drm/amdgpu: the warning dereferencing obj for > nbio_v7_4 > > if ras_manager obj null, don't print NBIO err data > > Signed-off-by: Jesse Zhang <Jesse.Zhang@xxxxxxx> > --- > drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c > b/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c > index fe18df10daaa..26e5885db9b7 100644 > --- a/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c > +++ b/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c > @@ -383,7 +383,7 @@ static void > nbio_v7_4_handle_ras_controller_intr_no_bifring(struct amdgpu_device > else > WREG32_SOC15(NBIO, 0, mmBIF_DOORBELL_INT_CNTL, > bif_doorbell_intr_cntl); > > - if (!ras->disable_ras_err_cnt_harvest) { > + if (!ras->disable_ras_err_cnt_harvest && obj) { We may need to check the ras pointer as well? Such as change to " if (ras && !ras->disable_ras_err_cnt_harvest && obj) {" [Zhang, Jesse(Jie)] Thanks, will update the patch . Tim Huang > /* > * clear error status after ras_controller_intr > * according to hw team and count ue number > -- > 2.25.1 >