[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v7 3/6] memory: Introduce memory_region_transaction_do_commit
From: |
Peter Xu |
Subject: |
Re: [PATCH v7 3/6] memory: Introduce memory_region_transaction_do_commit() |
Date: |
Fri, 10 Mar 2023 09:51:32 -0500 |
On Fri, Mar 10, 2023 at 10:24:22AM +0800, Chuang Xu wrote:
> Split memory_region_transaction_do_commit() from
> memory_region_transaction_commit().
>
> We'll call do_commit() in address_space_to_flatview() in the later patch.
>
> Signed-off-by: Chuang Xu <xuchuangxclwt@bytedance.com>
[...]
> +void memory_region_transaction_commit(void)
> +{
> + assert(memory_region_transaction_depth);
> + assert(qemu_mutex_iothread_locked());
This context has nothing to assert BQL, so this can be dropped I think (you
have one in do_commit).
> +
> + --memory_region_transaction_depth;
> + if (!memory_region_transaction_depth) {
> + memory_region_transaction_do_commit();
> + }
> }
With above dropped:
Reviewed-by: Peter Xu <peterx@redhat.com>
--
Peter Xu
- [PATCH v7 0/6] migration: reduce time of loading non-iterable vmstate, Chuang Xu, 2023/03/09
- [PATCH v7 1/6] memory: Reference as->current_map directly in memory commit, Chuang Xu, 2023/03/09
- [PATCH v7 2/6] rcu: Introduce rcu_read_is_locked(), Chuang Xu, 2023/03/09
- [PATCH v7 3/6] memory: Introduce memory_region_transaction_do_commit(), Chuang Xu, 2023/03/09
- Re: [PATCH v7 3/6] memory: Introduce memory_region_transaction_do_commit(),
Peter Xu <=
- [PATCH v7 4/6] memory: Add sanity check in address_space_to_flatview, Chuang Xu, 2023/03/09
- [PATCH v7 5/6] migration: Reduce time of loading non-iterable vmstate, Chuang Xu, 2023/03/09
- [PATCH v7 6/6] memory: Introduce address_space_to_flatview_rcu(), Chuang Xu, 2023/03/09