On Mon, Dec 4, 2023 at 9:23 AM Wafer <wa...@jaguarmicro.com> wrote:
> vcpu_dirty_stat_collect() has an unused parameter so remove it.
>
> Signed-off-by: Wafer <wa...@jaguarmicro.com>
> ---
> migration/dirtyrate.c | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/migration/dirtyrate.c b/migration/dirtyrate.c
> index 036ac017fc..62d86b8be2 100644
> --- a/migration/dirtyrate.c
> +++ b/migration/dirtyrate.c
> @@ -129,8 +129,7 @@ static DirtyPageRecord *vcpu_dirty_stat_alloc(VcpuStat
> *stat)
> return g_new0(DirtyPageRecord, nvcpu);
> }
>
> -static void vcpu_dirty_stat_collect(VcpuStat *stat,
> - DirtyPageRecord *records,
> +static void vcpu_dirty_stat_collect(DirtyPageRecord *records,
> bool start)
> {
> CPUState *cpu;
> @@ -158,7 +157,7 @@ retry:
> WITH_QEMU_LOCK_GUARD(&qemu_cpu_list_lock) {
> gen_id = cpu_list_generation_id_get();
> records = vcpu_dirty_stat_alloc(stat);
> - vcpu_dirty_stat_collect(stat, records, true);
> + vcpu_dirty_stat_collect(records, true);
> }
>
> duration = dirty_stat_wait(calc_time_ms, init_time_ms);
> @@ -172,7 +171,7 @@ retry:
> cpu_list_unlock();
> goto retry;
> }
> - vcpu_dirty_stat_collect(stat, records, false);
> + vcpu_dirty_stat_collect(records, false);
> }
>
> for (i = 0; i < stat->nvcpu; i++) {
> --
> 2.27.0
>
>
Reviewed-by: Hyman Huang <yong.hu...@smartx.com>
Thanks
--
Best regards
