[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 6/7] migration: Dump sub-cmd name in loadvm_process_command t
From: |
Peter Xu |
Subject: |
Re: [PATCH 6/7] migration: Dump sub-cmd name in loadvm_process_command tp |
Date: |
Thu, 9 Dec 2021 09:44:43 +0800 |
On Wed, Dec 08, 2021 at 06:41:22PM +0000, Dr. David Alan Gilbert wrote:
> * Peter Xu (peterx@redhat.com) wrote:
> > It'll be easier to read the name rather than index of sub-cmd when
> > debugging.
> >
> > Signed-off-by: Peter Xu <peterx@redhat.com>
> > ---
> > migration/savevm.c | 2 +-
> > migration/trace-events | 2 +-
> > 2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/migration/savevm.c b/migration/savevm.c
> > index d59e976d50..17b8e25e00 100644
> > --- a/migration/savevm.c
> > +++ b/migration/savevm.c
> > @@ -2271,7 +2271,7 @@ static int loadvm_process_command(QEMUFile *f)
> > return qemu_file_get_error(f);
> > }
> >
> > - trace_loadvm_process_command(cmd, len);
> > + trace_loadvm_process_command(mig_cmd_args[cmd].name, len);
> > if (cmd >= MIG_CMD_MAX || cmd == MIG_CMD_INVALID) {
>
> No! you can't do that name lookup before the limit check.
Heh, right. :)
I guess it shouldn't matter in reality as we don't worry too much on untrusted
or uncompatible src qemu, but it's very reasonable to fix it.
Thanks!
--
Peter Xu
- [PATCH 1/7] migration: Drop dead code of ram_debug_dump_bitmap(), (continued)
- [PATCH 1/7] migration: Drop dead code of ram_debug_dump_bitmap(), Peter Xu, 2021/12/07
- [PATCH 3/7] migration: Drop postcopy_chunk_hostpages(), Peter Xu, 2021/12/07
- [PATCH 4/7] migration: Do chunk page in postcopy_each_ram_send_discard(), Peter Xu, 2021/12/07
- [PATCH 5/7] migration: Drop return code for disgard ram process, Peter Xu, 2021/12/07
- [PATCH 6/7] migration: Dump sub-cmd name in loadvm_process_command tp, Peter Xu, 2021/12/07
- [PATCH 7/7] migration: Finer grained tracepoints for POSTCOPY_LISTEN, Peter Xu, 2021/12/07