[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_stat
From: |
Peter Xu |
Subject: |
Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup() |
Date: |
Fri, 15 Mar 2024 10:57:16 -0400 |
On Fri, Mar 15, 2024 at 03:31:28PM +0100, Cédric Le Goater wrote:
> On 3/15/24 14:11, Peter Xu wrote:
> > On Fri, Mar 15, 2024 at 01:20:49PM +0100, Cédric Le Goater wrote:
> > > +static void qemu_savevm_wait_unplug(MigrationState *s, int state)
> >
> > One more trivial comment: I'd even consider dropping "state" altogether, as
> > this should be the only state this function should be invoked. So we can
> > perhaps assert it instead of passing it over?
>
> Yes. If you prefer this implementation I will change.
I am fine with either approach, we can wait for 1-2 days to see whether
others want to say. Otherwise the other approach actually looks better to
me in that it avoids SETUP->UNPLUG->SETUP jumps.
And then we wait to see whether UNPLUG can be dropped for either way to go,
perhaps starting from adding it into deprecation list if no objections from
the relevant folks.
Thanks,
--
Peter Xu
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), (continued)
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), Peter Xu, 2024/03/12
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), Cédric Le Goater, 2024/03/12
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), Fabiano Rosas, 2024/03/12
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), Cédric Le Goater, 2024/03/15
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), Peter Xu, 2024/03/15
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), Cédric Le Goater, 2024/03/15
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), Peter Xu, 2024/03/15
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), Cédric Le Goater, 2024/03/15
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), Peter Xu, 2024/03/15
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), Cédric Le Goater, 2024/03/15
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(),
Peter Xu <=
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), Cédric Le Goater, 2024/03/15
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), Peter Xu, 2024/03/15
- Re: [PATCH v4 10/25] migration: Add Error** argument to qemu_savevm_state_setup(), Cédric Le Goater, 2024/03/19
[PATCH v4 08/25] migration: Always report an error in ram_save_setup(), Cédric Le Goater, 2024/03/06
[PATCH v4 06/25] vfio: Always report an error in vfio_save_setup(), Cédric Le Goater, 2024/03/06
[PATCH v4 12/25] migration: Add Error** argument to .load_setup() handler, Cédric Le Goater, 2024/03/06
[PATCH v4 11/25] migration: Add Error** argument to .save_setup() handler, Cédric Le Goater, 2024/03/06