On 2023.05.31 12:27:11 +0300, Jani Nikula wrote: > On Wed, 31 May 2023, Zhenyu Wang <[email protected]> wrote: > > On 2023.05.31 02:04:11 +0000, Zhi Wang wrote: > >> Remove unused variable gma_bottom in scan_workload() and scan_wa_ctx(). > >> commit be1da7070aea ("drm/i915/gvt: vGPU command scanner") introduces > >> gma_bottom in several functions to calculate the size of the command > >> buffer. However, some of them are set but actually unused. > >> > >> When compiling the code with ccflags -Wunused-but-set-variable, gcc > >> throws warnings. > >> > >> Remove unused variables to avoid the gcc warnings. Tested via compiling > >> the code with ccflags -Wunused-but-set-variable. > >> > >> Fixes: be1da7070aea ("drm/i915/gvt: vGPU command scanner") > >> Suggested-by: Jani Nikula <[email protected]> > >> Cc: Zhenyu Wang <[email protected]> > >> Cc: [email protected] > >> Signed-off-by: Zhi Wang <[email protected]> > >> --- > > > > Good with me. I think I also caught this before but never send the change.. > > Reviewed-by: Zhenyu Wang <[email protected]> > > I'd like to pick this up via drm-intel-next if that's all right. >
Sorry for missed this. Pls feel free to do so. I'll sync to check anything left in gvt tree. Thanks.
signature.asc
Description: PGP signature
