[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v5 07/26] monitor: unify global init
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [RFC v5 07/26] monitor: unify global init |
Date: |
Fri, 15 Dec 2017 12:47:11 +0000 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Fri, Dec 15, 2017 at 04:11:41PM +0800, Peter Xu wrote:
> On Wed, Dec 13, 2017 at 03:48:06PM +0000, Stefan Hajnoczi wrote:
> > On Tue, Dec 05, 2017 at 01:51:41PM +0800, Peter Xu wrote:
> > > diff --git a/vl.c b/vl.c
> > > index 1ad1c04637..1ec995a6ae 100644
> > > --- a/vl.c
> > > +++ b/vl.c
> > > @@ -3144,7 +3144,6 @@ int main(int argc, char **argv, char **envp)
> > > qemu_init_exec_dir(argv[0]);
> > >
> > > module_call_init(MODULE_INIT_QOM);
> > > - monitor_init_qmp_commands();
> > >
> > > qemu_add_opts(&qemu_drive_opts);
> > > qemu_add_drive_opts(&qemu_legacy_drive_opts);
> > > @@ -4690,6 +4689,8 @@ int main(int argc, char **argv, char **envp)
> > > default_drive(default_floppy, snapshot, IF_FLOPPY, 0, FD_OPTS);
> > > default_drive(default_sdcard, snapshot, IF_SD, 0, SD_OPTS);
> > >
> > > + monitor_init_globals();
> >
> > Why did you move monitor_init_qmp_commands() down here?
> >
> > There are many function calls between the old position and the new
> > position. Did you check all of them to make sure they don't touch the
> > monitor which is now totally uninitialized?
>
> Yeh, this patch is a bit hairy, but I really think we should do it.
> Because there are too many places (as you have seen) that we inited
> monitor stuff in different places.
But why did you move it down here? Can you replace the
monitor_init_qmp_commands() call instead?
Stefan
signature.asc
Description: PGP signature
- Re: [Qemu-devel] [RFC v5 05/26] qjson: add "opaque" field to JSONMessageParser, (continued)
[Qemu-devel] [RFC v5 06/26] monitor: move the cur_mon hack deeper for QMP, Peter Xu, 2017/12/05
[Qemu-devel] [RFC v5 07/26] monitor: unify global init, Peter Xu, 2017/12/05
- Re: [Qemu-devel] [RFC v5 07/26] monitor: unify global init, Stefan Hajnoczi, 2017/12/13
- Re: [Qemu-devel] [RFC v5 07/26] monitor: unify global init, Peter Xu, 2017/12/15
- Re: [Qemu-devel] [RFC v5 07/26] monitor: unify global init,
Stefan Hajnoczi <=
- Re: [Qemu-devel] [RFC v5 07/26] monitor: unify global init, Peter Xu, 2017/12/15
- Re: [Qemu-devel] [RFC v5 07/26] monitor: unify global init, Stefan Hajnoczi, 2017/12/16
- Re: [Qemu-devel] [RFC v5 07/26] monitor: unify global init, Peter Xu, 2017/12/17
- Re: [Qemu-devel] [RFC v5 07/26] monitor: unify global init, Stefan Hajnoczi, 2017/12/18
- Re: [Qemu-devel] [RFC v5 07/26] monitor: unify global init, Peter Xu, 2017/12/18
[Qemu-devel] [RFC v5 08/26] monitor: let mon_list be tail queue, Peter Xu, 2017/12/05
[Qemu-devel] [RFC v5 09/26] monitor: create monitor dedicate iothread, Peter Xu, 2017/12/05