On 2023.05.31 12:27:11 +0300, Jani Nikula wrote:
> On Wed, 31 May 2023, Zhenyu Wang <zhenyuw@xxxxxxxxxxxxxxx> 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 <jani.nikula@xxxxxxxxx>
> >> Cc: Zhenyu Wang <zhenyuw@xxxxxxxxxxxxxxx>
> >> Cc: intel-gvt-dev@xxxxxxxxxxxxxxxxxxxxx
> >> Signed-off-by: Zhi Wang <zhi.a.wang@xxxxxxxxx>
> >> ---
> >
> > Good with me. I think I also caught this before but never send the change..
> > Reviewed-by: Zhenyu Wang <zhenyuw@xxxxxxxxxxxxxxx>
>
> 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.
Attachment:
signature.asc
Description: PGP signature
